From 1129ecfd70d64f95ac63ca4e91c3d2c5d4782b6e Mon Sep 17 00:00:00 2001
From: Gal Sagie
Date: Tue, 21 Jul 2015 09:11:12 +0300
Subject: [PATCH] [ovn] Add 'name' field to logical router port NB schema
Signed-off-by: Gal Sagie
---
ovn/ovn-nb.ovsschema | 1 +
ovn/ovn-nb.xml | 9 +
2 fi
Hi,
I am using openvswitch-2.0.0 ovs.
1. Have Switch connected to OF controller.
2. Node is connected.
3. Run iperf command from node to trigger a lot of packet-in.
"iperf -c -t 1 -u -b 1000MB -l 466" where has static arp
configured.
4. After >1hr. Switch disconnect from OF controller.
5. Che
Add autogenerated files to the CLEANFILES so that they are not kept around
after clean/distclean.
Signed-off-by: Aaron Conole
---
vtep/automake.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/vtep/automake.mk b/vtep/automake.mk
index c712b0c..f44752d 100644
--- a/vtep/au
Eliminate the $(srcdir) requirement on vswitch-idl.ovsidl
Also, add autogenerated files to the CLEANFILES so that they are not kept
around after clean/distclean.
Signed-off-by: Aaron Conole
---
lib/automake.mk | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/lib/automak
Add autogenerated files to the CLEANFILES so that they are not kept around
after clean/distclean
Signed-off-by: Aaron Conole
---
tests/automake.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/automake.mk b/tests/automake.mk
index 8b54764..1fdb71f 100644
--- a/tests/automake.mk
+++
For some time, it seems that the distcheck make target has not functioned
properly. This patch works around the automake imposed distcheck read-only
directory, as well as signaling to travis-ci that sudo is required.
Tested on travis-ci, and Ubuntu 14.04LTS
---
This is a re-write from scratch of t
Signed-off-by: Aaron Conole
---
.travis.yml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/.travis.yml b/.travis.yml
index 36d95bd..70cc14b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -5,6 +5,8 @@ compiler:
before_install: ./.travis/prepare.sh
+sudo: true
+
env:
- OPTS="--disabl
Signed-off-by: Aaron Conole
---
datapath-windows/include/automake.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/datapath-windows/include/automake.mk
b/datapath-windows/include/automake.mk
index 31f2fe2..b8dcf83 100644
--- a/datapath-windows/include/automake.mk
+++ b/datapath-windows/i
GRE64 was introduced to extend gre key from 32-bit to 64-bit using
gre-key and sequence number field. But GRE64 is not standard
protocol. There are not many users of this protocol. Therefore we
have decided to deprecate it.
Signed-off-by: Pravin B Shelar
---
NEWS | 1 +
datapath
The build system was set up to install the ovs-sim manpage, but not
ovs-sim itself. I noticed this because the fedora rpm build is
currently broken because the man page was installed but not packaged
yet. Fix this up by not installing the manpage.
Signed-off-by: Russell Bryant
---
utilities/au
On Mon, Jul 20, 2015 at 1:49 PM, Ben Pfaff wrote:
> Due to the way that Autotest works, this text was ignored, but it's still
> useless and looks weird and we might as well remove it.
>
> Signed-off-by: Ben Pfaff
Acked-by: Andy Zhou
___
dev mailing li
Due to the way that Autotest works, this text was ignored, but it's still
useless and looks weird and we might as well remove it.
Signed-off-by: Ben Pfaff
---
tests/ofproto-macros.at | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/ofproto-macros.at b/tests/ofproto-macros.at
index 77e0
On the master branch, it appears my change wasn't necessary. I had merged
4249b54 and run into the problem. However, 10 pushes later, 3d8dede seems to
have fixed the problem without my change. Sorry about that.
Thanks,
Kyle
On 07/20/2015 03:11 PM, Kyle Upton wrote:
> Without the change, I se
np~ ;D
Thx~
On Mon, Jul 20, 2015 at 1:30 PM, Ben Pfaff wrote:
> On Mon, Jul 20, 2015 at 01:27:35PM -0700, Alex Wang wrote:
> > On Mon, Jul 20, 2015 at 1:24 PM, Ben Pfaff wrote:
> >
> > > correctly. The version on master (commit 5b3c194faa61)
> > > did not include the $(srcdir) prefixes becaus
On Mon, Jul 20, 2015 at 01:27:35PM -0700, Alex Wang wrote:
> On Mon, Jul 20, 2015 at 1:24 PM, Ben Pfaff wrote:
>
> > correctly. The version on master (commit 5b3c194faa61)
> > did not include the $(srcdir) prefixes because master had a previous commit
> > ab70cd304210e (Makefiles: Stop distribut
On Mon, Jul 20, 2015 at 1:24 PM, Ben Pfaff wrote:
> correctly. The version on master (commit 5b3c194faa61)
> did not include the $(srcdir) prefixes because master had a previous commit
> ab70cd304210e (Makefiles: Stop distributing files because building them
> requires Python.) that is not on br
On Mon, Jul 20, 2015 at 01:13:08PM -0700, Alex Wang wrote:
> Yeah, this fixes the issue,
>
> Do you want to push it?
I sent a proper patch:
http://openvswitch.org/pipermail/dev/2015-July/057798.html
___
dev mailing list
dev@openvswitch.org
http:
Commit e21a4c0271e6f (lib/automake.mk: Fix dependency typos.) was
crossported from master to fix some observed Debian build failures, but it
was not crossported correctly. The version on master (commit 5b3c194faa61)
did not include the $(srcdir) prefixes because master had a previous commit
ab70cd
Yeah, this fixes the issue,
Do you want to push it?
Thanks,
Alex Wang,
On Mon, Jul 20, 2015 at 1:07 PM, Ben Pfaff wrote:
> OK, I think I see the problem then, can you try this?
>
> diff --git a/lib/automake.mk b/lib/automake.mk
> index fdc518e..bd23ef7 100644
> --- a/lib/automake.mk
> +++ b/li
OK, I think I see the problem then, can you try this?
diff --git a/lib/automake.mk b/lib/automake.mk
index fdc518e..bd23ef7 100644
--- a/lib/automake.mk
+++ b/lib/automake.mk
@@ -486,14 +486,14 @@ $(srcdir)/lib/ofp-errors.inc: \
$(srcdir)/lib/ofp-errors.h \
$(srcdir
Also, here is the compilation error, while running `fakeroot debian/rules
binary DEB_BUILD_OPTIONS='parallel=8 nocheck'` using ovs branch-2.4:
libtool: compile: gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I.. -I ../include -I
./include -I ../lib -I ./lib -I/usr/include -D_FORTIFY_SOURCE=2 -Wstrict-p
roto
Sorry for the unclarity, I'm building branch-2.4.
Master is fine,
Thanks,
Alex Wang,
On Mon, Jul 20, 2015 at 12:11 PM, Kyle Upton
wrote:
> Without the change, I see the following while doing a build in our
> modified version of ovs. The input source file is lib/ofp-errors.c instead
> of ../lib
Without the change, I see the following while doing a build in our modified
version of ovs. The input source file is lib/ofp-errors.c instead of
../lib/ofp-errors.c. Same thing occurs with ofp-msgs.c. With the change, I
don't see this issue and my thinking was that the object and not the sourc
Under certain circumstances, we might need to cancel a pending IRP
that has been submitted and not yet responded. This might occur when
the request takes too long to complete or when the process which
initiated the request terminated, leaving the request outstanding.
This patch provides this missi
In order to support IRP cancelling mechanism for pending IRPs, all
tunnel filter requests, VXLAN create/delete tunnel, need to be
processed iteratively.
Signed-off-by: Sorin Vinturis
---
datapath-windows/ovsext/TunnelFilter.c | 107 +
1 file changed, 30 insertions
This patch series adds support for IRP cancelling mechanism to the OVS
extension.
Sorin Vinturis (2):
[PATCH v2 1/2] datapath-windows: Process tunnel filter requests iteratively
[PATCH v2 2/2] datapath-windows: Support for IRP cancelling mechanism
_
On 20 July 2015 at 10:51, Ben Pfaff wrote:
> On Mon, Jul 20, 2015 at 10:12:31AM -0700, Joe Stringer wrote:
>> On 17 July 2015 at 12:58, Ben Pfaff wrote:
>> > On Fri, Jul 17, 2015 at 11:23:31AM -0700, Joe Stringer wrote:
>> >> Signed-off-by: Joe Stringer
>> >> ---
>> >> acinclude.m4 | 2 +-
>> >>
On Sat, Jul 18, 2015 at 05:05:54PM -0400, Aaron Conole wrote:
> Ben Pfaff writes:
>
> > Can you explain the following change? What effect does it have, and why
> > is it needed only in one fork of the "if"?
> >
> >> --- a/Makefile.am
> >> +++ b/Makefile.am
> >> @@ -44,7 +44,7 @@ endif
> >> if W
OK, now I need an explanation myself. The Debian builds worked fine for
me both before and after this patch. What problem do you see?
On Mon, Jul 20, 2015 at 11:17:41AM -0700, Alex Wang wrote:
> Could I get an explanation on why there is a typo.
>
> This is breaking debian build on my setup,
>
Thanks, applied to master.
On Mon, Jul 20, 2015 at 11:24:31AM -0700, Alex Wang wrote:
> Acked-by: Alex Wang
>
> On Mon, Jul 20, 2015 at 11:20 AM, Ben Pfaff wrote:
>
> > The prototype said "static" but the definition did not.
> >
> > Reported by sparse.
> >
> > Signed-off-by: Ben Pfaff
> > ---
Thanks, applied to master.
On Mon, Jul 20, 2015 at 11:23:31AM -0700, Alex Wang wrote:
> Acked-by: Alex Wang
>
> On Mon, Jul 20, 2015 at 11:20 AM, Ben Pfaff wrote:
>
> > The function was removed some time ago, but not the prototype.
> >
> > Signed-off-by: Ben Pfaff
> > ---
> > ofproto/ofproto
Acked-by: Alex Wang
On Mon, Jul 20, 2015 at 11:20 AM, Ben Pfaff wrote:
> The prototype said "static" but the definition did not.
>
> Reported by sparse.
>
> Signed-off-by: Ben Pfaff
> ---
> lib/db-ctl-base.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/db-ctl-b
Acked-by: Alex Wang
On Mon, Jul 20, 2015 at 11:20 AM, Ben Pfaff wrote:
> The function was removed some time ago, but not the prototype.
>
> Signed-off-by: Ben Pfaff
> ---
> ofproto/ofproto-dpif.h | 5 -
> 1 file changed, 5 deletions(-)
>
> diff --git a/ofproto/ofproto-dpif.h b/ofproto/ofp
The prototype said "static" but the definition did not.
Reported by sparse.
Signed-off-by: Ben Pfaff
---
lib/db-ctl-base.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index e3c0373..09efd8d 100644
--- a/lib/db-ctl-base.c
+++ b/lib/db
The function was removed some time ago, but not the prototype.
Signed-off-by: Ben Pfaff
---
ofproto/ofproto-dpif.h | 5 -
1 file changed, 5 deletions(-)
diff --git a/ofproto/ofproto-dpif.h b/ofproto/ofproto-dpif.h
index 7bb45e3..0209565 100644
--- a/ofproto/ofproto-dpif.h
+++ b/ofproto/ofpr
On Mon, Jul 20, 2015 at 02:06:26PM -0400, Russell Bryant wrote:
> On 07/20/2015 01:46 PM, Ben Pfaff wrote:
> Regarding this patch, to be honest, the choice of "bridge mappings" is
> just borrowed from the existing OVS support in OpenStack. What's
> implemented here matches how that
Could I get an explanation on why there is a typo.
This is breaking debian build on my setup,
Thanks,
Alex Wang,
On Tue, Jul 14, 2015 at 10:43 PM, Ben Pfaff wrote:
> On Thu, Jul 09, 2015 at 03:22:46PM -0400, Kyle Upton wrote:
> > Fix dependency issues with lib/ofp-errors.lo and lib/ofp-msgs.lo
On 07/20/2015 01:46 PM, Ben Pfaff wrote:
Regarding this patch, to be honest, the choice of "bridge mappings" is
just borrowed from the existing OVS support in OpenStack. What's
implemented here matches how that works. It expects the bridge to be
created already, but automatica
Très Cher(e),
Excusez moi de cette manière de vous contacter car nous ne nous connaissons
pas, je viens d'écrire votre adresse par pure hasard tout en demandant a Dieu
de me guider vers une personne de bonne moralité, je me suis dis que vous êtes
la personne qu'il me faut. En bref, je me nomme
On Mon, Jul 20, 2015 at 10:12:31AM -0700, Joe Stringer wrote:
> On 17 July 2015 at 12:58, Ben Pfaff wrote:
> > On Fri, Jul 17, 2015 at 11:23:31AM -0700, Joe Stringer wrote:
> >> Signed-off-by: Joe Stringer
> >> ---
> >> acinclude.m4 | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
On Mon, Jul 20, 2015 at 02:15:22PM -0300, Marcelo Ricardo Leitner wrote:
> On Fri, Jul 17, 2015 at 02:55:33PM +0200, Thomas Graf wrote:
>
> [ snip ]
>
> > @@ -2373,6 +2470,12 @@ static void vxlan_setup(struct net_device *dev)
> > netif_keep_dst(dev); ---> (a)
> > dev->
> >> Regarding this patch, to be honest, the choice of "bridge mappings" is
> >> just borrowed from the existing OVS support in OpenStack. What's
> >> implemented here matches how that works. It expects the bridge to be
> >> created already, but automatically creates patch ports to/from the
> >>
On Fri, Jul 17, 2015 at 02:55:33PM +0200, Thomas Graf wrote:
[ snip ]
> @@ -2373,6 +2470,12 @@ static void vxlan_setup(struct net_device *dev)
> netif_keep_dst(dev); ---> (a)
> dev->priv_flags |= IFF_LIVE_ADDR_CHANGE;
>
> + /* If in flow based mode, keep the
On 17 July 2015 at 12:58, Ben Pfaff wrote:
> On Fri, Jul 17, 2015 at 11:23:31AM -0700, Joe Stringer wrote:
>> Signed-off-by: Joe Stringer
>> ---
>> acinclude.m4 | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/acinclude.m4 b/acinclude.m4
>> index 14907ab..4f1e66c 10064
Applied to master and branch-2.4,
Thanks,
Alex Wang,
On Mon, Jul 20, 2015 at 9:58 AM, Alex Wang wrote:
> Thx for the review, I'll adopt the comment~
>
> On Mon, Jul 20, 2015 at 9:47 AM, Ben Pfaff wrote:
>
>> On Mon, Jul 20, 2015 at 08:50:39AM -0700, Alex Wang wrote:
>> > On Mon, Jul 20, 2015 a
Thx for the review, I'll adopt the comment~
On Mon, Jul 20, 2015 at 9:47 AM, Ben Pfaff wrote:
> On Mon, Jul 20, 2015 at 08:50:39AM -0700, Alex Wang wrote:
> > On Mon, Jul 20, 2015 at 8:17 AM, Ben Pfaff wrote:
> >
> > > On Mon, Jul 20, 2015 at 01:22:32AM -0700, Alex Wang wrote:
> > > > Commit 6f
On Mon, Jul 20, 2015 at 08:50:39AM -0700, Alex Wang wrote:
> On Mon, Jul 20, 2015 at 8:17 AM, Ben Pfaff wrote:
>
> > On Mon, Jul 20, 2015 at 01:22:32AM -0700, Alex Wang wrote:
> > > Commit 6fd6ed7 (ofpbuf: Simplify ofpbuf API.) introduced the
> > > 'header' and 'msg' pointers to 'struct ofpbuf'.
Added ovs dev list to let others sanity check me. :-) The context is
discussing how OpenStack will use ACLs in OVN_Northbound.
We don't have Neutron code for this yet, but we have the "Security
Groups" section of this doc:
http://docs.openstack.org/developer/networking-ovn/design/data_model.htm
On Mon, Jul 20, 2015 at 8:17 AM, Ben Pfaff wrote:
> On Mon, Jul 20, 2015 at 01:22:32AM -0700, Alex Wang wrote:
> > Commit 6fd6ed7 (ofpbuf: Simplify ofpbuf API.) introduced the
> > 'header' and 'msg' pointers to 'struct ofpbuf'. However, we
> > forget to update the 'msg' pointer when resizing ofp
On 07/17/2015 10:53 AM, Russell Bryant wrote:
> On 07/16/2015 07:56 PM, Ben Pfaff wrote:
>> On Thu, Jul 16, 2015 at 04:55:11PM -0700, Ben Pfaff wrote:
>>> On Thu, Jul 16, 2015 at 06:06:10PM -0400, Russell Bryant wrote:
Add a new OVN configuration entry in the Open_vSwitch database called
On Mon, Jul 20, 2015 at 01:22:32AM -0700, Alex Wang wrote:
> Commit 6fd6ed7 (ofpbuf: Simplify ofpbuf API.) introduced the
> 'header' and 'msg' pointers to 'struct ofpbuf'. However, we
> forget to update the 'msg' pointer when resizing ofpbuf.
>
> This bug could cause serious issue. For example,
On Fri, 17 Jul 2015 13:56:21 -0300, Flavio Leitner wrote:
> When ifdown isn't executed (system didn't shut down properly),
> the interfaces remain in the openvswitch's database. For the
> internal ports or devices that are available when openvswitch
> service starts that's not an issue.
>
> Howev
On 07/17/15 at 11:25pm, Julian Anastasov wrote:
>
> Hello,
>
> On Fri, 17 Jul 2015, Thomas Graf wrote:
>
> > Add a new flowi_tunnel structure which is a subset of ip_tunnel_key to
> > allow routes to match on tunnel metadata. For now, the tunnel id is
> > added to flowi_tunnel which allows
>
> Hi All,
>
> Now i started using vhost-user and observing good performance.
Great!
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
On 07/17/15 at 11:41am, Alexei Starovoitov wrote:
> On 7/17/15 5:55 AM, Thomas Graf wrote:
> >@@ -2373,6 +2470,12 @@ static void vxlan_setup(struct net_device *dev)
> > netif_keep_dst(dev);
> > dev->priv_flags |= IFF_LIVE_ADDR_CHANGE;
> >
> >+/* If in flow based mode, keep the dst inclu
Hi All,
Now i started using vhost-user and observing good performance.
Packet loss is still happening, but is around 0.006%.
[root@warmtongue ~]# ovs-ofctl dump-ports ovs
OFPST_PORT reply (xid=0x2): 6 ports
port 4: rx pkts=24717275, bytes=?, drop=?, errs=?, frame=?, over=?, crc=?
t
Hi Cascardo,
Appreciating very much your help on this !
Please find attached tcpdump (wireshark files) and below requested MAC
address details.
Port MAC Address.
enp9s0f0 a0:36:9f:4f:38:94
enp9s0f3 a0:36:9f:4f:38:97
TG1 a0:36:9f:4f:38:95
TG2
Commit 6fd6ed7 (ofpbuf: Simplify ofpbuf API.) introduced the
'header' and 'msg' pointers to 'struct ofpbuf'. However, we
forget to update the 'msg' pointer when resizing ofpbuf.
This bug could cause serious issue. For example, in the function
ofputil_encode_nx_packet_in(), the 'msg' pointer is p
58 matches
Mail list logo
