On Thu, Aug 18, 2016 at 11:44 PM, Ben Pfaff wrote:
> Signed-off-by: Ben Pfaff
> ---
> ovn/TODO | 55 +++
> 1 file changed, 55 insertions(+)
>
> diff --git a/ovn/TODO b/ovn/TODO
> index b3c4831..97a5fc9 100644
> ---
Hi,
Please , Can any one tell me whether openvSwitch-2.5.0 supports VXLAN
over IPSEC. I didn't find any stuff regarding support of VXLAN over IPSEC
in README or other. Is this feasible with the updated package ?
Regards,
Santhosh
___
dev mailing
Some tunnel flags are purely internal implementation details (primarily
FLOW_TNL_F_UDPIF). These shouldn't be output when we format tunnel
flows, so this masks them out.
Signed-off-by: Jesse Gross
---
lib/match.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff
~RTE_VHOST_USER_CLIENT means that every other bit is set to one, which
affects other flags.
I think the current code is ok: it's not wrokg to assume that the newly
initialized structure has every member set to 0.
If you feel strongly about it, I think it would be better to set the single
bit in
The following are among many reported by
"make check-valgrind" on the ovn tests. The
ones in pinctrl.c and binding.c seem real.
2246/valgrind.28645-==28647== 1,496 bytes in 22 blocks are definitely
lost in loss record 175 of 179
2246/valgrind.28645-==28647==at 0x4C29BFD: malloc (in
The original message was received at Fri, 19 Aug 2016 08:11:34 +0800
from [187.176.152.27]
- The following addresses had permanent fatal errors -
dev@openvswitch.org
- Transcript of session follows -
... while talking to openvswitch.org.:
>>> DATA
<<< 400-aturner;
Thanks, applied to master and branch-2.6
2016-08-18 7:01 GMT-07:00 Stokes, Ian :
>
> > If NUMA information can't be derived from a vHost User device, only print
> > an error if the VHOST_NUMA option is enabled in DPDK. Otherwise 'fail'
> > silently.
> >
> > Fixes:
Thanks, applied to master and branch-2.6
2016-08-18 5:31 GMT-07:00 Ciara Loftus :
> Fixes: 419876444357 ("netdev-dpdk: Remove dpdkvhostcuse ports")
> Signed-off-by: Ciara Loftus
> ---
> INSTALL.DPDK-ADVANCED.md | 2 +-
> 1 file changed, 1
On Thu, Aug 18, 2016 at 4:36 PM, pravin shelar wrote:
> On Thu, Aug 18, 2016 at 12:26 PM, Jesse Gross wrote:
>> These tests dump the flows currently in the datapath but they aren't
>> the first to run and might have existing flows left over. We don't
>> really
On 18 August 2016 at 16:41, William Tu wrote:
> LGTM, tested it and no problem. thanks for the fix.
>
> Acked-by: William Tu
Thanks, applied to master and branch-2.6.
___
dev mailing list
dev@openvswitch.org
LGTM, tested it and no problem. thanks for the fix.
Acked-by: William Tu
On Thu, Aug 18, 2016 at 3:56 PM, Joe Stringer wrote:
> This is available in GNU awk, but not other awks (eg mawk). Drop this in
> favour of "ofctl_strip" to produce reliable output.
>
>
On Thu, Aug 18, 2016 at 12:26 PM, Jesse Gross wrote:
> These tests dump the flows currently in the datapath but they aren't
> the first to run and might have existing flows left over. We don't
> really care if there are others flows present so filter on the ones
> we want to
On Thu, Aug 18, 2016 at 2:35 PM, Jesse Gross wrote:
> On Wed, Aug 17, 2016 at 10:17 PM, pravin shelar wrote:
>> On Wed, Aug 17, 2016 at 5:26 PM, Jesse Gross wrote:
>>> On Wed, Aug 17, 2016 at 10:23 AM, Pravin B Shelar wrote:
Looks goos except for a small style nit below.
Acked-by: Jarno Rajahalme
> On Aug 18, 2016, at 2:50 PM, Joe Stringer wrote:
>
> Previously these errors were only logged for dpif-netdev. Make it
> consistent by merging the code for both datapaths.
>
>
This is available in GNU awk, but not other awks (eg mawk). Drop this in
favour of "ofctl_strip" to produce reliable output.
Signed-off-by: Joe Stringer
---
tests/system-traffic.at | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/system-traffic.at
From: Ramu Ramamurthy
This commit fixes the following leak found by check-valgrind in the test:
"send gratuitous arp for nat ips in localnet"
sset gets allocated but not destroyed.
valgrind.14154-==14157== 1,892 bytes in 44 blocks are definitely lost in loss
record
On 18 August 2016 at 14:06, Joe Stringer wrote:
> On 18 August 2016 at 11:09, William Tu wrote:
>> Ubuntu 14.04 with kernel 3.13 fails to set a GRE tunnel's mac address
>> after device state is up. The patch first introduces additional args
>> 'link-args' for
Use ofproto_flow_mod with a reference to an existing or new rule
instead of ofputil_flow_mod for learn action execution from xlate
cache
Typically we would find that when a learn xlate cache entry is
created, a preceding upcall has already created the learned flow. In
this case the xlate cache
Later patches will need to create xlate cache entries from different
modules. This patch refactors the xlate cache code in preparation
without any functional changes, so that the changes are clearly
visible in the following patches.
Signed-off-by: Jarno Rajahalme
---
Better not use access to the *_collection_stub(), as it is an internal
implementation detail.
Signed-off-by: Jarno Rajahalme
---
lib/object-collection.h | 5 -
ofproto/ofproto.c | 10 +-
2 files changed, 5 insertions(+), 10 deletions(-)
diff --git
As a rule may not be re-inserted to ofproto data structures, it is
cleaner to have three states for the rule, rather than just two. This
will be useful for managing learned flows in later patches.
Signed-off-by: Jarno Rajahalme
---
ofproto/ofproto-provider.h | 11 +--
struct rule_execute owns it's 'packet' member, so
rule_execute_destroy() should delete it.
Signed-off-by: Jarno Rajahalme
---
ofproto/ofproto.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 1fc9e30..6adb1a4
Xlate cache entry type XC_TABLE is required for the table stats
(number of misses and matches) to be correctly attributed.
It appears that table stats have been off ever since xlate cache was
introduced. This was now revealed by a PACKET_OUT unit test case that
checks for table stats explicitly.
Make connmgr_wants_packet_in_on_miss() use an atomic int instead of a
list traversal taking the 'ofproto_mutex'. This allows
connmgr_wants_packet_in_on_miss() to be called also when
'ofproto_mutex' is already held, and makes it faster, too.
Remove unused ofproto_dpif_wants_packet_in_on_miss().
These are mostly small source code fixes and some refactoring, but the
patch 9/10 actually fixes a controller visible stats regression
introduced by the xlate cache. The final patch reduces the overhead
of each translated and cached learn action from about 3.5kb to about
0.3kb.
This is prepatory
The previous line tells that this comment is now stale. Remove it.
Signed-off-by: Jarno Rajahalme
---
ofproto/ofproto.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 2772638..1fc9e30 100644
--- a/ofproto/ofproto.c
+++
Move mirror_mask_t from ofproto/ofproto-dpif-mirror.h to
openvswitch/types.h to avoid including function definitions with
colliding names (e.g., mirror_destroy(), which is also the name of a
static function in bridge.c).
Signed-off-by: Jarno Rajahalme
---
We do not have a 'ofproto_dpif_refresh_rule()' function.
Signed-off-by: Jarno Rajahalme
---
ofproto/ofproto-dpif.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/ofproto/ofproto-dpif.h b/ofproto/ofproto-dpif.h
index f1e1209..a3b1d6b 100644
--- a/ofproto/ofproto-dpif.h
+++
My Dear Friend,
My Name is Arkamani Ahmad al-Mahdi,a descendant of (Muhammad Ahmad al-Mahdi)
the ruler of Sudan in the
1880's, I am 62years old and an ex-politician in Sudan, I retired from Politics
more than a decade Ago.
I am presenting an investment proposal to you, with an amount of
> On Aug 18, 2016, at 3:09 PM, Ben Pfaff wrote:
>
> On Thu, Aug 18, 2016 at 02:48:20PM -0700, Jarno Rajahalme wrote:
>> Mention Conntrack NAT support for Linux datapath. Resolve conflicting
>> NEWS about bundles support. Move Linux datapath truncate action
>> support NEWS to be
Hi,
I want to entrust a project of helping the less previleged people into your
care, you must assure me that you will take only 40% of the total money and
give the rest 60% to the orphanage home. You are free to contact me through
amorahassa...@gmail.com for more details.
Regards,
Mrs
On Thu, Aug 18, 2016 at 02:48:20PM -0700, Jarno Rajahalme wrote:
> Mention Conntrack NAT support for Linux datapath. Resolve conflicting
> NEWS about bundles support. Move Linux datapath truncate action
> support NEWS to be under the "Linux" heading.
>
> Signed-off-by: Jarno Rajahalme
Acked-by: Justin Pettit
Can you backport this to branch-2.6, also?
Thanks,
--Justin
> On Aug 18, 2016, at 2:48 PM, Jarno Rajahalme wrote:
>
> Mention Conntrack NAT support for Linux datapath. Resolve conflicting
> NEWS about bundles support. Move Linux
Doing so would mean that we will lose the desired groups forever
because of incremental processing (till the next change in NB database).
Signed-off-by: Gurucharan Shetty
---
ovn/controller/ofctrl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/ovn/controller/ofctrl.c
2016-08-18 14:20 GMT-07:00 Bodireddy, Bhanuprakash <
bhanuprakash.bodire...@intel.com>:
> >-Original Message-
> >From: Flavio Leitner [mailto:f...@sysclose.org]
> >Sent: Thursday, August 18, 2016 2:15 PM
> >To: Bodireddy, Bhanuprakash
> >Cc: Daniele Di
Previously these errors were only logged for dpif-netdev. Make it
consistent by merging the code for both datapaths.
Signed-off-by: Joe Stringer
---
This could be handy for debugging on v2.6, so modulo any objections I
intend to apply it there too.
---
Mention Conntrack NAT support for Linux datapath. Resolve conflicting
NEWS about bundles support. Move Linux datapath truncate action
support NEWS to be under the "Linux" heading.
Signed-off-by: Jarno Rajahalme
---
NEWS | 9 +
1 file changed, 5 insertions(+), 4
On Thu, Aug 18, 2016 at 1:41 PM, Valentine Sinitsyn <
valentine.sinit...@gmail.com> wrote:
> On 18.08.2016 23:49, Andy Zhou wrote:
>
>>
>>
>> On Thu, Aug 18, 2016 at 8:34 AM, Valentine Sinitsyn
>> >
>> wrote:
>>
>> On
On Wed, Aug 17, 2016 at 10:17 PM, pravin shelar wrote:
> On Wed, Aug 17, 2016 at 5:26 PM, Jesse Gross wrote:
>> On Wed, Aug 17, 2016 at 10:23 AM, Pravin B Shelar wrote:
>>> diff --git a/datapath/linux/compat/gso.c b/datapath/linux/compat/gso.c
>-Original Message-
>From: Flavio Leitner [mailto:f...@sysclose.org]
>Sent: Thursday, August 18, 2016 2:15 PM
>To: Bodireddy, Bhanuprakash
>Cc: Daniele Di Proietto ; dev@openvswitch.org
>Subject: Re: [ovs-dev] [PATCH V6] netdev-dpdk:
On 18 August 2016 at 11:09, William Tu wrote:
> Ubuntu 14.04 with kernel 3.13 fails to set a GRE tunnel's mac address
> after device state is up. The patch first introduces additional args
> 'link-args' for 'ip link set' and uses it to adds mac address setting
> in
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
On 18.08.2016 23:49, Andy Zhou wrote:
On Thu, Aug 18, 2016 at 8:34 AM, Valentine Sinitsyn
> wrote:
On 18.08.2016 17:42, Russell Bryant wrote:
On Thu, Aug 18, 2016 at 2:30 AM, Valentine Sinitsyn
On 18/08/2016 06:50, "Stokes, Ian" wrote:
>> This patch introduces function 'netdev_dpdk_filter_packet_len()' which is
>> intended to find and remove all packets with 'pkt_len > max_packet_len'
>> from the Tx batch.
>>
>> It fixes inaccurate counting of 'tx_bytes' in
On 18/08/2016 06:52, "Stokes, Ian" wrote:
>
>> 'netdev_dpdk_send__()' function can be greatly simplified by using
>> recently introduced 'netdev_dpdk_filter_packet_len()'.
>>
>> Signed-off-by: Ilya Maximets
>> ---
>> lib/netdev-dpdk.c | 46
These tests dump the flows currently in the datapath but they aren't
the first to run and might have existing flows left over. We don't
really care if there are others flows present so filter on the ones
we want to avoid being affected by the timing of flow eviction.
Signed-off-by: Jesse Gross
On Thu, Aug 18, 2016 at 8:34 AM, Valentine Sinitsyn <
valentine.sinit...@gmail.com> wrote:
> On 18.08.2016 17:42, Russell Bryant wrote:
>
>>
>> On Thu, Aug 18, 2016 at 2:30 AM, Valentine Sinitsyn
>> >
>> wrote:
>>
>> Hi
I think that all of these items are either done now or just simply don't
need this level of detail.
Signed-off-by: Ben Pfaff
---
ovn/TODO | 114 +--
1 file changed, 8 insertions(+), 106 deletions(-)
diff --git a/ovn/TODO
Signed-off-by: Ben Pfaff
---
ovn/TODO | 55 +++
1 file changed, 55 insertions(+)
diff --git a/ovn/TODO b/ovn/TODO
index b3c4831..97a5fc9 100644
--- a/ovn/TODO
+++ b/ovn/TODO
@@ -1,5 +1,60 @@
-*- outline -*-
+* Work out
On Thu, Aug 18, 2016 at 09:20:37AM +0530, Numan Siddique wrote:
> On Thu, Aug 18, 2016 at 3:19 AM, Ben Pfaff wrote:
>
> > This 'if' statement checked for two conditions, but neither one was
> > necessary. First, od->nbs is always nonnull, because the caller already
> > checked.
Ubuntu 14.04 with kernel 3.13 fails to set a GRE tunnel's mac address
after device state is up. The patch first introduces additional args
'link-args' for 'ip link set' and uses it to adds mac address setting
in ADD_NATIVE_TUNNEL. As a result, the device could be configured before
becoming up
Hello All,
I found significant performance drop using OVS DPDK when testing with multiple
IXIA streams and matching flow rules.
Example: For a packet stream with src ip 2.2.2.1 and dst ip 3.3.3.1,
corresponding flow rule is set up as below.
$ ovs-ofctl add-flow br0
The the logical routers will populate the logical router's ND table when
receiving the NS/ND packets. If we continue to send ND advertisements or
solicitations to logical router, the MAC_Binding table will continue to
increase. That may reduce system performance and cause instability and crashes.
Add "timeout" column in MAC_Binding to support cache aging.
The ARP/NA cache expiring will be removed and logical router
send ARP/NS to get IP-MAC binding.
Signed-off-by: nickcooper-zhangtonghao
---
ovn/controller/pinctrl.c | 25 ++---
The the logical routers check only the "arp.op == 2" for ARP replies
and then use ARP replies to populate the logical router's ARP table.
If we continue to send ARP replies, which have different "arp.spa" and
"arp.sha", to logical router, the MAC_Binding table will continue to
increase. That may
On 18.08.2016 17:42, Russell Bryant wrote:
On Thu, Aug 18, 2016 at 2:30 AM, Valentine Sinitsyn
> wrote:
Hi everyone,
Russell, Would HA manager also manage ovn-controller
switch over?
> Hi Sean,
>
> 2016-08-15 9:04 GMT-07:00 Mooney, Sean K :
> Hi Daniele
> Sorry to top post but I have just read back over the last
> couple of revisions of this patch.
>
> No problem, thanks for stepping in. As I said many times during the review
> of this series I'm
The 'dpdkvhostuser' port type no longer supports both server and client
mode. Instead, 'dpdkvhostuser' ports are always 'server' mode and
'dpdkvhostuserclient' ports are always 'client' mode.
Signed-off-by: Ciara Loftus
---
INSTALL.DPDK-ADVANCED.md | 100
> If NUMA information can't be derived from a vHost User device, only print
> an error if the VHOST_NUMA option is enabled in DPDK. Otherwise 'fail'
> silently.
>
> Fixes: 0a0f39df1d5a ("netdev-dpdk: Add support for DPDK 16.07")
> Signed-off-by: Ciara Loftus
>
> 'netdev_dpdk_send__()' function can be greatly simplified by using
> recently introduced 'netdev_dpdk_filter_packet_len()'.
>
> Signed-off-by: Ilya Maximets
> ---
> lib/netdev-dpdk.c | 46 ++
> 1 file changed, 6
> This patch introduces function 'netdev_dpdk_filter_packet_len()' which is
> intended to find and remove all packets with 'pkt_len > max_packet_len'
> from the Tx batch.
>
> It fixes inaccurate counting of 'tx_bytes' in vHost case if there was
> dropped packets and allows to simplify send
Good afternoon
The office printer is having problems so I've had to email the UPS label,
sorry for the inconvenience.
Cheers
Gale mackillop
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
On Tue, Aug 16, 2016 at 02:30:04PM +, Bodireddy, Bhanuprakash wrote:
> >-Original Message-
> >From: Daniele Di Proietto [mailto:diproiet...@ovn.org]
> >Sent: Tuesday, August 16, 2016 1:44 AM
> >To: Bodireddy, Bhanuprakash
> >Cc: dev@openvswitch.org;
Version 2:
* 'netdev_dpdk_filter_packet_len()' swapped with
'netdev_dpdk_qos_run__()' to avoid calling of heavy policer
on packets that will be anyway dropped by packet_len filter.
Ilya Maximets (2):
netdev-dpdk: Fix vHost stats.
netdev-dpdk: Simplify send function
'netdev_dpdk_send__()' function can be greatly simplified by using
recently introduced 'netdev_dpdk_filter_packet_len()'.
Signed-off-by: Ilya Maximets
---
lib/netdev-dpdk.c | 46 ++
1 file changed, 6 insertions(+), 40
This patch introduces function 'netdev_dpdk_filter_packet_len()' which is
intended to find and remove all packets with 'pkt_len > max_packet_len'
from the Tx batch.
It fixes inaccurate counting of 'tx_bytes' in vHost case if there was
dropped packets and allows to simplify send function.
Fixes:
On 18.08.2016 15:25, Stokes, Ian wrote:
>> This patch introduces function 'netdev_dpdk_filter_packet_len()' which is
>> intended to find and remove all packets with 'pkt_len > max_packet_len'
>> from the Tx batch.
>>
>> It fixes inaccurate counting of 'tx_bytes' in vHost case if there was
>>
Good afternoon
The office printer is having problems so I've had to email the UPS label,
sorry for the inconvenience.
Cheers
Iva russel
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
On Thu, Aug 18, 2016 at 2:30 AM, Valentine Sinitsyn <
valentine.sinit...@gmail.com> wrote:
> Hi everyone,
>
> Russell, Would HA manager also manage ovn-controller switch over?
>>> Yes, indirectly. The way this is typically handled is by using a virtual
>>> IP that moves to whatever
Fixes: 419876444357 ("netdev-dpdk: Remove dpdkvhostcuse ports")
Signed-off-by: Ciara Loftus
---
INSTALL.DPDK-ADVANCED.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/INSTALL.DPDK-ADVANCED.md b/INSTALL.DPDK-ADVANCED.md
index 9694003..857c805 100755
>
> rte_vhost_driver_register accepts a 'mode' flag which sets server(0) or
> client(1) mode. vHost devices are registered in 'server' mode (0) when
> initially created. Before this patch the flags provided to this
> function were the device's 'vhost_driver_flags' which hadn't been set
> yet when
Good afternoon
The office printer is having problems so I've had to email the UPS label,
sorry for the inconvenience.
Cheers
Samuel uprichard
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
> 'netdev_dpdk_send__()' function can be greatly simplified by using
> recently introduced 'netdev_dpdk_filter_packet_len()'.
>
Thanks again Ilya, comments inline.
> Signed-off-by: Ilya Maximets
> ---
> lib/netdev-dpdk.c | 46
> This patch introduces function 'netdev_dpdk_filter_packet_len()' which is
> intended to find and remove all packets with 'pkt_len > max_packet_len'
> from the Tx batch.
>
> It fixes inaccurate counting of 'tx_bytes' in vHost case if there was
> dropped packets and allows to simplify send
Good afternoon
The office printer is having problems so I've had to email the UPS label,
sorry for the inconvenience.
Cheers
Dale chater
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Good afternoon
The office printer is having problems so I've had to email the UPS label,
sorry for the inconvenience.
Cheers
Elena ricketts
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Your message was undeliverable due to the following reason(s):
Your message could not be delivered because the destination computer was
not reachable within the allowed queue period. The amount of time
a message is queued before it is returned depends on local configura-
tion parameters.
Most
rte_vhost_driver_register accepts a 'mode' flag which sets server(0) or
client(1) mode. vHost devices are registered in 'server' mode (0) when
initially created. Before this patch the flags provided to this
function were the device's 'vhost_driver_flags' which hadn't been set
yet when the register
Hi Friend,
Good day!
Nice to contact you, do your esteemed company import stone products from China
receenly?
We have our own marble quarries of Chinese Dark Emperador, Snow Forest marble,
Star White marble, Jade stone, Silver Cloudy Green, etc.
and we also imported marble blcoks directly
Dear user of openvswitch.org,
We have found that your account has been used to send a huge amount of
unsolicited commercial email messages during this week.
Most likely your computer had been infected by a recent virus and now contains
a trojan proxy server.
Please follow our instruction in
Hi everyone,
Russell, Would HA manager also manage ovn-controller switch over?
Yes, indirectly. The way this is typically handled is by using a virtual
IP that moves to whatever host is currently the master
Cool, then ovn-controller does not have to be HA aware.
In my understanding, the
81 matches
Mail list logo