Re: [meta-arago][scarthgap][PATCH v5] coremark: add Coremark CPU benchmark

On 11/19/2025 12:04 PM, Denys Dmytriyenko wrote:
> On Wed, Nov 19, 2025 at 11:27:01AM -0600, Ryan Eatmon via lists.yoctoproject.org wrote:
>>
>>
>> On 11/19/2025 6:41 AM, sadik wrote:
>>> Add CoreMark benchmark recipe to measure CPU
>>> performance on TI platforms,Integrate CoreMark
>>> into ti-test package group alongside arm-benchmarks.
>>>
>>> CoreMark is an industry-standard CPU benchmark from
>>> EEMBC(Embedded Microprocessor Benchmark Consortium)
>>> that provides a standardized method to test the
>>> performance of processors.
>>>
>>> Signed-off-by: sadik <[email protected]>
>>> ---
>>>
>>> Changes in v5:
>>> - change subject line to include recipe name
>>> - append _git to recipe name for tracking of version
>>>
>>> Link to v4:https://lists.yoctoproject.org/g/meta-arago/message/16784
>>>
>>> Changes in v4:
>>> - change logs moved after the "---" line
>>> - change the summary of the coremark in recipe
>>>
>>> Link to v3:https://lists.yoctoproject.org/g/meta-arago/message/16776
>>>
>>> Changes in v3:
>>> - sent patch via UTF-8 encoding
>>>
>>> Changes in v2:
>>> - Include Coremark licese file in licenses
>>> - Use oe_runmake with proper compiler flags for cross-compilation
>>> - Removed unnecessary TARGET_CC_ARCH, COMPATIBLE_HOST, and SRC_URI checksum
>>> - Move coremark in alphabetical order in TI_TEST_TI_TOOLS
>>>
>>> Link to v1:https://lists.yoctoproject.org/g/meta-arago/message/16707
>>>
>>> ---
>>>   meta-arago-test/licenses/Coremark             | 91 +++++++++++++++++++
>>>   .../coremark/coremark_git.bb                  | 18 ++++
>>>   .../recipes-core/packagegroups/ti-test.bb     |  1 +
>>>   3 files changed, 110 insertions(+)
>>>   create mode 100644 meta-arago-test/licenses/Coremark
>>>   create mode 100644 meta-arago-test/recipes-benchmark/coremark/coremark_git.bb
>>>
>>> diff --git a/meta-arago-test/recipes-benchmark/coremark/coremark_git.bb b/meta-arago-test/recipes-benchmark/coremark/coremark_git.bb
>>> new file mode 100644
>>> index 00000000..7655a445
>>> --- /dev/null
>>> +++ b/meta-arago-test/recipes-benchmark/coremark/coremark_git.bb
>>> @@ -0,0 +1,18 @@
>>> +SUMMARY = "Industry-standard benchmark that measures CPU performance"
>>> +
>>> +LICENSE = "Coremark"
>>> +LIC_FILES_CHKSUM = "file://LICENSE.md;md5=0a18b17ae63deaa8a595035f668aebe1"
>>> +
>>> +SRC_URI = "git://github.com/eembc/coremark.git;branch=main;protocol=https"
>>> +SRCREV = "1f483d5b8316753a742cbf5590caf5bd0a4e4777"
>>> +
>>> +S = "${WORKDIR}/git"
>>
>> This patch looks better, and I'll start testing it today.  But I
>> also need a master version of this patch since this recipe will not
>> work in master as is.
>>
>> The only change that I think is needed is to remove the line:
>>
>> S = "${WORKDIR}/git"
> 
> It still doesn't set the version, as _git suffix in the filename isn't for
> that.
> 
> I see when CoreMark sources were imported into GitHub, they were tagged as
> 1.01 corresponding to a 2009 release, and no new releases or tags were made
> since then:
> 
> https://github.com/eembc/coremark/releases/tag/v1.01
> 
> This means we should probably add
> 
> PV = "1.01+git"
> 
> to base off of the release and track individual commits since then.

Good catch.  I should add this as a check to the check_repo step in PRC. 
  Making a note...


> 
>>> +do_compile() {
>>> +    oe_runmake PORT_DIR=linux CC="${CC}" XCFLAGS="${CFLAGS}" LFLAGS_END="${LDFLAGS}" compile
>>> +}
>>> +
>>> +do_install() {
>>> +    install -d ${D}${bindir}
>>> +    install -m 0755 ${S}/coremark.exe ${D}${bindir}/coremark
>>> +}
>>> diff --git a/meta-arago-test/recipes-core/packagegroups/ti-test.bb b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
>>> index c1166bc2..e05add8b 100644
>>> --- a/meta-arago-test/recipes-core/packagegroups/ti-test.bb
>>> +++ b/meta-arago-test/recipes-core/packagegroups/ti-test.bb
>>> @@ -89,6 +89,7 @@ TI_TEST_BASE:append:armv7ve = " \
>>>   TI_TEST_TI_TOOLS = " \
>>>       arm-benchmarks \
>>>       arm-ddr-bandwidth \
>>> +    coremark \
>>>       cpuloadgen \
>>>       input-utils \
>>>       ltp-ddt \

-- 
Ryan Eatmon                [email protected]
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS