Currently we have "localnet" in OVN to connect VIF to physical network, but
there seems to be no way to support intra-hypervisor communication with
exiting model mentioned in ovn-sb.xml:
localnet
A connection to a locally accessible network from each
ov
Hi OVS & OVN developers,
OVN is a SDN solution, in some case, customer need to bridge or routing
between a SDN network and a non-SDN network.
As I know, OVN has support east-west distribute router, and OVS has support
SNAT functionality.
How to connect OVN network to non-OVN network?
1. Will us
On Tue, Jan 12, 2016 at 9:25 PM, Alexandru Ardelean
wrote:
> On Tue, Jan 12, 2016 at 6:23 PM, Ben Pfaff wrote:
>
>> On Tue, Jan 12, 2016 at 09:26:35AM +0200, Alexandru Ardelean wrote:
>> > But as it turns out, OVS is getting bigger and bigger with each release,
>> > which means fewer and fewer d
+Adding Michael
Regards
Srinivasan
On 1/12/16, 4:16 PM, "dev on behalf of Srivatsan, Srinivasan"
wrote:
>Hi,
>
>I am Srinivasan and am trying to add Network Time Protocol support for
>OpenSwitch. Openswitch uses OVSDB as the central db for status/configuration
>and inter module communica
Hi,
I am Srinivasan and am trying to add Network Time Protocol support for
OpenSwitch. Openswitch uses OVSDB as the central db for status/configuration
and inter module communication. From the NTP point of view, we are trying to
define how can modulate NTP to accommodate into the ovsdb schema d
30+ Day Old Patches
===
ID State Date Name
-- -
548008 New2015-11-24 11:19:01 [ovs-dev,v4,2/4] ovn: New flows for DHCP
tranffic
548010 New2015-11-24 11:19:02 [ovs-dev,v4,3/4] ovn: Process dhcp
packet-ins and
On 12 January 2016 at 05:52, Chandran, Sugesh wrote:
> Hi,
> We found that there is a serious performance drop in OVS userspace-datapath
> after the commit "tunneling: add IPv6 support to netdev_tunnel_config(commit
> ID :- 3ae91c019019889fbe93aa7dfd6e3747da362b24)" in the OVS tree. The
> PHY-
On 13.01.2016 00:39, Hannes Frederic Sowa wrote:
+ if (__this_cpu_inc_return(ovs_recursion) > ovs_recursion_limit) {
+ net_crit_ratelimited("ovs: recursion limit reached on datapath %s,
probable configuration error\n",
+ovs_dp_name(vport->d
It was seen that defective configurations of openvswitch could overwrite
the STACK_END_MAGIC and cause a hard crash of the kernel because of too
many recursions within ovs.
This problem arises due to the high stack usage of openvswitch. The rest
of the kernel is fine with the current limit of 10 (
Hi Janusz,
[auto build test ERROR on net-next/master]
[also build test ERROR on v4.4 next-20160112]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Janusz-Wolak/Openvswitch-datapath-c-Fixed
Hi Janusz,
[auto build test ERROR on net-next/master]
[also build test ERROR on v4.4 next-20160112]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Janusz-Wolak/Openvswitch-datapath-c-Fixed
On Tue, Jan 12, 2016 at 01:48:44PM +, Sugesh Chandran wrote:
> Adding a new field called protocol in flow tunnel structure to verify the
> validity
> of tunnel metadata. This field avoids the need of resetting and validating the
> entire ipv4/ipv6 tunnel destination address which caused a seri
This patch forces appending "parent_process_name(PID)" when invoking
ovs-vsctl, in order to assist debugging. The patch is for Linux only.
For example:
User adds br0 by "ovs-vsctl add-br0", the log shows:
"ovs-vsctl (invoked by base(1528)): ovs-vsctl add-br br0"
Signed-off-by: William Tu
On Tue, Jan 12, 2016 at 1:55 PM, Janusz Wolak wrote:
> Removed: block comments trailing without separate line,
> missing blanks after decarations, comparasion to NULL, assignment in if
> condition, not necessary spaces after a casts, multiple assignments, line
> over 80 characters, alignment not m
It is not appropriate to submit net-next changes at this time.
If you wish to contribute to the networking kernel code, you should
read the netdev mailing lists, and read important announcements such
as this:
http://marc.info/?l=linux-netdev&m=145248145925834&w=2
THanks.
___
Fixed a coding style issue.
Signed-off-by: Janusz Wolak
---
net/openvswitch/actions.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/net/openvswitch/actions.c b/net/openvswitch/actions.c
index c88d0f2..28bc395 100644
--- a/net/openvswitch/actions.c
+++ b/net/openvswitc
Removed: block comments trailing without separate line,
missing blanks after decarations, comparasion to NULL, assignment in if
condition, not necessary spaces after a casts, multiple assignments, line
over 80 characters, alignment not matched to open parenthesis, kmalloc with
multiplying instead k
Fixed a coding style issue.
Signed-off-by: Janusz Wolak
---
net/openvswitch/conntrack.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/net/openvswitch/conntrack.c b/net/openvswitch/conntrack.c
index e004067..fab7b55 100644
--- a/net/openvswitch/conntrack.c
+++ b/net/openvswitch/conntrack.c
@
On Tue, Jan 12, 2016 at 12:37 PM, Ben Pfaff wrote:
> On Tue, Jan 12, 2016 at 12:34:58PM -0800, Andy Zhou wrote:
> > Remove unused implementation of ovsdb_idl_row_apply_diff().
> >
> > Reported-by: Ben Pfaff
> > Signed-off-by: Andy Zhou
>
> Thanks!
>
> Acked-by: Ben Pfaff
>
Thanks, pushed to m
On 12 January 2016 at 12:39, Flavio Leitner wrote:
> On Tue, 12 Jan 2016 11:32:41 -0800
> Joe Stringer wrote:
>
>> DPDK build was broken after commit 2f8932e8403a ("poll: Suppress
>> logging for pmd threads.") due to the following error:
>>
>> lib/netdev-dpdk.c:245:13: error: static declaration o
On Tue, Jan 12, 2016 at 11:21 AM, Ben Pfaff wrote:
> There's an entire function that's #if 0'd out in ovsdb-idl.c. Why?
>
Sorry, this should have been deleted. I sent out a patch that removes this
unused function.
https://patchwork.ozlabs.org/patch/566735
__
On Tue, Jan 12, 2016 at 11:06 AM, Joe Stringer wrote:
> On 11 January 2016 at 20:13, Pravin B Shelar wrote:
>> STT unregisters nf-hook when there are no other STT devices
>> left in the namespace. On some kernel versions the nf-unreg API
>> take RTNL lock, but it is already taken in the tunnel de
On Tue, 12 Jan 2016 11:32:41 -0800
Joe Stringer wrote:
> DPDK build was broken after commit 2f8932e8403a ("poll: Suppress
> logging for pmd threads.") due to the following error:
>
> lib/netdev-dpdk.c:245:13: error: static declaration of ‘thread_is_pmd’
> follows non-static declaration
> lib/ovs
On Tue, Jan 12, 2016 at 12:34:58PM -0800, Andy Zhou wrote:
> Remove unused implementation of ovsdb_idl_row_apply_diff().
>
> Reported-by: Ben Pfaff
> Signed-off-by: Andy Zhou
Thanks!
Acked-by: Ben Pfaff
___
dev mailing list
dev@openvswitch.org
http:
Remove unused implementation of ovsdb_idl_row_apply_diff().
Reported-by: Ben Pfaff
Signed-off-by: Andy Zhou
---
lib/ovsdb-idl.c | 57 -
1 file changed, 57 deletions(-)
diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
index a3dfcb9..7e84138
DPDK build was broken after commit 2f8932e8403a ("poll: Suppress logging
for pmd threads.") due to the following error:
lib/netdev-dpdk.c:245:13: error: static declaration of ‘thread_is_pmd’
follows non-static declaration
lib/ovs-thread.h:526:6: note: previous declaration of ‘thread_is_pmd’
was he
"Traynor, Kevin" writes:
>> -Original Message-
>> From: Aaron Conole [mailto:acon...@redhat.com]
>> Sent: Monday, January 4, 2016 9:47 PM
>> To: dev@openvswitch.org; Flavio Leitner; Traynor, Kevin
>> Subject: [PATCH v2 3/3] netdev-dpdk: Autofill lcore coremask if absent
>>
>> The user has
Octal constants must be written as 0o077 instead of 0077 to be
compatible with both Python 2 and 3.
Signed-off-by: Russell Bryant
---
Makefile.am | 3 ++-
debian/ovs-monitor-ipsec | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index
Python 3 removed support for "old-style classes". Classes should always
inherit from object to get consistent behavior between Python 2 and 3.
Enable a flake8 warning to help prevent regressions in the future.
Signed-off-by: Russell Bryant
---
Makefile.am | 3 ++-
debian/ovs-monit
In Python 2, dict.items(), dict.keys(), and dict.values() returned a
list. dict.iteritems(), dict.iterkeys(), and dict.itervalues() returned
an iterator.
As of Python 3, dict.iteritems(), dict.itervalues(), and dict.iterkeys()
are gone. items(), keys(), and values() now return an iterator.
In t
Fix imports of xmlrpclib to be compatible with Python 3. Python 2 had
xmlrpclib (client) and SimpleXMLRPCServer (server). In Python 3, these
have been renamed to xmlrpc.client and xmlrpc.server.
The solution implemented here is to use the six library. It may seem
excessive for this particular i
Python 2 had range() and xrange(). xrange() is more efficient, but
behaves differently so range() was retained for compatibility. Python 3
only has range() and it behaves like Python 2's xrange().
Remove explicit use of xrange() and use six.moves.range() to
make sure we're using xrange() from Py
v1->v2:
- Applied first 4 patches.
- Fix conflicts with master.
- Add python-six dependency to Debian and RHEL/Fedora packaging.
- Add Python/python-six dependency to Installation Requirements
section of INSTALL.md.
I got the ovs Python code ported to Python 3 in a branch. I've been submit
This patch add caller's filename and line number when calling ovs-vsctl.
For example, a script "vsctl.sh" adding br0 at line 9 shows:
ovs-vsctl add-br br0 -- comment (FILE:./vsctl.sh,LINE:9)
Signed-off-by: William Tu
---
debian/ifupdown.sh | 9 -
ovn/utilities/o
On Tue, Jan 12, 2016 at 11:28:51AM -0800, William Tu wrote:
> This patch forces appending "-- comment parent_process_name(PID)"
> when invoking ovs-vsctl, in order to assist debugging.
> For example:
> User adds br0 by "ovs-vsctl add-br0", the log shows:
> "ovs-vsctl: ovs-vsctl add-br br0 -
On 12 January 2016 at 05:12, Flavio Leitner wrote:
> On Tue, 12 Jan 2016 09:23:12 +0300
> Ilya Maximets wrote:
>
>> On 12.01.2016 07:13, Ben Pfaff wrote:
>> > On Mon, Jan 11, 2016 at 06:24:10PM -0800, Joe Stringer wrote:
>> >> DPDK build was broken after commit 2f8932e8403a ("poll: Suppress
>> >>
This patch forces appending "-- comment parent_process_name(PID)"
when invoking ovs-vsctl, in order to assist debugging.
For example:
User adds br0 by "ovs-vsctl add-br0", the log shows:
"ovs-vsctl: ovs-vsctl add-br br0 -- comment bash(1528)"
Signed-off-by: William Tu
---
utilities/ovs-v
On Tue, Jan 12, 2016 at 6:23 PM, Ben Pfaff wrote:
> On Tue, Jan 12, 2016 at 09:26:35AM +0200, Alexandru Ardelean wrote:
> > But as it turns out, OVS is getting bigger and bigger with each release,
> > which means fewer and fewer devices can support it.
>
> When I run "strip" on the version from m
On Thu, Dec 10, 2015 at 04:58:46PM +, Ansari, Shad wrote:
> (Resending patch with corrected Signed-off-by email address)
>
> Recent IDL change tracking patches allow quick traversal of changed
> rows. This patch adds additional support to track changed columns.
> It allows an IDL client to eff
There's an entire function that's #if 0'd out in ovsdb-idl.c. Why?
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Ben Pfaff writes:
> On Tue, Jan 12, 2016 at 11:36:09AM -0500, Aaron Conole wrote:
>> Ben Pfaff writes:
>> > This avoids a segmentation fault in case of memory allocation failure.
>> >
>> > Found by inspection.
>> >
>> > Signed-off-by: Ben Pfaff
>> > ---
>>
>> LGTM
>>
>> Acked-by: Aaron Conole
On 12 January 2016 at 11:04, Ben Pfaff wrote:
> On Mon, Jan 11, 2016 at 04:43:52PM -0800, Joe Stringer wrote:
>> This extraneous .RE caused the indentation for the subsequent actions to
>> drop back an extra step, fix it.
>>
>> Fixes: 8e53fe8cf7a1 ("Add connection tracking mark support.")
>> Signe
On 01/12/2016 01:55 PM, Ben Pfaff wrote:
> On Tue, Jan 12, 2016 at 10:47:48AM -0800, Joe Stringer wrote:
>> On 12 January 2016 at 10:39, Ben Pfaff wrote:
>>> When the source directory and build directory differ, xml2nroff needs
>>> to pull include files from the source directory, but it was blindl
On 11 January 2016 at 20:13, Pravin B Shelar wrote:
> STT unregisters nf-hook when there are no other STT devices
> left in the namespace. On some kernel versions the nf-unreg API
> take RTNL lock, but it is already taken in the tunnel device
> destroy code path which results in deadlock. To fix t
On Mon, Jan 11, 2016 at 04:43:52PM -0800, Joe Stringer wrote:
> This extraneous .RE caused the indentation for the subsequent actions to
> drop back an extra step, fix it.
>
> Fixes: 8e53fe8cf7a1 ("Add connection tracking mark support.")
> Signed-off-by: Joe Stringer
Acked-by: Ben Pfaff
___
On Tue, Jan 05, 2016 at 04:45:12PM +0530, saloni.jai...@gmail.com wrote:
> From: Saloni Jain
>
> On change in a table state, the controller needs to be informed with
> the OFPT_TABLE_STATUS message. The message is sent with reason
> OFPTR_VACANCY_DOWN or OFPTR_VACANCY_UP in case of change in rema
On Tue, Jan 12, 2016 at 10:47:48AM -0800, Joe Stringer wrote:
> On 12 January 2016 at 10:39, Ben Pfaff wrote:
> > When the source directory and build directory differ, xml2nroff needs
> > to pull include files from the source directory, but it was blindly
> > using the current working directory (t
On 12 January 2016 at 10:39, Ben Pfaff wrote:
> When the source directory and build directory differ, xml2nroff needs
> to pull include files from the source directory, but it was blindly
> using the current working directory (the build directory) instead.
>
> Signed-off-by: Ben Pfaff
> Fixes: 7b
When the source directory and build directory differ, xml2nroff needs
to pull include files from the source directory, but it was blindly
using the current working directory (the build directory) instead.
Signed-off-by: Ben Pfaff
Fixes: 7ba0c32f610 ("ovn-nbctl: add db commands help and manpage")
On Tue, Jan 12, 2016 at 10:12:05AM +0200, Liran Schour wrote:
> Andy Zhou wrote on 11/01/2016 10:57:30 PM:
> >
> > On Mon, Jan 11, 2016 at 5:05 AM, Liran Schour wrote:
> > Andy Zhou wrote on 09/01/2016 02:02:32 AM:
> > >
> > > On Fri, Jan 8, 2016 at 8:58 AM, Liran Schour
> wrote:
> > > Andy
On Tue, Jan 12, 2016 at 11:36:09AM -0500, Aaron Conole wrote:
> Ben Pfaff writes:
> > This avoids a segmentation fault in case of memory allocation failure.
> >
> > Found by inspection.
> >
> > Signed-off-by: Ben Pfaff
> > ---
>
> LGTM
>
> Acked-by: Aaron Conole
Thanks, I applied this to mast
Testing under 2012 gave some more inisight on an old bug.
If a PNP event with the value of NetEventSwitchActivate was triggered
we were calling OvsQuerySwitchActivationComplete which does an OID request
to the underlying drivers, however this triggered a hang because as per
documentation:
https://
On 01/11/2016 08:39 PM, Ben Pfaff wrote:
> On Wed, Jan 06, 2016 at 03:50:25PM -0500, Russell Bryant wrote:
>> Fix imports of xmlrpclib to be compatible with Python 3. Python 2 had
>> xmlrpclib (client) and SimpleXMLRPCServer (server). In Python 3, these
>> have been renamed to xmlrpc.client and x
On 01/11/2016 09:09 PM, Ben Pfaff wrote:
> On Wed, Jan 06, 2016 at 03:50:26PM -0500, Russell Bryant wrote:
>> Python 2 had range() and xrange(). xrange() is more efficient, but
>> behaves differently so range() was retained for compatibility. Python 3
>> only has range() and it behaves like Pytho
Dear Sir/Madam,
This is Mr. Walter from Global Star Trading Co.Ltd.we are doing Net market
research.
We are interested in purchasing your products and we sincerely hope to
establish a long-term business relation with your esteemed company.
Please kindly send me your latest catalog. Also, inform
Ben Pfaff writes:
> This avoids a segmentation fault in case of memory allocation failure.
>
> Found by inspection.
>
> Signed-off-by: Ben Pfaff
> ---
LGTM
Acked-by: Aaron Conole
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman
On Tue, Jan 12, 2016 at 02:15:39PM +0300, Ilya Maximets wrote:
> On arm/arm64:
> gcc: error: unrecognized command line option '-mssse3'
>
> Signed-off-by: Ilya Maximets
Applied, thanks!
___
dev mailing list
dev@openvswitch.org
http://openvswitch.
On Tue, Jan 12, 2016 at 09:26:35AM +0200, Alexandru Ardelean wrote:
> But as it turns out, OVS is getting bigger and bigger with each release,
> which means fewer and fewer devices can support it.
When I run "strip" on the version from master, the binary is about 2 MB
(on i386). For version 2.0.0
There is no need to set the mask on the outer header IP TTL [1]. The only
requirement
is that the TTL will be non-zero. Clear the mask in tnl_wc_init().
[1] OVS VXLAN decap rule has full match on TTL for the outer headers?
http://www.spinics.net/lists/netdev/msg351961.html
Cc: Jesse Gross
C
It is possible to pass some fields to the kernel with a zero mask, but
ovs-dpctl doesn't currently allow it. Change the code to allow it to
mimic what vswitchd is allowed to do.
Signed-off-by: Haggai Eran
---
lib/odp-util.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --g
These two patches allow passing tunnelled flows without a match on the
outer header TTL to the datapath. The first patch simply modifies ovs-dpctl
to pass along fields that don't have a mask set. The second patch changes
the behavior of ofproto to clear the mask specifically on the TTL.
Regards,
H
> -Original Message-
> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Aaron Conole
> Sent: Monday, January 11, 2016 6:51 PM
> To: Zoltan Kiss
> Cc: dev@openvswitch.org; Flavio Leitner
> Subject: Re: [ovs-dev] [PATCH v2 2/3] netdev-dpdk: Convert initialization
> from cmdline to
"Traynor, Kevin" writes:
>> -Original Message-
>> From: Aaron Conole [mailto:acon...@redhat.com]
>> Sent: Monday, January 4, 2016 9:47 PM
>> To: dev@openvswitch.org; Flavio Leitner; Traynor, Kevin
>> Subject: [PATCH v2 2/3] netdev-dpdk: Convert initialization from cmdline to
>> db
>>
>> E
> -Original Message-
> From: Aaron Conole [mailto:acon...@redhat.com]
> Sent: Monday, January 4, 2016 9:47 PM
> To: dev@openvswitch.org; Flavio Leitner; Traynor, Kevin
> Subject: [PATCH v2 3/3] netdev-dpdk: Autofill lcore coremask if absent
>
> The user has control over the DPDK internal
Hi,
We found that there is a serious performance drop in OVS userspace-datapath
after the commit "tunneling: add IPv6 support to netdev_tunnel_config(commit ID
:- 3ae91c019019889fbe93aa7dfd6e3747da362b24)" in the OVS tree. The PHY-PHY
throughput dropped almost 25% for 64 byte packets due to thi
Adding a new field called protocol in flow tunnel structure to verify the
validity
of tunnel metadata. This field avoids the need of resetting and validating the
entire ipv4/ipv6 tunnel destination address which caused a serious performance
drop.
Signed-off-by: Sugesh Chandran
---
lib/flow.c
> -Original Message-
> From: Aaron Conole [mailto:acon...@redhat.com]
> Sent: Monday, January 4, 2016 9:47 PM
> To: dev@openvswitch.org; Flavio Leitner; Traynor, Kevin
> Subject: [PATCH v2 2/3] netdev-dpdk: Convert initialization from cmdline to
> db
>
> Existing DPDK integration is provid
On Tue, 12 Jan 2016 09:23:12 +0300
Ilya Maximets wrote:
> On 12.01.2016 07:13, Ben Pfaff wrote:
> > On Mon, Jan 11, 2016 at 06:24:10PM -0800, Joe Stringer wrote:
> >> DPDK build was broken after commit 2f8932e8403a ("poll: Suppress
> >> logging for pmd threads.") due to the following error:
> >>
On arm/arm64:
gcc: error: unrecognized command line option '-mssse3'
Signed-off-by: Ilya Maximets
---
acinclude.m4 | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/acinclude.m4 b/acinclude.m4
index 9d652c2..6771f48 100644
--- a/acinclude.m4
+++ b/acinclude.m4
@@ -216
Dear Sir/Madam,
Good day to you.We are an imports and exports company located in Xi'an Shaanxi
.China ,Due to business needs,we are looking for casting manufacturer.Through
the Internet ,we know that your company are this kind of enterprise with actual
strength. So if you have any interest to w
Thanks Zoltan for your questions.
Anyone else has any new feedback?
> -Original Message-
> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Fischetti,
> Antonio
> Sent: Friday, December 18, 2015 3:44 PM
> To: Zoltan Kiss; dev@openvswitch.org
> Subject: Re: [ovs-dev] Wildcard Ma
Andy Zhou wrote on 11/01/2016 10:57:30 PM:
>
> On Mon, Jan 11, 2016 at 5:05 AM, Liran Schour wrote:
> Andy Zhou wrote on 09/01/2016 02:02:32 AM:
> >
> > On Fri, Jan 8, 2016 at 8:58 AM, Liran Schour
wrote:
> > Andy Zhou wrote on 07/01/2016 01:04:17 AM:
> > >
> > > I have some comments on the
72 matches
Mail list logo
