Re: [PATCH] openvswitch: Allow attaching helper in later commit

2019-10-03 Thread Yi-Hung Wei
On Thu, Oct 3, 2019 at 8:31 AM David Miller wrote: > > From: Yi-Hung Wei > Date: Mon, 30 Sep 2019 12:39:04 -0700 > > > - if ((nf_ct_is_confirmed(ct) ? !cached : info->commit) && > > + if ((nf_ct_is_confirmed(ct) ? !cached | add_helper : > > I would suggest using "||" inste

Re: [PATCH] openvswitch: Allow attaching helper in later commit

2019-10-03 Thread David Miller
From: Yi-Hung Wei Date: Mon, 30 Sep 2019 12:39:04 -0700 > - if ((nf_ct_is_confirmed(ct) ? !cached : info->commit) && > + if ((nf_ct_is_confirmed(ct) ? !cached | add_helper : I would suggest using "||" instea of "|" here since you are computing a boolean.

[PATCH] openvswitch: Allow attaching helper in later commit

2019-09-30 Thread Yi-Hung Wei
This patch allows to attach conntrack helper to a confirmed conntrack entry. Currently, we can only attach alg helper to a conntrack entry when it is in the unconfirmed state. This patch enables an use case that we can firstly commit a conntrack entry after it passed some initial conditions. Aft