On Fri, Apr 22, 2022 at 9:24 AM Dumitru Ceara <dce...@redhat.com> wrote:
> > @@ -37,14 +30,6 @@ jobs:
> >            - compiler:     clang
> >              opts:         --disable-ssl
> >
> > -          - compiler:     gcc
> > -            testsuite:    test
> > -            kernel:       3.16
> > -          - compiler:     clang
> > -            testsuite:    test
> > -            kernel:       3.16
> > -            asan:         asan
> > -
>
> Maybe it's enough to just remove the 'kernel' argument here?  It's
> useful to have address sanitizer enabled test jobs too.  Arguably, when
> we added it we should've maybe not link it to a given kernel but, in any
> case, I think we shouldn't remove it completely now.
>
> What do you think of keeping it as:
>
> - compiler:     clang
>   testsuite:    test
>   asan:         asan

+1

>
> I might be wrong but it seems to me we need something similar for afxdp too.

The Ubuntu 18.04 kernel does not have AF_XDP support iirc, so either
we keep on building a 5.3 kernel and compile ovs with afxdp against
this kernel.
Or we may switch to Ubuntu 20.04 in GHA.


-- 
David Marchand

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to