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