Hi Andy,
Please find my response inline:
On Tue, Jul 18, 2017 at 12:10 AM, Andy Zhou wrote:
> On Sat, Jul 15, 2017 at 10:58 PM, Arunkumar Rg
> wrote:
> > Hi Andy Zhou,
> >
> > Thanks for looking into this!
> >
> > Please find my replies inline:
> >
> >
Yes!, The problem was that we are snooping packets on the external side of
networks, and eventually we not only see the gARPs or packets we are
looking for, but we also see other ARP requests of external ports (used for
tunneling) looking for each othes. This didn't happen to me on my system
but
On Wed, Jul 19, 2017 at 5:37 AM, Justin Pettit wrote:
>
> > On Jul 18, 2017, at 4:44 PM, Joe Stringer wrote:
> >
> > On 18 July 2017 at 16:29, Justin Pettit wrote:
> >> Signed-off-by: Justin Pettit
> >
> > Acked-by: Joe Stringer
Signed-off-by: Tonghao Zhang
---
AUTHORS.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/AUTHORS.rst b/AUTHORS.rst
index 10c84c1..177cef8 100644
--- a/AUTHORS.rst
+++ b/AUTHORS.rst
@@ -312,6 +312,7 @@ Tsvi Slonim
Signed-off-by: Tonghao Zhang
---
lib/ovs-router.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/lib/ovs-router.c b/lib/ovs-router.c
index b387657..ce2f80b 100644
--- a/lib/ovs-router.c
+++ b/lib/ovs-router.c
@@ -44,8 +44,6 @@
#include "unixctl.h"
#include
> On Jul 19, 2017, at 7:32 AM, Justin Pettit wrote:
>
>>
>> On Jul 18, 2017, at 4:15 AM, Eelco Chaudron wrote:
>>
>> On 18/07/17 08:28, Justin Pettit wrote:
On Jul 17, 2017, at 10:06 PM, Numan Siddique wrote:
On Tue,
Signed-off-by: Tonghao Zhang
---
AUTHORS.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/AUTHORS.rst b/AUTHORS.rst
index 10c84c1..177cef8 100644
--- a/AUTHORS.rst
+++ b/AUTHORS.rst
@@ -312,6 +312,7 @@ Tsvi Slonim
ovs-router module uses the netdev_open() to get routes.
But this module always calls the netdev_open() with type
which is NULL. This module may open the eth0, vethx,
vxlan_sys_4789, br0 if these network devices exist. And
these device will be opened as 'system' type.
When debugging, somewhere
Signed-off-by: Tonghao Zhang
---
lib/ovs-router.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/lib/ovs-router.c b/lib/ovs-router.c
index b387657..ce2f80b 100644
--- a/lib/ovs-router.c
+++ b/lib/ovs-router.c
@@ -44,8 +44,6 @@
#include "unixctl.h"
#include
Hi all,
I just test ovs-dpdk with restart. I found when the dpdk start the PHYSICAL NIC
can't send/recv packet
I have a vhostuserclient port and dpdk port(82599EB) bridge with br0
It spends 1.4s for restarting the ovs-vswitchd.
1. It sends icmp packet(1 packet/second) from virtual machine to
On Tue, Jul 18, 2017 at 10:55 AM, Miguel Angel Ajo wrote:
> An specific ovn/l3ha test looks for port releases in the MASTER
> gateway when another gateway on the Gateway_Chassis list is gone.
>
> But we didn't clean the log before taking out the chassis, so
> any previous
On 18 July 2017 at 17:40, Joe Stringer wrote:
> On 18 July 2017 at 10:49, Sugesh Chandran wrote:
>> +static bool
>> +validate_and_combine_post_tnl_actions(struct xlate_ctx *ctx,
>> + const struct xport *xport,
>> +
On 18 July 2017 at 10:49, Sugesh Chandran wrote:
> This patch set removes the recirculation of encapsulated tunnel packets
> if possible. It is done by computing the post tunnel actions at the time of
> translation. The combined nested action set are programmed in the
Can we tallk?
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> On Jul 18, 2017, at 4:44 PM, Joe Stringer wrote:
>
> On 18 July 2017 at 16:29, Justin Pettit wrote:
>> Signed-off-by: Justin Pettit
>
> Acked-by: Joe Stringer
Thanks. I pushed these to branch-2.7.
--Justin
> On Jul 18, 2017, at 4:27 PM, Joe Stringer wrote:
>
> On 13 July 2017 at 23:42, Justin Pettit wrote:
>> The "ct" action is not supposed to make the various ct match fields
>> available except for the pipeline instantiated through the "table"
>> argument to the
On 18 July 2017 at 16:29, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
Acked-by: Joe Stringer
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On 18 July 2017 at 16:29, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
Acked-by: Joe Stringer
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> On Jul 18, 2017, at 4:15 AM, Eelco Chaudron wrote:
>
> On 18/07/17 08:28, Justin Pettit wrote:
>>> On Jul 17, 2017, at 10:06 PM, Numan Siddique wrote:
>>>
>>> On Tue, Jul 18, 2017 at 9:42 AM, Justin Pettit wrote:
>>>
On Jul
Signed-off-by: Justin Pettit
---
NEWS | 4
configure.ac | 2 +-
debian/changelog | 7 +++
3 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/NEWS b/NEWS
index eb691dbc3251..3ceb55f4c623 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,7 @@
+v2.7.3 -
On 18 July 2017 at 08:42, Greg Rose wrote:
> Upstream commit:
> commit 8b97ac5bda17cfaa257bcab6180af0f43a2e87e0
> Author: Greg Rose
> Date: Fri Jul 14 12:42:49 2017 -0700
>
> openvswitch: Fix for force/commit action failures
>
>
Signed-off-by: Justin Pettit
---
NEWS | 4 ++--
debian/changelog | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/NEWS b/NEWS
index 6f0768b2657f..eb691dbc3251 100644
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,6 @@
-v2.7.2 - xx xxx
+v2.7.2 - 18 Jul
On 13 July 2017 at 23:42, Justin Pettit wrote:
> The "ct" action is not supposed to make the various ct match fields
> available except for the pipeline instantiated through the "table"
> argument to the "ct" action. This commit fixes a few issues related to
> that and updates
On 10 July 2017 at 12:39, Eric Garver wrote:
> This series enables support for layer3 tunnels VXLAN-GPE and GRE in the kernel
> datapath. It includes new system-traffic test cases. The first two patches fix
> an issue translating packet_type from flows for the kernel.
>
> Note:
For non-Ethernet flows, when fixing up the netlink message we need make
sure to pass down a valid Ethertype. The kernel does not understand
packet_type so it's implicitly encoded by the absence of _ETHERNET and
exact match of _ETHERTYPE. Without this change match_validate in the
kernel complains
Rather than open-coding access to netlink attribute pointers in
put_exclude_packet_type(), make use of the netlink attribute helpers.
This simplifies the following bugfix.
Signed-off-by: Joe Stringer
---
lib/dpif-netlink.c | 10 +-
1 file changed, 5 insertions(+), 5
I have donated my funds to you email for details: marinasander...@gmail.com
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Análisis, variaciones y ajustes en las políticas de crédito
Dirección Avanzada de Crédito y Cobranza
Dar la importancia que se merece al área de Crédito y Cobranza le garantizará
excelentes resultados en la operación de su empresa. Participe en esta
capacitación que traemos para usted y
On 07/18/2017 01:23 PM, Joe Stringer wrote:
On 18 July 2017 at 13:08, Greg Rose wrote:
> On 07/18/2017 10:16 AM, Darrell Ball wrote:
>>
>>
>>
>> On 7/18/17, 8:42 AM, "ovs-dev-boun...@openvswitch.org on behalf of Greg
>> Rose"
On 18 July 2017 at 13:08, Greg Rose wrote:
> On 07/18/2017 10:16 AM, Darrell Ball wrote:
>>
>>
>>
>> On 7/18/17, 8:42 AM, "ovs-dev-boun...@openvswitch.org on behalf of Greg
>> Rose"
>> wrote:
>>
>>
On 07/18/2017 10:16 AM, Darrell Ball wrote:
On 7/18/17, 8:42 AM, "ovs-dev-boun...@openvswitch.org on behalf of Greg Rose"
wrote:
From: Joe Stringer
Add a new test check if the conntrack force
On 7/17/17, 11:19 PM, "Ilya Maximets" wrote:
On 18.07.2017 06:48, Darrell Ball wrote:
>
>
> On 7/13/17, 8:07 AM, "ovs-dev-boun...@openvswitch.org on behalf of Ilya
Maximets"
wrote:
Thank you Joe,
I sent out the v3 patch . Please have a look
Regards
_Sugesh
> -Original Message-
> From: Joe Stringer [mailto:j...@ovn.org]
> Sent: Tuesday, July 18, 2017 6:29 PM
> To: Chandran, Sugesh
> Cc: Zoltán Balogh ; ovs
The following tunnel combine patch series avoids the packets recirculation
after the tunnel push. So it is necessary to populate all relevant packet meta
data fields for the following combined action-set.
Consider a chained tunnel test case shown below,
PKT-IN --> TUNNEL_PUSH --> MOD_PKT_HDR -->
The tunnel mask in the translation state should be cleared along with other
context fields. It is necessary in 'apply_nested_clone_actions' as it will be
used to combine post tunnel output actions with tunnel push. This will assure
right openflow state while executing the translation.
Hi (again) Michal,
Michal Weglicki writes:
> Upgrading to DPDK 17.05.1 stable release adds new
> significant features relevant to OVS, including,
> but not limited to:
> - tun/tap PMD,
> - VFIO hotplug support,
> - Generic flow API.
>
> Following changes are applied:
On 18 July 2017 at 01:59, Chandran, Sugesh wrote:
>
>
> Regards
> _Sugesh
>
>
>> -Original Message-
>> From: Joe Stringer [mailto:j...@ovn.org]
>> Sent: Tuesday, July 18, 2017 6:42 AM
>> To: Chandran, Sugesh
>> Cc: Zoltán Balogh
On 7/18/17, 8:42 AM, "ovs-dev-boun...@openvswitch.org on behalf of Greg Rose"
wrote:
From: Joe Stringer
Add a new test check if the conntrack force direction change and
commit is working
Hi
Good day.
Would you be interested to get in touch with Key Decision Makers across B2B
industry sector as mentioned below? We provide complete contact information
like dual verified Opt-in Email address, direct Phone no and other business
information.
Below are the Industry sector,
Michal Weglicki writes:
> Upgrading to DPDK 17.05.1 stable release adds new
> significant features relevant to OVS, including,
> but not limited to:
> - tun/tap PMD,
> - VFIO hotplug support,
> - Generic flow API.
>
> Following changes are applied:
> - netdev-dpdk:
Upstream commit:
commit 8b97ac5bda17cfaa257bcab6180af0f43a2e87e0
Author: Greg Rose
Date: Fri Jul 14 12:42:49 2017 -0700
openvswitch: Fix for force/commit action failures
When there is an established connection in direction A->B, it is
possible to
From: Joe Stringer
Add a new test check if the conntrack force direction change and
commit is working correctly.
This test was used to find and root cause BZ 18090854.
Signed-off-by: Joe Stringer
Signed-off-by: Greg Rose
---
V2 - Fix commit
Daniel Alvarez Sanchez writes:
> When the interfaces list is retrieved through getiffaddrs(), there
> might be elements with iface_name set to NULL.
>
> This patch checks ifa_name to be not NULL before comparing it to the
> actual device name in the loop that calculates how
Thanks Aaron, I just sent a new version of it which also checks for
ifa_netmask
as the existing code does currently. It will save some memory in cases where
either name or netmask are NULL (which I guess it shouldn't happen but it
can
happen).
Thanks again,
Daniel
On Mon, Jul 17, 2017 at 9:07
When the interfaces list is retrieved through getiffaddrs(), there
might be elements with iface_name set to NULL.
This patch checks ifa_name to be not NULL before comparing it to the
actual device name in the loop that calculates how many interfaces
exist with that same name.
Also, this patch
On 07/17/2017 12:35 PM, Joe Stringer wrote:
On 17 July 2017 at 12:27, Darrell Ball wrote:
>
>
> On 7/17/17, 11:26 AM, "Greg Rose" wrote:
>
> On 07/17/2017 11:23 AM, Darrell Ball wrote:
> >
> >
> > On 7/17/17, 10:44 AM,
An specific ovn/l3ha test looks for port releases in the MASTER
gateway when another gateway on the Gateway_Chassis list is gone.
But we didn't clean the log before taking out the chassis, so
any previous unrelated occurrence could make the test fail
while it was ok.
Signed-off-by: Miguel Angel
We're now filtering out packets that we don't care about before
comparing with expected data.
Also, we're less strict on the number of gARPs we expect to
receive by using uniq filter, and looking for just one (or
none where it should be none).
Signed-off-by: Miguel Angel Ajo
This makes tests easier to debug because the sync operation won't hang
up when an ovn-controller or ovn-northd is not responding for any
reason.
Signed-off-by: Miguel Angel Ajo
---
tests/ovn.at | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
The bundle pointer wasn't properly re-pointed each time
we added a new slave port to the openflow action.
Signed-off-by: Miguel Angel Ajo
---
ovn/controller/physical.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/ovn/controller/physical.c
The bundle pointer wasn't properly re-pointed each time
we added a new slave port to the openflow action.
Signed-off-by: Miguel Angel Ajo
---
ovn/controller/physical.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/ovn/controller/physical.c
From: Ben Pfaff
GNU (at least) printf interprets -I as an option, but we want to print it
literally, so use %s.
CC: YAMAMOTO Takashi
Fixes: 27d41afaa446 ("acinclude.m4: Avoid echo -n")
Signed-off-by: Ben Pfaff
Acked-by: Justin Pettit
From: Ben Pfaff
update_in_band_remotes() created an in-band manager and then tried to work
with it without first checking whether creation had succeeded. If it
failed, this led to a segfault.
Reported-by: Numan Siddique
Reported-at:
From: Ben Pfaff
I hadn't even heard of this feature before, but it seems to be at least
semi-standard to support Linux link-local address scopes via a % suffix,
e.g. fe80::1234%eth0 for a link-local address scoped to eth0. This commit
adds support.
I'd appreciate feedback from
This is an revival from a thread I initiated earlier this year [0], that
I had to postpone due to other priorities.
First, I'd like to thanks reviewers of my first proposal, this new
version tries to address the comments made:
1.This is Nova's role and not Libvirt's to query hosts supported
From: Antonio Fischetti
With the command:
ovs-appctl dpctl/ct-bkts
shows the number of connections per bucket.
By using a threshold:
ovs-appctl dpctl/ct-bkts gt=N
for each bucket shows the number of connections when they
are greater than N.
Signed-off-by: Antonio
Thanks Darrel for your feedback, I'll post a v3 soon.
Antonio
> -Original Message-
> From: Darrell Ball [mailto:db...@vmware.com]
> Sent: Monday, July 17, 2017 6:03 PM
> To: Fischetti, Antonio ; d...@openvswitch.org
> Subject: Re: [ovs-dev] [PATCH v2] dpctl:
On 18/07/17 08:28, Justin Pettit wrote:
On Jul 17, 2017, at 10:06 PM, Numan Siddique wrote:
On Tue, Jul 18, 2017 at 9:42 AM, Justin Pettit wrote:
On Jul 17, 2017, at 8:51 PM, Guru Shetty wrote:
On 17 July 2017 at 12:51, Justin Pettit
Regards
_Sugesh
> -Original Message-
> From: Joe Stringer [mailto:j...@ovn.org]
> Sent: Tuesday, July 18, 2017 6:42 AM
> To: Chandran, Sugesh
> Cc: Zoltán Balogh ; ovs dev
> ; Andy Zhou
>
oops, thank you.
On Tue, Jul 18, 2017 at 2:12 AM, Ben Pfaff wrote:
> GNU (at least) printf interprets -I as an option, but we want to print it
> literally, so use %s.
>
> CC: YAMAMOTO Takashi
> Fixes: 27d41afaa446 ("acinclude.m4: Avoid echo -n")
> Signed-off-by:
Account tijdelijk geblokkeerd
Dit e-mailadres is geïdentificeerd als het versturen van een ongebruikelijke
hoeveelheid e-mails in een korte periode en is geblokkeerd op het IT-relais.
Dit account kan worden gecompromitteerd als u niet weet of er hoogvolume
berichten van dit adres worden
> On Jul 17, 2017, at 10:06 PM, Numan Siddique wrote:
>
> On Tue, Jul 18, 2017 at 9:42 AM, Justin Pettit wrote:
>
> > On Jul 17, 2017, at 8:51 PM, Guru Shetty wrote:
> >
> >
> >>
> >> On 17 July 2017 at 12:51, Justin Pettit
From: Nissan, Caroline
Sent: Tuesday, July 18, 2017 2:16 AM
To: Nissan, Caroline
Subject: RE: ICT Service Desk
From: Nissan, Caroline
Sent: Tuesday, July 18, 2017 2:06 AM
To: Nissan, Caroline
Subject: ICT Service Desk
Account
When calling the flow_free() to free the flow, we call many times
(cpu_possible_mask, eg. 128 as default) cpumask_next(). That will
take up our CPU usage if we call the flow_free() frequently.
When we put all packets to userspace via upcall, and OvS will send
them back via netlink to
In the ovs_flow_stats_update(), we only use the node
var to alloc flow_stats struct. But this is not a
common case, it is unnecessary to call the numa_node_id()
everytime. This patch is not a bugfix, but there maybe
a small increase.
Signed-off-by: Tonghao Zhang
---
On 18.07.2017 06:48, Darrell Ball wrote:
>
>
> On 7/13/17, 8:07 AM, "ovs-dev-boun...@openvswitch.org on behalf of Ilya
> Maximets" i.maxim...@samsung.com> wrote:
>
> If RSS hash exists in a packet it can be reused instead of
> 5 tuple
From: Venkata Anil
This change adds commands to set, get and delete gateway chassis
for logical router port.
Signed-off-by: Venkata Anil Kommaddi
---
ovn/utilities/ovn-nbctl.8.xml | 21 +
ovn/utilities/ovn-nbctl.c | 178
Thanks Jan, I have folded it to my local branch and will include it in next
version.
-Original Message-
From: Jan Scheurich [mailto:jan.scheur...@ericsson.com]
Sent: Tuesday, July 18, 2017 12:30 AM
To: Yang, Yi Y ; d...@openvswitch.org
Cc: Mengke Liu
68 matches
Mail list logo