Daniele Di Proietto writes:
> On 29/11/2016 07:57, "Aaron Conole" wrote:
>
>>Hi Ilya,
>>
>>Ilya Maximets writes:
>>
>>> On 28.11.2016 21:55, Aaron Conole wrote:
Ilya Maximets writes:
>
On 5 December 2016 at 13:59, Ben Pfaff wrote:
> Guru, did you ever have a look at this? It's always nice to integrate
> with more systems.
>
Right. I did work with Nimay to get this patch sent to the mailing list as
a RFC. The main reason it was a RFC was because of the
>
> Thanks for the patch.
>
> I experience a crash with this patch applied by starting ovs and
> immediately adding a vhostuserclient port. It's not reproducible 100%
> of the times.
>
> Program received signal SIGSEGV, Segmentation fault.
> rte_eth_xstats_get (port_id=3 '\003',
On Tue, 6 Dec 2016 07:17:20 +, Yang, Yi Y wrote:
> So my advice about this is we can push patch [2] to Linux net-next
> first, then apply patch series
> https://mail.openvswitch.org/pipermail/ovs-dev/2016-June/316879.html
> from Cascardo and apply [1], that can cover all the cases Pravin
>
On Mon, Dec 05, 2016 at 08:39:57PM -0500, Russell Bryant wrote:
> On Sun, Oct 23, 2016 at 1:50 PM, Ben Pfaff wrote:
>
> > Adding ovn-trace calls makes failures easier to understand and diagnose.
> >
> > Signed-off-by: Ben Pfaff
> > ---
> > tests/ovn.at | 96
I've been doing a bad job as Debian maintainer for Open vSwitch. I'd
very much appreciate help from a co-maintainer who can invest more time
in doing a good job with Debian uploads and bug tracking.
Thanks,
Ben.
___
dev mailing list
On Tue, Dec 06, 2016 at 12:04:00AM -0500, Hui Kang wrote:
> In some netbsd version, RTF_LLINFO is undefined
>
> Signed-off-by: Hui Kang
Applied to master, thanks!
Should I backport this to OVS 2.6?
(Are you the same Hui Kang already listed in AUTHORS.rst? Should we
On Tue, Dec 6, 2016 at 12:01 PM, Ben Pfaff wrote:
> On Tue, Dec 06, 2016 at 12:04:00AM -0500, Hui Kang wrote:
>> In some netbsd version, RTF_LLINFO is undefined
>>
>> Signed-off-by: Hui Kang
>
> Applied to master, thanks!
>
> Should I backport this to OVS
Alin,
Thanks for the review.
Yes, my main intent was to introduce something simple that we can iterate
on. I did look at the safe-string functions. Unfortunately, I couldn¹t
find Rtl* functions that I could reuse here. That¹s the reason why I had
to introduce the two str functions.
I will apply
En línea y en Vivo / Para todo su Equipo con una sola Conexión
Devolución de Impuestos Y Las Reacciones de las Autoridades fiscales
Cómo proteger sus derechos en la PRODECON
21 de diciembre - Online en Vivo - 10:00 a 13:00 y de 15:00 a 18:00 Hrs
Requerimientos EXCESIVOS, desistimientos
is used by the CONTAINER_OF macro. More specifically by the
offsetof macro -
https://msdn.microsoft.com/en-us/library/dz4y9b9a(v=vs.120).aspx
I will remove the NetProto.h include.
On 12/5/16, 8:11 AM, "Alin Serdean"
wrote:
>Looks good, but I have a question
On Tue, Dec 06, 2016 at 09:45:07AM +0300, Ilya Maximets wrote:
> The copy should be used here.
> Additionally, 'strlen' changed to the faster check.
>
> Fixes: 821b86649a90 ("netdev-dpdk: Don't try to unregister empty vhost_id.")
> Signed-off-by: Ilya Maximets
> ---
>
I posted Github PR #163 because the moderator for this list still hasn't
released my equivalent patch, which appeared to trigger some kind of
filter. Please review:
https://github.com/openvswitch/ovs/pull/163
Thanks,
Ben.
___
dev mailing list
Thanks for the clarification. I agree that there is a bug in 2.5.1. I
applied this patch to branch-2.5.
On Tue, Dec 06, 2016 at 05:38:31PM +, Ken Ajiro wrote:
>
> Ben,
>
> Sorry, it was my mistake. I met this issue with OVS 2.5.1.
> In OVS 2.5.1, packet_in scheduling is implemented
On Tue, Dec 06, 2016 at 12:11:04PM -0500, Hui Kang wrote:
> On Tue, Dec 6, 2016 at 12:01 PM, Ben Pfaff wrote:
> > On Tue, Dec 06, 2016 at 12:04:00AM -0500, Hui Kang wrote:
> >> In some netbsd version, RTF_LLINFO is undefined
> >>
> >> Signed-off-by: Hui Kang
I am unfortunately not very up-to-date with Mesos, but I reckon it is fair
to assume that in the current status N/S connectivity to containers should
be handled outside of mesos infrastructure, ie: OVN can define a topology
that enables external users to reach the LSPs in the LS specified in the
On Tue, Dec 06, 2016 at 03:51:03PM +0100, Jiri Benc wrote:
> On Tue, 6 Dec 2016 07:17:20 +, Yang, Yi Y wrote:
> > So my advice about this is we can push patch [2] to Linux net-next
> > first, then apply patch series
> > https://mail.openvswitch.org/pipermail/ovs-dev/2016-June/316879.html
> >
The commit message was a bit stale, so I sent a v12.
On 12/6/16, 9:06 PM, "ovs-dev-boun...@openvswitch.org on behalf of Darrell
Ball" wrote:
This patch adds datapaths of interest support where only datapaths of
local
"ovs-appctl ofproto/trace" is invaluable for debugging, but as the users of
Open vSwitch have evolved it has failed to keep up with the times. It's
pretty easy to design OpenFlow tables and pipelines that resubmit dozens of
times. Each resubmit causes an additional tab of indentation, so the
An upcoming commit will rewrite much of the ofproto/trace functionality.
As the mechanism behind this grows and evolves, it makes sense to move it
into its own file, especially since ofproto-dpif.c is too big anyway.
Signed-off-by: Ben Pfaff
---
ofproto/automake.mk | 2
This patch adds datapaths of interest support where only datapaths of
local interest are monitored by the ovn-controller ovsdb client. The
idea is to do a flood fill in ovn-controller of datapath associations
calculated by northd. A new column is added to the SB database
datapath_binding table -
the do_handshake() function throws the exception OpenSSL.SSL.SysCallError when
the peer's SSL connection is closed,
And the recv() function also throws the exception OpenSSL.SSL.SysCallError when
the peer's SSL connection is abnormally closed,
we need to catch the exception and return error's
On Wed, Dec 07, 2016 at 02:07:05PM +0800, Guoshuai Li wrote:
> Signed-off-by: Guoshuai Li
> ---
> IntegrationGuide.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/IntegrationGuide.rst b/IntegrationGuide.rst
> index d34e9ed..11e77ba 100644
> ---
Until now, ofproto-dpif.c has hidden the definitions of several structures,
such as struct ofproto_dpif and struct rule_dpif. This kind of information
hiding is often beneficial, because it forces code outside the file with
the definition to use the documented interfaces. However, in this case
The 'flow_cookie' member of struct rule is read during flow translation
without holding a mutex, breaking the documented OVS_GUARDED access rule.
However, the flow_cookie member is actually never modified after a rule is
initialized, so this commit changes the access rules to reflect that.
On Fri, Dec 2, 2016 at 11:38 PM, Guoshuai Li wrote:
> PCS man page says role=Stopped/Started/Master/Slave.
> A role can be master or slave (if no role is specified, it defaults to
> 'started').
>
> Command line "$pcs constraint colocation add ovndb_servers-master with
> master
On 12/6/16, 7:08 PM, "ovs-dev-boun...@openvswitch.org on behalf of Russell
Bryant" wrote:
On Tue, Dec 6, 2016 at 3:35 PM, Ben Pfaff wrote:
> On Tue, Dec 06, 2016 at 02:45:19PM -0500, Russell Bryant
On Mon, Dec 5, 2016 at 2:38 PM, Ben Pfaff wrote:
> GCC 6.1 warns that -Wformat-security has no effect without -Wformat, so
> this commit fixes the problem.
>
> The change to _OVS_CHECK_CC_OPTION is needed so that the cache variable
> name doesn't end up with a space in it, which
On 12/4/16, 9:48 PM, "ovs-dev-boun...@openvswitch.org on behalf of Mickey
Spiegel"
wrote:
On Sun, Dec 4, 2016 at 4:13 PM, Darrell Ball wrote:
> This patch adds datapaths of interest support
This patch adds datapaths of interest support where only datapaths of
local interest are monitored by the ovn-controller ovsdb client. The
idea is to do a flood fill in ovn-controller of datapath associations
calculated by northd. A new column is added to the SB database
datapath_binding table -
Based on the description, the amended constraint looks correct.
On Tue, Dec 6, 2016 at 11:49 PM, Babu Shanmugam wrote:
> + Andrew.
>
> Hi Andrew,
>
> Please see below. Your comments are most welcome.
>
> --
>
> Babu
>
> On Saturday 03 December 2016 01:08 PM, Guoshuai Li
Sounds good :).
> -Original Message-
> From: Sairam Venugopal [mailto:vsai...@vmware.com]
> Sent: Tuesday, December 6, 2016 8:53 PM
> To: Alin Serdean ;
> d...@openvswitch.org
> Subject: Re: [ovs-dev] [PATCH 3/4] datapath-windows: Conntrack - Introduce
>
On Tue, Dec 06, 2016 at 03:51:03PM +0100, Jiri Benc wrote:
> On Tue, 6 Dec 2016 07:17:20 +, Yang, Yi Y wrote:
> > So my advice about this is we can push patch [2] to Linux net-next
> > first, then apply patch series
> > https://mail.openvswitch.org/pipermail/ovs-dev/2016-June/316879.html
> >
En línea y en Vivo / Para todo su Equipo con una sola Conexión
Devolución de Impuestos Y Las Reacciones de las Autoridades fiscales
Cómo proteger sus derechos en la PRODECON
21 de diciembre - Online en Vivo - 10:00 a 13:00 y de 15:00 a 18:00 Hrs
Requerimientos EXCESIVOS,
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
---
On Mon, Dec 5, 2016 at 2:38 PM, Ben Pfaff wrote:
> Signed-off-by: Ben Pfaff
> ---
>
Acked-by: Andy Zhou
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On Mon, Dec 5, 2016 at 2:38 PM, Ben Pfaff wrote:
> -Wnull-dereference is also new in GCC 6 but upon testing it produces far
> too many false positives to be usable for OVS.
>
> Signed-off-by: Ben Pfaff
>
Acked-by: Andy Zhou
On Tue, Dec 06, 2016 at 07:55:36AM -0800, Guru Shetty wrote:
> On 5 December 2016 at 13:59, Ben Pfaff wrote:
>
> > Guru, did you ever have a look at this? It's always nice to integrate
> > with more systems.
> >
>
> Right. I did work with Nimay to get this patch sent to the
On Tue, Dec 06, 2016 at 02:48:43PM +0300, Ilya Maximets wrote:
> HRW algorithm uses hash function which is dependent from the build
> environment. This leads to constant fails of the testsuite
> with CFLAGS='-march=native' if CPU supports hashing instructions:
Thanks, applied to master.
On Tue, Dec 06, 2016 at 07:36:09PM +0800, Guoshuai Li wrote:
> In 341 line already have this code
Thanks, applied to master.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
On Mon, Dec 05, 2016 at 06:30:17PM -0800, Mickey Spiegel wrote:
> On Sun, Dec 4, 2016 at 11:17 PM, Ben Pfaff wrote:
>
> > This will have its first real user in an upcoming commit.
> >
>
> I am not sure this is really necessary. See my comments on patch 6.
OK, I'll read those
Unified code for the threading part sounds good.
Can you please add an item to https://github.com/openvswitch/ovs-issues/issues
so we can tackle it later on?
I'm okay with leaving the init/cleanup as it is for the moment.
Thanks,
Alin.
> -Original Message-
> From: Sairam Venugopal
On Tue, Dec 06, 2016 at 11:30:36PM +, Alin Serdean wrote:
> 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
On 06/12/2016 04:48, "Ilya Maximets" wrote:
>On 03.12.2016 05:14, Daniele Di Proietto wrote:
>> +/* Assign pmds to queues. If 'pinned' is true, assign pmds to pinned
>> + * queues and marks the pmds as isolated. Otherwise, assign non isolated
>> + * pmds to
On Tue, Dec 06, 2016 at 09:38:09AM -0500, Eric Garver wrote:
> On Tue, Dec 06, 2016 at 07:17:20AM +, Yang, Yi Y wrote:
> > Hi, guys
>
> Hi Yi,
>
> > This patch isn't updated from June on, Cascardo said he/Eric is still
> > working on this, but six months passed, we don't see any following
>
On Mon, Dec 5, 2016 at 2:22 AM, Ben Pfaff wrote:
> On Sun, Dec 04, 2016 at 04:13:44PM -0800, Darrell Ball wrote:
> > This patch adds datapaths of interest support where only datapaths of
> > local interest are monitored by the ovn-controller ovsdb client. The
> > idea is to do a
On Tue, Dec 06, 2016 at 01:01:22AM -0800, nickcooper-zhangtonghao wrote:
> Signed-off-by: nickcooper-zhangtonghao
Applied to master, thanks!
___
dev mailing list
d...@openvswitch.org
The usage should not repeat the command name, but most of the dpctl
commands' usage did repeat it. This fixes the problem.
Signed-off-by: Ben Pfaff
---
lib/dpctl.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/lib/dpctl.c b/lib/dpctl.c
On Tue, Dec 06, 2016 at 02:45:19PM -0500, Russell Bryant wrote:
> On Mon, Dec 5, 2016 at 2:22 AM, Ben Pfaff wrote:
>
> > On Sun, Dec 04, 2016 at 04:13:44PM -0800, Darrell Ball wrote:
> > > This patch adds datapaths of interest support where only datapaths of
> > > local interest
On Tue, Dec 06, 2016 at 12:35:16PM -0800, Ben Pfaff wrote:
> On Tue, Dec 06, 2016 at 02:45:19PM -0500, Russell Bryant wrote:
> > On Mon, Dec 5, 2016 at 2:22 AM, Ben Pfaff wrote:
> >
> > > On Sun, Dec 04, 2016 at 04:13:44PM -0800, Darrell Ball wrote:
> > > > This patch adds
On Tue, Dec 06, 2016 at 03:23:45PM -0500, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > The usage should not repeat the command name, but most of the dpctl
> > commands' usage did repeat it. This fixes the problem.
> >
> > Signed-off-by: Ben Pfaff
> > ---
> >
On Tue, Dec 06, 2016 at 01:01:21AM -0800, nickcooper-zhangtonghao wrote:
> Signed-off-by: nickcooper-zhangtonghao
Applied to master, thanks!
___
dev mailing list
d...@openvswitch.org
On Tue, Dec 06, 2016 at 01:01:20AM -0800, nickcooper-zhangtonghao wrote:
> The ovs-sandbox runs in the "dummy mode" by default.
> In this mode of testing, no packets travel across
> physical or virtual networks. But sometimes, we may
> create veth network devices and add them to ovs bridge
> for
Ben Pfaff writes:
> The usage should not repeat the command name, but most of the dpctl
> commands' usage did repeat it. This fixes the problem.
>
> Signed-off-by: Ben Pfaff
> ---
> lib/dpctl.c | 20 ++--
> 1 file changed, 10 insertions(+), 10
On Thu, Dec 1, 2016 at 10:37 AM, Lance Richardson
wrote:
> This series adds support for SSL connections to the northbound
> and southbound OVN database servers and removes the previous
> default TCP connection type.
>
> Lance Richardson (3):
> ovn-nb: remote connection
On Tue, Dec 06, 2016 at 01:01:19AM -0800, nickcooper-zhangtonghao wrote:
> When the datapath, whose type is "netdev", processes packets
> in userspce action, it may cause a segmentation fault. In the
> dp_execute_userspace_action(), we pass the "wc" argument to
> dp_netdev_upcall() using NULL. In
On 12/6/16, 2:03 PM, "ovs-dev-boun...@openvswitch.org on behalf of Russell
Bryant" wrote:
On Sun, Dec 4, 2016 at 7:13 PM, Darrell Ball wrote:
> This patch adds datapaths of interest support where
On Tue, Dec 6, 2016 at 4:35 AM, Guoshuai Li wrote:
> OVSDB resource is multi-state resource, pacemaker monitor actions are
> insufficient to monitor a multi-state resource.
> the usual one will cover the slave role, so it is need to configure
> monitoring for the master.
>
>
Regards
_Sugesh
> -Original Message-
> From: Ben Pfaff [mailto:b...@ovn.org]
> Sent: Monday, December 5, 2016 7:44 PM
> To: Chandran, Sugesh
> Cc: d...@openvswitch.org; ro...@mellanox.com; ktray...@redhat.com;
> fla...@flaviof.com
> Subject: Re: [ovs-dev]
When the datapath, whose type is "netdev", processes packets
in userspce action, it may cause a segmentation fault. In the
dp_execute_userspace_action(), we pass the "wc" argument to
dp_netdev_upcall() using NULL. In the dp_netdev_upcall() call tree,
the "wc" will be used. For example,
Signed-off-by: nickcooper-zhangtonghao
---
ofproto/ofproto-dpif-upcall.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c
index 84f1de2..6cb9c2e 100644
--- a/ofproto/ofproto-dpif-upcall.c
The ovs-sandbox runs in the "dummy mode" by default.
In this mode of testing, no packets travel across
physical or virtual networks. But sometimes, we may
create veth network devices and add them to ovs bridge
for developing and testing. It's necessary to add an option.
Signed-off-by:
Processing commands for cont...@bugs.debian.org:
> user debian...@lists.debian.org
Setting user to debian...@lists.debian.org (was a...@debian.org).
> usertags 831924 piuparts
Usertags were: qa-ftbfs-20160720 qa-indep qa-ftbfs.
Usertags are now: qa-ftbfs-20160720 qa-indep piuparts qa-ftbfs.
>
+ Andrew.
Hi Andrew,
Please see below. Your comments are most welcome.
--
Babu
On Saturday 03 December 2016 01:08 PM, Guoshuai Li wrote:
PCS man page says role=Stopped/Started/Master/Slave.
A role can be master or slave (if no role is specified, it defaults to
'started').
Command line
On 03.12.2016 05:14, Daniele Di Proietto wrote:
> +/* Assign pmds to queues. If 'pinned' is true, assign pmds to pinned
> + * queues and marks the pmds as isolated. Otherwise, assign non isolated
> + * pmds to unpinned queues.
> + *
> + * The function doesn't touch the pmd threads, it just
In 341 line already have this code
---
ovn/utilities/ovndb-servers.ocf | 1 -
1 file changed, 1 deletion(-)
diff --git a/ovn/utilities/ovndb-servers.ocf b/ovn/utilities/ovndb-servers.ocf
index 72c0cf4..1cf6f20 100755
--- a/ovn/utilities/ovndb-servers.ocf
+++ b/ovn/utilities/ovndb-servers.ocf
@@
HRW algorithm uses hash function which is dependent from the build
environment. This leads to constant fails of the testsuite
with CFLAGS='-march=native' if CPU supports hashing instructions:
[---]
./bundle.at:233: ovs-appctl
On Sun, Dec 4, 2016 at 7:13 PM, Darrell Ball wrote:
> This patch adds datapaths of interest support where only datapaths of
> local interest are monitored by the ovn-controller ovsdb client. The
> idea is to do a flood fill in ovn-controller of datapath associations
>
On Tue, Dec 6, 2016 at 3:36 PM, Ben Pfaff wrote:
> On Tue, Dec 06, 2016 at 12:35:16PM -0800, Ben Pfaff wrote:
> > On Tue, Dec 06, 2016 at 02:45:19PM -0500, Russell Bryant wrote:
> > > On Mon, Dec 5, 2016 at 2:22 AM, Ben Pfaff wrote:
> > >
> > > > On Sun, Dec 04, 2016
On Mon, Dec 5, 2016 at 2:38 PM, Ben Pfaff wrote:
> AC_LANG_PROGRAM(,) uses a program like this:
>
> int main() { return 0; }
>
> but that triggers warnings for -Wstrict-prototypes and for
> -Wold-style-definition, since this definition of main() lacks a prototype
> and is
70 matches
Mail list logo