On Fri, Oct 25, 2013 at 03:02:00PM -0700, Joe Stringer wrote:
> On Fri, Oct 25, 2013 at 1:12 PM, Ben Pfaff wrote:
>
> > From: Venkitachalam Gopalakrishnan
> >
> > Open vSwitch has never implemented this request and reply, even though they
> > have been in OpenFlow since version 1.0. This commit
Dear Sir/Madam,
Glad to learn that you may purchase injection tools, molded plastic products or
mould core part. We are a medium-sized manufacturer of plastic injection tools,
molded plastic products and mould core parts in China. By writing this email we
are in view of establishing a good bus
Commit 348f01e3e3 (cfm: Eight byte MPIDs in extended mode.) allows
eight byte MPIDs when running in extended mode. This commit explains
this change in the vswitch.conf.db.
Signed-off-by: Alex Wang
---
vswitchd/vswitch.xml | 21 ++---
1 file changed, 14 insertions(+), 7 deletio
Adds additional indices to classifier tables to be able to
stop matching as soon as it is evident that a match will
not be found or if a single matching rule is found.
Especaially avoids folding in the transport port masks,
if they need not be examined.
The struct flow is re-organized to allow has
From: Mark Hamilton
For example, if the value is ipv4(src=192.168.0.1/255.255.0.0)
the output from top is ipv4(src=192.168.0.0/255.255.0.0). Prior
to this change, the original flow field content was shown
unaltered even though flow statistics where aggregated among
several flows.
If the flow fie
On Fri, Oct 25, 2013 at 10:49 AM, Romain Lenglet wrote:
> On Oct 24, 2013, at 5:46 PM, Jesse Gross wrote:
>
>> On Thu, Oct 24, 2013 at 3:39 PM, Romain Lenglet wrote:
>>> - Original Message -
From: "Jesse Gross"
To: "Romain Lenglet"
Cc: dev@openvswitch.org
Sent: Tues
I pushed this patch to master.
Thanks.
On Fri, Oct 25, 2013 at 2:41 PM, Jesse Gross wrote:
> On Fri, Oct 25, 2013 at 1:42 PM, Pravin B Shelar wrote:
>> OVS already can handle all types of segmentation offloads that
>> are supported by the kernel.
>> Following patch specifically enables UDP and I
Acked-by: Joe Stringer
On Fri, Oct 25, 2013 at 1:12 PM, Ben Pfaff wrote:
> Message types are defined in ofp-msgs.h these days.
>
> Signed-off-by: Ben Pfaff
> ---
> include/openflow/openflow-1.2.h |9 -
> 1 file changed, 9 deletions(-)
>
> diff --git a/include/openflow/openflow-1.
On Fri, Oct 25, 2013 at 1:42 PM, Pravin B Shelar wrote:
> OVS already can handle all types of segmentation offloads that
> are supported by the kernel.
> Following patch specifically enables UDP and IPV6 segmentation
> offloads.
>
> Signed-off-by: Pravin B Shelar
Acked-by: Jesse Gross
_
On Fri, Oct 25, 2013 at 02:04:44PM -0700, Jarno Rajahalme wrote:
>
> On Oct 25, 2013, at 1:56 PM, Ben Pfaff wrote:
>
> > On Fri, Oct 25, 2013 at 01:44:17PM -0700, Jarno Rajahalme wrote:
> >>
> >> On Oct 25, 2013, at 1:32 PM, Ben Pfaff wrote:
> >>
> >>> On Thu, Oct 24, 2013 at 02:08:24PM -0700
Applied, thanks.
On Fri, Oct 25, 2013 at 02:01:01PM -0700, Jarno Rajahalme wrote:
> Nice clean-up too :-)
>
> Acked-by: Jarno Rajahalme
>
> On Oct 25, 2013, at 1:53 PM, Ben Pfaff wrote:
>
> > Until now this code has called ofpacts_check() directly, but that omits
> > the additional check that
On Oct 25, 2013, at 1:56 PM, Ben Pfaff wrote:
> On Fri, Oct 25, 2013 at 01:44:17PM -0700, Jarno Rajahalme wrote:
>>
>> On Oct 25, 2013, at 1:32 PM, Ben Pfaff wrote:
>>
>>> On Thu, Oct 24, 2013 at 02:08:24PM -0700, Jarno Rajahalme wrote:
On Oct 21, 2013, at 3:52 PM, Ben Pfaff
Nice clean-up too :-)
Acked-by: Jarno Rajahalme
On Oct 25, 2013, at 1:53 PM, Ben Pfaff wrote:
> Until now this code has called ofpacts_check() directly, but that omits
> the additional check that the ofproto_check_ofpacts() wrapper includes.
>
> Reported-by: Jarno Rajahalme
> Signed-off-by:
On Fri, Oct 25, 2013 at 01:44:17PM -0700, Jarno Rajahalme wrote:
>
> On Oct 25, 2013, at 1:32 PM, Ben Pfaff wrote:
>
> > On Thu, Oct 24, 2013 at 02:08:24PM -0700, Jarno Rajahalme wrote:
> >>
> >>
> >> On Oct 21, 2013, at 3:52 PM, Ben Pfaff wrote:
> >>
> >>> Commit e3b5693319c (Fix table chec
Until now this code has called ofpacts_check() directly, but that omits
the additional check that the ofproto_check_ofpacts() wrapper includes.
Reported-by: Jarno Rajahalme
Signed-off-by: Ben Pfaff
---
ofproto/ofproto.c | 16 +++-
1 file changed, 7 insertions(+), 9 deletions(-)
d
OVS already can handle all types of segmentation offloads that
are supported by the kernel.
Following patch specifically enables UDP and IPV6 segmentation
offloads.
Signed-off-by: Pravin B Shelar
---
datapath/vport-internal_dev.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff
On Oct 25, 2013, at 1:32 PM, Ben Pfaff wrote:
> On Thu, Oct 24, 2013 at 02:08:24PM -0700, Jarno Rajahalme wrote:
>>
>>
>> On Oct 21, 2013, at 3:52 PM, Ben Pfaff wrote:
>>
>>> Commit e3b5693319c (Fix table checking for goto table instruction.) moved
>>> action checking into modify_flows__(),
On Thu, Oct 24, 2013 at 02:08:24PM -0700, Jarno Rajahalme wrote:
>
>
> On Oct 21, 2013, at 3:52 PM, Ben Pfaff wrote:
>
> > Commit e3b5693319c (Fix table checking for goto table instruction.) moved
> > action checking into modify_flows__(), for good reason, but as a side
> > effect made modify_f
Message types are defined in ofp-msgs.h these days.
Signed-off-by: Ben Pfaff
---
include/openflow/openflow-1.2.h |9 -
1 file changed, 9 deletions(-)
diff --git a/include/openflow/openflow-1.2.h b/include/openflow/openflow-1.2.h
index 541b143..249e861 100644
--- a/include/openflow/o
v1->v2:
- Rebase and fix a conflict
- Add signed-off-by
- Make two separate patches into a 2-patch series, because they need
to be applied in the correct order.
Ben Pfaff (1):
openflow-1.2.h: Remove redundant definition of
OFPST12_GROUP_FEATURES.
Venkitachalam Gopalakrishnan (1):
From: Venkitachalam Gopalakrishnan
Open vSwitch has never implemented this request and reply, even though they
have been in OpenFlow since version 1.0. This commit adds an
implementation.
Signed-off: Venkitachalam Gopalakrishnan
Co-authored-by: Ben Pfaff
Signed-off-by: Ben Pfaff
---
include
On Fri, Oct 25, 2013 at 10:25 AM, Jarno Rajahalme wrote:
>
> On Oct 25, 2013, at 9:27 AM, Jesse Gross wrote:
>
>> On Fri, Oct 25, 2013 at 9:13 AM, Jarno Rajahalme
>> wrote:
>>>
On Oct 25, 2013, at 9:06 AM, Jesse Gross wrote:
> On Thu, Oct 24, 2013 at 9:01 AM, Jarno Rajahalme
>>
Thanks a lot, I added this to the patch.
On Fri, Oct 25, 2013 at 11:40:22AM -0700, Venkitachalam Gopalakrishnan wrote:
> Signed-off: Venkitachalam Gopalakrishnan
>
> On Oct 24, 2013, at 3:55 PM, Ben Pfaff wrote:
>
> > On Thu, Oct 17, 2013 at 02:20:33PM -0700, Ben Pfaff wrote:
> >> On Thu, Oct
On Tue, Oct 22, 2013 at 10:03:08AM -0700, Joe Stringer wrote:
> jsonrpc_session_recv() handles echo replies prior to this.
>
> Found by inspection.
>
> Signed-off-by: Joe Stringer
Applied, thanks.
___
dev mailing list
dev@openvswitch.org
http://openvs
Signed-off: Venkitachalam Gopalakrishnan
On Oct 24, 2013, at 3:55 PM, Ben Pfaff wrote:
> On Thu, Oct 17, 2013 at 02:20:33PM -0700, Ben Pfaff wrote:
>> On Thu, Oct 17, 2013 at 09:35:27AM -0700, Ben Pfaff wrote:
>>> From: Venkitachalam Gopalakrishnan
>>
>> Thanks gops.
>>
>> I'm spending a lit
jsonrpc_session_recv() handles echo replies prior to this.
Found by inspection.
Signed-off-by: Joe Stringer
---
lib/ovsdb-idl.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
index 4631291..3366523 100644
--- a/lib/ovsdb-idl.c
+++ b/lib/ovsdb-idl.c
@@
On Fri, Oct 25, 2013 at 10:46:13AM -0700, Gurucharan Shetty wrote:
> >
> > I changed my directory name to include a directory with ">>" in it.
> > And this test failed for me.
> > It was not supposed to fail, right?
> Me inserting a ">>" is not actually unicode but rather just 2 ">". So
> non_ascii
On Fri, Oct 25, 2013 at 9:22 AM, Ben Pfaff wrote:
> This suppresses a testsuite failure when the testsuite is run from a
> directory whose name contains a non-ASCII character. I'd rather fix the
> problem but webpages like the following make it sound difficult or
> impossible on Python 2.x: http:
On Oct 24, 2013, at 5:46 PM, Jesse Gross wrote:
> On Thu, Oct 24, 2013 at 3:39 PM, Romain Lenglet wrote:
>> - Original Message -
>>> From: "Jesse Gross"
>>> To: "Romain Lenglet"
>>> Cc: dev@openvswitch.org
>>> Sent: Tuesday, October 22, 2013 3:46:54 PM
>>> Subject: Re: [ovs-dev] [PATCH
Never mind.
The change is to skip the test and not just
On Fri, Oct 25, 2013 at 10:29 AM, Gurucharan Shetty wrote:
> On Fri, Oct 25, 2013 at 9:22 AM, Ben Pfaff wrote:
>> The Debian autobuild machines run their builds inside a top-level
>> directory named /«BUILDDIR». That name contains two non-
On Thu, Oct 24, 2013 at 7:59 PM, Ben Pfaff wrote:
> On Thu, Oct 24, 2013 at 06:27:10PM -0700, Ansis Atteka wrote:
>> It is more obvious to figure out what exactly test is doing
>> if ovs-vsctl commands contain full table names.
>>
>> Signed-off-by: Ansis Atteka
>
> Acked-by: Ben Pfaff
Thanks, I
On Fri, Oct 25, 2013 at 9:22 AM, Ben Pfaff wrote:
> The Debian autobuild machines run their builds inside a top-level
> directory named /«BUILDDIR». That name contains two non-ASCII characters
> that cause the Python code in the interface-reconfigure and ovs-monitor-ipsec
> tests to fail. The pr
On Oct 25, 2013, at 9:27 AM, Jesse Gross wrote:
> On Fri, Oct 25, 2013 at 9:13 AM, Jarno Rajahalme
> wrote:
>>
>>> On Oct 25, 2013, at 9:06 AM, Jesse Gross wrote:
>>>
On Thu, Oct 24, 2013 at 9:01 AM, Jarno Rajahalme
wrote:
diff --git a/datapath/flow_netlink.c b/datapath/flo
On Fri, Oct 25, 2013 at 9:13 AM, Jarno Rajahalme wrote:
>
>> On Oct 25, 2013, at 9:06 AM, Jesse Gross wrote:
>>
>>> On Thu, Oct 24, 2013 at 9:01 AM, Jarno Rajahalme
>>> wrote:
>>> diff --git a/datapath/flow_netlink.c b/datapath/flow_netlink.c
>>> index 515a9f6..fc6f42e 100644
>>> --- a/datapath
These patches fix testsuite failures that appeared on the Debian
autobuilder machines, e.g.
https://buildd.debian.org/status/fetch.php?pkg=openvswitch&arch=amd64&ver=1.9.3%2Bgit20131021-1&stamp=1382373784
These patches should also apply to master but I started with branch-1.9
because that
The Debian autobuild machines run their builds inside a top-level
directory named /«BUILDDIR». That name contains two non-ASCII characters
that cause the Python code in the interface-reconfigure and ovs-monitor-ipsec
tests to fail. The problem seems to be related to the locale, because
switching
This suppresses a testsuite failure when the testsuite is run from a
directory whose name contains a non-ASCII character. I'd rather fix the
problem but webpages like the following make it sound difficult or
impossible on Python 2.x: http://stackoverflow.com/a/11742928
Signed-off-by: Ben Pfaff
-
> On Oct 25, 2013, at 9:06 AM, Jesse Gross wrote:
>
>> On Thu, Oct 24, 2013 at 9:01 AM, Jarno Rajahalme
>> wrote:
>> diff --git a/datapath/flow_netlink.c b/datapath/flow_netlink.c
>> index 515a9f6..fc6f42e 100644
>> --- a/datapath/flow_netlink.c
>> +++ b/datapath/flow_netlink.c
>> @@ -154,8 +1
On Thu, Oct 24, 2013 at 9:01 AM, Jarno Rajahalme wrote:
> diff --git a/datapath/flow_netlink.c b/datapath/flow_netlink.c
> index 515a9f6..fc6f42e 100644
> --- a/datapath/flow_netlink.c
> +++ b/datapath/flow_netlink.c
> @@ -154,8 +155,10 @@ static bool match_validate(const struct sw_flow_match
> *
From: Joe Stringer
OpenFlow 1.1 and 1.2, and 1.3 differ in their handling of MPLS actions in the
presence of VLAN tags. To allow correct behaviour to be committed in
each situation, this patch adds a second round of VLAN tag action
handling to commit_odp_actions(), which occurs after MPLS actions
From: Joe Stringer
This patch modifies the push_mpls behaviour to allow
pushing of an MPLS LSE either before any VLAN tag that may be present.
Pushing the MPLS LSE before any VLAN tag that is present is the
behaviour specified in OpenFlow 1.3.
Pushing the MPLS LSE after the any VLAN tag that is
Hi,
This series implements MPLS actions and matches based on work by
Ravi K, Leo Alterman, Yamahata-san and Joe Stringer.
This series provides two changes
* Patches 1 - 2
Provide user-space support for the VLAN/MPLS tag insertion order
up to and including OpenFlow 1.2, and the different ord
Allow datapath to recognize and extract MPLS labels into flow keys
and execute actions which push, pop, and set labels on packets.
Based heavily on work by Leo Alterman, Ravi K, Isaku Yamahata and Joe Stringer.
Cc: Ravi K
Cc: Leo Alterman
Cc: Isaku Yamahata
Cc: Joe Stringer
Signed-off-by: Sim
Break out deacceleration portion of vlan_push into vlan_put
so that it may be re-used by mpls_push.
For both vlan_push and mpls_push if there is an accelerated VLAN tag
present then it should be deaccelerated, adding it to the data of
the skb, before the new tag is added.
Signed-off-by: Simon Hor
On Thu, Oct 24, 2013 at 01:19:34PM -0700, Jarno Rajahalme wrote:
> Signed-off-by: Jarno Rajahalme
This looks good to me.
Reviewed-by: Simon Horman
> ---
> v3: Fix byte order errors.
>
> include/openflow/nicira-ext.h | 24
> lib/ofp-actions.c | 82
>
45 matches
Mail list logo
