diff mbox

[ovs-dev] datapath-windows: post event during vport delete

Message ID 1460586791-6089-1-git-send-email-nithin@vmware.com
State Accepted
Headers show

Commit Message

Nithin Raju April 13, 2016, 10:33 p.m. UTC 
Got left out during the previous round of refactoring.

Signed-off-by: Nithin Raju <nithin@vmware.com>
---
 datapath-windows/ovsext/Vport.c | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Sairam Venugopal April 13, 2016, 10:51 p.m. UTC | #1 
Acked-by: Sairam Venugopal <vsairam@vmware.com>


On 4/13/16, 3:33 PM, "Nithin Raju" <nithin@vmware.com> wrote:

>Got left out during the previous round of refactoring.
>
>Signed-off-by: Nithin Raju <nithin@vmware.com>
>---
> datapath-windows/ovsext/Vport.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
>diff --git a/datapath-windows/ovsext/Vport.c
>b/datapath-windows/ovsext/Vport.c
>index 4da2b42..882b41f 100644
>--- a/datapath-windows/ovsext/Vport.c
>+++ b/datapath-windows/ovsext/Vport.c
>@@ -291,7 +291,15 @@ HvDeletePort(POVS_SWITCH_CONTEXT switchContext,
>      * delete will delete the vport.
>     */
>     if (vport) {
>+        OVS_EVENT_ENTRY event;
>+
>+        event.portNo = vport->portNo;
>+        event.ovsType = vport->ovsType;
>+        event.upcallPid = vport->upcallPid;
>+        RtlCopyMemory(&event.ovsName, &vport->ovsName, sizeof
>event.ovsName);
>+        event.type = OVS_EVENT_LINK_DOWN;
>         OvsRemoveAndDeleteVport(NULL, switchContext, vport, TRUE, FALSE);
>+        OvsPostEvent(&event);
>     } else {
>         OVS_LOG_WARN("Vport not present.");
>     }
>-- 
>2.6.2
>
>_______________________________________________
>dev mailing list
>dev@openvswitch.org
>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailma
>n_listinfo_dev&d=BQIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=Dc
>ruz40PROJ40ROzSpxyQSLw6fcrOWpJgEcEmNR3JEQ&m=hmZ8WN48otl6pam9IA3tTuWkGk5ENf
>VELpG58nYGo-c&s=HBowTCMiyT-6rBFhIW-mwJjgN61G0j2XVY5Br99doGU&e=
Ben Pfaff April 14, 2016, 8:57 p.m. UTC | #2 
On Wed, Apr 13, 2016 at 03:33:11PM -0700, Nithin Raju wrote:
> Got left out during the previous round of refactoring.
> 
> Signed-off-by: Nithin Raju <nithin@vmware.com>

Applied, thanks!
diff mbox

Patch

diff --git a/datapath-windows/ovsext/Vport.c b/datapath-windows/ovsext/Vport.c
index 4da2b42..882b41f 100644
--- a/datapath-windows/ovsext/Vport.c
+++ b/datapath-windows/ovsext/Vport.c
@@ -291,7 +291,15 @@  HvDeletePort(POVS_SWITCH_CONTEXT switchContext,
      * delete will delete the vport.
     */
     if (vport) {
+        OVS_EVENT_ENTRY event;
+
+        event.portNo = vport->portNo;
+        event.ovsType = vport->ovsType;
+        event.upcallPid = vport->upcallPid;
+        RtlCopyMemory(&event.ovsName, &vport->ovsName, sizeof event.ovsName);
+        event.type = OVS_EVENT_LINK_DOWN;
         OvsRemoveAndDeleteVport(NULL, switchContext, vport, TRUE, FALSE);
+        OvsPostEvent(&event);
     } else {
         OVS_LOG_WARN("Vport not present.");
     }