On 4/26/2022 9:35 AM, Gregory Rose wrote:
On 4/22/2022 8:34 AM, David Marchand wrote:
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.
Hi David and Dimitru,
thanks for the input - I'll see if I can get that working.
- Greg
A quick update - I was able to reinstate the address sanitizer test by
dropping the kernel argument but no luck with afxdp. Perhaps we could
limit afxdp support to branch 2.16 or earlier until we get a mainstream
Ubuntu distro that supports it working.
My biggest problem to date is fixing up the debian packaging and build
to not include the dkms driver package. They're pretty tightly coupled
and I'm no expert in the debian rules for packaging. However, I am
making some progress - one of the main issues is that every time I make
a mistake I have to reclone the repo because debian seems to scatter
little traps around the code that are not cleaned out when you run
'fakeroot debian/rules clean'.
Stay tuned and I'll send a new series of patches after I get this debian
packaging figured out.
Thanks,
- Greg
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
