Acked-by: Eitan Eliahu <elia...@vmware.com>

-----Original Message-----
From: Samuel Ghinet [mailto:sghi...@cloudbasesolutions.com] 
Sent: Tuesday, September 30, 2014 7:47 AM
To: dev@openvswitch.org
Cc: Alin Serdean; Nithin Raju; Eitan Eliahu; Ankur Sharma
Subject: [PATCH 04/14] datapath-windows: Update OVS_SWITCH_CONTEXT: external 
and internal port

The fields externalVport and internalVport of the OVS_SWITCH_CONTEXT struct are 
currently defined as PVOID. However, all over the code they are used as 
POVS_VPORT_ENTRY. In order to improve clarity and reduce the need for useless 
casts to POVS_VPORT_ENTRY, this patch changes the type from PVOID to 
POVS_VPORT_ENTRY.

This patch does not cleanup the code that already uses casts to 
POVS_VPORT_ENTRY. This cleanup can be done later on as well.

Signed-off-by: Samuel Ghinet <sghi...@cloudbasesolutions.com>
---
 datapath-windows/ovsext/Switch.h | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/datapath-windows/ovsext/Switch.h b/datapath-windows/ovsext/Switch.h
index 5199268..f7acd87 100644
--- a/datapath-windows/ovsext/Switch.h
+++ b/datapath-windows/ovsext/Switch.h
@@ -53,6 +53,8 @@
 
 #define OVS_HASH_BASIS   0x13578642
 
+typedef struct _OVS_VPORT_ENTRY *POVS_VPORT_ENTRY;
+
 typedef struct _OVS_DATAPATH
 {
    PLIST_ENTRY             flowTable;       // Contains OvsFlows.
@@ -104,8 +106,8 @@ typedef struct _OVS_SWITCH_CONTEXT
 
     NDIS_SWITCH_PORT_ID     externalPortId;
     NDIS_SWITCH_PORT_ID     internalPortId;
-    PVOID                   externalVport;  // the virtual adapter vport
-    PVOID                   internalVport;
+    POVS_VPORT_ENTRY        externalVport;  // the virtual adapter vport
+    POVS_VPORT_ENTRY        internalVport;
 
     PVOID                  *vportArray;
     PLIST_ENTRY             nameHashArray;  // based on ovsName
--
1.8.3.msysgit.0

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to