On Wed, Nov 13, 2013 at 01:25:22PM +0900, Simon Horman wrote:
> Add table config to to struct ofproto and set it
> when a table mod message is received.
> 
> This is in preparation for changing the behaviour of the switch
> based on table config.
> 
> Cc: Andy Zhou <[email protected]>
> Signed-off-by: Simon Horman <[email protected]>

The 'config' member seems like a perfect place to use an atomic.  A
rwlock is overkill.

I don't see anything in handle_table_mod() that validates that the
table_id is in the supported range.

Thanks,

Ben.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to