[ovs-dev] [PATCH] ovn: Change type of 'chassis' column in 'Binding' table.

Ben Pfaff blp at nicira.com
Thu May 21 14:40:42 UTC 2015


On Tue, May 19, 2015 at 02:29:45PM -0700, Alex Wang wrote:
> This commit changes the type of 'chassis' column in 'Binding' table
> from string to weak reference of 'Chassis' table entry.  This will
> make accessing the chassis from binding more efficient.
> 
> Signed-off-by: Alex Wang <alexw at nicira.com>

This code seems more straightforward, thanks.

Now we have four pieces of code that search for a chassis record for the
current chassis: register_chassis(), chassis_destroy(), binding_run(),
binding_destroy().  I think we could write a helper function.  I also
don't think that it's a good idea to assert-fail if the chassis is
missing.

Acked-by: Ben Pfaff <blp at nicira.com>



More information about the dev mailing list