Thx applied to master and branch-2.3
On Tue, Dec 23, 2014 at 3:17 PM, Andy Zhou <az...@nicira.com> wrote:
> Thanks for making the change.
>
> Acked-by: Andy Zhou <az...@nicira.com>
>
> On Tue, Dec 23, 2014 at 2:21 PM, Alex Wang <al...@nicira.com> wrote:
> > Issues a ERR log when the 'recirc_id' is not freed by the
> > owning 'ofproto'.
> >
> > Signed-off-by: Alex Wang <al...@nicira.com>
> > ---
> > ofproto/ofproto-dpif.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
> > index cdd302a..b5fe73f 100644
> > --- a/ofproto/ofproto-dpif.c
> > +++ b/ofproto/ofproto-dpif.c
> > @@ -5458,6 +5458,13 @@ ofproto_dpif_free_recirc_id(struct ofproto_dpif
> *ofproto, uint32_t recirc_id)
> > cmap_remove(&backer->recirc_map, &node->cmap_node,
> node->recirc_id);
> > ovs_mutex_unlock(&backer->recirc_mutex);
> > recirc_id_free(backer->rid_pool, node->recirc_id);
> > +
> > + if (node->ofproto != ofproto) {
> > + VLOG_ERR("recirc_id %"PRIu32", freed by incorrect ofproto
> (%s),"
> > + " expect ofproto (%s)", node->recirc_id, ofproto->
> up.name,
> > + node->ofproto->up.name);
> > + }
> > +
> > /* RCU postpone the free, since other threads may be referring
> > * to 'node' at same time. */
> > ovsrcu_postpone(free, node);
> > --
> > 1.7.9.5
> >
>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
