On Mon, May 12, 2014 at 08:51:49AM -0700, Ben Pfaff wrote:
> On Mon, May 12, 2014 at 08:56:37AM +0900, Simon Horman wrote:
> > Some concern has been raised by Ben Pfaff that atomic_uint64_t may not
> > be portable. In particular on 32bit platforms that do not have atomic
> > 64bit integers.
> >
>
Implementation of the pop_eth and push_eth actions in the kernel, and
layer 3 flow support.
Signed-off-by: Lorand Jakab
---
datapath/actions.c| 35 +
datapath/datapath.h | 1 +
datapath/flow.c | 43 ++
datapath/flow
This series implements support for layer 3 ports, of which we have one example
so far, the LISP vport. LISP support is currently implemented with a hack, by
adding/removing the Ethernet header within the datapath/vport-lisp.c file. By
removing the assumption that all packets/flows have Ethernet h
These actions will allow L2->L3 and L3->L2 switching, and are supposed
to be added to flows installed in the datapath transparently by
ovs-vswitchd.
Signed-off-by: Lorand Jakab
---
include/linux/openvswitch.h | 16 ++--
lib/dpif-netdev.c | 2 ++
lib/dpif.c |
This commit relaxes the assumption that all packets have an Ethernet
header, and adds support for layer 3 flows. For each packet received on
the Linux kernel datapath the l2 and l3 members of struct ofpbuf are
intialized appropriately, and some functions now expect this (notably
flow_extract()), i
On Tue, May 13, 2014 at 9:55 AM, Alexandru Copot wrote:
> On Tue, May 13, 2014 at 4:18 AM, Ben Pfaff wrote:
>> On Sun, May 11, 2014 at 09:39:32PM +0300, Alexandru Copot wrote:
>>> On Sat, May 10, 2014 at 5:40 AM, Ben Pfaff wrote:
>>> > Signed-off-by: Ben Pfaff
>>>
>>> >
>>> > diff --git a/lib/o
Visual Studio 2013's behavior is to crash when 0 is passed as second
argument to strftime.
Signed-off-by: Gurucharan Shetty
---
lib/timeval.c | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/lib/timeval.c b/lib/timeval.c
index d2a4380..014fd7d 100644
--- a/lib/t
Some ovsdb-tool related unit tests fail with bad checksum errors
while reading transactions from database. It is most likely because
of the CR at the end of line. Using binary mode solves the problem.
Signed-off-by: Gurucharan Shetty
---
ovsdb/log.c |3 +++
1 file changed, 3 insertions(+)
d
The first argument passed to 'sign' command is used as prefix
for both req.pem and cert.pem.
Signed-off-by: Gurucharan Shetty
---
utilities/ovs-pki.in |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/utilities/ovs-pki.in b/utilities/ovs-pki.in
index 501b06e..89af405 100755
In Windows, we use kernel assigned TCP port for ssl/tcp and
unixctl. In tests, we parse the log files of ovsdb-server.log,
test-sflow.log and test-netflow.log to get this port. In all
the above cases, tcp port is allocated first and then the unixctl port.
So a 'head -1' on the result should be safe
Windows port does not yet implement ovsdb-server's "--run" option.
Till it is implemented, skip the tests.
Signed-off-by: Gurucharan Shetty
---
tests/ovsdb-server.at |1 +
1 file changed, 1 insertion(+)
diff --git a/tests/ovsdb-server.at b/tests/ovsdb-server.at
index 86179bd..08c56ca 100644
Signed-off-by: Gurucharan Shetty
---
tests/automake.mk |7 ++-
tests/library.at |6 --
2 files changed, 10 insertions(+), 3 deletions(-)
diff --git a/tests/automake.mk b/tests/automake.mk
index bf80702..c115289 100644
--- a/tests/automake.mk
+++ b/tests/automake.mk
@@ -233,10 +2
setvbuf() in Windows treats _IOLBF to be the same as _IOFBF. So
we cannot have size as zero (otherwise, there is a crash).
Workaround is to set _IONBF. I don't see unit test failures
because of the change.
Signed-off-by: Gurucharan Shetty
---
tests/test-ovsdb.c |2 +-
1 file changed, 1 inse
Ex: ovstest.c
Signed-off-by: Gurucharan Shetty
---
lib/util.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/util.c b/lib/util.c
index 769558e..74dc889 100644
--- a/lib/util.c
+++ b/lib/util.c
@@ -461,7 +461,7 @@ set_program_name__(const char *argv0, const char *versi
The default behavior for programs is to display a popup
after an assert/abort etc. This is not an ideal behavior because
this needs user intervention.
set_program_name, though not an ideal place to disable this, is
a useful place because it is called by all programs including
unit test binaries.
This lets us generate certs for unit tests on Windows
Signed-off-by: Gurucharan Shetty
---
utilities/ovs-pki.in |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/utilities/ovs-pki.in b/utilities/ovs-pki.in
index 89af405..7f56b4e 100755
--- a/utilities/ovs-pki.in
+++ b/u
On Mon, May 12, 2014 at 10:46 PM, Simon Horman wrote:
> On Mon, May 12, 2014 at 04:32:29PM -0700, Daniele Di Proietto wrote:
>> You’re right, of course.
>>
>> Sorry I didn’t see it the first time.
>>
>> On May 12, 2014, at 4:28 PM, Jesse Gross wrote:
>>
>> > On Mon, May 12, 2014 at 4:06 PM, Danie
From: Wilfried Klaebe
Date: Sun, 11 May 2014 00:12:32 +
> net: get rid of SET_ETHTOOL_OPS
>
> Dave Miller mentioned he'd like to see SET_ETHTOOL_OPS gone.
> This does that.
>
> Mostly done via coccinelle script:
> @@
> struct ethtool_ops *ops;
> struct net_device *dev;
> @@
> - SET_ET
As suggested by others, we can use the classifier, instead of the
hash table, as the only flow container in dpif-netdev
Signed-off-by: Daniele Di Proietto
---
lib/dpif-netdev.c | 68 +++
1 file changed, 34 insertions(+), 34 deletions(-)
diff -
One minor comment below, otherwise looks good to me,
Jarno
> On May 9, 2014, at 7:40 PM, Ben Pfaff wrote:
>
> This also adds the definitions of a few other OXM headers we didn't have
> yet.
>
> EXT-109.
> Signed-off-by: Ben Pfaff
> ---
> include/openflow/openflow-1.2.h | 15 +--
Repost it since the original one was blocked by some spam filters.
@Jesse, could you help review it from technical point of view? If this
looks good I can then make it optional and configurable. Thanks a lot!
-Han
On Mon, May 12, 2014 at 4:04 PM, Han Zhou wrote:
> This patch implements vxlan-soe
I suppose so. Does this affect finding rules when there's a new userspace
and older kernel?
On 13 May 2014 16:39, Andy Zhou wrote:
> Their is no use case of partial mask in the user space.
>
> Now that both user space and kernel data path always export recirc_id
> and dp_hash, may be we can drop
On Wed, May 07, 2014 at 05:02:06PM +1200, Simon Horman wrote:
> On Thu, May 01, 2014 at 08:24:21AM -0700, Ben Pfaff wrote:
> > On Thu, May 01, 2014 at 08:21:36AM -0700, Jarno Rajahalme wrote:
> > >
> > > > On May 1, 2014, at 7:53 AM, Ben Pfaff wrote:
> > > >
> > > >> On Mon, Apr 28, 2014 at 03:5
23 matches
Mail list logo
