[ovs-dev] [RFC] datapath-windows: Allow validation ports processing
Nithin Raju
nithin at vmware.com
Tue Jul 19 17:10:44 UTC 2016
Alin,
Do you intend to add them to the vport hash tables? What is the use of
doing this?
Also, will it not create a collision in the hash table? When the vport
gets added from userspace, we should make sure that we don¹t use the
validation port as a valid port.
-- Nithin
-----Original Message-----
From: dev <dev-bounces at openvswitch.org> on behalf of Alin Serdean
<aserdean at cloudbasesolutions.com>
Date: Thursday, July 14, 2016 at 8:38 PM
To: "dev at openvswitch.org" <dev at openvswitch.org>
Subject: [ovs-dev] [RFC] datapath-windows: Allow validation ports
processing
>Currently we do not validate ports prior a VM being started. This patch
>allows ports to be validated during creation.
>
>Validation ports have the same lifecycle as an operational port so allow
>them to have the same treatment as an operational port.
>
>Beside the above this also allows us to add OVS ports prior the VM to
>being
>started.
>
>Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
>---
> datapath-windows/ovsext/Oid.c | 8 --------
> datapath-windows/ovsext/Vport.c | 4 ----
> 2 files changed, 12 deletions(-)
>
>diff --git a/datapath-windows/ovsext/Oid.c b/datapath-windows/ovsext/Oid.c
>index 7c7ffe7..d5ba821 100644
>--- a/datapath-windows/ovsext/Oid.c
>+++ b/datapath-windows/ovsext/Oid.c
>@@ -159,14 +159,6 @@ OvsProcessSetOidPort(POVS_SWITCH_CONTEXT
>switchObject,
> goto done;
> }
>
>- if (portParam->IsValidationPort) {
>- /* Validation ports are used internally by the Hyper-V switch
>- * to validate and verify settings. We must skip handling them,
>- * and return STATUS_SUCCESS as the OID result
>- */
>- return NDIS_STATUS_SUCCESS;
>- }
>-
> switch(setInfo->Oid) {
> case OID_SWITCH_PORT_CREATE:
> status = HvCreatePort(switchObject, portParam, 0);
>diff --git a/datapath-windows/ovsext/Vport.c
>b/datapath-windows/ovsext/Vport.c
>index 1462453..294fd7f 100644
>--- a/datapath-windows/ovsext/Vport.c
>+++ b/datapath-windows/ovsext/Vport.c
>@@ -1451,10 +1451,6 @@ OvsAddConfiguredSwitchPorts(POVS_SWITCH_CONTEXT
>switchContext)
> for (arrIndex = 0; arrIndex < portArray->NumElements; arrIndex++) {
> portParam = NDIS_SWITCH_PORT_AT_ARRAY_INDEX(portArray,
>arrIndex);
>
>- if (portParam->IsValidationPort) {
>- continue;
>- }
>-
> status = HvCreatePort(switchContext, portParam, 0);
> if (status != STATUS_SUCCESS && status !=
>STATUS_DATA_NOT_ACCEPTED) {
> break;
>--
>1.9.5.msysgit.0
>_______________________________________________
>dev mailing list
>dev at openvswitch.org
>https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailma
>n_listinfo_dev&d=CwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=pN
>HQcdr7B40b4h6Yb7FIedI1dnBsxdDuTLBYD3JqV80&m=h3ZpmSAbU9vCcm137IPWUXX5sGlUrq
>mrTf_ZBjJIXFo&s=7mvYeMxSHNmNI3UrtU57bmhSN0qnni8TYfNtl8sGEm4&e=
More information about the dev
mailing list