On Wed, Nov 12, 2025 at 12:05 PM David Marchand <[email protected]>
wrote:
> Let's enhance our coverage further in a way similar to
> commit 585c8088eb23 ("dpif-netdev: Enhance checksum coverage.").
>
> Add more knobs in the netdev-dummy driver for faking outer checksums
> support and force incorrect packet with all combinations of possible
> Rx flags.
>
> On the unit tests side, IPv4 and IPv6 vxlan tunnels are tested for
> encapsulating ARP, IP, IP/TCP, IP/UDP, IP6/NDP, IP6/TCP and IP6/UDP
> traffic.
>
> Signed-off-by: David Marchand <[email protected]>
> ---
>
> ...
> +dnl Simple ARP.
> +$PYTHON3 -c dnl
> +"from scapy.all import Ether, ARP; "dnl
> +"p = Ether(dst='ff:ff:ff:ff:ff:ff'); "dnl
> +"p /= ARP(pdst='192.168.123.1', hwsrc='8a:bf:7e:2f:05:84'); "dnl
> +"print(p.build().hex())" > arp_frame
>
I think the code changes look reasonable, but is it acceptable to include
scapy as a dependency in the build tests? It appears to only be currently
used in the two MFEX autovalidator tests, and isn't listed on the build
requirements page.
-M
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
