openvswitch: Inherit master's labels.
authorJarno Rajahalme <jarno@ovn.org>
Thu, 9 Feb 2017 19:21:58 +0000 (11:21 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 10 Feb 2017 03:59:34 +0000 (22:59 -0500)
commit09aa98ad496d6b11a698b258bc64d7f64c55d682
treec58988ce19e23909d9e24ab7b0a20a35991c9dda
parent6ffcea79957df43caeaa6d1de5062556a5afc262
openvswitch: Inherit master's labels.

We avoid calling into nf_conntrack_in() for expected connections, as
that would remove the expectation that we want to stick around until
we are ready to commit the connection.  Instead, we do a lookup in the
expectation table directly.  However, after a successful expectation
lookup we have set the flow key label field from the master
connection, whereas nf_conntrack_in() does not do this.  This leads to
master's labels being inherited after an expectation lookup, but those
labels not being inherited after the corresponding conntrack action
with a commit flag.

This patch resolves the problem by changing the commit code path to
also inherit the master's labels to the expected connection.
Resolving this conflict in favor of inheriting the labels allows more
information be passed from the master connection to related
connections, which would otherwise be much harder if the 32 bits in
the connmark are not enough.  Labels can still be set explicitly, so
this change only affects the default values of the labels in presense
of a master connection.

Fixes: 7f8a436eaa2c ("openvswitch: Add conntrack action")
Signed-off-by: Jarno Rajahalme <jarno@ovn.org>
Acked-by: Pravin B Shelar <pshelar@ovn.org>
Acked-by: Joe Stringer <joe@ovn.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/openvswitch/conntrack.c