On Thu, Sep 1, 2016 at 5:01 PM, Joe Stringer <j...@ovn.org> wrote:
> The upstream code uses NF_INET_PRE_ROUTING hook for the nf_conntrack_in()
> call, which does deeper (eg l4proto) validation. It was previously
> thought that using the NF_INET_ROUTING hook for this function on older
> kernels would trigger kernel panics due to a dependency on the
> unpopulated skb->dev, however during recent testing on a variety of
> platforms (Centos7.[12], Ubuntu 1[46].04, Fedora23) using the latest
> distribution kernels and the OVS kernel module testsuite, no such kernel
> panics were observed. Therefore it appears to be safe to bring this in
> line with upstream without any other workarounds.
>
> Reported-by: Jesse Gross <je...@kernel.org>
> Signed-off-by: Joe Stringer <j...@ovn.org>

If you are confident that it doesn't cause problems on older kernels,
the change looks obviously correct to me relative to upstream.

Acked-by: Jesse Gross <je...@kernel.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to