Thanks Alin. > On Jun 18, 2015, at 2:00 PM, Alin Serdean <[email protected]> > wrote: > > Delete internal port > Test connectivity > Add internal port > Test connectivity > > Delete external port > Test connectivity > Add external port > Test connectivity > > Delete internal port > Delete external port > Test connectivity > Add internal port > Test connectivity > Add external port > Test connectivity > > Uninstall extension verify port deletion. > > Alin. > > -----Mesaj original----- > De la: dev [mailto:[email protected]] În numele Nithin Raju > Trimis: Thursday, June 18, 2015 11:30 PM > Către: Sorin Vinturis > Cc: [email protected] > Subiect: Re: [ovs-dev] [PATCH] datapath-windows: Remove the external/internal > port only if it is removed on the Hyper-V switch > >> On Jun 18, 2015, at 11:52 AM, Sorin Vinturis >> <[email protected]> wrote: >> >> Disconnecting the NIC from HV switch and then unloading the driver >> hits an ASSERT (ASSERT(switchContext->numPhysicalNics);). >> >> Also if the external or internal port is deleted from the userspace, >> the latter ports would not be re-added until the extension would be reloaded. >> >> This patch ensures that the internal or external ports are being >> deleted only by the Hyper-V switch mechanisms, which also solves the >> issue raised by the above assert. >> >> Signed-off-by: Sorin Vinturis <[email protected]> >> Co-authored-by: Alin Gabriel Serdean <[email protected]> >> Reported-by: Sorin Vinturis <[email protected]> > > Just curious, was any unit testing performed to test the specific scenario we > are claiming to fix? > > thanks, > -- Nithin > _______________________________________________ > dev mailing list > [email protected] > https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailman_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=pNHQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=q1cb-T2pctLQnNJ2OwKlv_RsKbtpJ8FPjBJ-W5p_9vU&s=nWK_z-UHuGNF1B0MBF0RTfbwqrzr85wc920kbDvQWCU&e= >
_______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
