On Tue, Oct 01, 2013 at 01:36:11PM -0700, Alex Wang wrote:
> This commit moves the main logic of send_packet() function into
> the ofproto-dpif-xlate module.  Also, modification is made to
> guarantee the thread safety of ofproto-dpif-xlate module.
> 
> Signed-off-by: Alex Wang <al...@nicira.com>

xlate_send_packet() locks xlate_rwlock before it looks up the xport,
but at the end it unlocks xlate_rwlock before it dereferences
xport->xbridge->dpif.  Is the latter dereference safe, that is, does
anything guarantee that 'xport' and 'xbridge' aren't destroyed?

In ofproto/ofproto-dpif-xlate.h, we usually put a blank line before
the #endif.

Thanks,

Ben.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to