Hi Andy,
On 17.03.2017 02:28, Andy Zhou wrote:
On Thu, Mar 16, 2017 at 1:52 PM, Ben Pfaff wrote:
On Thu, Mar 16, 2017 at 11:38:19PM +0500, Valentine Sinitsyn wrote:
On 16.03.2017 20:56, Ben Pfaff wrote:
On Tue, Mar 14, 2017 at 07:08:54PM +0500, Valentine Sinitsyn wrote:
Recently, I was eval
I am using 2.0.2 at the moment.
On Thu, Mar 16, 2017 at 5:18 PM, Ben Pfaff wrote:
> On Thu, Mar 16, 2017 at 04:12:54PM +0100, Aqsa Malik wrote:
> > We have a setup using 2 ovs-switches that are rewriting the rules (the ip
> > addresses)
> > Its an ipv6 network but when i ping from one edge node
Hi Billy,
On 03/01/2017 01:36 PM, Billy O'Mahony wrote:
Hi All,
I'm creating this patch on the basis of performance results outlined below. In
summary it appears that enabling INDIRECT_DESC on DPDK vHostUser ports leads to
very large increase in performance when using linux stack applications i
Hi,
On 17.03.2017 02:24, Han Zhou wrote:
On Thu, Mar 16, 2017 at 1:06 PM, Valentine Sinitsyn
mailto:valentine.sinit...@gmail.com>> wrote:
Hi Han,
Thanks for the quick answer.
On 17.03.2017 00:34, Han Zhou wrote:
On Thu, Mar 16, 2017 at 3:58 AM, Valentine Sinitsyn
mailto:valentine.sinit..
On 03/17/2017 10:48 AM, Maxime Coquelin wrote:
Hi Billy,
On 03/01/2017 01:36 PM, Billy O'Mahony wrote:
Hi All,
I'm creating this patch on the basis of performance results outlined
below. In
summary it appears that enabling INDIRECT_DESC on DPDK vHostUser ports
leads to
very large increase in
On Fri, Mar 17, 2017 at 11:57 AM, Ben Pfaff wrote:
> On Thu, Mar 16, 2017 at 09:18:00PM -0400, Eric Garver wrote:
>> On Thu, Mar 16, 2017 at 03:23:38PM -0700, Ben Pfaff wrote:
>> > Thanks for all the iteration on this patch.
>> >
>> > I've folded in the following incremental and applied this to ma
On Mon, Mar 13, 2017 at 03:36:50PM +0200, Roi Dayan wrote:
> This patch series introduces rule offload functionality to dpif-netlink
> via netdev ports new flow offloading API. The user can specify whether to
> enable rule offloading or not via OVS configuration. Netdev providers
> are able to impl
Signed-off-by: Ben Pfaff
---
Documentation/faq/openflow.rst | 8
1 file changed, 8 insertions(+)
diff --git a/Documentation/faq/openflow.rst b/Documentation/faq/openflow.rst
index 376e64eb4482..214e38e6a9aa 100644
--- a/Documentation/faq/openflow.rst
+++ b/Documentation/faq/openflow.rst
I recommend upgrading, because that version is 3 1/2 years old.
On Fri, Mar 17, 2017 at 09:21:03AM +0100, Aqsa Malik wrote:
> I am using 2.0.2 at the moment.
>
> On Thu, Mar 16, 2017 at 5:18 PM, Ben Pfaff wrote:
>
> > On Thu, Mar 16, 2017 at 04:12:54PM +0100, Aqsa Malik wrote:
> > > We have a s
On Fri, Mar 17, 2017 at 08:39:01AM -0700, Ben Pfaff wrote:
> On Wed, Mar 01, 2017 at 05:48:00PM -0500, Eric Garver wrote:
> > - Example:
> > ovs-vsctl set Port p1 vlan_mode=dot1q-tunnel tag=100
> >Pushes another VLAN 100 header on packets (tagged and untagged) on
> >ingress, and pops
On Wed, Mar 01, 2017 at 05:48:00PM -0500, Eric Garver wrote:
> - Example:
> ovs-vsctl set Port p1 vlan_mode=dot1q-tunnel tag=100
>Pushes another VLAN 100 header on packets (tagged and untagged) on
>ingress, and pops it on egress.
> - Customer VLAN check:
> ovs-vsctl set Port p1
Previously, this function could modify the pkt_mark field as part of IPsec
integration. It no longer does that, so there's no longer any need for it
to save and restore pkt_mark, and this commit removes that.
CC: Ansis Atteka
Signed-off-by: Ben Pfaff
---
ofproto/ofproto-dpif-xlate.c | 3 ---
1
Acked-by: Jarno Rajahalme
> On Mar 17, 2017, at 10:55 AM, Ben Pfaff wrote:
>
> Previously, this function could modify the pkt_mark field as part of IPsec
> integration. It no longer does that, so there's no longer any need for it
> to save and restore pkt_mark, and this commit removes that.
>
On Fri, Mar 03, 2017 at 01:00:05PM +, László Sürü wrote:
> hereby I'm sending the implementation of link liveness propagation
> functionality
> including the additional functionalities requested last year (last activity
> at Wed Mar 16 23:13:24 UTC 2016).
>
> The idea is to use OFPPS_LIVE b
On Thu, Mar 09, 2017 at 10:53:37AM +0800, Guoshuai Li wrote:
> Co-authored-by: Dong Jun
> Signed-off-by: Dong Jun
> Signed-off-by: Guoshuai Li
> Acked-by: Numan Siddique
Applied to master, thanks!
___
dev mailing list
d...@openvswitch.org
https://mai
Signed-off-by: Joe Stringer
---
CONTRIBUTING.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst
index 99ae164ee67d..ea471bfab413 100644
--- a/CONTRIBUTING.rst
+++ b/CONTRIBUTING.rst
@@ -30,5 +30,5 @@ range from bug reports and code revi
On Tue, Mar 07, 2017 at 05:11:27AM -0800, nickcooper-zhangtonghao wrote:
> This patch changes the stp_port_get_role and removes
> the stp_port_get_id, because stp/show has locked the
> mutex before calling the stp_port_get_role, and
> stp_port_get_id will not be used.
>
> Signed-off-by: nickcooper
On Fri, Mar 17, 2017 at 2:50 AM, Valentine Sinitsyn <
valentine.sinit...@gmail.com> wrote:
> Did you restart controllers or ovn-northd after running full tests before
binding more ports. It would be interesting to learn how long does it take
warm up IDL in controllers/northd in your setup.
>
No nee
On Tue, Mar 07, 2017 at 05:11:28AM -0800, nickcooper-zhangtonghao wrote:
> The stp/show command will help users and developers get
> more details about stp. This patch works together with
> the previous patch "stp: Change the api for next patch."
>
> Signed-off-by: nickcooper-zhangtonghao
Thanks
This commit provides more visibility into conditions where learn limits
are reached when the functionality from patch 4c71600d2256
("ofp-actions: Add limit to learn action.") is used.
VMWare-BZ: #1832142
Signed-off-by: Joe Stringer
---
ofproto/ofproto.c | 6 ++
1 file changed, 6 insertions(+
It's slightly more cognitive load to read an inverse condition and then
invert it again to understand the 'else' condition.
Signed-off-by: Joe Stringer
---
ofproto/ofproto.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index c6d8
Joe Stringer writes:
> Signed-off-by: Joe Stringer
> ---
> CONTRIBUTING.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Acked-by: Aaron Conole
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/
On Tue, Mar 07, 2017 at 05:11:29AM -0800, nickcooper-zhangtonghao wrote:
> Signed-off-by: nickcooper-zhangtonghao
Thanks for the bug fix. I applied this to master, branch-2.7, and
branch-2.6.
___
dev mailing list
d...@openvswitch.org
https://mail.openv
On Tue, Mar 07, 2017 at 05:11:30AM -0800, nickcooper-zhangtonghao wrote:
> Signed-off-by: nickcooper-zhangtonghao
Thanks, applied to master, branch-2.7, and branch-2.6.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listi
On 2 February 2017 at 20:48, Mickey Spiegel wrote:
> This patch extends gratuitous ARP support for NAT addresses so that it
> applies to centralized NAT rules on a distributed router, in addition to
> the existing gratuitous ARP support for NAT addresses on gateway routers.
>
> Gratuitous ARP pac
On Wed, Mar 08, 2017 at 08:01:20PM -0800, nickcooper-zhangtonghao wrote:
> Some debian distribution may not contain the ifconfig.
> We use the ip command instead of ifconfig in debian/ifupdown.sh
>
> Reported-at:
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329503.html
> CC: procha
Add macro OVS_CHECK_8021AD().
Signed-off-by: Eric Garver
---
tests/system-common-macros.at | 5 +
1 file changed, 5 insertions(+)
diff --git a/tests/system-common-macros.at b/tests/system-common-macros.at
index 10a08f44567d..3cb6b11ea44f 100644
--- a/tests/system-common-macros.at
+++ b/test
This series add tests for 802.1ad and dot1q-tunnel.
Eric Garver (5):
system-common-macros: Add macro to check for 802.1ad support.
system-traffic: 802.1ad: Add conntrack ping tests for CVLANs.
system-traffic: 802.1ad: Add vlan_limit test case.
system-traffic: 802.1ad: Add push/pop test cas
Signed-off-by: Eric Garver
---
tests/system-traffic.at | 107
1 file changed, 107 insertions(+)
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index 9861fb15e347..72e421150bea 100644
--- a/tests/system-traffic.at
+++ b/tests/system
Verify that vlan_limit works as expected and preserves legacy dl_type
matching behavior.
Signed-off-by: Eric Garver
---
tests/system-traffic.at | 30 ++
1 file changed, 30 insertions(+)
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index 72e421150bea
Two test cases to push and pop an outer tag between two "customer"
bridges. One to push/pop 0x88a8. One to push/pop a second 0x8100.
Signed-off-by: Eric Garver
---
tests/system-traffic.at | 112
1 file changed, 112 insertions(+)
diff --git a/test
Test case for dot1q-tunnel between two "customer" bridges.
Signed-off-by: Eric Garver
---
tests/system-traffic.at | 63 +
1 file changed, 63 insertions(+)
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index fd026ca2a040..1816b1a88
On Thu, Mar 09, 2017 at 03:49:49AM -0800, nickcooper-zhangtonghao wrote:
> This patch makes the stp tests more stable and
> adds the test for fdb/mdb flushing when topology
> is changed.
>
> Signed-off-by: nickcooper-zhangtonghao
Thanks for the tests! I applied this to master.
_
Thanks for testing. I applied this to master and branch-2.7.
On Thu, Mar 09, 2017 at 01:30:55PM +0800, nickcooper-zhangtonghao wrote:
> I have tested it. It is ok now. Simon, what do you think about it?
>
>
> Tested-by: nickcooper-zhangtonghao
>
> > On Mar 9, 2017, at 2:22 AM, Ben Pfaff wrot
OK, thanks, we'll go with that approach then.
On Fri, Mar 10, 2017 at 12:31:20PM +0300, Ilya Maximets wrote:
> I've sent the patch for removing highlighting at all here:
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329651.html
>
> Only 'windows.rst' uses this functionality. So, I t
On Fri, Mar 10, 2017 at 12:25:52PM +0300, Ilya Maximets wrote:
> Current documentation uses syntax highlighting in 'sphinx'
> via 'pygments' library. This leads to build failures on the
> systems with old version of this library.
>
> In fact that only 'windows.rst' uses highlighting it's a
> very
On Thu, Mar 09, 2017 at 02:55:24PM +0300, Ilya Maximets wrote:
> Sphinx 1.1.3 on RHEL7 is able to properly build the documentation.
>
> Discovering fixed because 'sphinx-build v1.1.3' doesn't support
> '--version' option.
>
> Acked-by: Stephen Finucane
> Signed-off-by: Ilya Maximets
> ---
> Ver
On Thu, Mar 09, 2017 at 07:00:28PM -0800, Daniele Di Proietto wrote:
> 2017-03-07 15:56 GMT-08:00 Ben Pfaff :
> > Various printf() format specifiers in the tree had minor technical issues
> > which the Mac OS build reported, e.g. here:
> > https://s3.amazonaws.com/archive.travis-ci.org/jobs/2087183
Reported-by: Timothy Redaelli
Reported-at:
https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329604.html
Signed-off-by: Ben Pfaff
---
lib/route-table.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/lib/route-table.c b/lib/route-table.c
index ae8af438d864..67fd
On Thu, Mar 09, 2017 at 05:44:29PM +0100, Timothy Redaelli wrote:
> Currently to build ovs, you need the following version:
>
> - GCC 4.6 or later (old GCCs have some bugs with unnamed fields)
> - python six library 1.4.0 or later (for six.moves.range)
> - kernel 2.6.36 or later (for RTA_MARK)
>
On 15 March 2017 at 16:01, Joe Stringer wrote:
> Commit 04f48a68c428 ("ofp-actions: Fix variable length meta-flow OXMs."), on
> branch-2.7 as 9554b03d6ab7, attempted to address incorrect encode and decode
> of
> variable length metaflow fields where the OXM/NXM encoding of the variable
> length f
Clang 3.4 and later should now be widespread, so it's not worth suggesting
where to find it.
OVS needs a variety of shared libraries at runtime and it's not worth
mentioning each one by name.
The Linux kernel datapath module is available from a variety of places, so
don't say you have to use the
On Mon, Mar 13, 2017 at 05:43:45PM +0100, Timothy Redaelli wrote:
> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1339097
> Signed-off-by: Timothy Redaelli
> ---
> Changes in v2:
> - Don't use mask, but use an example MAC address for dl_dst
>
Thanks, applied to master.
__
On Fri, Mar 17, 2017 at 04:13:10PM -0400, Eric Garver wrote:
> This series add tests for 802.1ad and dot1q-tunnel.
>
> Eric Garver (5):
> system-common-macros: Add macro to check for 802.1ad support.
> system-traffic: 802.1ad: Add conntrack ping tests for CVLANs.
> system-traffic: 802.1ad: A
On 17 March 2017 at 14:47, Ben Pfaff wrote:
> On Fri, Mar 17, 2017 at 04:13:10PM -0400, Eric Garver wrote:
>> This series add tests for 802.1ad and dot1q-tunnel.
>>
>> Eric Garver (5):
>> system-common-macros: Add macro to check for 802.1ad support.
>> system-traffic: 802.1ad: Add conntrack pi
On 17 March 2017 at 12:02, Aaron Conole wrote:
> Joe Stringer writes:
>
>> Signed-off-by: Joe Stringer
>> ---
>> CONTRIBUTING.rst | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>
> Acked-by: Aaron Conole
Thanks, applied to master and branch-2.7.
__
On Fri, Mar 17, 2017 at 12:47 PM, Guru Shetty wrote:
>
>
> On 2 February 2017 at 20:48, Mickey Spiegel wrote:
>
>> This patch extends gratuitous ARP support for NAT addresses so that it
>> applies to centralized NAT rules on a distributed router, in addition to
>> the existing gratuitous ARP sup
This patch extends gratuitous ARP support for NAT addresses so that it
applies to centralized NAT rules on a distributed router, in addition to
the existing gratuitous ARP support for NAT addresses on gateway routers.
Gratuitous ARP packets for centralized NAT rules on a distributed router
are onl
This patch extends gratuitous ARP support for NAT addresses so that it
applies to distributed NAT rules on a distributed logical router.
Gratuitous ARP packets for distributed NAT rules are only generated on
the chassis where the 'logical_port' specified in the NAT rule resides.
Gratuitous ARPs ar
On Wed, Mar 08, 2017 at 03:53:10AM -0800, nickcooper-zhangtonghao wrote:
> The OvS BPDU packets have not right format. The STP works
> well in OvS bridges when stp enabled, because they set the
> time(e.g. max age, hello time and forward delay) in ticks.
> But they should be sec. If so, OvS STP ca
On Fri, Mar 17, 2017 at 11:38:34AM -0700, Joe Stringer wrote:
> This commit provides more visibility into conditions where learn limits
> are reached when the functionality from patch 4c71600d2256
> ("ofp-actions: Add limit to learn action.") is used.
>
> VMWare-BZ: #1832142
> Signed-off-by: Joe S
On Fri, Mar 17, 2017 at 11:38:35AM -0700, Joe Stringer wrote:
> It's slightly more cognitive load to read an inverse condition and then
> invert it again to understand the 'else' condition.
>
> Signed-off-by: Joe Stringer
It's also somewhat odd to handle the error condition before the common
cas
I'm seeing failures on Travis that boil down to this:
/home/travis/build/blp/ovs-reviews/Documentation/intro/install/documentation.rst:34:
ERROR: Error in "note" directive:
invalid option block.
.. note::
These instructions provide information on building the documentation
lo
> From: "Ben Pfaff"
> To: d...@openvswitch.org
> Cc: "Lance Richardson"
> Sent: Friday, March 17, 2017 7:09:51 PM
> Subject: sphinx on os x on travis?
>
> I'm seeing failures on Travis that boil down to this:
>
>
> /home/travis/build/blp/ovs-reviews/Documentation/intro/install/documentatio
> From: "Lance Richardson"
> To: "Ben Pfaff"
> Cc: d...@openvswitch.org
> Sent: Friday, March 17, 2017 7:39:03 PM
> Subject: Re: sphinx on os x on travis?
>
> > From: "Ben Pfaff"
> > To: d...@openvswitch.org
> > Cc: "Lance Richardson"
> > Sent: Friday, March 17, 2017 7:09:51 PM
> > Subject: sp
On Fri, Mar 17, 2017 at 07:44:53PM -0400, Lance Richardson wrote:
> > From: "Lance Richardson"
> > To: "Ben Pfaff"
> > Cc: d...@openvswitch.org
> > Sent: Friday, March 17, 2017 7:39:03 PM
> > Subject: Re: sphinx on os x on travis?
> >
> > > From: "Ben Pfaff"
> > > To: d...@openvswitch.org
> > >
- Original Message -
> From: "Ben Pfaff"
> To: "Lance Richardson"
> Cc: d...@openvswitch.org
> Sent: Friday, March 17, 2017 10:08:38 PM
> Subject: Re: sphinx on os x on travis?
>
> On Fri, Mar 17, 2017 at 07:44:53PM -0400, Lance Richardson wrote:
> > > From: "Lance Richardson"
> > > T
57 matches
Mail list logo