Looks good.
--Justin
On May 1, 2013, at 5:13 PM, Ben Pfaff wrote:
> MAX_PORTS is currently USHRT_MAX (also 65535). I think that's a
> coincidence; I don't remember MAX_PORTS being mentioned when the new
> dpif_channel code was written.
>
> Signed-off-by: Ben Pfaff
> ---
> lib/dpif-linux.c |
On Wed, May 01, 2013 at 09:53:42PM -0700, Jesse Gross wrote:
> On Wed, May 1, 2013 at 3:57 PM, Simon Horman wrote:
> > On Wed, May 01, 2013 at 11:16:40AM -0700, Jesse Gross wrote:
> >> On Wed, May 1, 2013 at 12:50 AM, Simon Horman wrote:
> >> > On Tue, Apr 30, 2013 at 09:19:51AM -0700, Jesse Gros
On Wed, May 1, 2013 at 3:57 PM, Simon Horman wrote:
> On Wed, May 01, 2013 at 11:16:40AM -0700, Jesse Gross wrote:
>> On Wed, May 1, 2013 at 12:50 AM, Simon Horman wrote:
>> > On Tue, Apr 30, 2013 at 09:19:51AM -0700, Jesse Gross wrote:
>> >> On Mon, Apr 29, 2013 at 8:21 PM, Simon Horman wrote:
On Wed, May 01, 2013 at 09:42:18PM -0700, Jesse Gross wrote:
> On Wed, May 1, 2013 at 1:11 AM, Simon Horman wrote:
> > learn_parse() needs to know the current ethertype, including any
> > updates made by MPLS push and pop actions in order to determine if
> > access to fields is valid or not.
> >
>
On Wed, May 01, 2013 at 09:19:46PM -0700, Jesse Gross wrote:
> On Wed, May 1, 2013 at 1:11 AM, Simon Horman wrote:
> > Some actions require an MPLS dl_type while others require a non-MPLS
> > dl_type. Enforce this for actions handled in ofpact_check__().
> >
> > Update the following actions to req
On Wed, May 1, 2013 at 1:11 AM, Simon Horman wrote:
> learn_parse() needs to know the current ethertype, including any
> updates made by MPLS push and pop actions in order to determine if
> access to fields is valid or not.
>
> Changes made to ethertype my VLAN push and pop actions are excluded
>
The ethertype should always be updated on mpls_pop
as there may be a transition between MPLS unicast (0x8847) and
MPLS multicast (0x8848).
Ben Pfaff tells me that this is consistent with the
behaviour described in EXT-194 of the JIRA bug tracker.
Signed-off-by: Simon Horman
---
v2
* Add note a
On Wed, May 1, 2013 at 1:11 AM, Simon Horman wrote:
> Some actions require an MPLS dl_type while others require a non-MPLS
> dl_type. Enforce this for actions handled in ofpact_check__().
>
> Update the following actions to require an non-MPLS dl_type:
>
> set_ipv4_src
> set_ipv4_dst
> set_ipv4_ds
Thanx Ansis for reply.
On Wed, May 1, 2013 at 10:26 PM, Ansis Atteka wrote:
>
>
>
> On Tue, Apr 30, 2013 at 11:27 PM, freak 62 wrote:
>
>> Hi all,
>>
>>
>> I am using Ubuntu 12.10. I am trying to launch VM on the bridge of
>> OVS.
>> Can I do that with virt-manager ? or Is there any o
On Wed, May 1, 2013 at 6:49 PM, Simon Horman wrote:
> Note that OVS_KEY_ATTR_MPLS may be an array of ovs_key_mpls
> and that the acceptable length may be restricted by the implementation.
>
> Currently the user-space datapath and proposed kernel datapath
> implementation restrict the length to a s
On Wed, May 1, 2013 at 3:30 PM, Thomas F Herbert
wrote:
> All,
>
> I would like to introduce once again the subject of stacked vlans. We have
> implemented a network edge device using 802.1ad. It is currently implemented
> on Linux with iproute2 and a patched kernel but we would like to
> re-impl
On Wed, May 1, 2013 at 6:31 PM, Jesse Gross wrote:
> On Wed, May 1, 2013 at 4:02 PM, Pravin B Shelar wrote:
>> VPORT_F_TUN_ID is last remaining flag, once we remove it, flags
>> field from vport-ops can be removed. Since it does not complicate
>> much code, we decided to remove this flag and for
Note that OVS_KEY_ATTR_MPLS may be an array of ovs_key_mpls
and that the acceptable length may be restricted by the implementation.
Currently the user-space datapath and proposed kernel datapath
implementation restrict the length to a single element.
Also update the mpls_top_lse name of the eleme
On Wed, May 1, 2013 at 4:02 PM, Pravin B Shelar wrote:
> VPORT_F_TUN_ID is last remaining flag, once we remove it, flags
> field from vport-ops can be removed. Since it does not complicate
> much code, we decided to remove this flag and force vport modules
> assign correct tun_key.
>
> Signed-off
On Wed, May 1, 2013 at 4:02 PM, Pravin B Shelar wrote:
> vport->init and exit() functios are defined by gre and netdev vport
> only and both can be moved to first port create.
>
> Following patch does same, it moves vport init to respectve vport
> create and get rid of vport->init() and vport->exi
On Wed, May 1, 2013 at 4:02 PM, Pravin B Shelar wrote:
> Following patch changes vport->send return type so that vport
> layer can do error accounting.
>
> Signed-off-by: Pravin B Shelar
This is definitely an improvement:
Acked-by: Jesse Gross
Just one question:
> diff --git a/datapath/vport.
On Wed, May 1, 2013 at 4:02 PM, Pravin B Shelar wrote:
> After flow based tunneling, kernel tunneling is greatly simplified.
> There is no need to have extra tunning layer between vport and
> perticular protocol.
> Following patch removes tunneling struct which make code easy to read.
>
> Signed-o
When ovs-vswitchd deletes a port with dpif_linux_port_del(), that function
uses del_channel() to delete the corresponding channel, including closing
its Netlink socket fd. However, if the vport gets removed by some other
process (e.g. "ip link delete" for veths) then this function never gets
calle
This series fixes a fd leak in the dpif-linux implementation.
The first patch is a simple partial fix suitable for backporting.
The remaining patches are a complete solution.
Ben Pfaff (4):
dpif-linux: Close channel Netlink sockets when a port number gets
recycled.
dpif-linux: Factor out p
Until now, the code in dpif_linux_recv_set() has assumed it is starting
from a clean slate with no channels at all. This commit generalizes it to
compare the existing set of channels against the ones that should exist
given the vports that the kernel has, adding any new ones that are missing
and r
These helpers simplify new code to be added in an upcoming commit.
Signed-off-by: Ben Pfaff
---
lib/dpif-linux.c | 46 +-
1 files changed, 33 insertions(+), 13 deletions(-)
diff --git a/lib/dpif-linux.c b/lib/dpif-linux.c
index 933aabe..b80c27a 1006
When ovs-vswitchd deletes a port with dpif_linux_port_del(), that function
uses del_channel() to delete the corresponding channel, including closing
its Netlink socket fd. However, if the vport gets removed by some other
process (e.g. "ip link delete" for veths) then this function never gets
calle
MAX_PORTS is currently USHRT_MAX (also 65535). I think that's a
coincidence; I don't remember MAX_PORTS being mentioned when the new
dpif_channel code was written.
Signed-off-by: Ben Pfaff
---
lib/dpif-linux.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/dpif-li
On Thu, May 02, 2013 at 08:27:48AM +0900, Simon Horman wrote:
> On Wed, May 01, 2013 at 03:53:37PM -0700, Ben Pfaff wrote:
> > On Thu, May 02, 2013 at 07:40:53AM +0900, Simon Horman wrote:
> > > On Wed, May 01, 2013 at 09:54:03AM -0700, Ben Pfaff wrote:
> > > > On Wed, May 01, 2013 at 05:06:02PM +0
On Wed, May 01, 2013 at 03:53:37PM -0700, Ben Pfaff wrote:
> On Thu, May 02, 2013 at 07:40:53AM +0900, Simon Horman wrote:
> > On Wed, May 01, 2013 at 09:54:03AM -0700, Ben Pfaff wrote:
> > > On Wed, May 01, 2013 at 05:06:02PM +0900, Simon Horman wrote:
> > > > The ethertype should always be update
VPORT_F_TUN_ID is last remaining flag, once we remove it, flags
field from vport-ops can be removed. Since it does not complicate
much code, we decided to remove this flag and force vport modules
assign correct tun_key.
Signed-off-by: Pravin B Shelar
---
v1-v3:
New patch.
---
datapath/vport-gre
vport->init and exit() functios are defined by gre and netdev vport
only and both can be moved to first port create.
Following patch does same, it moves vport init to respectve vport
create and get rid of vport->init() and vport->exit() fnctions.
Signed-off-by: Pravin B Shelar
---
v2-v3:
- Fix
Following patch changes vport->send return type so that vport
layer can do error accounting.
Signed-off-by: Pravin B Shelar
---
v1-v3:
New patch.
---
datapath/tunnel.c | 33 ++---
datapath/vport-gre.c| 14 --
datapath/vport-lisp.c | 13 ++
After flow based tunneling, kernel tunneling is greatly simplified.
There is no need to have extra tunning layer between vport and
perticular protocol.
Following patch removes tunneling struct which make code easy to read.
Signed-off-by: Pravin B Shelar
---
v2-v3:
- Fixed error handling in ovs_t
On Wed, May 01, 2013 at 11:16:40AM -0700, Jesse Gross wrote:
> On Wed, May 1, 2013 at 12:50 AM, Simon Horman wrote:
> > On Tue, Apr 30, 2013 at 09:19:51AM -0700, Jesse Gross wrote:
> >> On Mon, Apr 29, 2013 at 8:21 PM, Simon Horman wrote:
> >> > On Fri, Apr 26, 2013 at 04:03:21PM -0700, Jesse Gro
On Thu, May 02, 2013 at 07:40:53AM +0900, Simon Horman wrote:
> On Wed, May 01, 2013 at 09:54:03AM -0700, Ben Pfaff wrote:
> > On Wed, May 01, 2013 at 05:06:02PM +0900, Simon Horman wrote:
> > > The ethertype should always be updated on mpls_pop
> > > as there may be a transition between MPLS unica
On Wed, May 01, 2013 at 09:54:03AM -0700, Ben Pfaff wrote:
> On Wed, May 01, 2013 at 05:06:02PM +0900, Simon Horman wrote:
> > The ethertype should always be updated on mpls_pop
> > as there may be a transition between MPLS unicast (0x8847) and
> > MPLS multicast (0x8848).
> >
> > Signed-off-by: S
All,
I would like to introduce once again the subject of stacked vlans. We
have implemented a network edge device using 802.1ad. It is currently
implemented on Linux with iproute2 and a patched kernel but we would
like to re-implement it with openvswitch.
I have read earlier discussion in t
On May 1, 2013, at 2:33 PM, Ben Pfaff wrote:
> On Wed, May 01, 2013 at 02:31:31PM -0700, Justin Pettit wrote:
>> Signed-off-by: Justin Pettit
>
> Looks good, thanks.
Thanks. I pushed this to master, branch-1.11, and branch-1.10.
--Justin
___
dev
On May 1, 2013, at 2:45 PM, Ben Pfaff wrote:
> On Wed, May 01, 2013 at 02:42:07PM -0700, Justin Pettit wrote:
>> This reverts commit 9f31efc731090e8d33236c55a4adbf461f3acbd1.
>>
>> This is the 1.11.0 release, so this patch doesn't belong.
>> ---
>> This only applies to "branch-1.11".
>
> Looks
On Wed, May 01, 2013 at 02:42:07PM -0700, Justin Pettit wrote:
> This reverts commit 9f31efc731090e8d33236c55a4adbf461f3acbd1.
>
> This is the 1.11.0 release, so this patch doesn't belong.
> ---
> This only applies to "branch-1.11".
Looks good.
___
dev
This reverts commit 9f31efc731090e8d33236c55a4adbf461f3acbd1.
This is the 1.11.0 release, so this patch doesn't belong.
---
This only applies to "branch-1.11".
---
NEWS |4
configure.ac |2 +-
debian/changelog |7 ---
3 files changed, 1 insertions(+), 12 delet
On Wed, May 01, 2013 at 02:31:31PM -0700, Justin Pettit wrote:
> Signed-off-by: Justin Pettit
Looks good, thanks.
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Signed-off-by: Justin Pettit
---
NEWS |2 +-
debian/changelog |2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/NEWS b/NEWS
index 2c57d9b..3a7123b 100644
--- a/NEWS
+++ b/NEWS
@@ -24,7 +24,7 @@ v1.11.0 - xx xxx
will have to manually configure it
On Wed, May 01, 2013 at 01:21:54PM -0700, Duffie Cooley wrote:
> Forgot to add myself to the AUTHORS file.
I already did this (I folded it into the previous patch).
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Forgot to add myself to the AUTHORS file.
Signed-off-by: Duffie Cooley
---
AUTHORS | 1 +
1 file changed, 1 insertion(+)
diff --git a/AUTHORS b/AUTHORS
index b314ce6..8dd06b5 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -27,6 +27,7 @@ David Erickson deric...@stanford.edu
David S. Miller
I couldn't apply this from the email for some reason (whitespace
damage?) but since it was a one-liner I fixed it up.
Thanks a lot! Applied to master and branch-1.11.
On Wed, May 01, 2013 at 11:50:08AM -0700, Duffie Cooley wrote:
> Found a bug in the ovs-sandbox script. We should use the well
>
Found a bug in the ovs-sandbox script. We should use the well defined
$schema var to create the db. Not the $srcdir/dir stuff.
Thanks,
Duffie
Signed-off-by: Duffie Cooley
---
tutorial/ovs-sandbox | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tutorial/ovs-sandbox b/tut
POSIX says that multithreaded programs must not use sigprocmask() but must
use pthread_sigmask() instead. This commit makes that replacement.
The actual use of signals in Open vSwitch is still not thread safe
following this commit, but this change is a necessary prerequisite for
fixing the other
Signed-off-by: Ben Pfaff
---
lib/socket-util.c | 47 +--
1 files changed, 37 insertions(+), 10 deletions(-)
diff --git a/lib/socket-util.c b/lib/socket-util.c
index 4f9b5b8..4b559cb 100644
--- a/lib/socket-util.c
+++ b/lib/socket-util.c
@@ -185,23 +1
getmntent_r() is a GNU extension so we test for its existence and just
disable this feature of system stats if it is not present, because this
feature is not very important.
Signed-off-by: Ben Pfaff
---
configure.ac|2 +-
vswitchd/system-stats.c |8 +---
2 files changed,
Signed-off-by: Ben Pfaff
---
lib/dynamic-string.c | 12 ++--
lib/table.c |5 +++--
ovsdb/ovsdb-tool.c|3 ++-
utilities/ovs-ofctl.c |4 +++-
4 files changed, 14 insertions(+), 10 deletions(-)
diff --git a/lib/dynamic-string.c b/lib/dynamic-string.c
index bd1cf
strsignal() isn't thread-safety but sys_siglist[] is. Also, according to
the glibc manual, BSD has sys_siglist[] but not strsignal().
Signed-off-by: Ben Pfaff
---
configure.ac |3 ++-
lib/signals.c |8 ++--
2 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/configure.ac
Signed-off-by: Ben Pfaff
---
lib/fatal-signal.c |4 +++-
lib/process.c | 10 --
lib/signals.c | 29 +++--
lib/signals.h |4 +++-
4 files changed, 29 insertions(+), 18 deletions(-)
diff --git a/lib/fatal-signal.c b/lib/fatal-signal.c
inde
Without this change, an initialization such as
const struct hmap map = HMAP_INITIALIZER(&map);
yields a compiler warning "initialization discards qualifiers from pointer
target type".
Signed-off-by: Ben Pfaff
---
lib/hmap.h |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff -
This makes this code more obviously thread-safe.
Signed-off-by: Ben Pfaff
---
lib/netdev.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/netdev.c b/lib/netdev.c
index 156984a..434e257 100644
--- a/lib/netdev.c
+++ b/lib/netdev.c
@@ -1115,7 +1115,7 @@ netdev_set_qo
This changes allows the compiler to put 'names' in a read-only section.
Signed-off-by: Ben Pfaff
---
lib/ofp-util.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index 42c3613..4d3fe9e 100644
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
These data structures are never modified so it seems logical for them
to be const.
Signed-off-by: Ben Pfaff
---
lib/vconn-provider.h | 20 ++--
lib/vconn-stream.c | 20 ++--
lib/vconn.c | 30 +++---
3 files changed, 35 inse
These data structures are never modified so this seems like a
logical change.
Signed-off-by: Ben Pfaff
---
lib/vlog.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/vlog.c b/lib/vlog.c
index 8bc9938..275d02a 100644
--- a/lib/vlog.c
+++ b/lib/vlog.c
@@ -49,14 +4
In preparation for multithreading OVS, it makes sense to mark any
static data that we can "const", because read-only access to data
is obviously thread-safe.
Ben Pfaff (9):
dpif-linux: Make dummy_action const in dpif_linux_init_flow_put().
Make most "struct option" instances "const".
hmap: M
This makes this code more obviously thread-safe.
Signed-off-by: Ben Pfaff
---
lib/netdev-linux.c | 30 ++
1 files changed, 14 insertions(+), 16 deletions(-)
diff --git a/lib/netdev-linux.c b/lib/netdev-linux.c
index c0e0c40..722b88b 100644
--- a/lib/netdev-linux.c
These data structures are never modified so this seems like a
logical change.
Signed-off-by: Ben Pfaff
---
lib/stream-fd.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/stream-fd.c b/lib/stream-fd.c
index 1b16f3e..90d328a 100644
--- a/lib/stream-fd.c
+++ b/lib/s
This makes this code more obviously thread-safe.
Signed-off-by: Ben Pfaff
---
lib/dpif-linux.c |6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/lib/dpif-linux.c b/lib/dpif-linux.c
index b863a2e..2aa09ff 100644
--- a/lib/dpif-linux.c
+++ b/lib/dpif-linux.c
@@ -794,7 +
Reducing non-const static data makes code more obviously thread-safe.
Although option parsing does not normally need to be thread-safe, I
don't know of a drawback to making its data const.
Signed-off-by: Ben Pfaff
---
ovsdb/ovsdb-client.c |4 ++--
ovsdb/ovsdb-server.c |2 +-
On Wed, May 1, 2013 at 12:50 AM, Simon Horman wrote:
> On Tue, Apr 30, 2013 at 09:19:51AM -0700, Jesse Gross wrote:
>> On Mon, Apr 29, 2013 at 8:21 PM, Simon Horman wrote:
>> > On Fri, Apr 26, 2013 at 04:03:21PM -0700, Jesse Gross wrote:
>> >> On Thu, Apr 25, 2013 at 12:36 AM, Simon Horman wrote
The ports that exist within a dpif have already been translated through
netdev_vport_get_dpif_port(), so there is no value to translating them
again in the interfaces that query or dump ports (and possibly a drawback
if somehow the translation could change).
After this change, dpif-netdev translat
Returning a static data buffer makes code more brittle and definitely
not thread-safe, so this commit switches to using a caller-provided
buffer instead.
Signed-off-by: Ben Pfaff
---
lib/dpif-linux.c |4 +++-
lib/dpif-netdev.c |9 ++---
lib/netdev-vport.c | 18 ++
Signed-off-by: Ben Pfaff
---
lib/netdev-vport.c | 14 --
1 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/lib/netdev-vport.c b/lib/netdev-vport.c
index 08169a1..a9911d3 100644
--- a/lib/netdev-vport.c
+++ b/lib/netdev-vport.c
@@ -126,7 +126,6 @@ netdev_vport_get_dpif_p
On Tue, Apr 30, 2013 at 11:27 PM, freak 62 wrote:
> Hi all,
>
>
> I am using Ubuntu 12.10. I am trying to launch VM on the bridge of
> OVS.
> Can I do that with virt-manager ? or Is there any other way??
>
Virt-manager is one of the UI applications that use libvirt. If I recall
correctly
The code's tricky, but I looked over it a couple of times, and it looks
reasonable to me.
Thanks,
--Justin
On Apr 25, 2013, at 11:23 AM, Ben Pfaff wrote:
> In the case where execute_controller_action() returned true to
> handle_flow_miss(), indicating that the packet had been sent to the
> c
On Wed, May 01, 2013 at 05:06:02PM +0900, Simon Horman wrote:
> The ethertype should always be updated on mpls_pop
> as there may be a transition between MPLS unicast (0x8847) and
> MPLS multicast (0x8848).
>
> Signed-off-by: Simon Horman
The comment on pop_mpls() needs an update.
EXT-194 in th
On Wed, May 01, 2013 at 05:05:26PM +0900, Simon Horman wrote:
> Supplying this data is somewhat misleading as it
> is completely ignored and is not included in
> the synthesised packet.
>
> Signed-off-by: Simon Horman
Applied to master, thanks!
___
dev
On Tue, Apr 30, 2013 at 11:27 PM, freak 62 wrote:
> Hi all,
>
>
> I am using Ubuntu 12.10. I am trying to launch VM on the bridge of
> OVS.
> Can I do that with virt-manager ? or Is there any other way??
> Can I run the existing VM's to my openvswitch bridge ?
> I tried to edit e
This moves generic action execution code out of lib/dpif-netedev.c
and into a new file, lib/execute-actions.c.
This is in preparation for using execute_set_action()
in lib/odp-util.c to handle recirculation/
Signed-off-by: Simon Horman
---
v7
* No change
v6
* As suggested by Jesse Gross
- C
Recirculation is a technique to allow a frame to re-enter
frame processing. This is intended to be used after actions
have been applied to the frame with modify the frame in
some way that makes it possible for richer processing to occur.
An example is and indeed targeted use case is MPLS. If an MP
Add set skb_mark support to execute_set_action.
This also adds support for the user-space datapath
to honour such actions if they occur before recirculation,
which will be added by a subsequent patch.
This is in preparation for using execute_set_action()
to handle recirculation.
Signed-off-by: Si
Add set skb_priority support to execute_set_action.
This also adds support for the user-space datapath
to honour such actions if they occur before recirculation,
which will be added by a subsequent patch.
This is in preparation for using execute_set_action()
to handle recirculation.
Signed-off-by
Add set tunnel support to execute_set_action.
This also adds support for the user-space datapath
to honour such actions if they occur before recirculation,
which will be added by a subsequent patch.
This is in preparation for using execute_set_action()
to handle recirculation.
Signed-off-by: Simo
Hi,
this short series enhances the checking of dl_type pre-requisites.
All of the cases arise from the addition of MPLS.
The second patch is dependent on the first patch,
but other than that this series is not dependent on any patches.
I would like it considered for merging.
Simon Horman (2):
Some actions require an MPLS dl_type while others require a non-MPLS
dl_type. Enforce this for actions handled in ofpact_check__().
Update the following actions to require an non-MPLS dl_type:
set_ipv4_src
set_ipv4_dst
set_ipv4_dscp
set_l4_src_port
set_l4_dst_port
dec_ttl
Update the following ac
learn_parse() needs to know the current ethertype, including any
updates made by MPLS push and pop actions in order to determine if
access to fields is valid or not.
Changes made to ethertype my VLAN push and pop actions are excluded
from this logic as they do not effect the behaviour of learn_par
The ethertype should always be updated on mpls_pop
as there may be a transition between MPLS unicast (0x8847) and
MPLS multicast (0x8848).
Signed-off-by: Simon Horman
---
lib/packets.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/lib/packets.c b/lib/packets.c
index 77aa7
Supplying this data is somewhat misleading as it
is completely ignored and is not included in
the synthesised packet.
Signed-off-by: Simon Horman
---
This patch is independent of other MPLS patches and I would
like it considered for merging.
---
tests/ofproto-dpif.at | 2 +-
1 file changed, 1
On Tue, Apr 30, 2013 at 09:19:51AM -0700, Jesse Gross wrote:
> On Mon, Apr 29, 2013 at 8:21 PM, Simon Horman wrote:
> > On Fri, Apr 26, 2013 at 04:03:21PM -0700, Jesse Gross wrote:
> >> On Thu, Apr 25, 2013 at 12:36 AM, Simon Horman wrote:
> >> > On Tue, Apr 23, 2013 at 02:00:19PM -0700, Joseph G
*,
On May 1, 2013 8:49 AM, wrote:
> Send dev mailing list submissions to
> dev@openvswitch.org
>
> To subscribe or unsubscribe via the World Wide Web, visit
> http://openvswitch.org/mailman/listinfo/dev
> or, via email, send a message with subject or body 'help' to
> dev-r
80 matches
Mail list logo
