diff mbox series

[ovs-dev,v2,3/3] ci: Add kernel and userspace ASAN/UBSAN tests.

Message ID 2dc68595e326af56d92827a965ee3caf761542c9.1704882144.git.echaudro@redhat.com
State Accepted
Commit 8b51b2bcbd6ee19662976c1a63862a57360ccfa5
Delegated to: Eelco Chaudron
Headers show
Series [ovs-dev,v2,1/3] tests: Set handle_segv fpr UBSAN to allow SIGSEGV tests. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test fail github build: failed
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Eelco Chaudron Jan. 10, 2024, 10:22 a.m. UTC
This patch adds ASAN and UBSAN GitHub action tests for both
the userspace and kernel datapaths.

Signed-off-by: Eelco Chaudron <echaudro@redhat.com>
---
 .github/workflows/build-and-test.yml | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Eelco Chaudron Jan. 10, 2024, 12:12 p.m. UTC | #1
Recheck-request: github-robot

On 10 Jan 2024, at 11:22, Eelco Chaudron wrote:

> This patch adds ASAN and UBSAN GitHub action tests for both
> the userspace and kernel datapaths.
>
> Signed-off-by: Eelco Chaudron <echaudro@redhat.com>
> ---
>  .github/workflows/build-and-test.yml | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
> index 1b9dc4ef3..7a90a30ab 100644
> --- a/.github/workflows/build-and-test.yml
> +++ b/.github/workflows/build-and-test.yml
> @@ -172,6 +172,15 @@ jobs:
>              testsuite:    check-kernel
>              test_range:   "100-"
>
> +          - compiler:     clang
> +            sanitizers:   address,undefined
> +            testsuite:    check-kernel
> +            test_range:   "-100"
> +          - compiler:     clang
> +            sanitizers:   address,undefined
> +            testsuite:    check-kernel
> +            test_range:   "100-"
> +
>            - compiler:     gcc
>              testsuite:    check-offloads
>              test_range:   "-100"
> @@ -183,6 +192,11 @@ jobs:
>              dpdk:         dpdk
>              testsuite:    check-system-userspace
>
> +          - compiler:     clang
> +            sanitizers:   address,undefined
> +            dpdk:         dpdk
> +            testsuite:    check-system-userspace
> +
>            - compiler:     gcc
>              dpdk:         dpdk
>              testsuite:    check-system-tso
> -- 
> 2.43.0
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Simon Horman Jan. 11, 2024, 12:01 p.m. UTC | #2
On Wed, Jan 10, 2024 at 11:22:24AM +0100, Eelco Chaudron wrote:
> This patch adds ASAN and UBSAN GitHub action tests for both
> the userspace and kernel datapaths.
> 
> Signed-off-by: Eelco Chaudron <echaudro@redhat.com>

Acked-by: Simon Horman <horms@ovn.org>
Eelco Chaudron Jan. 18, 2024, 8:02 a.m. UTC | #3
On 11 Jan 2024, at 13:01, Simon Horman wrote:

> On Wed, Jan 10, 2024 at 11:22:24AM +0100, Eelco Chaudron wrote:
>> This patch adds ASAN and UBSAN GitHub action tests for both
>> the userspace and kernel datapaths.
>>
>> Signed-off-by: Eelco Chaudron <echaudro@redhat.com>
>
> Acked-by: Simon Horman <horms@ovn.org>

Thanks, I’ve applied the series:

* 8b51b2bcb ci: Add kernel and userspace ASAN/UBSAN tests.
* fc13c0d65 ci: Combine the ubsan and asan sanitizer runs.
* 48d4f6963 tests: Set handle_segv for UBSAN to allow SIGSEGV tests.
diff mbox series

Patch

diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
index 1b9dc4ef3..7a90a30ab 100644
--- a/.github/workflows/build-and-test.yml
+++ b/.github/workflows/build-and-test.yml
@@ -172,6 +172,15 @@  jobs:
             testsuite:    check-kernel
             test_range:   "100-"
 
+          - compiler:     clang
+            sanitizers:   address,undefined
+            testsuite:    check-kernel
+            test_range:   "-100"
+          - compiler:     clang
+            sanitizers:   address,undefined
+            testsuite:    check-kernel
+            test_range:   "100-"
+
           - compiler:     gcc
             testsuite:    check-offloads
             test_range:   "-100"
@@ -183,6 +192,11 @@  jobs:
             dpdk:         dpdk
             testsuite:    check-system-userspace
 
+          - compiler:     clang
+            sanitizers:   address,undefined
+            dpdk:         dpdk
+            testsuite:    check-system-userspace
+
           - compiler:     gcc
             dpdk:         dpdk
             testsuite:    check-system-tso