[ovs-dev] [PATCH v2] ipf: fix only nat the first fragment in the reass process

Paolo Valerio pvalerio at redhat.com
Thu Aug 12 17:33:52 UTC 2021


Hi,

thanks for working on this

wenxu at ucloud.cn writes:

> From: wenxu <wenxu at ucloud.cn>
>
> The ipf collect original fragment packets and reass a new pkt
> to do the conntrack logic. After finsh the conntrack things
> copy the ct meta info to each orignal packet and modify the
> l4 header in the first fragment. It should modify the ip src/
> dst info for all the fragments.
>
> Signed-off-by: wenxu <wenxu at ucloud.cn>
> Co-authored-by: luke.li <luke.li at ucloud.cn>
> Signed-off-by: luke.li <luke.li at ucloud.cn>
> ---

The patch LGTM, and works as expected.
Any chance to add a Fixes tag?

I think it should be:

4ea96698f667 ("Userspace datapath: Add fragmentation handling.")

besides that:

Acked-by: Paolo Valerio <pvalerio at redhat.com>



More information about the dev mailing list