Acked-by: Sairam Venugopal
On 1/26/17, 6:30 PM, "ovs-dev-boun...@openvswitch.org on behalf of Alin
Serdean" wrote:
>Multiple VTEP and multiple physical NICs are supported on Hyper-V now.
>Update the documentation
>
>Signed-off-by: Alin Gabriel Serdean
>---
>Intended for master and branch-
wcscat_s and wcscpy_s requires number of elements as argument. wchar_t
uses 2 bytes for storage and using sizeof(internal_port_query) causes
access violation error on Windows 2012 R2 (64 bit). This patch introduces
a #define WMI_QUERY_COUNT set to 2048 and uses that instead.
Signed-off-by: Sairam
Thanks for the patch.
Acked-by: Sairam Venugopal
On 12/15/16, 6:50 PM, "ovs-dev-boun...@openvswitch.org on behalf of Alin
Serdean" wrote:
>Unfortunately, WSAPoll misbehaves on Windows please view detailed behavior
>on:
>https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_o
Yes, my initial thought was to define 2048 and re-use it. I added in the define
in case we use something other than 2048 in the future.
I am okay with either one of the approaches. I can send out a new patch with -
#define WMI_QUERY_ARRAY_SIZE set to 2048.
We can do an incremental for checking
Close the 'input_file' after data is read.
Signed-off-by: Alin Gabriel Serdean
---
tests/test-rstp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/test-rstp.c b/tests/test-rstp.c
index 2ee8c7e..7bcff83 100644
--- a/tests/test-rstp.c
+++ b/tests/test-rstp.c
@@ -691,6 +691,7 @@ test_rs
'client_id' is of type unsigned int. Change the printf specifier to match it.
Signed-off-by: Alin Gabriel Serdean
---
tests/dpdk/ring_client.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/dpdk/ring_client.c b/tests/dpdk/ring_client.c
index 8a43c64..b153713 100644
---
Found by inspection.
Signed-off-by: Alin Gabriel Serdean
---
ovn/utilities/ovn-trace.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ovn/utilities/ovn-trace.c b/ovn/utilities/ovn-trace.c
index 307556b..2e2859a 100644
--- a/ovn/utilities/ovn-trace.c
+++ b/ovn/utilities/ovn-t
> -Original Message-
> From: Sairam Venugopal [mailto:vsai...@vmware.com]
> Sent: Friday, January 27, 2017 8:01 AM
> To: Alin Serdean ;
> d...@openvswitch.org
> Subject: Re: [ovs-dev] [PATCH] windows: wmi add include
>
> Hi Alin,
>
> I investigated the issue and have sent out a patch for
Thanks for catching this!
It slipped my mind when I wrote the code.
Mind adding a define for the number of element instead and passing that in?
i.e.
#define WMI_QUERY 2048
wchar_t internal_port_query[WMI_QUERY] = L"SELECT * from "
L"Msvm_EthernetPortAllocationSettingData WHERE Elemen
Acked-by: Sairam Venugopal
On 1/26/17, 2:15 PM, "ovs-dev-boun...@openvswitch.org on behalf of Alin
Serdean" wrote:
>Add 'util.h' to includes otherwise the definition of the function
>ovs_format_message will be unknown and an abort with the following stacktrace
>will be triggered:
>*1
Hi Alin,
I investigated the issue and have sent out a patch for this -
https://patchwork.ozlabs.org/patch/720476/
This is the associated stack trace:
https://github.com/openvswitch/ovs-issues/issues/121
I will ack this change.
Thanks,
Sairam
On 1/26/17, 4:47 PM, "Alin Serdean" wrote:
>
wcscat_s and wcscpy_s requires number of elements as argument. wchar_t
uses 2 bytes for storage and using sizeof(internal_port_query) causes
access violation error on Windows 2012 R2 (64 bit). This patch introduces
a count_of() function to return the count of the wchar_t array.
Signed-off-by: Sair
On 26/01/2017 12:35, "Ansis Atteka" wrote:
>
>
>On 26 January 2017 at 21:24, Aaron Conole
> wrote:
>
>Daniele Di Proietto writes:
>
>> On 25/01/2017 00:01, "Ansis Atteka" wrote:
>>
>>>On Jan 25, 2017 4:22 AM, "Daniele Di Proietto"
>>>wrote:
>>>
>>>Current SELinux policy in RHEL and Fedo
It feels weird reviewing my own code from past. What if I make a pass of fixing
the problems I see and send a new version of the remaining patches for review?
Jarno
> On Jan 24, 2017, at 10:45 PM, Andy Zhou wrote:
>
> From: Jarno Rajahalme
>
> Add DPIF-level infrastructure for meters. All
Multiple VTEP and multiple physical NICs are supported on Hyper-V now.
Update the documentation
Signed-off-by: Alin Gabriel Serdean
---
Intended for master and branch-2.7
---
Documentation/faq/releases.rst | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/Documentation/
We shouldn't if you look at the userspace code it is set as BE:
https://github.com/openvswitch/ovs/blob/bef503e8c094456ca82fefd044a8c3341d9d21ac/lib/odp-util.c#L2038
I also checked/rechecked with the debugger. Here is a snippet of debugging with
the patchset applied:
OvsDoEncapVxlan
2: kd> ??tun
On Wed, Jan 25, 2017 at 4:50 PM, Jarno Rajahalme wrote:
> Trusting you address the comments below:
>
> Acked-by: Jarno Rajahalme
>
Thanks for the review. Pushed with fixes.
>
>
> > On Jan 24, 2017, at 10:45 PM, Andy Zhou wrote:
> >
> > One common use case of 'struct dp_packet_batch' is to pro
Currently in OVN, the "nat-addresses" in the "options" column of a
logical switch port of type "router" must be specified manually.
Typically the user would specify as "nat-addresses" all of the NAT
external IP addresses and load balancer IP addresses that have
already been specified separately on
This patch implements the flows required in the ingress and egress
pipeline stages in order to support NAT on a distributed logical router.
NAT functionality is associated with the logical router gateway port.
The flows that carry out NAT functionality all have match conditions on
inport or outpor
This optional patch addresses offline comments that the documentation
in ovn-nb.xml should not describe southbound constructs or flow
details, since it is user facing documentation.
Signed-off-by: Mickey Spiegel
Acked-by: Gurucharan Shetty
---
ovn/ovn-nb.xml | 36 ++-
This patch adds the new optional arguments "logical_port" and
"external_mac" to lr-nat-add, and displays that information in
lr-nat-list.
Signed-off-by: Mickey Spiegel
Acked-by: Gurucharan Shetty
---
ovn/utilities/ovn-nbctl.8.xml | 27 +++---
ovn/utilities/ovn-nbctl.c | 54 +
Currently, for performance reasons on gateway routers, ct_snat
that does not specify an IP address does not immediately trigger
recirculation. On gateway routers, ct_snat that does not specify
an IP address happens in the UNSNAT pipeline stage, which is
followed by the DNAT pipeline stage that tri
This will make it easy for distributed NAT to reuse some of the
existing code for NAT flows, while leaving load balancing and defrag
as functionality specific to gateway routers. There is no intent to
change any functionality in this patch.
Signed-off-by: Mickey Spiegel
Acked-by: Gurucharan Shet
PATCH v13 -> PATCH v14
Rebase.
In patch 5 (rewrite redirect-chassis description in ovn-nb.xml),
emphasized the need to set the peer logical switch port's "addresses"
column to the string "router". Added a little more detail explaining
what this does.
Currently OVN supports NAT functionality by c
Answer inline.
Thanks,
Alin.
> -Original Message-
> From: Sairam Venugopal [mailto:vsai...@vmware.com]
> Sent: Friday, January 27, 2017 2:36 AM
> To: Alin Serdean ;
> d...@openvswitch.org
> Subject: Re: [ovs-dev] [PATCH] windows: wmi add include
>
> Hi Alin,
>
> Does this fix the access
Hi Alin,
Does this fix the access violation error in wcscat_s inside create_wmi_port()?
I ran into this issue when creating a bridge previously.
I will apply this patch and re-test.
Thanks,
Sairam
On 1/26/17, 2:15 PM, "ovs-dev-boun...@openvswitch.org on behalf of Alin
Serdean" wrote:
>Ad
Previous commit 336a5584 has a variable declared inline, while
-Wdeclaration-after-statement flag is enabled on this branch. This
causes build failure:
lib/lacp.c:620:9: error: mixing declarations and code
The declaration of the variable "enable" has been moved in this change
to comply with th
Previous commit 336a5584 has a variable declared inline, while
-Wdeclaration-after-statement flag is enabled on this branch. This
causes build failure:
lib/lacp.c:620:9: error: mixing declarations and code
The declaration of the variable "enable" has been moved in this change
to comply with th
From: Alin Serdean
This patch extracts the code that tries to get a continuous IPv4 header
buffer from the function 'OvsUpdateIPv4Header' and moves it to a new
function 'OvsGetHeaderBySize'.
The new function can be used later when trying to change the UDP/TCP/MPLS
etc., headers.
Signed-off-by:
Forgot to add version number.
Please ignore this patch.
Alin.
> -Original Message-
> From: Alin Serdean
> Sent: Friday, January 27, 2017 1:41 AM
> To: d...@openvswitch.org
> Cc: Alin Serdean
> Subject: [PATCH 4/4] datapath-windows: Add support for
> OVS_KEY_ATTR_TCP set action
>
> From
From: Alin Serdean
This patch adds support for set action with OVS_KEY_ATTR_TCP attribute
(change TCP source or destination port).
If the source or destination TCP port was changed, update the TCP checksum.
A sample flow can look like the following:
set(tcp(src=80,dst=443))
Signed-off-by: Alin
Forgot to add version number.
Please ignore this patch.
Alin.
> -Original Message-
> From: Alin Serdean
> Sent: Friday, January 27, 2017 1:38 AM
> To: d...@openvswitch.org
> Cc: Alin Serdean
> Subject: [PATCH v3 2/4] datapath-windows: Add function to get continuous
> buffer from context
Forgot to add version number.
Please ignore this patch.
Alin.
> -Original Message-
> From: Alin Serdean
> Sent: Friday, January 27, 2017 1:40 AM
> To: d...@openvswitch.org
> Cc: Alin Serdean
> Subject: [PATCH 3/4] datapath-windows: Add support for
> OVS_KEY_ATTR_UDP set action
>
> From
From: Alin Serdean
This patch adds support for set action with OVS_KEY_ATTR_UDP attribute
(change UDP source or destination port).
If the source or destination UDP port was changed, update the UDP checksum.
A sample flow can look like the following:
set(udp(src=67,dst=68))
Signed-off-by: Alin
From: Alin Serdean
This patch adds support for set action with OVS_KEY_ATTR_TCP attribute
(change TCP source or destination port).
If the source or destination TCP port was changed, update the TCP checksum.
A sample flow can look like the following:
set(tcp(src=80,dst=443))
Signed-off-by: Alin
From: Alin Serdean
This patch adds support for set action with OVS_KEY_ATTR_UDP attribute
(change UDP source or destination port).
If the source or destination UDP port was changed, update the UDP checksum.
A sample flow can look like the following:
set(udp(src=67,dst=68))
Signed-off-by: Alin
From: Alin Serdean
This patch extracts the code that tries to get a continuous IPv4 header
buffer from the function 'OvsUpdateIPv4Header' and moves it to a new
function 'OvsGetHeaderBySize'.
The new function can be used later when trying to change the UDP/TCP/MPLS
etc., headers.
Signed-off-by:
From: Alin Serdean
bufferStart can be used directly to access the data of the net buffer.
Add the MDL offset to save unnecessary additions.
Signed-off-by: Alin Gabriel Serdean
Acked-by: Sairam Venugopal
---
v3: add acked
v2: no change
---
datapath-windows/ovsext/Actions.c | 8
1 file
On Wed, Jan 25, 2017 at 4:16 PM, Jarno Rajahalme wrote:
> With two small nits below:
>
> Acked-by: Jarno Rajahalme
>
Thanks.
>
> > On Jan 24, 2017, at 10:45 PM, Andy Zhou wrote:
> >
> > Currently, there is no way to specify the packet size when injecting
> > a packet via "netdev-dummy/receive
From: Alin Serdean
Until now we used 'Open vSwitch' as the company/organization name.
The project is now under The Linux Foundation ownership.
This patch updates the MSI and driver attributes to reflect that ownership.
Signed-off-by: Alin Gabriel Serdean
Acked-by: Ben Pfaff
Acked-by: Sairam
On Thu, Jan 26, 2017 at 02:36:41PM -0800, Joe Stringer wrote:
> On 26 January 2017 at 11:28, Ben Pfaff wrote:
> > Found by travis.
> >
> > Signed-off-by: Ben Pfaff
>
> Acked-by: Joe Stringer
Thanks, applied to master and branch-2.7.
___
dev mailing l
On 26 January 2017 at 11:28, Ben Pfaff wrote:
> Found by travis.
>
> Signed-off-by: Ben Pfaff
Acked-by: Joe Stringer
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On 01/22/2017 09:07 PM, Jan Scheurich wrote:
> Hi Kevin
>
>>> The performance results are very impressive - it looks like ~50%
>>> performance improvement after about 10K flows.
>>>
>>> Did you measure any negative effects when the the emc is not full?
>> Hi Kevin,
>>
>> When the EMC is not full t
Add 'util.h' to includes otherwise the definition of the function
ovs_format_message will be unknown and an abort with the following stacktrace
will be triggered:
*1 ovs-vswitchd.exe!_output_l(_iobuf * stream=0x00bb711dcbf0, const
char * format=0x7ff6016a2c12, localeinfo_struct * plo
UNITED STATES OF AMERICA LOTTO INC.
ADRESSE: OSTWESTLANDSTRASSE 1325
STADT: SILBERNER FRUHLING
STATE/PROVINCE: MARYLAND
POSTCODE: 20110
LAND: USA
INC.
REF NR: .575061725
STAPEL NR: .6485769789/646
On 26 January 2017 at 04:03, Daniele Di Proietto
wrote:
>
>
>
>
>
> On 25/01/2017 00:01, "Ansis Atteka" wrote:
>
> >
> >
> >On Jan 25, 2017 4:22 AM, "Daniele Di Proietto"
> wrote:
> >
> >Current SELinux policy in RHEL and Fedora doesn't allow the creation of
> >TAP devices.
> >
> >A tap device
On 26 January 2017 at 21:24, Aaron Conole wrote:
> Daniele Di Proietto writes:
>
> > On 25/01/2017 00:01, "Ansis Atteka" wrote:
> >
> >>On Jan 25, 2017 4:22 AM, "Daniele Di Proietto"
> wrote:
> >>
> >>Current SELinux policy in RHEL and Fedora doesn't allow the creation of
> >>TAP devices.
> >>
Wait for clients to read from the pipe before disconnecting the server.
Signed-off-by: Alin Gabriel Serdean
---
Intended for master and branch-2.7
---
python/ovs/stream.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/python/ovs/stream.py b/python/ovs/stream.py
index 58c4925..be69534 10
Currently in OVN, the "nat-addresses" in the "options" column of a
logical switch port of type "router" must be specified manually.
Typically the user would specify as "nat-addresses" all of the NAT
external IP addresses and load balancer IP addresses that have
already been specified separately on
This patch implements the flows required in the ingress and egress
pipeline stages in order to support NAT on a distributed logical router.
NAT functionality is associated with the logical router gateway port.
The flows that carry out NAT functionality all have match conditions on
inport or outpor
This optional patch addresses offline comments that the documentation
in ovn-nb.xml should not describe southbound constructs or flow
details, since it is user facing documentation.
Signed-off-by: Mickey Spiegel
Acked-by: Gurucharan Shetty
---
ovn/ovn-nb.xml | 25 ++---
1 fi
This patch adds the new optional arguments "logical_port" and
"external_mac" to lr-nat-add, and displays that information in
lr-nat-list.
Signed-off-by: Mickey Spiegel
Acked-by: Gurucharan Shetty
---
ovn/utilities/ovn-nbctl.8.xml | 27 +++---
ovn/utilities/ovn-nbctl.c | 54 +
Currently, for performance reasons on gateway routers, ct_snat
that does not specify an IP address does not immediately trigger
recirculation. On gateway routers, ct_snat that does not specify
an IP address happens in the UNSNAT pipeline stage, which is
followed by the DNAT pipeline stage that tri
This will make it easy for distributed NAT to reuse some of the
existing code for NAT flows, while leaving load balancing and defrag
as functionality specific to gateway routers. There is no intent to
change any functionality in this patch.
Signed-off-by: Mickey Spiegel
Acked-by: Gurucharan Shet
PATCH v12 -> PATCH v13
Addressed minor comments from shettyg on nat flows patch.
Currently OVN supports NAT functionality by connecting each distributed
logical router to a centralized "l3gateway" router that resides on a
single chassis. NAT is only carried out in the "l3gateway" router.
This p
Found by travis.
Signed-off-by: Ben Pfaff
---
lib/automake.mk | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lib/automake.mk b/lib/automake.mk
index 0acbac9..abc9d0d 100644
--- a/lib/automake.mk
+++ b/lib/automake.mk
@@ -1,4 +1,4 @@
-# Copyright (C) 2009, 2010, 2011, 2012,
Aaron Conole writes:
> When the ovsdb library pkgconfig changes were introduced, they placed
> generated output in the src directory. This is incorrect, however, as
> the output files should actually be placed in the build directory. It
> is only seen when running `make distcheck` after enablin
When the ovsdb library pkgconfig changes were introduced, they placed
generated output in the src directory. This is incorrect, however, as
the output files should actually be placed in the build directory. It
is only seen when running `make distcheck` after enabling shared
libraries (ex: `./conf
Daniele Di Proietto writes:
> On 25/01/2017 00:01, "Ansis Atteka" wrote:
>
>>On Jan 25, 2017 4:22 AM, "Daniele Di Proietto" wrote:
>>
>>Current SELinux policy in RHEL and Fedora doesn't allow the creation of
>>TAP devices.
>>
>>A tap device is used by dpif-netdev to create internal devices.
>>
Wait for clients to read from the pipe before disconnecting the server.
Found while testing.
Signed-off-by: Alin Gabriel Serdean
---
Intended for master and branch-2.7
---
lib/stream-windows.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/lib/stream-windows.c b/lib/stream-windows.c
ind
Daniele Di Proietto writes:
> The OVSDPDKBond case wasn't handled in the rhel ifdown script.
>
> Fixes: f6bf8880613a ("rhel: Add support DPDK port creation via network
> scripts")
> Signed-off-by: Daniele Di Proietto
> ---
D'oh!
Acked-by: Aaron Conole
On Thu, Jan 26, 2017 at 06:37:42PM +, Alin Serdean wrote:
> Found by inspection.
>
> Signed-off-by: Alin Gabriel Serdean
Thanks, applied to master and branch-2.7.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listin
Found by inspection.
Signed-off-by: Alin Gabriel Serdean
---
lib/.gitignore | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/.gitignore b/lib/.gitignore
index 9f6e924..0680af6 100644
--- a/lib/.gitignore
+++ b/lib/.gitignore
@@ -9,6 +9,7 @@
/ofp-actions.inc2
/ofp-errors.inc
/ofp-msgs.i
Thanks Guru! Can you apply it on branch-2.7 as well?
Thanks,
Alin.
From: Guru Shetty [mailto:g...@ovn.org]
Sent: Thursday, January 26, 2017 8:03 PM
To: Alin Serdean
Cc: d...@openvswitch.org
Subject: Re: [ovs-dev] [PATCH] test windows: appctl - route/add with gateway
On 26 January 2017 at 09:5
Skip the test if the service 'ovsdb-server' is already defined.
The arguments of the service are incomplete: in the former state
it will try to create the pidfile and unixctl in the configuration path.
This patch adds those arguments.
Signed-off-by: Alin Gabriel Serdean
---
Intended for master a
It turns out that, since groff 1.20 around 2009, groff comes with a
preprocessor named "preconv" that can fix encoding issues. Use it instead
of the existing hack.
Signed-off-by: Ben Pfaff
---
build-aux/extract-ofp-fields | 8
lib/automake.mk | 4 ++--
2 files changed, 2 i
On 26 January 2017 at 09:57, Alin Serdean
wrote:
> This test passes on Windows, change the test accordingly.
>
> Signed-off-by: Alin Gabriel Serdean
>
Thanks, applied.
> ---
> tests/ovs-router.at | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/tests/ovs-router.at b/tests/ovs-router.a
This test passes on Windows, change the test accordingly.
Signed-off-by: Alin Gabriel Serdean
---
tests/ovs-router.at | 1 -
1 file changed, 1 deletion(-)
diff --git a/tests/ovs-router.at b/tests/ovs-router.at
index 93a730a..4e0abdf 100644
--- a/tests/ovs-router.at
+++ b/tests/ovs-router.at
@@
On 26 January 2017 at 09:47, Mickey Spiegel wrote:
>
> On Thu, Jan 26, 2017 at 8:53 AM, Guru Shetty wrote:
>
>>
>>
>> On 21 January 2017 at 16:52, Mickey Spiegel
>> wrote:
>>
>>> This patch implements the flows required in the ingress and egress
>>> pipeline stages in order to support NAT on a
>
> 2017-01-25 7:52 GMT-08:00 Loftus, Ciara :
> >> 2017-01-22 11:45 GMT-08:00 Jan Scheurich :
> >> >
> >> >> It's not a big deal, since the most important use case we have for
> >> >> dpif-netdev is with dpdk, but I'd still like the code to behave
> >> >> similarly on different platforms. How abo
Unconditional insertion of EMC entries results in EMC thrashing at high
numbers of parallel flows. When this occurs, the performance of the EMC
often falls below that of the dpcls classifier, rendering the EMC
practically useless.
Instead of unconditionally inserting entries into the EMC when a mi
On Thu, Jan 26, 2017 at 9:20 AM, Guru Shetty wrote:
>
>
> On 21 January 2017 at 16:52, Mickey Spiegel wrote:
>
>> This patch adds the new optional arguments "logical_port" and
>> "external_mac" to lr-nat-add, and displays that information in
>> lr-nat-list.
>>
>> Signed-off-by: Mickey Spiegel
>
On Thu, Jan 26, 2017 at 8:53 AM, Guru Shetty wrote:
>
>
> On 21 January 2017 at 16:52, Mickey Spiegel wrote:
>
>> This patch implements the flows required in the ingress and egress
>> pipeline stages in order to support NAT on a distributed logical router.
>>
>> NAT functionality is associated w
On 21 January 2017 at 16:52, Mickey Spiegel wrote:
> This optional patch addresses offline comments that the documentation
> in ovn-nb.xml should not describe southbound constructs or flow
> details, since it is user facing documentation.
>
> Signed-off-by: Mickey Spiegel
>
Acked-by: Gurucharan
On 21 January 2017 at 16:52, Mickey Spiegel wrote:
> This patch adds the new optional arguments "logical_port" and
> "external_mac" to lr-nat-add, and displays that information in
> lr-nat-list.
>
> Signed-off-by: Mickey Spiegel
>
Acked-by: Gurucharan Shetty
On a different note, can the exte
On 21 January 2017 at 16:52, Mickey Spiegel wrote:
> This patch implements the flows required in the ingress and egress
> pipeline stages in order to support NAT on a distributed logical router.
>
> NAT functionality is associated with the logical router gateway port.
> The flows that carry out N
From: Sugesh Chandran
Openvswitch datapath recirculates packets for tunneling, i.e.
the incoming packets are encapsulated at first pass. Further actions are
applied on encapsulated packets on the second pass after recirculating.
The proposed patch compute and append the post tunnel actions at the
From: Sugesh Chandran
Following tunneling specific test cases are failing due to the introduction of
new CLONE action for the tunnel push.
769: tunnel_push_pop_ipv6 - action FAILED
(tunnel-push-pop-ipv6.at:92)
767: tunnel_push_pop - actionFAILED
(tunne
From: Sugesh Chandran
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
This patch set removes the recirculation of encapsulated tunnel packets by
computing the post tunnel actions at the time of translation.
V3
- Use only single
View The Enclosed Attached And Redeem Your Reward. ___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Currently in OVN, the "nat-addresses" in the "options" column of a
logical switch port of type "router" must be specified manually.
Typically the user would specify as "nat-addresses" all of the NAT
external IP addresses and load balancer IP addresses that have
already been specified separately on
This patch implements the flows required in the ingress and egress
pipeline stages in order to support NAT on a distributed logical router.
NAT functionality is associated with the logical router gateway port.
The flows that carry out NAT functionality all have match conditions on
inport or outpor
This patch adds the new optional arguments "logical_port" and
"external_mac" to lr-nat-add, and displays that information in
lr-nat-list.
Signed-off-by: Mickey Spiegel
---
ovn/utilities/ovn-nbctl.8.xml | 27 +++---
ovn/utilities/ovn-nbctl.c | 54 ++
This optional patch addresses offline comments that the documentation
in ovn-nb.xml should not describe southbound constructs or flow
details, since it is user facing documentation.
Signed-off-by: Mickey Spiegel
---
ovn/ovn-nb.xml | 25 ++---
1 file changed, 10 insertions(+),
Currently, for performance reasons on gateway routers, ct_snat
that does not specify an IP address does not immediately trigger
recirculation. On gateway routers, ct_snat that does not specify
an IP address happens in the UNSNAT pipeline stage, which is
followed by the DNAT pipeline stage that tri
This will make it easy for distributed NAT to reuse some of the
existing code for NAT flows, while leaving load balancing and defrag
as functionality specific to gateway routers. There is no intent to
change any functionality in this patch.
Signed-off-by: Mickey Spiegel
Acked-by: Gurucharan Shet
PATCH v11 -> PATCH v12
Added a patch to automatically add router NAT and load balancer IP
addresses to "options:nat-addresses" of type "router" lsps, in order
to trigger gratuitous ARPs. This patch is for existing gateway
routers. A separate patch (not yet ready) is being developed to
extend grat
87 matches
Mail list logo