[ovs-dev] [PATCH v2] tests: Add check for correct l3l4 conntrack frag reassembly

Gregory Rose gvrose8192 at gmail.com
Fri Oct 4 14:05:26 UTC 2019


On 10/3/2019 6:56 PM, Darrell Ball wrote:
> Thanks for the patch
>
> This approach will not work for the userspace datapath
>
> Few issues off the top of my head:
>
> 1/ packet-out frees the packet (which is a fragment in this case) 
> after completion
>    hence multiple packet-outs need to be part of a single Openflow 
> bundle command as in other similar tests
>    This test involves 2 fragments for the first 2 packets.
>
> 2/ Userpsace datapath checks UDP checksums; for V6 packets, they need 
> to always be correct and they are not presently
>
> 3/ UDP header lengths cannot be larger than the memory allocated for 
> the packet, else sanity checking will filter
>     out the packet
>
> Alternatively, if you want to use this simplified approach, you can 
> disable the test for the userspace datapath.

There's no reason that I know of to run this test in the userspace 
datapath.  It should be disabled.

How do you do that?

- Greg




More information about the dev mailing list