> On Feb 8, 2017, at 3:06 PM, Joe Stringer <j...@ovn.org> wrote: > > On 8 February 2017 at 11:32, Jarno Rajahalme <ja...@ovn.org> wrote: >> Refactoring conntrack labels initialization makes chenges in later > > *changes > >> patches easier to review. >> >> Signed-off-by: Jarno Rajahalme <ja...@ovn.org> > > Minor other nit: > >> >> cl = nf_ct_labels_find(ct); >> if (!cl) { >> nf_ct_labels_ext_add(ct); >> cl = nf_ct_labels_find(ct); >> } >> + >> + return cl; >> +} >> + >> +/* Initialize labels for a new, to be committed conntrack entry. Note that > > Maybe insert 'yet': > > Initialize labels for a new, yet to be committed conntrack entry. Note that
Fixed these typos for v4. Jarno