[ovs-dev] [PATCH] netdev-dpdk: Do not flush tx queue which is shared among CPUs since it is always flushed

2015-06-04 Thread Wei li
When tx queue is shared among CPUS,the pkts always be flush in 'netdev_dpdk_eth_send' So it is unnecessarily for flushing in netdev_dpdk_rxq_recv Otherwise tx will be accessed without locking Signed-off-by: Wei li --- lib/netdev-dpdk.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(

[ovs-dev] [PATCH] Do not flush tx queue which is shared among CPUs since it is always flushed

2015-06-04 Thread Wei li
When tx queue is shared among CPUS,the pkts always be flush in 'netdev_dpdk_eth_send' So it is unnecessarily for flushing in netdev_dpdk_rxq_recv Otherwise tx will be accessed without locking Signed-off-by: Wei li --- lib/netdev-dpdk.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(

Re: [ovs-dev] [PATCH net-next V10 3/4] 802.1AD: Flow handling, actions and vlan parsing

2015-06-04 Thread Thomas F Herbert
On 6/4/15 4:39 PM, Pravin Shelar wrote: On Thu, Jun 4, 2015 at 11:18 AM, Thomas F Herbert wrote: On 6/4/15 1:45 PM, Pravin Shelar wrote: On Tue, Jun 2, 2015 at 10:50 AM, Thomas F Herbert wrote: Add support for 802.1ad including the ability to push and pop double tagged vlans. Signed-off-b

[ovs-dev] Mail System Error - Returned Mail

2015-06-04 Thread MAILER-DAEMON
The original message was received at Thu, 4 Jun 2015 23:22:24 +0200 from openvswitch.org [139.206.246.16] - The following addresses had permanent fatal errors - dev@openvswitch.org - Transcript of session follows - ... while talking to mail server 92.169.105.202: >>> MAIL FROM:"MA

[ovs-dev] [PATCH] tunneling: Fix a tunnel name display bug

2015-06-04 Thread Andy Zhou
Currently, 'ovs-appctl tnl/ports/show' command won't display gre port name correctly. Since netdev_vport_get_dpif_port() will not always set the 'namebuf' it receives. Should use the name by its return value instead. Found by inspection. Also extend the test case to cover this command. Signed-off

Re: [ovs-dev] [PATCH net-next V10 3/4] 802.1AD: Flow handling, actions and vlan parsing

2015-06-04 Thread Pravin Shelar
On Thu, Jun 4, 2015 at 11:18 AM, Thomas F Herbert wrote: > On 6/4/15 1:45 PM, Pravin Shelar wrote: >> >> On Tue, Jun 2, 2015 at 10:50 AM, Thomas F Herbert >> wrote: >>> >>> Add support for 802.1ad including the ability to push and pop double >>> tagged vlans. >>> >>> Signed-off-by: Thomas F Herbe

Re: [ovs-dev] [PATCH net-next V10 3/4] 802.1AD: Flow handling, actions and vlan parsing

2015-06-04 Thread Thomas F Herbert
On 6/4/15 1:45 PM, Pravin Shelar wrote: On Tue, Jun 2, 2015 at 10:50 AM, Thomas F Herbert wrote: Add support for 802.1ad including the ability to push and pop double tagged vlans. Signed-off-by: Thomas F Herbert --- net/openvswitch/flow.c | 82 ++-

Re: [ovs-dev] [PATCH v7] netdev-dpdk: add dpdk vhost-user ports

2015-06-04 Thread Pravin Shelar
On Thu, Jun 4, 2015 at 6:53 AM, Loftus, Ciara wrote: > >> On Fri, May 22, 2015 at 8:40 AM, Ciara Loftus wrote: >> > This patch adds support for a new port type to the userspace >> > datapath called dpdkvhostuser. ... ... >> > + " --vhost_sock_dir DIR override default directory >>

Re: [ovs-dev] [PATCH net-next V10 3/4] 802.1AD: Flow handling, actions and vlan parsing

2015-06-04 Thread Pravin Shelar
On Tue, Jun 2, 2015 at 10:50 AM, Thomas F Herbert wrote: > Add support for 802.1ad including the ability to push and pop double > tagged vlans. > > Signed-off-by: Thomas F Herbert > --- > net/openvswitch/flow.c | 82 > ++ > net/openvswitch/flow.h

Re: [ovs-dev] [PATCH v3] dpif-netdev: Fix non-pmd thread queue id.

2015-06-04 Thread Daniele Di Proietto
Thanks for the reviews! On 03/06/2015 23:35, "Ethan Jackson" wrote: >Acked-by: Ethan Jackson > >Merged. > >Ethan > >On Wed, Jun 3, 2015 at 10:45 AM, Flavio Leitner wrote: >> On Wed, Jun 03, 2015 at 03:55:16PM +0100, Daniele Di Proietto wrote: >>> Non pmd threads have a core_id == UINT32_MAX, w

Re: [ovs-dev] [PATCH v3] dpif-netdev: Fix non-pmd thread queue id.

2015-06-04 Thread Gray, Mark D
> Subject: [PATCH v3] dpif-netdev: Fix non-pmd thread queue id. > > Non pmd threads have a core_id == UINT32_MAX, while queue ids used by > netdevs range from 0 to the number of CPUs. Therefore core ids cannot be > used directly to select a queue. > > This commit introduces a simple mapping to f

Re: [ovs-dev] [PATCH v7] netdev-dpdk: add dpdk vhost-user ports

2015-06-04 Thread Loftus, Ciara
> On Fri, May 22, 2015 at 8:40 AM, Ciara Loftus wrote: > > This patch adds support for a new port type to the userspace > > datapath called dpdkvhostuser. > > > > A new dpdkvhostuser port will create a unix domain socket which > > when provided to QEMU is used to facilitate communication between

[ovs-dev] [PATCH v8] netdev-dpdk: add dpdk vhost-user ports

2015-06-04 Thread Ciara Loftus
This patch adds support for a new port type to the userspace datapath called dpdkvhostuser. A new dpdkvhostuser port will create a unix domain socket which when provided to QEMU is used to facilitate communication between the virtio-net device on the VM and the OVS port on the host. vhost-cuse ('

Re: [ovs-dev] 答复: dpdk vhost-user patch questions.

2015-06-04 Thread Gray, Mark D
> > It should be once, from the host memory address space to the guest > > memory address space (and vice-versa in the other direction). > > I can not understand 'be once' > > In vhost, VM share *down* the memory to host, why host 'copy the pkts > from the host memory address space to the guest

[ovs-dev] Attention Timepiece Enthusiasts and Collectors

2015-06-04 Thread Pasquale Brady
Swiss-quality branded watches at clearance prices here http://nkbdesignandmanagement.com/xryugj/y6zou.html";>More Details ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ofproto-dpif: Avoid creating OpenFlow ports for duplicate tunnels.

2015-06-04 Thread Andy Zhou
By bumb luck, I noticed that 'ofproto->backer->tnl_count' will affect how ovs_native_tunneling_is_on() gives its answer. The following incremental diff seems to help 'tunnel_push_pop" test to pass. BTW, I am not suggesting this is the right fix, since it looks ugly. diff --git a/ofproto/ofproto-d