Xin Long <[email protected]> writes: > Ilya found a failure in running check-kernel tests with at_groups=144 > (144: conntrack - FTP SNAT orig tuple) in OVS repo. After his further > investigation, the root cause is that the labels sent to userspace > for related ct are incorrect. > > The labels for unconfirmed related ct should use its master's labels. > However, the changes made in commit 8c8b73320805 ("openvswitch: set > IPS_CONFIRMED in tmpl status only when commit is set in conntrack") > led to getting labels from this related ct. > > So fix it in ovs_ct_get_labels() by changing to copy labels from its > master ct if it is a unconfirmed related ct. Note that there is no > fix needed for ct->mark, as it was already copied from its master > ct for related ct in init_conntrack(). > > Fixes: 8c8b73320805 ("openvswitch: set IPS_CONFIRMED in tmpl status > only when commit is set in conntrack") > Reported-by: Ilya Maximets <[email protected]> > Signed-off-by: Xin Long <[email protected]> > ---
Reviewed-by: Aaron Conole <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
