diff mbox series

[meta-arago,scarthgap] arm-ddr-bandwidth: Bandwidth script for testing

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

Commit Message

Ryan Eatmon June 3, 2024, 4:10 p.m. UTC 
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

Comments

Denys Dmytriyenko June 4, 2024, 4:05 p.m. UTC | #1 
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
Ryan Eatmon June 4, 2024, 4:38 p.m. UTC | #2 
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
Denys Dmytriyenko June 4, 2024, 5:40 p.m. UTC | #3 
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 mbox series

Patch

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 \