Re: [ovs-dev] [PATCH v2] dpif: Return ENODEV from dpif_port_query_by_name() if there's no port.

2017-01-06 Thread Ben Pfaff
On Fri, Jan 06, 2017 at 08:43:03PM +, Daniele Di Proietto wrote: > > > > > > On 06/01/2017 11:34, "Ben Pfaff" wrote: > > >On Fri, Jan 06, 2017 at 10:59:07AM -0800, Daniele Di Proietto wrote: > >> bridge_delete_or_reconfigure() deletes every interface that's not dumped > >>

Re: [ovs-dev] [PATCH v2] dpif: Return ENODEV from dpif_port_query_by_name() if there's no port.

2017-01-06 Thread Daniele Di Proietto
On 06/01/2017 11:34, "Ben Pfaff" wrote: >On Fri, Jan 06, 2017 at 10:59:07AM -0800, Daniele Di Proietto wrote: >> bridge_delete_or_reconfigure() deletes every interface that's not dumped >> by OFPROTO_PORT_FOR_EACH(). ofproto_dpif.c:port_dump_next(), used by >>

Re: [ovs-dev] [PATCH v2] dpif: Return ENODEV from dpif_port_query_by_name() if there's no port.

2017-01-06 Thread Ben Pfaff
On Fri, Jan 06, 2017 at 10:59:07AM -0800, Daniele Di Proietto wrote: > bridge_delete_or_reconfigure() deletes every interface that's not dumped > by OFPROTO_PORT_FOR_EACH(). ofproto_dpif.c:port_dump_next(), used by > OFPROTO_PORT_FOR_EACH, checks if the ofport is in the datapath by > calling

[ovs-dev] [PATCH v2] dpif: Return ENODEV from dpif_port_query_by_name() if there's no port.

2017-01-06 Thread Daniele Di Proietto
bridge_delete_or_reconfigure() deletes every interface that's not dumped by OFPROTO_PORT_FOR_EACH(). ofproto_dpif.c:port_dump_next(), used by OFPROTO_PORT_FOR_EACH, checks if the ofport is in the datapath by calling port_query_by_name(). If port_query_by_name() returns an error, the dump is