Re: [ovs-dev] [PATCH 2/2] ovn/TODO: Add items proposed for 2.7 in OVN IRC meeting.

2016-08-18 Thread Numan Siddique
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 > ---

[ovs-dev] Will openvSwitch-2.5.0 supports VXLAN over IPSEC ?

2016-08-18 Thread santhu vaddepally
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

[ovs-dev] [PATCH] match: Only print external tunnel flags.

2016-08-18 Thread Jesse Gross
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

Re: [ovs-dev] [PATCH] netdev-dpdk: Provide explicit flag to rte_vhost_driver_register

2016-08-18 Thread Daniele Di Proietto
~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

[ovs-dev] [bug] ovn: make check-valgrind issues

2016-08-18 Thread Ramu Ramamurthy
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

[ovs-dev] Returned mail: see transcript for details

2016-08-18 Thread Returned mail
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;

Re: [ovs-dev] [PATCH] netdev-dpdk: Fix occurance of error log

2016-08-18 Thread Daniele Di Proietto
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:

Re: [ovs-dev] [PATCH] INSTALL.DPDK-ADVANCED.md: Fix incorrect numbering

2016-08-18 Thread Daniele Di Proietto
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

Re: [ovs-dev] [PATCH] tests: Avoid race in tunnel-push-pop tests.

2016-08-18 Thread Jesse Gross
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

Re: [ovs-dev] [PATCH] system-traffic: Don't rely on --field-separator.

2016-08-18 Thread Joe Stringer
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

Re: [ovs-dev] [PATCH] system-traffic: Don't rely on --field-separator.

2016-08-18 Thread William Tu
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. > >

Re: [ovs-dev] [PATCH] tests: Avoid race in tunnel-push-pop tests.

2016-08-18 Thread pravin shelar
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

Re: [ovs-dev] [PATCH v2] datapath: compat: backport LCO optimization.

2016-08-18 Thread pravin shelar
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:

Re: [ovs-dev] [PATCH] upcall: Log failure to flow_put for dpif-netlink.

2016-08-18 Thread Jarno Rajahalme
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. > >

[ovs-dev] [PATCH] system-traffic: Don't rely on --field-separator.

2016-08-18 Thread Joe Stringer
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

[ovs-dev] [PATCH] ovn: fix valgrind leak

2016-08-18 Thread Ramu Ramamurthy
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

Re: [ovs-dev] [PATCHv2] system-traffic: Add link-args for ADD_NATIVE_TUNNEL.

2016-08-18 Thread Joe Stringer
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

[ovs-dev] [PATCH 10/10] ofproto: Use ofproto_flow_mod for learn execution from xlate cache.

2016-08-18 Thread Jarno Rajahalme
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

[ovs-dev] [PATCH 08/10] ofproto-dpif-xlate: Expose xlate cache.

2016-08-18 Thread Jarno Rajahalme
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 ---

[ovs-dev] [PATCH 06/10] object-collection: Remove access to stub.

2016-08-18 Thread 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

[ovs-dev] [PATCH 03/10] ofproto: Change rule's 'removed' member to a tri-state 'state'.

2016-08-18 Thread Jarno Rajahalme
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 +--

[ovs-dev] [PATCH 05/10] ofproto: Clarify rule_execute packet ownership.

2016-08-18 Thread Jarno Rajahalme
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

[ovs-dev] [PATCH 09/10] ofproto-dpif-xlate: Add xlate cache type XC_TABLE.

2016-08-18 Thread Jarno Rajahalme
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.

[ovs-dev] [PATCH 07/10] connmgr: Make connmgr_wants_packet_in_on_miss() lock-free.

2016-08-18 Thread Jarno Rajahalme
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().

[ovs-dev] [PATCH 00/10] ofproto fixes.

2016-08-18 Thread Jarno Rajahalme
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

[ovs-dev] [PATCH 04/10] ofproto: Remove stale comment.

2016-08-18 Thread Jarno Rajahalme
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 +++

[ovs-dev] [PATCH 01/10] types.h: Move mirror_mask_t here.

2016-08-18 Thread Jarno Rajahalme
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 ---

[ovs-dev] [PATCH 02/10] ofproto-dpif: Remove prototype for an undefined function.

2016-08-18 Thread 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 +++

[ovs-dev] INVESTMENT

2016-08-18 Thread Mr. Arkamani
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

Re: [ovs-dev] [PATCH] NEWS: NAT and bundles fixes for 2.6 release.

2016-08-18 Thread Jarno Rajahalme
> 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

[ovs-dev] [CudaMailTagged] Care

2016-08-18 Thread Amora Hassan
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

Re: [ovs-dev] [PATCH] NEWS: NAT and bundles fixes for 2.6 release.

2016-08-18 Thread Ben Pfaff
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

Re: [ovs-dev] [PATCH] NEWS: NAT and bundles fixes for 2.6 release.

2016-08-18 Thread Justin Pettit
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

[ovs-dev] [PATCH] ovn-controller: Don't clear desired groups during backlog.

2016-08-18 Thread Gurucharan Shetty
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

Re: [ovs-dev] [PATCH V6] netdev-dpdk: Set pmd thread priority

2016-08-18 Thread Daniele Di Proietto
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

[ovs-dev] [PATCH] upcall: Log failure to flow_put for dpif-netlink.

2016-08-18 Thread Joe Stringer
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. ---

[ovs-dev] [PATCH] NEWS: NAT and bundles fixes for 2.6 release.

2016-08-18 Thread Jarno Rajahalme
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

Re: [ovs-dev] ovsdb active backup deployment

2016-08-18 Thread Andy Zhou
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

Re: [ovs-dev] [PATCH v2] datapath: compat: backport LCO optimization.

2016-08-18 Thread Jesse Gross
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

Re: [ovs-dev] [PATCH V6] netdev-dpdk: Set pmd thread priority

2016-08-18 Thread Bodireddy, Bhanuprakash
>-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:

Re: [ovs-dev] [PATCHv2] system-traffic: Add link-args for ADD_NATIVE_TUNNEL.

2016-08-18 Thread Joe Stringer
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

[ovs-dev] (no subject)

2016-08-18 Thread dev-bounces
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] ovsdb active backup deployment

2016-08-18 Thread Valentine Sinitsyn
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

Re: [ovs-dev] [PATCH v2 1/2] netdev-dpdk: Fix vHost stats.

2016-08-18 Thread Daniele Di Proietto
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

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Simplify send function for ETH devices.

2016-08-18 Thread Daniele Di Proietto
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

[ovs-dev] [PATCH] tests: Avoid race in tunnel-push-pop tests.

2016-08-18 Thread Jesse Gross
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

Re: [ovs-dev] ovsdb active backup deployment

2016-08-18 Thread Andy Zhou
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

[ovs-dev] [PATCH 1/2] ovn/TODO: Update for items that have been completed.

2016-08-18 Thread Ben Pfaff
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

[ovs-dev] [PATCH 2/2] ovn/TODO: Add items proposed for 2.7 in OVN IRC meeting.

2016-08-18 Thread Ben Pfaff
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

Re: [ovs-dev] [PATCH] ovn-northd: Remove unnecessary 'if' test from build_acls().

2016-08-18 Thread Ben Pfaff
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.

[ovs-dev] [PATCHv2] system-traffic: Add link-args for ADD_NATIVE_TUNNEL.

2016-08-18 Thread William Tu
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

[ovs-dev] OVS DPDK performance drop with multiple flows

2016-08-18 Thread Bodireddy, Bhanuprakash
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

[ovs-dev] [PATCH v2 1/2] ovn: ND security vulnerability.

2016-08-18 Thread nickcooper-zhangtonghao
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.

[ovs-dev] [PATCH v2 2/2] ovn: add "timeout" column in MAC_Binding

2016-08-18 Thread nickcooper-zhangtonghao
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 ++---

[ovs-dev] [PATCH v2 0/2] ovn: ARP reply security vulnerability.

2016-08-18 Thread nickcooper-zhangtonghao
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

Re: [ovs-dev] ovsdb active backup deployment

2016-08-18 Thread Valentine Sinitsyn
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?

Re: [ovs-dev] [PATCH v4 3/3] netdev-dpdk: vHost client mode and reconnect

2016-08-18 Thread Loftus, Ciara
> 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

[ovs-dev] [PATCH] netdev-dpdk: Add new 'dpdkvhostuserclient' port type

2016-08-18 Thread Ciara Loftus
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

Re: [ovs-dev] [PATCH] netdev-dpdk: Fix occurance of error log

2016-08-18 Thread 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: 0a0f39df1d5a ("netdev-dpdk: Add support for DPDK 16.07") > Signed-off-by: Ciara Loftus >

Re: [ovs-dev] [PATCH v2 2/2] netdev-dpdk: Simplify send function for ETH devices.

2016-08-18 Thread Stokes, Ian
> '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

Re: [ovs-dev] [PATCH v2 1/2] netdev-dpdk: Fix vHost stats.

2016-08-18 Thread Stokes, Ian
> 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

[ovs-dev] Emailing: Label

2016-08-18 Thread Gale mackillop
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

Re: [ovs-dev] [PATCH V6] netdev-dpdk: Set pmd thread priority

2016-08-18 Thread Flavio Leitner
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;

[ovs-dev] [PATCH v2 0/2] Fix vHost statistics

2016-08-18 Thread Ilya Maximets
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

[ovs-dev] [PATCH v2 2/2] netdev-dpdk: Simplify send function for ETH devices.

2016-08-18 Thread Ilya Maximets
'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

[ovs-dev] [PATCH v2 1/2] netdev-dpdk: Fix vHost stats.

2016-08-18 Thread Ilya Maximets
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:

Re: [ovs-dev] [PATCH 1/2] netdev-dpdk: Fix vHost stats.

2016-08-18 Thread Ilya Maximets
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 >>

[ovs-dev] Emailing: Label

2016-08-18 Thread Iva russel
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

Re: [ovs-dev] ovsdb active backup deployment

2016-08-18 Thread Russell Bryant
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

[ovs-dev] [PATCH] INSTALL.DPDK-ADVANCED.md: Fix incorrect numbering

2016-08-18 Thread Ciara Loftus
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

Re: [ovs-dev] [PATCH] netdev-dpdk: Provide explicit flag to rte_vhost_driver_register

2016-08-18 Thread Loftus, Ciara
> > 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

[ovs-dev] Emailing: Label

2016-08-18 Thread Samuel uprichard
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

Re: [ovs-dev] [PATCH 2/2] netdev-dpdk: Simplify send function for ETH devices.

2016-08-18 Thread Stokes, Ian
> '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

Re: [ovs-dev] [PATCH 1/2] netdev-dpdk: Fix vHost stats.

2016-08-18 Thread Stokes, Ian
> 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

[ovs-dev] Emailing: Label

2016-08-18 Thread Dale chater
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

[ovs-dev] Emailing: Label

2016-08-18 Thread Elena ricketts
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

[ovs-dev] delivery failed

2016-08-18 Thread Returned mail
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

[ovs-dev] [PATCH] netdev-dpdk: Provide explicit flag to rte_vhost_driver_register

2016-08-18 Thread Ciara Loftus
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

[ovs-dev] Chinese marble slabs

2016-08-18 Thread a...@stoneartchina.com
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

[ovs-dev] MAIL SYSTEM ERROR - RETURNED MAIL

2016-08-18 Thread bruce
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

Re: [ovs-dev] ovsdb active backup deployment

2016-08-18 Thread Valentine Sinitsyn
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