From 421e24220fc9493683f75fafc7e2ced8ba60bf30 Mon Sep 17 00:00:00 2001 From: Nithin Raju Date: Fri, 26 Jun 2015 11:51:29 -0700 Subject: datapath-windows: Rename 'vport->isPresentOnHv' to 'isAbsentOnHv' Looking at the code, the flag 'vport->isPresentOnHv' is actually indicating if the vport is present on the Hyper-V switch or not, but the logic seems to be inverse. 'isPresentOnHv == TRUE' indicates that the vport is not present on the Hyper-V switch. Eg. VXLAN port, would have isPresentOnHv == TRUE. In this patch, we rename the variable to reflect its meaning. vport->isAbsentOnHv is TRUE iff: - vport is bridge internal port - vport is tunnel port - vport was added from Hyper-V and also from OVS, but got deleted from Hyper-V Signed-off-by: Nithin Raju Acked-by: Alin Gabriel Serdean Signed-off-by: Ben Pfaff --- datapath-windows/ovsext/Vport.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'datapath-windows/ovsext/Vport.h') diff --git a/datapath-windows/ovsext/Vport.h b/datapath-windows/ovsext/Vport.h index 3ea3d0303..4c6e858b1 100644 --- a/datapath-windows/ovsext/Vport.h +++ b/datapath-windows/ovsext/Vport.h @@ -132,7 +132,7 @@ typedef struct _OVS_VPORT_ENTRY { BOOLEAN isExternal; UINT32 upcallPid; /* netlink upcall port id */ PNL_ATTR portOptions; - BOOLEAN isPresentOnHv; /* Is this port present on the + BOOLEAN isAbsentOnHv; /* Is this port present on the Hyper-V switch? */ } OVS_VPORT_ENTRY, *POVS_VPORT_ENTRY; -- cgit v1.2.1