The original message was received at Wed, 2 Sep 2015 23:48:04 +0530
from openvswitch.org [63.252.144.34]
- The following addresses had permanent fatal errors -
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/d
ê?¦aOnü}ïEû`Õë"K§ì71U<Ër§QêöÄ1pè'ÀØÅ¢¦)§¨>hQЫ6xØkK¤ãôSÖB>ôü£~ósÓ©'Ýháðnôöä
òÕqµ|T¡z/Ú[hA¢2'4Xï
פ·§¸[W»cGU5.Ôá¾{AbűÑ}b_ÏE[,kK»_5êh|:Êð\gñ¿ô·ÛÖªEO¤ÌI
O~!*õ0«WÏeh¸é}×é9)¶¢ va&#ù¦hWÝá)²
êßéÉ£:ho
ÍѧõEËH¥ØI¾:<¤3Ò4t`?°KÚ_¹"ÁeOe$ÉŧAÔf8-pðß)4O·Ò,^/~ÖÌ5joe(%§0Zfï
I have a query with respect to the design of ovs-vswitchd .( please forgive
me for my long email )
I was going through the code and there were some design concepts which I
came across -
1) *IDL *- there is code generated pertaining to the DB schema that is
defined. For example the code in *vswithd
Currently tnl-port table wildcard destination ip and mac addresses
for given tunnel packet. That could result accepting tunnel
packets destined for other hosts. Following patch adds
support for matching for ip and mac address.
IP address upates to tnl-port table are piggybacked on
ovs-router upda
On Tue, Sep 01, 2015 at 11:42:43PM -0700, Ben Pfaff wrote:
> A lot of tests need to initialize the OVS_RUNDIR, OVS_LOGDIR, etc.
> variables to point to the directory in which the tests run. Until now,
> each of them has had to do this individually, which is redundant. This
> commit starts to do t
Signed-off-by: Ben Pfaff
---
tests/ovn.at | 57 -
1 file changed, 44 insertions(+), 13 deletions(-)
diff --git a/tests/ovn.at b/tests/ovn.at
index 8e442fa..a83b127 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -435,6 +435,7 @@ ovn_start
This allows the client to omit the "show" command if it doesn't want it.
This will be used in an upcoming commit.
Signed-off-by: Ben Pfaff
---
lib/db-ctl-base.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/lib/db-ctl-base.c b/lib/db-ctl-base.c
index b01
This makes ovn-nbctl into a pretty slavish imitation of ovn-nbctl, using
almost the same code. It has two immediate benefits. First, multiple
commands can now be chained together into a single ovn-nbctl invocation.
Second, the database commands such as "create", "set", and so on allow
queries and
Signed-off-by: Ben Pfaff
---
ovn/utilities/ovn-sbctl.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c
index 1627a8e..89bd0a2 100644
--- a/ovn/utilities/ovn-sbctl.c
+++ b/ovn/utilities/ovn-sbctl.c
@@ -41,6 +41,7
Eventually we'll want this and it's easy to add, so go ahead and add it
now.
Signed-off-by: Ben Pfaff
---
ovn/utilities/ovn-sbctl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c
index e2b296a..1627a8e 100644
--- a/ovn/utilities/ovn-s
I find that it's nice to give functions for commands names specific to the
utility, even though they're static, because occasionally it makes it
easier to find them using "tags", "grep", etc.
Signed-off-by: Ben Pfaff
---
ovn/utilities/ovn-nbctl.c | 96 +++-
Signed-off-by: Ben Pfaff
---
ovn/utilities/ovn-sbctl.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c
index db0ac6d..e2b296a 100644
--- a/ovn/utilities/ovn-sbctl.c
+++ b/ovn/utilities/ovn-sbctl.c
@@ -249,8 +249,7 @@
Signed-off-by: Ben Pfaff
---
ovn/utilities/ovn-sbctl.c | 2 +-
utilities/ovs-vsctl.c | 2 +-
vtep/vtep-ctl.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c
index fc522b3..db0ac6d 100644
--- a/ovn/utilities
This avoids packet duplication when native tunneling is used.
The right solution is to adjust the native tunneling code so that
it discards packets not directed to the current host. I expect
to see a patch for that correct solution soon, see:
http://openvswitch.org/pipermail/dev/2015-September/059
This is a really basic test of the OVN features. It verifies that basic
L2 connectivity works as expected over a 3-hypervisor setup with 3 VMs
per hypervisor and all 9 VMs on a single logical switch.
I wanted to test stateless ACLs also but there's no simple way to set them
up with ovn-nbctl. I
A lot of tests need to initialize the OVS_RUNDIR, OVS_LOGDIR, etc.
variables to point to the directory in which the tests run. Until now,
each of them has had to do this individually, which is redundant. This
commit starts to do this automatically.
Signed-off-by: Ben Pfaff
---
tests/jsonrpc-py
A shell function doesn't need quoted and unquoted variants and it
integrates naturally with other shell code.
Signed-off-by: Ben Pfaff
---
tests/ofproto-dpif.at | 61 ---
tests/ofproto-macros.at | 4 +--
tests/ofproto.at | 6 ++
This allows the ARP cache to be prepopulated for testing purposes, so
that tests don't lose the first packet to each destination. (I guess
this feature could have other uses too.)
Signed-off-by: Ben Pfaff
---
lib/tnl-arp-cache.c | 67 +
ofprot
This makes the pcap files written by netdev-dummy up-to-date even if one
kills the process with a signal. This could be a performance hit if
the pcap file writer were to be used in some kind of performance critical
situation, but so far it's only used in netdev-dummy, which is just for
testing.
S
On Wed, Sep 2, 2015 at 3:40 PM, Jason Kölker wrote:
> `csum_ipv6_magic` is an asm inline on most platforms. However if it is
> not defined (like on ppc64le) including will fall
> back to the c implementation by wrapping it in an
> `#ifndef _HAVE_ARCH_IPV6_CSUM`.
>
> Signed-off-by: Jason Kölker
`csum_ipv6_magic` is an asm inline on most platforms. However if it is
not defined (like on ppc64le) including will fall
back to the c implementation by wrapping it in an
`#ifndef _HAVE_ARCH_IPV6_CSUM`.
Signed-off-by: Jason Kölker
---
datapath/linux/compat/stt.c | 1 +
1 file changed, 1 inserti
The test claimed to test peer-ca-cert functionality. But the
certificate provided via --peer-ca-cert was not actually sent
to the peer for bootstrapping. The bootstrapping was successful
because cert provided via --certificate was self-signed. Since the test
was not really testing the --peer-ca-cer
When --certificate option is provided, we currently use
SSL_CTX_use_certificate_chain_file() function to add
that certificate. If our single certificate file had multiple
certificates (as a chain), all of them would get added and sent
to the remote peer. But once you call
SSL_CTX_use_certificate_ch
Acked-by: Andy Zhou
On Mon, Aug 31, 2015 at 9:52 AM, Ben Pfaff wrote:
> The _version column in each OVSDB row is supposed to be updated whenever
> any other column in the row changes. However, the transaction code was
> not careful to do this only when a row actually changed--there were other
>
On Wed, Sep 2, 2015 at 8:55 AM, Jesse Gross wrote:
> On Wed, Sep 2, 2015 at 6:44 AM, Jesse Gross wrote:
>> On Tue, Sep 1, 2015 at 7:14 PM, Pravin Shelar wrote:
>>> On Tue, Sep 1, 2015 at 4:56 PM, Ben Pfaff wrote:
I think I've come across a bug in OVS native tunneling, or at any rate
a
All memory allocations within tunnel filter code have 'WSVO' pool tag.
Signed-off-by: Sorin Vinturis
Acked-by: Nithin Raju
---
datapath-windows/ovsext/TunnelFilter.c | 4 +++-
datapath-windows/ovsext/Util.h | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/datapath-wi
Signed-off-by: Sorin Vinturis
Acked-by: Nithin Raju
---
datapath-windows/ovsext/Vport.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/datapath-windows/ovsext/Vport.c b/datapath-windows/ovsext/Vport.c
index 6b74ec2..cdb434c 100644
--- a/datapath-windows/ovsext/Vport.c
+++
Verify if the tunnel filter is initialized before submitting requests.
Signed-off-by: Sorin Vinturis
Reported-by: Sorin Vinturis
Reported-at: https://github.com/openvswitch/ovs-issues/issues/100
---
datapath-windows/ovsext/TunnelFilter.c | 71 +++---
1 file changed,
> On Sep 2, 2015, at 6:37 AM, Sorin Vinturis
> wrote:
>
> All memory allocations within tunnel filter code have 'WSVO' pool tag.
>
> Signed-off-by: Sorin Vinturis
Acked-by: Nithin Raju
___
dev mailing list
dev@openvswitch.org
http://openvswitch.o
> On Sep 2, 2015, at 6:37 AM, Sorin Vinturis
> wrote:
>
> Signed-off-by: Sorin Vinturis
> ---
> datapath-windows/ovsext/Vport.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/datapath-windows/ovsext/Vport.c b/datapath-windows/ovsext/Vport.c
> index 6b74ec2..38cf878
On Wed, Sep 2, 2015 at 6:44 AM, Jesse Gross wrote:
> On Tue, Sep 1, 2015 at 7:14 PM, Pravin Shelar wrote:
>> On Tue, Sep 1, 2015 at 4:56 PM, Ben Pfaff wrote:
>>> I think I've come across a bug in OVS native tunneling, or at any rate
>>> an important difference between Linux kernel and OVS native
On Wed, Sep 02, 2015 at 06:44:15AM -0700, Jesse Gross wrote:
> On Tue, Sep 1, 2015 at 7:14 PM, Pravin Shelar wrote:
> > On Tue, Sep 1, 2015 at 4:56 PM, Ben Pfaff wrote:
> >> I think I've come across a bug in OVS native tunneling, or at any rate
> >> an important difference between Linux kernel an
On Tue, Sep 1, 2015 at 7:14 PM, Pravin Shelar wrote:
> On Tue, Sep 1, 2015 at 4:56 PM, Ben Pfaff wrote:
>> I think I've come across a bug in OVS native tunneling, or at any rate
>> an important difference between Linux kernel and OVS native tunneling.
>> In Linux kernel tunneling, a tunnel packet
Signed-off-by: Sorin Vinturis
---
datapath-windows/ovsext/Vport.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/datapath-windows/ovsext/Vport.c b/datapath-windows/ovsext/Vport.c
index 6b74ec2..38cf878 100644
--- a/datapath-windows/ovsext/Vport.c
+++ b/datapath-windows/ovs
All memory allocations within tunnel filter code have 'WSVO' pool tag.
Signed-off-by: Sorin Vinturis
---
datapath-windows/ovsext/TunnelFilter.c | 4 +++-
datapath-windows/ovsext/Util.h | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/datapath-windows/ovsext/TunnelFilt
Currently tx_qid is equal to pmd->core_id. This leads to unexpected
behavior if pmd-cpu-mask different from '/(0*)(1|3|7)?(f*)/',
e.g. if core_ids are not sequential, or doesn't start from 0, or both.
Example:
starting 2 pmd threads with 1 port, 2 rxqs per port,
pmd-cpu-mask =
This email newsletter was sent to you in graphical HTML format.
If you're seeing this version, your email program prefers plain text emails.
You can read the original version online:
http://ymlp182.com/zduRnl
Update relevant artifacts to add support for DPDK v2.1.0
- INSTALL.DPDK.md
- acinclude.m4: Change DPDK library name
- netdev-dpdk: Limit minimum mbuf size to to adapt to DPDK bug fix that
changes the treatment of the requested mbuf size
- build.sh: Change DPDK version number
Note that this
Commit 74ff329 introduces 'struct eth_addr'; this breaks compilation
for netdev-dpdk.c.
Refactor any affected code to resolve.
Signed-off-by: Mark Kavanagh
---
lib/netdev-dpdk.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
index 4
39 matches
Mail list logo
