On 21 September 2016 at 03:26, Pravin B Shelar wrote:
> OVS IPsec tunnel support has issues:
> 1. It only works for GRE.
> 2. only works on Debian.
> 3. It does not allow user to match on packet-mark
>on packet received on tunnel ports.
>
> This patch deprecates support for IPsec tunnel port.
Makes sense to me, applied to master and branch-2.6
Thanks,
Daniele
On 23/09/2016 06:17, "Ilya Maximets" wrote:
>Current error message incorrect for the client mode.
>
>Fixes: c1ff66ac80b5 ("netdev-dpdk: vHost client mode and reconnect")
>Signed-off-by: Ilya Maximets
>---
> lib/netdev-dpdk.
Thanks for the fix, applied to master and branch-2.6
On 23/09/2016 06:17, "Ilya Maximets" wrote:
>'vhost_driver_flags' and 'vhost_id' are mutable and must be protected
>by 'dev->mutex'.
>
>Fixes: 2d24d165d6a5 ("netdev-dpdk: Add new 'dpdkvhostuserclient' port type")
>Signed-off-by: Ilya Maxime
On 23 September 2016 at 12:13, Justin Pettit wrote:
>
>> On Sep 23, 2016, at 10:10 AM, Joe Stringer wrote:
>>
>> On 23 September 2016 at 01:53, Justin Pettit wrote:
>>>
>>> diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h
>>> index 5ab026c..9d53623 100644
>>> --- a/incl
On Fri, Sep 23, 2016 at 12:54 PM, Ansis Atteka wrote:
> On Fri, Sep 23, 2016 at 1:12 AM, pravin shelar wrote:
>> On Thu, Sep 22, 2016 at 11:59 AM, Ansis Atteka wrote:
>>>
>>>
>>> On 20 September 2016 at 20:52, Pravin B Shelar wrote:
OVS IPsec tunnel support has issues:
1. It only
On Fri, Sep 23, 2016 at 1:12 AM, pravin shelar wrote:
> On Thu, Sep 22, 2016 at 11:59 AM, Ansis Atteka wrote:
>>
>>
>> On 20 September 2016 at 20:52, Pravin B Shelar wrote:
>>>
>>> OVS IPsec tunnel support has issues:
>>> 1. It only works for GRE.
>>>
>>> 2. only works on Debian.
>>>
>>> 3. It d
On Thu, Sep 22, 2016 at 1:07 PM, Ramu Ramamurthy
wrote:
> Currently, arping from a vtep-gw physical-switch port to
> a VIF IP address does not work.
>
> When a physical-switch-port arps for an IP address
> of a VIF, that arp packet comes into the VIF hypervisor via a
> vxlan tunnel. That arp pack
> On Sep 23, 2016, at 10:14 AM, Ben Pfaff wrote:
>
> On Fri, Sep 23, 2016 at 01:53:03AM -0700, Justin Pettit wrote:
>> Flush any existing conntrack entries for a zone when that zone is
>> allocated to a new logical port.
>>
>> Signed-off-by: Justin Pettit
>
> Also would be nice to have a test
> On Sep 23, 2016, at 9:56 AM, Ben Pfaff wrote:
>
> On Fri, Sep 23, 2016 at 01:53:01AM -0700, Justin Pettit wrote:
>> From: Ben Pfaff
>>
>> This will have a caller in the future.
>>
>> Signed-off-by: Ben Pfaff
>> Signed-off-by: Justin Pettit
>
> I guess I can't review this; presumably, you
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
Add macros OVS_CHECK_8021AD(), ADD_SVLAN() and ADD_CVLAN().
Signed-off-by: Eric Garver
---
tests/system-common-macros.at | 32 +++-
1 file changed, 31 insertions(+), 1 deletion(-)
diff --git a/tests/system-common-macros.at b/tests/system-common-macros.at
index 1dfdcf
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 83c7b8a2f4e5..bb0cb02804f9 100644
--- a/tests/system-traffic.at
+++ b/tests/system
Signed-off-by: Eric Garver
---
tests/system-traffic.at | 62 +
1 file changed, 62 insertions(+)
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index 4dabd90356a1..83c7b8a2f4e5 100644
--- a/tests/system-traffic.at
+++ b/tests/system-
This series adds 802.1ad test cases for the check-* make targets. It is
meant as a counterpart to the 802.1ad work currently going on and being
discussed on the dev list.
User space support for 802.1ad is being worked on by Xiao Liang (based
on Thomas F Herbert's work). Kernel support is already p
Passing sed arguments to OVS_TRAFFIC_VSWITCHD_STOP() was being ignored
for check-system-userspace. This is useful to selective ignore log
messages.
Signed-off-by: Eric Garver
---
tests/system-userspace-macros.at | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/system-use
Verify that vlan_limit works as expected and preserves legacy dl_type
matching behavior.
Signed-off-by: Eric Garver
---
tests/system-traffic.at | 36
1 file changed, 36 insertions(+)
diff --git a/tests/system-traffic.at b/tests/system-traffic.at
index bb0cb0
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 4d31e5452914..9b0ae5547
We need to check if a packet is double tagged. If so make sure to push
0x88a8 instead of 0x8100. Without this a simple port redirect of 802.1ad
frames means the outer tag gets translated from 0x88a8 to 0x8100 by the
userspace datapath.
This only affected kernels that don't use TP_STATUS_VLAN_TPID_
> On Sep 23, 2016, at 10:10 AM, Joe Stringer wrote:
>
> On 23 September 2016 at 01:53, Justin Pettit wrote:
>>
>> diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h
>> index 5ab026c..9d53623 100644
>> --- a/include/openflow/nicira-ext.h
>> +++ b/include/openflow/nicira-
On 22 September 2016 at 09:54, Ben Pfaff wrote:
> On Mon, Sep 19, 2016 at 03:52:55AM -0700, Gurucharan Shetty wrote:
> > Without initializing the databases, commands of the form
> > 'ovn-nbctl --wait=sb ls-add ls0' will simply hang.
> >
> > Signed-off-by: Gurucharan Shetty
>
> Thanks!
>
> Acked-
> On Sep 23, 2016, at 11:29 AM, Guru Shetty wrote:
>
>
>
> On 23 September 2016 at 11:25, Justin Pettit wrote:
>
> > On Sep 23, 2016, at 11:21 AM, Guru Shetty wrote:
> >
> >
> >
> > I changed the sentence to the following, since I was worried that "key" was
> > being used in different cont
On 23 September 2016 at 11:25, Justin Pettit wrote:
>
> > On Sep 23, 2016, at 11:21 AM, Guru Shetty wrote:
> >
> >
> >
> > I changed the sentence to the following, since I was worried that "key"
> was being used in different contexts in that paragraph:
> >
> > The name contains a prefix
> On Sep 23, 2016, at 11:21 AM, Guru Shetty wrote:
>
>
>
> I changed the sentence to the following, since I was worried that "key" was
> being used in different contexts in that paragraph:
>
> The name contains a prefix
> of ct-zone- followed by the name of the logical
>
>
>
>
> I changed the sentence to the following, since I was worried that "key"
> was being used in different contexts in that paragraph:
>
> The name contains a prefix
> of ct-zone- followed by the name of the logical
> port or the gateway router.
>
> Are you good with it?
> On Sep 23, 2016, at 11:12 AM, Guru Shetty wrote:
>
>
>
> On 23 September 2016 at 11:01, Justin Pettit wrote:
>
> > On Sep 23, 2016, at 10:19 AM, Guru Shetty wrote:
> >
> >
> >
> > On 23 September 2016 at 01:53, Justin Pettit wrote:
> > If ovn-controller is restarted, it may choose differ
On 23 September 2016 at 11:01, Justin Pettit wrote:
>
> > On Sep 23, 2016, at 10:19 AM, Guru Shetty wrote:
> >
> >
> >
> > On 23 September 2016 at 01:53, Justin Pettit wrote:
> > If ovn-controller is restarted, it may choose different conntrack zones
> > than had been previously used, which cou
> On Sep 23, 2016, at 10:19 AM, Guru Shetty wrote:
>
>
>
> On 23 September 2016 at 01:53, Justin Pettit wrote:
> If ovn-controller is restarted, it may choose different conntrack zones
> than had been previously used, which could cause the wrong conntrack
> entries to be associated with a log
On 23 September 2016 at 01:53, Justin Pettit wrote:
> If ovn-controller is restarted, it may choose different conntrack zones
> than had been previously used, which could cause the wrong conntrack
> entries to be associated with a logical port. This commit stores in the
> integration bridge's OV
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.
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/
On 23 September 2016 at 06:25, Ansis Atteka wrote:
>
>
> On 23 September 2016 at 03:25, Joe Stringer wrote:
>>
>> ovs-lib creates several directories directly from the script, but
>> doesn't make any attempt to ensure that the correct SELinux context is
>> applied to these directories. As a resul
On Fri, Sep 23, 2016 at 01:53:03AM -0700, Justin Pettit wrote:
> Flush any existing conntrack entries for a zone when that zone is
> allocated to a new logical port.
>
> Signed-off-by: Justin Pettit
Also would be nice to have a test.
Acked-by: Ben Pfaff
On Fri, Sep 23, 2016 at 01:53:02AM -0700, Justin Pettit wrote:
> If ovn-controller is restarted, it may choose different conntrack zones
> than had been previously used, which could cause the wrong conntrack
> entries to be associated with a logical port. This commit stores in the
> integration br
On 23 September 2016 at 01:53, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
> ---
> NEWS | 1 +
> debian/changelog | 10 ++
> include/openflow/nicira-ext.h | 9 +
> include/openvswitch/ofp-msgs.h | 4
> lib/dpif.c
On 23 September 2016 at 05:51, Eric Garver wrote:
> On Thu, Sep 22, 2016 at 06:48:09PM -0700, Joe Stringer wrote:
>> On 14 September 2016 at 17:26, Eric Garver wrote:
>> > Signed-off-by: Eric Garver
>>
>> Hi Eric,
>>
>> I tried this series again on a variety of kernels and these initial
>> ping
> On Sep 23, 2016, at 8:42 AM, Ben Pfaff wrote:
>
> On Fri, Sep 23, 2016 at 01:53:00AM -0700, Justin Pettit wrote:
>> Signed-off-by: Justin Pettit
>
> ct_dpif_flush() and dpif_ct_flush() are almost identical. Why do we
> need both?
>
> Acked-by: Ben Pfaff
Just an oversight. I'll add my lo
On Fri, Sep 23, 2016 at 01:53:01AM -0700, Justin Pettit wrote:
> From: Ben Pfaff
>
> This will have a caller in the future.
>
> Signed-off-by: Ben Pfaff
> Signed-off-by: Justin Pettit
I guess I can't review this; presumably, you've reviewed it.
___
Joe,
I have a pretty straight forward fix for the TPID issue. I'll be sending it to
ovs-dev shortly.
However, I had to make a small change to one of my tests to support it. So I'll
be sending out another rev of this test series as well.
Thanks.
Eric.
On Fri, Sep 23, 2016 at 08:51:42AM -0400, Er
On Fri, Sep 23, 2016 at 01:53:00AM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
ct_dpif_flush() and dpif_ct_flush() are almost identical. Why do we
need both?
Acked-by: Ben Pfaff
___
dev mailing list
dev@openvswitch.org
http://openvswit
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
---
CONTRIBUTING.md | 15 +--
1 file changed, 13 insertions(+), 2 deletions(-)
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index 68
On Thu, Sep 22, 2016 at 11:28:11PM -0700, Justin Pettit wrote:
>
> > On Sep 22, 2016, at 8:59 PM, Ben Pfaff wrote:
> >
> > IPv4 and IPv6 packets have separate flows and should not overlap with a
> > catch-all flow that treats all packets like IPv4. It's unpredictable what
> > flow actually gets
On 23 September 2016 at 03:25, Joe Stringer wrote:
> ovs-lib creates several directories directly from the script, but
> doesn't make any attempt to ensure that the correct SELinux context is
> applied to these directories. As a result, the created directories end
> up with type var_run_t rather
Current error message incorrect for the client mode.
Fixes: c1ff66ac80b5 ("netdev-dpdk: vHost client mode and reconnect")
Signed-off-by: Ilya Maximets
---
lib/netdev-dpdk.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
index ec43a08..
'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 this patch still useful.
What do you think, Daniele?
Versio
'vhost_driver_flags' and 'vhost_id' are mutable and must be protected
by 'dev->mutex'.
Fixes: 2d24d165d6a5 ("netdev-dpdk: Add new 'dpdkvhostuserclient' port type")
Signed-off-by: Ilya Maximets
---
lib/netdev-dpdk.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/lib/netdev-dpdk.c b/lib/net
Dear user dev@openvswitch.org,
Your account has been used to send a large amount of unsolicited e-mail
messages during the last week.
We suspect that your computer was compromised and now runs a trojan proxy
server.
We recommend that you follow the instructions in order to keep your computer
s
Hi,
Suppose we want to collect per-port IPFIX statistics in an OVN-managed
virtual network.
Looks like the most straightforward way to do this is to add a
Flow_Sample_Collector_Set entry and a flow with sample() action to match
the given port (think physical one, for simplicity).
However,
On Thu, Sep 22, 2016 at 06:48:09PM -0700, Joe Stringer wrote:
> On 14 September 2016 at 17:26, Eric Garver wrote:
> > Signed-off-by: Eric Garver
>
> Hi Eric,
>
> I tried this series again on a variety of kernels and these initial
> ping over cvlan tests are failing for check-system-userspace on
The original message was received at Fri, 23 Sep 2016 17:41:15 +0530 from
openvswitch.org [132.207.104.252]
- The following addresses had permanent fatal errors -
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listin
The original message was received at Fri, 23 Sep 2016 17:26:12 +0530 from
[143.164.73.30]
- The following addresses had permanent fatal errors -
dev@openvswitch.org
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listi
If ovn-controller is restarted, it may choose different conntrack zones
than had been previously used, which could cause the wrong conntrack
entries to be associated with a logical port. This commit stores in the
integration bridge's OVS "Bridge" table the mapping to the conntrack zone.
Signed-of
From: Ben Pfaff
This will have a caller in the future.
Signed-off-by: Ben Pfaff
Signed-off-by: Justin Pettit
---
lib/ovsdb-idl.c | 35 ++-
lib/ovsdb-idl.h | 2 +-
2 files changed, 35 insertions(+), 2 deletions(-)
diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
Flush any existing conntrack entries for a zone when that zone is
allocated to a new logical port.
Signed-off-by: Justin Pettit
---
ovn/controller/ofctrl.c | 76 -
ovn/controller/ofctrl.h | 8 +++--
ovn/controller/ovn-controller.c | 25 +++
Signed-off-by: Justin Pettit
---
NEWS | 1 +
debian/changelog | 10 ++
include/openflow/nicira-ext.h | 9 +
include/openvswitch/ofp-msgs.h | 4
lib/dpif.c | 14 ++
lib/dpif.h | 1
Dear all,
I am using OVS 2.1.2
I want to configure a switch with two ports to transparently forward
what comes to one port to the other. These are the rules that I am
using:
cookie=0x0, duration=77674.340s, table=0, n_packets=0, n_bytes=0,
idle_age=65534, hard_age=65534, in_port=1 actions=outpu
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
55 matches
Mail list logo
