On Mon, Jul 18, 2016 at 4:22 PM, Ryan Moats <rmo...@us.ibm.com> wrote:
> Russell Bryant <russ...@ovn.org> wrote on 07/18/2016 03:01:41 PM:
>
> > From: Russell Bryant <russ...@ovn.org>
> > To: dev@openvswitch.org
> > Cc: Ryan Moats/Omaha/IBM@IBMUS, Russell Bryant <russ...@ovn.org>
> > Date: 07/18/2016 03:01 PM
> > Subject: [PATCH] ovn-controller: Remove local_datapaths_by_uuid.
> >
> > binding.c included a static local_datapaths_by_uuid but it was not used
> > for anything. In fact, the hash node used when inserting into this hmap
> > is overwritten in another code path for a different hmap.
> >
> > Fixes: 263064aeaa31 ("Convert binding_run to incremental processing.")
> > Signed-off-by: Russell Bryant <russ...@ovn.org>
> > ---
>
> Well, that's odd... I thought I had removed everything that was orphans.
>
> Anyway, it's going to cause me to rebase the remaining incremental
> processing patches, but so be it....
>
> Acked-by: Ryan Moats <rmo...@us.ibm.com>
>
>
> Thanks, Ryan. I applied this to master.
I have one remaining cleanup patch for binding.c coming in a few minutes.
--
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
