mbox series

[0/3] Introducing opencl testing

Message ID 20250205200411.686617-1-a-christidis@ti.com
Headers show
Series Introducing opencl testing | expand

Message

Antonios Christidis Feb. 5, 2025, 8:04 p.m. UTC
From: Antonios Christidis <a0507488@uda0867391-2.dhcp.ti.com>

Hello ! 

This patch series, includes everything needed to start testing opencl. 
This includes: 
- Recipe to build opencl-cts 
- A patch for opencl-cts itself 
- A bbappend to bump the version of opencl-headers

Reasoning for the above 3 can be found on the individual patches. 

Antonios Christidis (3):
  meta-arago-test: Ignore Compiler Warnings Patch for OpenCL-CTS
  meta-arago-test: Introducing OpenCL testing
  meta-arago-test: opencl-headers version v2023.12.14->v2024.05.08

 .../recipes-core/packagegroups/ti-test.bb     |  1 +
 .../opencl/opencl-headers_%.bbappend          |  1 +
 .../files/0001-Ignore-Compiler-Warnings.patch | 35 +++++++++++
 ...Ignore-Compiler-Warnings-Patch-for-O.patch | 61 +++++++++++++++++++
 .../opencl/opencl-cts_2024.08.08.bb           | 27 ++++++++
 5 files changed, 125 insertions(+)
 create mode 100644 meta-arago-test/recipes-extended/opencl/opencl-headers_%.bbappend
 create mode 100644 meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch
 create mode 100644 meta-arago-test/recipes-graphics/opencl/files/0001-meta-arago-test-Ignore-Compiler-Warnings-Patch-for-O.patch
 create mode 100644 meta-arago-test/recipes-graphics/opencl/opencl-cts_2024.08.08.bb

Comments

Antonios Christidis Feb. 5, 2025, 8:12 p.m. UTC | #1
Forgot to tag the patches. All patches in this series are for scarthgap.

On 2/5/25 2:04 PM, Antonios Christidis via lists.yoctoproject.org wrote:
> From: Antonios Christidis <a0507488@ uda0867391-2. dhcp. ti. com> 
> Within the cmake build system, the compiler -Werror flag is enabeled. 
> This turns all warnings into errors and doens't allow the system to 
> compile correctly. Change the system
> ZjQcmQRYFpfptBannerStart
> This message was sent from outside of Texas Instruments.
> Do not click links or open attachments unless you recognize the source 
> of this email and know the content is safe.
> Report Suspicious
> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uRdqPBfPVqwaKqYFPP3NsvaOW-AC0f2W5AKZr2gZk4GyKmNn6P9WH_O3f5crwuRtqfpZzQALJjaRVfm6nsYFMQwAZs8g$> 
>
> ZjQcmQRYFpfptBannerEnd
> From: Antonios Christidis <a0507488@uda0867391-2.dhcp.ti.com>
>
> Within the cmake build system, the compiler -Werror flag is enabeled.  This
> turns all warnings into errors and doens't allow the system to compile
> correctly.  Change the system to ignore the 3 warning rules that show up when
> compiling.
>
> Signed-off-by: Antonios Christidis <a-christidis@ti.com>
> ---
>   .../files/0001-Ignore-Compiler-Warnings.patch | 35 +++++++++++++++++++
>   1 file changed, 35 insertions(+)
>   create mode 100644 meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch
>
> diff --git a/meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch b/meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch
> new file mode 100644
> index 00000000..c21db6fb
> --- /dev/null
> +++ b/meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch
> @@ -0,0 +1,35 @@
> +From 082ce10dd191f036f5a6e1fb31134e9736ba3643 Mon Sep 17 00:00:00 2001
> +From: Antonios Christidis <a-christidis@ti.com>
> +Date: Fri, 17 Jan 2025 15:05:39 -0600
> +Subject: [PATCH] Ignore Compiler Warnings
> +
> +Within the cmake build system, the compiler -Werror flag is enabeled.  This
> +turns all warnings into errors and doens't allow the system to compile
> +correctly.  Change the system to ignore the 3 warning rules that show up when
> +compiling.
> +
> +Upstream-Status: Inappropriate [oe specific - compilation errors due to cross-compiling]
> +
> +Singed-Off-By: Antonios Christidis <a-christidis@ti.com>
> +---
> + CMakeLists.txt | 4 +++-
> + 1 file changed, 3 insertions(+), 1 deletion(-)
> +
> +diff --git a/CMakeLists.txt b/CMakeLists.txt
> +index 5cfef6b3..71b6ddb4 100644
> +--- a/CMakeLists.txt
> ++++ b/CMakeLists.txt
> +@@ -109,7 +109,9 @@ if(CMAKE_COMPILER_IS_GNUCC OR "${CMAKE_CXX_COMPILER_ID}" MATCHES "(Apple)?Clang"
> +     add_cxx_flag_if_supported(-Wno-error=cpp) # Allow #warning directive
> +     add_cxx_flag_if_supported(-Wno-unknown-pragmas) # Issue #785
> +     add_cxx_flag_if_supported(-Wno-error=asm-operand-widths) # Issue #784
> +-
> ++    add_cxx_flag_if_supported(-Wno-maybe-uninitialized)
> ++    add_cxx_flag_if_supported(-Wno-stringop-truncation)
> ++    add_cxx_flag_if_supported(-Wno-strict-aliasing)
> +     # -msse -mfpmath=sse to force gcc to use sse for float math,
> +     # avoiding excess precision problems that cause tests like int2float
> +     # to falsely fail. -ffloat-store also works, but WG suggested
> +--
> +2.34.1
> +
> -- 
> 2.34.1
>
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#15814):https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-arago/message/15814__;!!G3vK!VU6Bx3Q5OIRRTq2hv8bOVDuwJFJNPEUWBXxrYPF1d_H8AjBdwcpTefEFc4WHcFSEFI5XC9ty1GZ516xqqesdeOnMeHGcxUKh3zxd$
> Mute This Topic:https://urldefense.com/v3/__https://lists.yoctoproject.org/mt/111019654/9164840__;!!G3vK!VU6Bx3Q5OIRRTq2hv8bOVDuwJFJNPEUWBXxrYPF1d_H8AjBdwcpTefEFc4WHcFSEFI5XC9ty1GZ516xqqesdeOnMeHGcxTPinxbc$
> Group Owner: meta-arago+owner@lists.yoctoproject.org
> Unsubscribe:https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-arago/unsub__;!!G3vK!VU6Bx3Q5OIRRTq2hv8bOVDuwJFJNPEUWBXxrYPF1d_H8AjBdwcpTefEFc4WHcFSEFI5XC9ty1GZ516xqqesdeOnMeHGcxQVuUZZq$  [a-christidis@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Ryan Eatmon Feb. 6, 2025, 4:08 p.m. UTC | #2
On 2/5/2025 2:12 PM, Antonios Christidis via lists.yoctoproject.org wrote:
> Forgot to tag the patches. All patches in this series are for scarthgap.

There needs to be a master version of this patch that adds the package 
to ti-test.


> On 2/5/25 2:04 PM, Antonios Christidis via lists.yoctoproject.org wrote:
>> From: Antonios Christidis <a0507488@ uda0867391-2. dhcp. ti. com> 
>> Within the cmake build system, the compiler -Werror flag is enabeled. 
>> This turns all warnings into errors and doens't allow the system to 
>> compile correctly. Change the system
>> ZjQcmQRYFpfptBannerStart
>> This message was sent from outside of Texas Instruments.
>> Do not click links or open attachments unless you recognize the source 
>> of this email and know the content is safe.
>> Report Suspicious
>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!uRdqPBfPVqwaKqYFPP3NsvaOW-AC0f2W5AKZr2gZk4GyKmNn6P9WH_O3f5crwuRtqfpZzQALJjaRVfm6nsYFMQwAZs8g$>
>> ZjQcmQRYFpfptBannerEnd
>> From: Antonios Christidis <a0507488@uda0867391-2.dhcp.ti.com>
>>
>> Within the cmake build system, the compiler -Werror flag is enabeled.  
>> This
>> turns all warnings into errors and doens't allow the system to compile
>> correctly.  Change the system to ignore the 3 warning rules that show 
>> up when
>> compiling.
>>
>> Signed-off-by: Antonios Christidis <a-christidis@ti.com>
>> ---
>>   .../files/0001-Ignore-Compiler-Warnings.patch | 35 +++++++++++++++++++
>>   1 file changed, 35 insertions(+)
>>   create mode 100644 
>> meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch
>>
>> diff --git 
>> a/meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch b/meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch
>> new file mode 100644
>> index 00000000..c21db6fb
>> --- /dev/null
>> +++ 
>> b/meta-arago-test/recipes-graphics/opencl/files/0001-Ignore-Compiler-Warnings.patch
>> @@ -0,0 +1,35 @@
>> +From 082ce10dd191f036f5a6e1fb31134e9736ba3643 Mon Sep 17 00:00:00 2001
>> +From: Antonios Christidis <a-christidis@ti.com>
>> +Date: Fri, 17 Jan 2025 15:05:39 -0600
>> +Subject: [PATCH] Ignore Compiler Warnings
>> +
>> +Within the cmake build system, the compiler -Werror flag is 
>> enabeled.  This
>> +turns all warnings into errors and doens't allow the system to compile
>> +correctly.  Change the system to ignore the 3 warning rules that show 
>> up when
>> +compiling.
>> +
>> +Upstream-Status: Inappropriate [oe specific - compilation errors due 
>> to cross-compiling]
>> +
>> +Singed-Off-By: Antonios Christidis <a-christidis@ti.com>
>> +---
>> + CMakeLists.txt | 4 +++-
>> + 1 file changed, 3 insertions(+), 1 deletion(-)
>> +
>> +diff --git a/CMakeLists.txt b/CMakeLists.txt
>> +index 5cfef6b3..71b6ddb4 100644
>> +--- a/CMakeLists.txt
>> ++++ b/CMakeLists.txt
>> +@@ -109,7 +109,9 @@ if(CMAKE_COMPILER_IS_GNUCC OR 
>> "${CMAKE_CXX_COMPILER_ID}" MATCHES "(Apple)?Clang"
>> +     add_cxx_flag_if_supported(-Wno-error=cpp) # Allow #warning 
>> directive
>> +     add_cxx_flag_if_supported(-Wno-unknown-pragmas) # Issue #785
>> +     add_cxx_flag_if_supported(-Wno-error=asm-operand-widths) # Issue 
>> #784
>> +-
>> ++    add_cxx_flag_if_supported(-Wno-maybe-uninitialized)
>> ++    add_cxx_flag_if_supported(-Wno-stringop-truncation)
>> ++    add_cxx_flag_if_supported(-Wno-strict-aliasing)
>> +     # -msse -mfpmath=sse to force gcc to use sse for float math,
>> +     # avoiding excess precision problems that cause tests like 
>> int2float
>> +     # to falsely fail. -ffloat-store also works, but WG suggested
>> +--
>> +2.34.1
>> +
>> -- 
>> 2.34.1
>>
>>
>>
>>
>>
>>
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#15818): 
> https://lists.yoctoproject.org/g/meta-arago/message/15818
> Mute This Topic: https://lists.yoctoproject.org/mt/111019823/6551054
> Group Owner: meta-arago+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub 
> [reatmon@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
> 
>