socksp)) {
> goto error;
> }
> +#endif
>
> error = vport_add_channel(dpif, vport.port_no, socksp);
> if (error) {
> VLOG_INFO("%s: could not add channels for port %s",
>
On 10/3/18, 4:30 AM, "ovs-dev-boun...@openvswitch.org on behalf of Anand Kumar"
wrote:
'OVS_IPHELPER_INSTANCE' is linked to ovsSortedIPNeighList.
So when an Iphelper instance is deleted, also delete the ip
neighboring entries associated with that instance.
Also fix accessi
Hi Anand, one thing looks off to me here.
In the signature of OvsCreateNlMsgFromCtLimit(), numAttrs is UINT32, but the
loop’s index variable is an UINT16:
for (UINT16 i = 0; i < numAttrs; i++)
Please address this discrepancy.
--
Thanks,
Shashank
On 8/28/18, 10:37 AM, "ovs-dev-boun...@openvs
On 08/21/2018 02:57 PM, Anand Kumar wrote:
This patch implements limiting conntrack entries
per zone using dpctl commands.
Example:
ovs-appctl dpctl/ct-set-limits default=5 zone=1,limit=2 zone=1,limit=3
ovs-appctl dpct/ct-del-limits zone=4
ovs-appctl dpct/ct-get-limits zone=1,2,3
- Also upda
On 07/09/2018 06:40 AM, Alin Gabriel Serdean wrote:
Do not assign an initial value to ` status`, it will be assigned
later on.
Signed-off-by: Alin Gabriel Serdean
---
datapath-windows/ovsext/Gre.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/datapath-windows/ovsext/
On 07/09/2018 06:40 AM, Alin Gabriel Serdean wrote:
Do not assign an initial value for the variable `entry`, it will
be reassigned later on.
Signed-off-by: Alin Gabriel Serdean
---
datapath-windows/ovsext/Conntrack.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/data
;
-
-mdl = NET_BUFFER_FIRST_MDL(nb);
+PMDL mdl = NET_BUFFER_FIRST_MDL(nb);
+PUINT8 bufferStart = (PUINT8)OvsGetMdlWithLowPriority(mdl);
-bufferStart = (PUINT8)OvsGetMdlWithLowPriority(mdl);
if (!bufferStart) {
return NDIS_STATUS_RESOURCES;
}
Acked-by: Shashank
FindVportByPortNo(switchContext, srcPortNo);
if (vport == NULL || vport->ovsState != OVS_STATE_CONNECTED) {
OvsCompleteNBLForwardingCtx(&ovsFwdCtx,
L"OVS-Dropped due to port removal");
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On Tue, Jun 26, 2018 at 7:08 PM Yifeng Sun wrote:
> This patch is a simple implementation for the proposal discussed in
>
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-August/337038.html
> and
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-October/340013.html.
>
>
> It enables ovs-
On 06/17/2018 10:37 PM, Anand Kumar wrote:
Conntrack 5-tuple consists of src address, dst address, src port,
dst port and protocol which will be unique to a ct session.
Use this information along with zone to compute hash.
Also re-factor conntrack code related to parsing netlink attributes.
This patch should be combined with the patch where NAT lock is removed
from CT. Keeping this separate will cause the previous patches in this
series to break NAT functionality.
Thanks,
Shashank
On 06/17/2018 10:37 PM, Anand Kumar wrote:
The 'ovsNatTable' and 'ovsUnNatTable' tables are shared
On 06/17/2018 10:37 PM, Anand Kumar wrote:
This patch primarily gets rid of NdisRWLock in conntrack for NAT
functionality along with some conntrack optimization. The subsequent
patch will have a lock implementation inside NAT module.
- Introduce a new function OvsGetTcpHeader() to retrieve TC
On 06/17/2018 10:37 PM, Anand Kumar wrote:
This patch mainly changes a ndis RW lock for conntrack entry to a
spinlock along with some minor refactor in conntrack. Using
spinlock instead of RW lock as RW locks causes performance hits
when acquired/released multiple times.
- Use NdisInterlocked
On Thu, Jun 7, 2018, 11:52 AM Anand Kumar wrote:
> - Move conntrack lock out of NAT module
> - Use spinlock instead of read/write lock for conntrack entry.
> - Update 'ctTotalRelatedEntries' using interlocked functions
> - Refactor conntrack code to make it more readable.
>
> Testing:
> Evaluated
From: ovs-dev-boun...@openvswitch.org on
behalf of Anand Kumar
Sent: Tuesday, May 15, 2018 4:38 PM
To: d...@openvswitch.org
Subject: [ovs-dev] [PATCH] datapath-windows: Add support for handling DEI
bit of VLAN header
The Drop eligible indicator
From: ovs-dev-boun...@openvswitch.org on
behalf of Sairam Venugopal
Sent: Tuesday, May 8, 2018 10:09 AM
To: d...@openvswitch.org
Subject: Re: [ovs-dev] [PATCH] Windows: Trigger an ARP request before sending
IPFIX packet
Will send out a V2 with the cor
From: aserd...@ovn.org
Sent: Thursday, April 5, 2018 2:32 AM
To: Shashank Ram; d...@openvswitch.org
Subject: RE: [ovs-dev] [PATCH] RFC datapath-windows: Create com device only
whenextension is enabled
> -Mesaj original-
> De l
From: ovs-dev-boun...@openvswitch.org on
behalf of Alin Gabriel Serdean
Sent: Tuesday, April 3, 2018 8:08 AM
To: d...@openvswitch.org
Cc: Alin Gabriel Serdean
Subject: [ovs-dev] [PATCH] RFC datapath-windows: Create com device only when
extension i
ase.tar.gz
>
> It makes it easier for people which are building on systems that
> don't have an internet connections.
>
> Another idea would be to have `--with-pthread-tar=` and point it
> to the tarball.
>
> Which one do you prefer?
>
> Thanks,
> Alin.
>
>
>
> Signed-off-by: Alin Gabriel Serdean
> ---
>
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
--with-openssl=C:/OpenSSL-Win32 --with-vstudiotarget=\"Debug\"
> - C:\MinGW\msys\1.0\bin\bash -lc "cd /c/openvswitch && make"
> - C:\MinGW\msys\1.0\bin\bash -lc "cd /c/openvswitch && make
> datapath_windows_analyze"
> --
> 2.16.1.windows.1
>
> ___
>
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Thanks Alin!
From: aserd...@ovn.org
Sent: Thursday, March 1, 2018 3:09:19 PM
To: aserd...@ovn.org; Anand Kumar; Shashank Ram; d...@openvswitch.org
Subject: RE: [ovs-dev] [PATCH v2] datapath-windows: Support to selectively
compile targets
Applied on
s,
we will not pass the "--with-vstudiotargetver" to the
configure script.
Signed-off-by: Shashank Ram
---
Documentation/intro/install/windows.rst | 5 -
Makefile.am | 15 +++
appveyor.yml| 4 ++--
m4
Sounds good Alin, I will update the documentation, and we can work on a patch
with the solution you suggested as an alternative workflow.
Thanks,
Shashank
From: Alin Serdean
Sent: Tuesday, February 27, 2018 9:38:22 AM
To: Shashank Ram; Alin Balutoiu
From: Alin Balutoiu
Sent: Tuesday, February 20, 2018 2:58:06 AM
To: Anand Kumar; Shashank Ram; Alin Serdean; d...@openvswitch.org
Subject: RE: [ovs-dev] [PATCH] datapath-windows: Support to selectively compile
targets
I agree with having the target specified during make and not during
From: aserd...@ovn.org
Sent: Thursday, February 8, 2018 10:43 AM
To: Shashank Ram; aserd...@ovn.org; d...@openvswitch.org
Subject: RE: [ovs-dev] [PATCH] datapath-windows: Support to selectively compile
targets
Trimming the message a bit.
-Mesaj
, February 8, 2018 9:32:59 AM
To: Shashank Ram; d...@openvswitch.org
Subject: RE: [ovs-dev] [PATCH] datapath-windows: Support to selectively compile
targets
It looks a bit complicated.
What do you think about the following approach:
git diff datapath-windows/automake.mk
diff --git a/datapath
s,
we will not pass the "--with-vstudiotargetver" to the
configure script.
Signed-off-by: Shashank Ram
---
Makefile.am | 15 +++
appveyor.yml | 4 ++--
m4/openvswitch.m4 | 28 +++-
3 files changed, 44 insertions(+), 3 deletions(-
Signed-off-by: Shashank Ram
---
Makefile.am | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 5988c02..d397f65 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -148,7 +148,7 @@ ro_shell = printf '\043 Generated automatically -- do not
m
From: Ben Pfaff
Sent: Friday, February 2, 2018 2:07 PM
To: Shashank Ram
Cc: d...@openvswitch.org
Subject: Re: [ovs-dev] [PATCH] Makefile.am: Use correct path separator for
Windows
On Thu, Feb 01, 2018 at 05:03:52PM -0800, Shashank Ram wrote:
> Sig
Signed-off-by: Shashank Ram
---
Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Makefile.am b/Makefile.am
index 5988c02..1d336b6 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -398,7 +398,7 @@ CLEANFILES += flake8-check
include $(srcdir)/manpages.mk
$(srcdir
SDKs.
Signed-off-by: Shashank Ram
---
datapath-windows/Package/package.VcxProj | 28 --
datapath-windows/ovsext/ovsext.vcxproj | 50 +++-
2 files changed, 49 insertions(+), 29 deletions(-)
diff --git a/datapath-windows/Package/package.VcxProj
b
Newer compilers expect the platorm architecture to be passed.
Signed-off-by: Shashank Ram
---
Makefile.am | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index ed4b7fd..5988c02 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -409,14 +409,15
= {{{ 0xff, 0xff, 0xff, 0xff, 0xff,
0xff }}};
*An extra curly brace is required for GCC in case the struct contains
a union.
Signed-off-by: Shashank Ram
Tested-by: Yi-Hung Wei
---
include/openvswitch/types.h | 5 ++---
1 file changed, 2 insertions(+), 3
On Thu, Nov 30, 2017 at 11:45 AM, Ben Pfaff wrote:
> On Wed, Nov 29, 2017 at 05:13:14PM -0800, Sairam Venugopal wrote:
> > Found when compiling the code with C++ binaries. Most of the issues are
> > due to missing explicit cast.
> >
> > Signed-off-by: Sairam Venugopal
> > Signed-off-by: Shireesh
ggested by: Shashank Ram r...@vmware.com
---
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On Nov 6, 2017 9:07 PM, "Alin Gabriel Serdean" wrote:
From: Alin Serdean
If we can't allocate the NBL just go to the cleanup sequence.
Found using WDK 10 static code analysis.
Signed-off-by: Alin Gabriel Serdean
---
Acked-b
,
Alin.
> -Original Message-
> From: Shashank Ram [mailto:r...@vmware.com]
> Sent: Tuesday, November 28, 2017 3:33 PM
> To: d...@openvswitch.org; Alin Serdean
>
> Subject: Re: [ovs-dev] [PATCH] datapath-windows: Account for VLAN tag in
>
Alin, do you think it makes sense to also apply this on 2.8?
--
Thanks,
Shashank
On 11/28/17, 11:47 AM, "ovs-dev-boun...@openvswitch.org on behalf of Shashank
Ram" wrote:
Alin, can we apply this patch? It’s been sitting around in the mailing list
for a week without any review
ot;ovs-dev-boun...@openvswitch.org on behalf of
Shashank Ram"
wrote:
Decap functions for tunneling protocols do not compute
the packet header offsets correctly when there is a VLAN
tag in the L2 header. This results in incorrect checksum
computation caus
tunneling protocols. Previously,
this would result in packet drops due to invalid checksums
being computed.
- Verified that non-VLAN tagged packets are handled correctly.
Signed-off-by: Shashank Ram
---
datapath-windows/ovsext/Geneve.c | 14 +
datapath-windows/ovsext/Geneve.h | 6
On Mon, Nov 6, 2017 at 7:33 AM, Alin Gabriel Serdean
wrote:
> From: Alin Serdean
>
> This fixes the static code analysis over the function
> 'OvsGetTcpPayloadLength'.
>
> Signed-off-by: Alin Gabriel Serdean
> ---
> v2: No chan
r + MPLS_HLEN, sizeof(*ethHdr));
> ethHdr->Type = mpls->mpls_ethertype;
>
> --
> 2.10.2.windows.1
>
> ___
> Acked-by: Shashank Ram
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
From: Alin Serdean
>
> The result of `RtlStringCbLengthW` is not currently checked and triggers
> a warning using the WDK 8.1 static analysis.
>
> This patch treats the result of `RtlStringCbLengthW`.
>
> Signed-off-by: Alin Gabriel Serdean
> ---
> v2: s/Fail/Failed. S
From: Alin Serdean
>
> Found by inspection.
>
> Signed-off-by: Alin Gabriel Serdean
> ---
> v2: s/Fail/Failed. Suggested by: Shashank Ram r...@vmware.com
> ---
>
Acked-by: Shashank Ram
___
dev mailing list
d.
lysis.
>
> Signed-off-by: Alin Gabriel Serdean
> ---
> v2: Change OvsGetMdlWithLowPrio to OvsGetMdlWithLowPriority.
> Suggested by: Shashank Ram
> ---
>
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Update the OvsCompleteNbl to take in a PVOID and explicitly cast to
POVS_SWITCH_CONTEXT. This is useful when finding declarations in Visual
Studio. The mismatch breaks this functionality.
Found by inspection.
Signed-off-by: Sairam Venugopal
---
Acked-by: Shashank Ram
On 10/17/17, 5:31 PM, "ovs-dev-boun...@openvswitch.org on behalf of Anand
Kumar"
wrote:
Set the geneve flags OVS_TNL_F_GENEVE_OPT and OVS_TNL_F_CRT_OPT
in OvsDecapGeneve, so that windows behavior is similiar to linux
https://github.com/openvswitch/ovs/blob/master/datapath/linux/c
during
driver unload.
Testint done:
Verified that cleaner threads run as expected without
crashing during driver unload.
Signed-off-by: Shashank Ram
---
datapath-windows/ovsext/Conntrack-related.c | 6 +-
datapath-windows/ovsext/Conntrack.c | 6 +-
datapath-windows/ovsext
;0'.
Signed-off-by: Alin Gabriel Serdean
---
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
does not
make sense to style the CT code differently than the rest of the Windows data
path code.
---
Acked-by: Shashank Ram
From: ovs-dev-boun...@openvswitch.org on
behalf of Shashank Ram
Sent: Tuesday, September 5, 2017 7:47:21 PM
To: Anand Kumar
Comments inline.
Subject: [ovs-dev] [PATCH] datapath-windows: Increment ct packet counters
based on ct_state.
For a given packet, packet counters in conntrack should be accounted only
once, even if the packet is processed multiple times by conntra
---
Subject: [ovs-dev] [PATCH v2] datapath-windows: Add an upper limit to
conntrack entries
The current implementation lacked an upper bound of number of entries in
the system. Set the size to ~2M (2^21) for the time being.
Signed-off-by: Sairam Venugopal
---
Acked-by: Shashank Ram
Subject: [ovs-dev] [PATCH] datapath-windows: Remove unnecessary call to
OvsInitForwardingCtx
Only curNbl in the forwarding context needs to be updated with
fragmented NBL.
Signed-off-by: Anand Kumar
___
Acked-by: Shashank Ram
/Conntrack.c | 10 ++
datapath-windows/ovsext/Conntrack.h | 1 +
2 files changed, 11 insertions(+)
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman
in
the system. Set the size to ~2M (2^21) for the time being.
Signed-off-by: Sairam Venugopal
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman
___
>From: ovs-dev-boun...@openvswitch.org on
>behalf of Sairam Venugopal
>Sent: Monday, August 28, 2017 4:56 PM
>To: d...@openvswitch.org
>Subject: [ovs-dev] [PATCH] datapath-windows: Add an upper limit to conntrack
> entries
>
>The current implementati
From: ovs-dev-boun...@openvswitch.org on
behalf of Sairam Venugopal
Sent: Monday, August 28, 2017 4:56 PM
To: d...@openvswitch.org
Subject: [ovs-dev] [PATCH] datapath-windows: Add an upper limit to conntrack
entries
The current implementation lack
> From: ovs-dev-boun...@openvswitch.org on
> behalf of Ben Pfaff
> Sent: Wednesday, August 2, 2017 11:35 AM
> To: Alin Serdean
> Cc: d...@openvswitch.org
> Subject: Re: [ovs-dev] [PATCH 00/40] Fix static code analysis warnings.
>
> On Fri, Jul 14, 2017
From: ovs-dev-boun...@openvswitch.org on
behalf of Ben Pfaff
Sent: Wednesday, August 2, 2017 11:35 AM
To: Alin Serdean
Cc: d...@openvswitch.org
Subject: Re: [ovs-dev] [PATCH 00/40] Fix static code analysis warnings.
On Fri, Jul 14, 2017 at 04:40:51AM +
From: ovs-dev-boun...@openvswitch.org on
behalf of Ben Pfaff
Sent: Wednesday, August 2, 2017 11:35 AM
To: Alin Serdean
Cc: d...@openvswitch.org
Subject: Re: [ovs-dev] [PATCH 00/40] Fix static code analysis warnings.
On Fri, Jul 14, 2017 at 04:40:51AM +
Moves function OvsCreateNewNBLsFromMultipleNBs() to BufferMgmt.c
to facilitate consumption from outside PacketIO.c.
Signed-off-by: Shashank Ram
---
datapath-windows/ovsext/BufferMgmt.c | 47
datapath-windows/ovsext/BufferMgmt.h | 4 +++
datapath-windows
icmp_id field during
SNAT/DNAT only for ICMP traffic
The current NAT module doesn't take the ICMP type/code into account
during the lookups. Fix this to make it similar with the other conntrack
module.
Signed-off-by: Anand Kumar
---
___
Acked-by: Sha
for the ct_endpoint's union, so that each member
of ct_endpoint's union are of same size.
Co-authored-by: Sairam Venugopal
Signed-off-by: Anand Kumar
---
___
Acked-by: Shashank Ram
___
dev mail
From: ovs-dev-boun...@openvswitch.org on
behalf of Anand Kumar
Sent: Tuesday, August 15, 2017 3:23 PM
To: d...@openvswitch.org
Subject: [ovs-dev] [PATCH v2] datapath-windows: Update Orig Tuple to use
ICMP Type and Code
- Also add in some padding
icmp_id field during
SNAT/DNAT only for ICMP traffic
The current NAT module doesn't take the ICMP type/id/code into account
during the lookups. Fix this to make it similar with the other conntrack
module.
Signed-off-by: Anand Kumar
___
Acked-by: Sha
From: ovs-dev-boun...@openvswitch.org on
behalf of Lance Richardson
Sent: Tuesday, August 8, 2017 11:59 AM
To: Ben Pfaff
Cc: d...@openvswitch.org
Subject: Re: [ovs-dev] DNS support options
> From: "Ben Pfaff"
> To: d...@openvswitch.org
> Sent: Tuesd
Guru, could you please apply this patch.
Thanks,
Shashank
From: Anand Kumar
Sent: Thursday, July 27, 2017 4:48:02 PM
To: Shashank Ram; d...@openvswitch.org
Subject: Re: [ovs-dev] [PATCH v2] datapath-windows: Refactor
OvsCreateNewNBLsFromMultipleNBs
y->key)) {
+if (OvsCtKeyAreSame(revCtxKey, entry->key)) {
found = entry;
reply = TRUE;
break;
--
2.9.3.windows.1
_______
Acked-by: Shashank Ram
__
, and it returns
a linked list of NBLs and sets the HEAD and TAIL of the
new list obtained from the curNbl. If the caller wants
to chain a new linked list at the HEAD or TAIL, it
can make use of the curNbl and lastNbl to do so.
Signed-off-by: Shashank Ram
---
datapath-windows/ovsext/PacketIO.c | 20
I will hold onto this patch a bit longer. Noticed that there are other bugs
that get exposed as a result of this bug fix. Will send out a v2.
Thanks,
Shashank
From: Shashank Ram
Sent: Friday, July 21, 2017 11:57:44 AM
To: d...@openvswitch.org
Cc
While creating a new NBL chain with 1 NB per NBL,
from an NBL that contains multiple NBs, the new NBL
chain was being broken, as a result leaking packets.
Signed-off-by: Shashank Ram
---
datapath-windows/ovsext/PacketIO.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
Alin, thanks a lot for this important series! Could you please test out all
these changes with Driver verifier and ensure that it does not introduce new
regressions such as memory leaks, deadlocks etc.
Thanks,
Shashank
From: ovs-dev-boun...@openvswitch.o
OvsIpFragmentEntryDelete(POVS_IPFRAG_ENTRY entry, BOOLEAN
checkExpiry);
/* Global and static variables */
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https
windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Comments inline.
Thanks,
Shashank
From: ovs-dev-boun...@openvswitch.org on
behalf of Alin Serdean
Sent: Thursday, July 13, 2017 9:40 PM
To: d...@openvswitch.org
Subject: [ovs-dev] [PATCH 40/40] datapath-windows: Fix static analysis warnings
around ovsI
Neigh(internalRow,
+status = OvsGetOrResolveIPNeigh(&instance->internalRow,
ipAddr, &ipNeigh);
OvsUpdateIPNeighEntry(ipAddr, &ipNeigh, status);
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
_
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
{
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
text == NULL || !NT_SUCCESS(status)) {
goto error;
}
NdisAcquireRWLockRead(gOvsSwitchContext->dispatchLock, &lockState, 0);
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
gt;> s/Fail/Failed
+return status;
+}
portFriendlyName->Length = (USHORT)len;
} else {
OVS_LOG_ERROR("Fail to convert interface LUID to alias, status:
%x",
--
2.10.2.windows.1
_____
il to register UNICAST IP change, status: %x.",
>> s/Fail/Failed
+ status);
}
register_cleanup:
if (status != STATUS_SUCCESS) {
--
2.10.2.windows.1
_______
Acked-by: Shashank Ram
___
ummy,
TRUE, &ipRouteNotificationHandle);
if (status != STATUS_SUCCESS) {
OVS_LOG_ERROR("Fail to regiter ip route change, status: %x.",
--
2.10.2.windows.1
___
Acke
gnedMemory(size_t size, UINT16 align);
+
VOID *OvsAllocateMemoryPerCpu(size_t size, size_t count, ULONG tag);
VOID OvsFreeMemory(VOID *ptr);
VOID OvsFreeMemoryWithTag(VOID *ptr, ULONG tag);
--
2.10.2.windows.1
_______
Acked-by: Shashank Ram
);
+DRIVER_CANCEL OvsTunnelFilterCancelIrp;
/*
* Callout driver global variables
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org
];
JhashFinal(&a, &b, &c);
}
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Comments inline.
Thanks,
Shashank
From: ovs-dev-boun...@openvswitch.org on
behalf of Alin Serdean
Sent: Thursday, July 13, 2017 9:40 PM
To: d...@openvswitch.org
Subject: [ovs-dev] [PATCH 01/40] datapath-windows: Use only non executable
memory
Use
) == NULL) {
return OvsCopySinglePacketNBL(context, nbl, nb, headRoom, copyNblInfo);
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org
(eventQueueLockArr[eventId])
static __inline VOID
OvsAcquireEventQueueLock(int eventId)
{
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman
dows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
);
OvsInitNBLContext(dstCtx, flags, NET_BUFFER_DATA_LENGTH(srcNb) - copySize,
OVS_DPPORT_NUMBER_INVALID);
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d
(DISPATCH_LEVEL)
+_IRQL_saves_global_(OldIrql, gOvsSwitchContext->pidHashLock)
+_Acquires_lock_(gOvsSwitchContext->pidHashLock)
static __inline VOID
OvsAcquirePidHashLock()
{
--
2.10.2.windows.1
___
Acked-by: Shasha
;
NTSTATUS
OvsInitEventQueue()
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
(DISPATCH_LEVEL)
+_IRQL_saves_global_(OldIrql, lockState)
+_Acquires_lock_(datapath->lock)
static __inline VOID
OvsAcquireDatapathWrite(OVS_DATAPATH *datapath,
LOCK_STATE_EX *lockState,
--
2.10.2.windows.1
___
Acked-by: Shashank
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
ws.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
);
-VOID OvsReleaseDatapath(OVS_DATAPATH *datapath,
-LOCK_STATE_EX *lockState);
-
-
NTSTATUS
OvsTunnelNotify(FWPS_CALLOUT_NOTIFY_TYPE notifyType,
const GUID *filterKey,
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
OvsAcquirePidHashLock()
--
2.10.2.windows.1
___
Acked-by: Shashank Ram
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
1 - 100 of 156 matches
Mail list logo
