[ovs-dev] Returned mail: Data format error

___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn: update docs for l2gateway port vlan tag

What a nice introduction! Russell, do you want to review this? On Thu, Sep 29, 2016 at 03:40:02PM -0700, Gabe Beged-Dov wrote: > Hi All, > > Since this is my first post to the list, I'd like to take the opportunity > to thank the community and especially the core committers > for all their

Re: [ovs-dev] [PATCH v2] ovn-vtep: fix arping from vtep-gw physical port

On Thu, Sep 29, 2016 at 7:31 PM, Ramu Ramamurthy wrote: > > Hello Ramu > > > > Can you describe your configuration for this test failure when logical > > switch > > arp responders are skipped for logical switch "router type" ports ? > > I know the existing OVN tests

Re: [ovs-dev] [PATCH v2] ovn-vtep: fix arping from vtep-gw physical port

> Hello Ramu > > Can you describe your configuration for this test failure when logical > switch > arp responders are skipped for logical switch "router type" ports ? > I know the existing OVN tests (both system and non-system) pass either way. > > Thanks Darrell > There is no test that failed.

Re: [ovs-dev] [PATCH v2] ovn-vtep: fix arping from vtep-gw physical port

On Mon, Sep 26, 2016 at 2:00 PM, Darrell Ball wrote: > > > On Mon, Sep 26, 2016 at 11:24 AM, Ramu Ramamurthy < > ramu.ramamur...@gmail.com> wrote: > >> -if (!strcmp(op->nbsp->type, "localnet")) { >> +/* Skip arp responder if the logical switch inport is not >> +

Re: [ovs-dev] Deferring ofproto_class::destruct vs. ovs-appctl exit

Jarno Rajahalme writes: > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c > index 83dcc9c..5b42b7e 100644 > --- a/ofproto/ofproto-dpif.c > +++ b/ofproto/ofproto-dpif.c > @@ -864,7 +864,7 @@ close_dpif_backer(struct dpif_backer *backer) > free(backer->type); >

Re: [ovs-dev] Deferring ofproto_class::destruct vs. ovs-appctl exit

Jarno Rajahalme writes: > Btw. I don’t see this problem triggered by the testsuite even if I set the > ‘ofproto->backer' pointer > to NULL right after the free() call. Do you see this happening on an > unmodified OVS 2.5? If so, > could you send the steps to reproduce. Just need

[ovs-dev] [PATCH] ovn: update docs for l2gateway port vlan tag

Hi All, Since this is my first post to the list, I'd like to take the opportunity to thank the community and especially the core committers for all their visible efforts as well as all the effort I know goes on behind the scenes to keep such an amazing project barrelling along, even around

Re: [ovs-dev] [PATCH] ovn: Do not reply to ARP or ND NS for a VM's own IP address.

On Thu, Sep 29, 2016 at 03:13:47PM -0700, Darrell Ball wrote: > On Thu, Sep 29, 2016 at 11:31 AM, Ben Pfaff wrote: > > > When a VM sends an ARP or an ND NS for its own IP address, it is trying to > > check for a duplicate address in the network. OVN needs to suppress the > > reply

Re: [ovs-dev] [PATCH] ovn: Do not reply to ARP or ND NS for a VM's own IP address.

On Thu, Sep 29, 2016 at 11:31 AM, Ben Pfaff wrote: > When a VM sends an ARP or an ND NS for its own IP address, it is trying to > check for a duplicate address in the network. OVN needs to suppress the > reply in such a case, otherwise the VM thinks that its address is a >

Re: [ovs-dev] [PATCH 1/3] tests: Use standard -q option to grep in place of GNU --quiet extension.

> On Sep 29, 2016, at 5:41 PM, Ben Pfaff wrote: > > Reported-by: Stuart Cardall > Reported-at: > http://openvswitch.org/pipermail/discuss/2016-September/022803.html > Signed-off-by: Ben Pfaff Acked-by: Flavio Fernandes

Re: [ovs-dev] [PATCH] ovn: Do not reply to ARP or ND NS for a VM's own IP address.

On Thu, Sep 29, 2016 at 12:15:36PM -0700, Han Zhou wrote: > On Thu, Sep 29, 2016 at 11:31 AM, Ben Pfaff wrote: > > > > When a VM sends an ARP or an ND NS for its own IP address, it is trying to > > check for a duplicate address in the network. OVN needs to suppress the > > reply in

[ovs-dev] [PATCH 3/3] ovs-ofctl: Tolerate differences in IPv6 formatting.

glibc formats single zeros as 0: fec0:0:1234:f045:8fff::fe4e:571 Musl formats single zeros as ::: fec0::1234:f045:8fff::fe4e:571 This patch makes the OVS testsuite tolerate either one. Reported-by: Stuart Cardall Reported-at:

[ovs-dev] [PATCH 2/3] tests: Use Linux-specific way to get parent PID, to avoid noncompliant "ps".

POSIX defines "ps" -o and -p options, but the "ps" implementation in busybox (used in Alpine Linux) doesn't support -p, which makes some tests fail for no good reason. Therefore, this commit makes the testsuite instead check for support for the Linux-specific /proc-based way to find the parent of

[ovs-dev] [PATCH 1/3] tests: Use standard -q option to grep in place of GNU --quiet extension.

Reported-by: Stuart Cardall Reported-at: http://openvswitch.org/pipermail/discuss/2016-September/022803.html Signed-off-by: Ben Pfaff --- AUTHORS | 1 + tests/ovn.at | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/AUTHORS

Re: [ovs-dev] [PATCH v9] ovn-nbctl: Add LB commands.

> On Sep 29, 2016, at 5:43 AM, nickcooper-zhangtonghao > wrote: > > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to multiple > switches or routers. A single load balancer can

Re: [ovs-dev] [PATCH v9] ovn-nbctl: Add LB commands.

On 29 September 2016 at 02:43, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to multiple > switches or routers. A single load balancer can have

Re: [ovs-dev] Deferring ofproto_class::destruct vs. ovs-appctl exit

Btw. I don’t see this problem triggered by the testsuite even if I set the ‘ofproto->backer' pointer to NULL right after the free() call. Do you see this happening on an unmodified OVS 2.5? If so, could you send the steps to reproduce. Just need to know if this is a potential crashing bug in

Re: [ovs-dev] Deferring ofproto_class::destruct vs. ovs-appctl exit

This may not be the cleanest solution, but how about changing the last line of close_dpif_backer() in ofproto/ofproto-dpif.c like this: diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 83dcc9c..5b42b7e 100644 --- a/ofproto/ofproto-dpif.c +++ b/ofproto/ofproto-dpif.c @@ -864,7

Re: [ovs-dev] [PATCH] ovn: Do not reply to ARP or ND NS for a VM's own IP address.

On Thu, Sep 29, 2016 at 11:31 AM, Ben Pfaff wrote: > > When a VM sends an ARP or an ND NS for its own IP address, it is trying to > check for a duplicate address in the network. OVN needs to suppress the > reply in such a case, otherwise the VM thinks that its address is a >

Re: [ovs-dev] [PATCH 2/2] netdev-dpdk: Fix coding style

2016-09-29 3:27 GMT-07:00 Mark Kavanagh : > Coding style violations of the following conventions are present in > netdev-dpdk.c: > - limit lines to 79 characters > - put a space after (but not before) the "sizeof" keyword. > - put a space between the () used

Re: [ovs-dev] [PATCH v2] netdev-dpdk: Introduce dpdk_mp_mutex.

2016-09-23 6:17 GMT-07:00 Ilya Maximets : > 'dpdk_mutex' protects two independent things: list of dpdk devices > and list of memory pools. Let's spit it in two to avoid global blocking > inside 'netdev_dpdk.*_reconfigure()' as possible. > > Signed-off-by: Ilya Maximets

[ovs-dev] Deferring ofproto_class::destruct vs. ovs-appctl exit

Hi, On 2.5, we are seeing the following problem when removing a bridge: - ofproto_destroy calls ofproto_flush__, which eventually calls ovsrcu_postpone(remove_rules_rcu) - ofproto_destroy also calls p->ofproto_class->destruct, which eventually leads to release of DPIF backer

Re: [ovs-dev] Neighborhood solicitation and stateful DHCPv6

On Wed, Sep 28, 2016 at 09:31:58PM +0500, Valentine Sinitsyn wrote: > We are trying to set up stateful DHCPv6 (i.e. the one with dhcpv6_stateless > = false) on current OVN master. > > Connected VMs don't get DHCPv6 options unless we tell OVN the IPv6 address > behind the logical port, and unless

[ovs-dev] [PATCH] ovn: Do not reply to ARP or ND NS for a VM's own IP address.

When a VM sends an ARP or an ND NS for its own IP address, it is trying to check for a duplicate address in the network. OVN needs to suppress the reply in such a case, otherwise the VM thinks that its address is a duplicate. Reported-by: Valentine Sinitsyn

Re: [ovs-dev] [PATCH v4] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

2016-09-29 3:28 GMT-07:00 Loftus, Ciara : > > > > Hi Ciara, > > thanks for the patch, it looks good to me. > > I only have a minor comment: > > I'd like the requested values to depend only on the current database > > state. With the current patch when a value is invalid

Re: [ovs-dev] tx and rx packet

On 29 September 2016 at 07:29, Ashish Kurian wrote: > Dear Developers, > > I am trying to send packets into s1-eth1 and it is receiving the packets in > correct numbers. Then I send those packets to the gtp tunnel port set. But > I am not getting any rx packet count on that

Re: [ovs-dev] Subject: [PATCH] ovs-monitor-ipsec: ipsec_vxlan support and IPsec tunnel mode options support.

On Thu, Sep 29, 2016 at 07:13:55AM +, Muthukrishnan Thangasamy wrote: > From 76e504a38a3c556884782323459cc6862a38747c Mon Sep 17 00:00:00 2001 > From: Muthukrishnan T > Date: Thu, 29 Sep 2016 12:21:29 +0530 > Subject: [PATCH] ovs-monitor-ipsec: ipsec_vxlan

Re: [ovs-dev] [PATCH] CONTRIBUTING.md: Improve recommended command for Fixes: header.

On Fri, Sep 23, 2016 at 01:18:16PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > It usually makes sense to CC the author of the commit being fixed. > > > > The CC tag wasn't mentioned before, so describe it too. > > > > Signed-off-by: Ben Pfaff > > --- > >

Re: [ovs-dev] [PATCH] FAQ: ERSPAN is no longer undocumented.

Thanks, applied to master. On Thu, Sep 29, 2016 at 07:33:05AM -0700, William Tu wrote: > Acked-by: William Tu > > On Thu, Sep 15, 2016 at 11:11 AM, Ben Pfaff wrote: > > See https://tools.ietf.org/html/draft-foschiano-erspan-01. Even though > > that draft has

Re: [ovs-dev] Query regarding the OF v1.5 specification requirement on: Flow Statistics Trigger(EXT-335)

On Thu, Sep 29, 2016 at 04:40:04PM +0530, Sravya Chava/HYD/TCS wrote: > Hi Ben en all, > > With reference to the OpenFlow Specification version 1.5 we have picked up > the Implementation for B.18.4 Flow Entry Statistics Trigger EXT-335. > we are reaching out to you to correct our understanding

Re: [ovs-dev] [PATCH] FAQ: ERSPAN is no longer undocumented.

Acked-by: William Tu On Thu, Sep 15, 2016 at 11:11 AM, Ben Pfaff wrote: > See https://tools.ietf.org/html/draft-foschiano-erspan-01. Even though > that draft has expired, it's genuine documentation. > > Signed-off-by: Ben Pfaff > --- > FAQ.md |

[ovs-dev] tx and rx packet

Dear Developers, I am trying to send packets into s1-eth1 and it is receiving the packets in correct numbers. Then I send those packets to the gtp tunnel port set. But I am not getting any rx packet count on that port. I am keeping the dump-flows s1 and packet counts output. Please look into this

[ovs-dev] Query regarding the OF v1.5 specification requirement on: Flow Statistics Trigger(EXT-335)

Hi Ben en all, With reference to the OpenFlow Specification version 1.5 we have picked up the Implementation for B.18.4 Flow Entry Statistics Trigger EXT-335. we are reaching out to you to correct our understanding on the above requirement and help us clarify our Open questions for the same.

Re: [ovs-dev] [PATCH v4] netdev-dpdk: Allow configurable queue sizes for 'dpdk' ports

> > Hi Ciara, > thanks for the patch, it looks good to me. > I only have a minor comment: > I'd like the requested values to depend only on the current database > state.  With the current patch when a value is invalid (not pow2 or bigger > than 4096) we keep the previous one. > Could you change

[ovs-dev] [PATCH 2/2] netdev-dpdk: Fix coding style

Coding style violations of the following conventions are present in netdev-dpdk.c: - limit lines to 79 characters - put a space after (but not before) the "sizeof" keyword. - put a space between the () used in a cast and the expression whose type is cast: (void *) 0. Resolve

[ovs-dev] [PATCH 1/2] netdev-dpdk: consistent naming for mbuf variables

Pointers to struct rte_mbuf are typically denoted within functions as 'pkt'; similarly, arrays of, and pointer-to-pointer to, struct rte_mbuf are denoted by 'pkts'. Update discrepancies to the above convention for consistency. Signed-off-by: Mark Kavanagh ---

[ovs-dev] [PATCH 0/2] netdev-dpdk: Code clean-up

Clean up coding style issues and inconsistencies that have built up in netdev-dpdk.c Mark Kavanagh (2): netdev-dpdk: consistent naming for mbuf variables netdev-dpdk: Fix coding style lib/netdev-dpdk.c | 115 +- 1 file changed, 61

Re: [ovs-dev] [PATCH v3 2/4] ovn: OCF script for OVN OVSDB servers

Yes Andrew. The database is never reset while testing in all the scenarios that we discussed. On Thursday 29 September 2016 09:14 AM, Andrew Beekhof wrote: no problem. it went well in testing? On Wed, Sep 28, 2016 at 2:32 PM, Babu Shanmugam wrote: Hi Andrew, I have

[ovs-dev] [PATCH v9] ovn-nbctl: Add LB commands.

This patch provides the command line to create a load balancer. You can create a load balancer independently and add it to multiple switches or routers. A single load balancer can have multiple vips. Add a name column for the load balancer. With --add-duplicate, the command really creates a new

Re: [ovs-dev] [PATCH/RFC 00/12] Programming Open vSwitch (-like) flows into hardware using SwitchDev

Hi Or, On Wed, Sep 28, 2016 at 04:54:40PM +0300, Or Gerlitz wrote: > On Wed, Sep 28, 2016 at 3:42 PM, Simon Horman > wrote: > > > A different approach, not implemented by this patch-set, is for user-space > > to program flows into hardware by some other means, for

[ovs-dev] Subject: [PATCH] ovs-monitor-ipsec: ipsec_vxlan support and IPsec tunnel mode options support.

From 76e504a38a3c556884782323459cc6862a38747c Mon Sep 17 00:00:00 2001 From: Muthukrishnan T Date: Thu, 29 Sep 2016 12:21:29 +0530 Subject: [PATCH] ovs-monitor-ipsec: ipsec_vxlan support and IPsec tunnel mode options support. Description : OpenvSwitch doesn’t