___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
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 visib
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 (both system and non-system)
> 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. B
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
>> + * associat
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);
> free(backer->d
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 to know if this
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 corners
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 in such a case
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
> duplicate.
>
> Reporte
> 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
> ---
> AUTHORS | 1 +
> tests/ovn.at | 4 ++--
> 2 files changed,
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 such a case, o
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: http://openvswitch.org/pipermail/discuss/2016-September/022803
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
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 b/AUTHORS
index 371c683..a7ae253 100644
---
> 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 have multiple vips.
> Add a name column for
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 mul
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 OVS
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 +8
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
> duplicate.
>
> Report
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 in a cast and the
> e
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
> ---
>
> I think, that thi
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 (close_d
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 t
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
Reported-at: http://openvswitch.org/pi
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 (not pow2 or
> bigger
> >
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 port. I am keeping the
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 support and IPsec tunnel mode
> opti
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
> > ---
>
> ACK.
Thanks, applied to master
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 expired, it's genuine documentation
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 on
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 | 5 ++---
> 1 file changed, 2 insertions(+), 3 de
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
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.
Te
>
> 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 dp
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 occ
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
---
lib/netdev-dpdk.c | 34
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 insertio
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 added "Signed-off-by" on
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 loa
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 example TC, and/or
> > the (ker
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 have support for IPSec Tunnel inter
42 matches
Mail list logo
