Message ID | 20240603161039.16176-2-reatmon@ti.com |
---|---|
State | Superseded |
Delegated to: | Ryan Eatmon |
Headers | show |
Series | [meta-arago,scarthgap] arm-ddr-bandwidth: Bandwidth script for testing | expand |
On Mon, Jun 03, 2024 at 11:10:39AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote: > We need to pull a single script in from the matrix-gui packages since > they are getting removed. The runLmDDRBandwidth.sh is required for the > ti-test flow. > > Signed-off-by: Ryan Eatmon <reatmon@ti.com> > --- > .../arm-ddr-bandwidth/arm-ddr-bandwidth.bb | 23 +++++++++++++++++++ > .../recipes-core/packagegroups/ti-test.bb | 1 + > 2 files changed, 24 insertions(+) > create mode 100644 meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb > > diff --git a/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb > new file mode 100644 > index 00000000..fc4aea94 > --- /dev/null > +++ b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb > @@ -0,0 +1,23 @@ > +SUMMARY = "Benchmark for checking DDR bandwidth" > +HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh" > +LICENSE = "CC-BY-SA-3.0" > +LIC_FILES_CHKSUM = "file://${WORKDIR}/LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9" Isn't the default here ^^^ relative to ${S} anyway? > + > +PV = "1.0" > + > +SRC_URI = " \ > + https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh;name=runLmDDRBandwidth \ > + https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \ > +" > +SRC_URI[runLmDDRBandwidth.sha256sum] = "67997d721b351344d8788200a15bb3640093045bc25f2b5307dd331ebd1556a3" > +SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3" > + > +S = "${WORKDIR}" > + > +do_install() { > + install -d ${D}${datadir}/matrix-gui-2.0/apps > + install -m 0644 ${S}/runLmDDRBandwidth.sh ${D}${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh > +} > + > +FILES:${PN} = "${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh" > + > diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > index 4dbd9fca..b71c374b 100644 > --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb > +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > @@ -69,6 +69,7 @@ TI_TEST_BASE:append:armv7ve = " \ > # timestamping > TI_TEST_TI_TOOLS = " \ > arm-benchmarks \ > + arm-ddr-bandwidth \ > cpuloadgen \ > input-utils \ > ltp-ddt \ > -- > 2.17.1
On 6/4/2024 11:05 AM, Denys Dmytriyenko wrote: > On Mon, Jun 03, 2024 at 11:10:39AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote: >> We need to pull a single script in from the matrix-gui packages since >> they are getting removed. The runLmDDRBandwidth.sh is required for the >> ti-test flow. >> >> Signed-off-by: Ryan Eatmon <reatmon@ti.com> >> --- >> .../arm-ddr-bandwidth/arm-ddr-bandwidth.bb | 23 +++++++++++++++++++ >> .../recipes-core/packagegroups/ti-test.bb | 1 + >> 2 files changed, 24 insertions(+) >> create mode 100644 meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb >> >> diff --git a/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb >> new file mode 100644 >> index 00000000..fc4aea94 >> --- /dev/null >> +++ b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb >> @@ -0,0 +1,23 @@ >> +SUMMARY = "Benchmark for checking DDR bandwidth" >> +HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh" >> +LICENSE = "CC-BY-SA-3.0" >> +LIC_FILES_CHKSUM = "file://${WORKDIR}/LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9" > > Isn't the default here ^^^ relative to ${S} anyway? Could be. I was making a number of changes to get this recipe "right". I'll test it and make sure it works without it. It might be that master now requires the UNPACKDIR and when I backported this to scarthgap I didn't check that it was not needed. > >> + >> +PV = "1.0" >> + >> +SRC_URI = " \ >> + https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh;name=runLmDDRBandwidth \ >> + https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \ >> +" >> +SRC_URI[runLmDDRBandwidth.sha256sum] = "67997d721b351344d8788200a15bb3640093045bc25f2b5307dd331ebd1556a3" >> +SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3" >> + >> +S = "${WORKDIR}" >> + >> +do_install() { >> + install -d ${D}${datadir}/matrix-gui-2.0/apps >> + install -m 0644 ${S}/runLmDDRBandwidth.sh ${D}${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh >> +} >> + >> +FILES:${PN} = "${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh" >> + >> diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> index 4dbd9fca..b71c374b 100644 >> --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb >> @@ -69,6 +69,7 @@ TI_TEST_BASE:append:armv7ve = " \ >> # timestamping >> TI_TEST_TI_TOOLS = " \ >> arm-benchmarks \ >> + arm-ddr-bandwidth \ >> cpuloadgen \ >> input-utils \ >> ltp-ddt \ >> -- >> 2.17.1
On Tue, Jun 04, 2024 at 11:38:32AM -0500, Ryan Eatmon wrote: > > > On 6/4/2024 11:05 AM, Denys Dmytriyenko wrote: > >On Mon, Jun 03, 2024 at 11:10:39AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote: > >>We need to pull a single script in from the matrix-gui packages since > >>they are getting removed. The runLmDDRBandwidth.sh is required for the > >>ti-test flow. > >> > >>Signed-off-by: Ryan Eatmon <reatmon@ti.com> > >>--- > >> .../arm-ddr-bandwidth/arm-ddr-bandwidth.bb | 23 +++++++++++++++++++ > >> .../recipes-core/packagegroups/ti-test.bb | 1 + > >> 2 files changed, 24 insertions(+) > >> create mode 100644 meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb > >> > >>diff --git a/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb > >>new file mode 100644 > >>index 00000000..fc4aea94 > >>--- /dev/null > >>+++ b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb > >>@@ -0,0 +1,23 @@ > >>+SUMMARY = "Benchmark for checking DDR bandwidth" > >>+HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh" > >>+LICENSE = "CC-BY-SA-3.0" > >>+LIC_FILES_CHKSUM = "file://${WORKDIR}/LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9" > > > >Isn't the default here ^^^ relative to ${S} anyway? > > Could be. I was making a number of changes to get this recipe > "right". I'll test it and make sure it works without it. It might > be that master now requires the UNPACKDIR and when I backported this > to scarthgap I didn't check that it was not needed. Yeah, it should also work for master, as long as LICENSE is in ${S} And I see there you set UNPACKDIR = "${S}" > >>+PV = "1.0" > >>+ > >>+SRC_URI = " \ > >>+ https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh;name=runLmDDRBandwidth \ > >>+ https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \ > >>+" > >>+SRC_URI[runLmDDRBandwidth.sha256sum] = "67997d721b351344d8788200a15bb3640093045bc25f2b5307dd331ebd1556a3" > >>+SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3" > >>+ > >>+S = "${WORKDIR}" > >>+ > >>+do_install() { > >>+ install -d ${D}${datadir}/matrix-gui-2.0/apps > >>+ install -m 0644 ${S}/runLmDDRBandwidth.sh ${D}${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh > >>+} > >>+ > >>+FILES:${PN} = "${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh" > >>+ > >>diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > >>index 4dbd9fca..b71c374b 100644 > >>--- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb > >>+++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb > >>@@ -69,6 +69,7 @@ TI_TEST_BASE:append:armv7ve = " \ > >> # timestamping > >> TI_TEST_TI_TOOLS = " \ > >> arm-benchmarks \ > >>+ arm-ddr-bandwidth \ > >> cpuloadgen \ > >> input-utils \ > >> ltp-ddt \ > >>-- > >>2.17.1
diff --git a/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb new file mode 100644 index 00000000..fc4aea94 --- /dev/null +++ b/meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb @@ -0,0 +1,23 @@ +SUMMARY = "Benchmark for checking DDR bandwidth" +HOMEPAGE = "https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/tree/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh" +LICENSE = "CC-BY-SA-3.0" +LIC_FILES_CHKSUM = "file://${WORKDIR}/LICENSE;md5=6e0ae7214f6c74c149cb25f373057fa9" + +PV = "1.0" + +SRC_URI = " \ + https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/arm_apps/arm_ddr_bandwidth/runLmDDRBandwidth.sh;name=runLmDDRBandwidth \ + https://git.ti.com/cgit/matrix-gui-v2/matrix-gui-v2-apps/plain/LICENSE;name=license \ +" +SRC_URI[runLmDDRBandwidth.sha256sum] = "67997d721b351344d8788200a15bb3640093045bc25f2b5307dd331ebd1556a3" +SRC_URI[license.sha256sum] = "7febd1df714fa4b1e44fe0b0f73ceac7f9b9f97326695a0cc7074bd6c8d263e3" + +S = "${WORKDIR}" + +do_install() { + install -d ${D}${datadir}/matrix-gui-2.0/apps + install -m 0644 ${S}/runLmDDRBandwidth.sh ${D}${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh +} + +FILES:${PN} = "${datadir}/matrix-gui-2.0/apps/runLmDDRBandwidth.sh" + diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb index 4dbd9fca..b71c374b 100644 --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb @@ -69,6 +69,7 @@ TI_TEST_BASE:append:armv7ve = " \ # timestamping TI_TEST_TI_TOOLS = " \ arm-benchmarks \ + arm-ddr-bandwidth \ cpuloadgen \ input-utils \ ltp-ddt \
We need to pull a single script in from the matrix-gui packages since they are getting removed. The runLmDDRBandwidth.sh is required for the ti-test flow. Signed-off-by: Ryan Eatmon <reatmon@ti.com> --- .../arm-ddr-bandwidth/arm-ddr-bandwidth.bb | 23 +++++++++++++++++++ .../recipes-core/packagegroups/ti-test.bb | 1 + 2 files changed, 24 insertions(+) create mode 100644 meta-arago-test/recipes-benchmark/arm-ddr-bandwidth/arm-ddr-bandwidth.bb