On Fri, Nov 16, 2012 at 12:03:00AM -0800, Justin Pettit wrote: > The port_destruct() function checks if the port still exists in the > datapath to see if additional cleanup is necessary. It used > dpif_port_query_by_number(), since the datapath port number is readily > available and cheap to use as a lookup handle. Unfortunately, that > function logs a warning message if a port with that number doesn't exist > (because the datapath did remove it), which is confusing. The > dpif_port_query_by_name() function doesn't log such an warning, so use > it instead. > > Signed-off-by: Justin Pettit <jpet...@nicira.com>
Why not use dpif_port_exists() here? It seems tailor-made for the job. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
