On Fri, Mar 26, 2021 at 1:12 AM Mark Gray <[email protected]> wrote: > > On 25/03/2021 14:56, Dumitru Ceara wrote: > > Spotted during code inspection. > > > > Signed-off-by: Dumitru Ceara <[email protected]> > > --- > > controller/lflow.c | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/controller/lflow.c b/controller/lflow.c > > index a3d84aff4..680b8cca1 100644 > > --- a/controller/lflow.c > > +++ b/controller/lflow.c > > @@ -716,15 +716,15 @@ consider_logical_flow__(const struct sbrec_logical_flow *lflow, > > struct lflow_ctx_in *l_ctx_in, > > struct lflow_ctx_out *l_ctx_out) > > { > > - /* Determine translation of logical table IDs to physical table IDs. */ > > - bool ingress = !strcmp(lflow->pipeline, "ingress"); > > - > > if (!get_local_datapath(l_ctx_in->local_datapaths, dp->tunnel_key)) { > > VLOG_DBG("lflow "UUID_FMT" is not for local datapath, skip", > > UUID_ARGS(&lflow->header_.uuid)); > > return true; > > } > > > > + /* Determine translation of logical table IDs to physical table IDs. */ > > + bool ingress = !strcmp(lflow->pipeline, "ingress"); > > + > > /* Determine translation of logical table IDs to physical table IDs. */ > > uint8_t first_ptable = (ingress > > ? OFTABLE_LOG_INGRESS_PIPELINE > > > Acked-by: Mark Gray <[email protected]> >
Thanks Dumitru and Mark. I applied this to master. Han _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
