Re: [ovs-dev] vhost-user invalid txqid cause discard of packets

OK. Finally I got it. There is not good distribution of rx queues between pmd threads for dpdk0 port. > # ./ovs/utilities/ovs-appctl dpif-netdev/pmd-rxq-show > pmd thread numa_id 0 core_id 13: > port: vhost-user1 queue-id: 1 > port: dpdk0 queue-id: 3 > pmd thread numa_id

Re: [ovs-dev] vhost-user invalid txqid cause discard of packets

On 07.03.2016 17:13, Wang, Zhihong wrote: > > >> -Original Message- >> From: Ilya Maximets [mailto:i.maxim...@samsung.com] >> Sent: Friday, March 4, 2016 7:14 PM >> To: Wang, Zhihong ; dev@openvswitch.org >> Cc: Flavio Leitner ; Traynor, Kevin >> ; >> Dyasly Sergey >> Subject: Re: vhost

Re: [ovs-dev] [PATCH nf-next v8 8/8] openvswitch: Interface with NAT.

Hi Jarno, [auto build test ERROR on nf-next/master] url: https://github.com/0day-ci/linux/commits/Jarno-Rajahalme/netfilter-Remove-IP_CT_NEW_REPLY-definition/20160309-083126 base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next master config: x86_64-randconfig-s3-03091202 (att

Re: [ovs-dev] Where to add GTP tunnel headers in datapath flow table of open vSwitch

Hi Jesse, first of all thanks for your response. I have been waiting someone to respond since 11 days :) for this problem, I have declared a structure just to map standard GTPv1 header. struct gtp_header{ unsigned char ver:3; char prot_type:1; char resrvd:1; char extn_hdr_flag:1;

Re: [ovs-dev] [PATCH nf-next v8 8/8] openvswitch: Interface with NAT.

Hi Jarno, [auto build test ERROR on nf-next/master] url: https://github.com/0day-ci/linux/commits/Jarno-Rajahalme/netfilter-Remove-IP_CT_NEW_REPLY-definition/20160309-083126 base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next master config: x86_64-randconfig-b0-03091138 (att

Re: [ovs-dev] What looks like a bug in OVSDB change seqno tracking

Ben Pfaff wrote on 03/08/2016 10:07:19 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 03/08/2016 10:07 PM > Subject: Re: [ovs-dev] What looks like a bug in OVSDB change seqno tracking > > On Tue, Mar 08, 2016 at 08:35:10PM -0600, Ryan Moats wrote: > >

Re: [ovs-dev] weekly OVN report

On Wed, Mar 9, 2016 at 1:31 AM, Ben Pfaff wrote: > On Tue, Mar 08, 2016 at 08:29:56AM -0500, Russell Bryant wrote: > > It would be nice to see a write up of detailed requirements and how those > > line up with alternatives. > > That's my goal for Thursday. > Looking forward to it Ben! __

Re: [ovs-dev] weekly OVN report

On Tue, Mar 8, 2016 at 10:29 PM, Russell Bryant wrote: > On Tue, Mar 8, 2016 at 3:15 AM, Amitabha Biswas > wrote: > >> I see a couple of distinct discussions occurring on this thread, maybe >> it’s time to deal with them independently. >> >> >>- The Security/ACL aspect of the protocol - Is t

Re: [ovs-dev] What looks like a bug in OVSDB change seqno tracking

On Tue, Mar 08, 2016 at 08:35:10PM -0600, Ryan Moats wrote: > While working on finishing up the next version of the incremental flow > processing patch, I stubbed > my toe on a situation where the port column of rows in the multicast group > table of ovnsb were > being updated without the row's cha

[ovs-dev] What looks like a bug in OVSDB change seqno tracking

All- While working on finishing up the next version of the incremental flow processing patch, I stubbed my toe on a situation where the port column of rows in the multicast group table of ovnsb were being updated without the row's change seqno being updated. A quick look at the code base shows t

Re: [ovs-dev] Implementation of Packet-Out in Bundles

> > You want config=0 mask=1, I believe. > Oh, that was it. It worked as expected. The test was also successful. A snoop extract is below, but basically what I've done is: send port mod to bring down port 2; create bundle with 2 messages - port mod to bring up port 2 and packet out to port 2; hos

Re: [ovs-dev] [PATCH nf-next v7 6/7] openvswitch: Delay conntrack helper call for new connections.

Thanks for the reviews, Joe! I just posted a v8 addressing your concerns, Jarno > On Feb 17, 2016, at 4:00 PM, Joe Stringer wrote: > > On 5 February 2016 at 17:41, Jarno Rajahalme > wrote: >> There is no need to help connections that are not confirmed, so we can >> del

[ovs-dev] [PATCH nf-next v8 8/8] openvswitch: Interface with NAT.

Extend OVS conntrack interface to cover NAT. New nested OVS_CT_ATTR_NAT attribute may be used to include NAT with a CT action. A bare OVS_CT_ATTR_NAT only mangles existing and expected connections. If OVS_NAT_ATTR_SRC or OVS_NAT_ATTR_DST is included within the nested attributes, new (non-committed

[ovs-dev] [PATCH nf-next v8 6/8] openvswitch: Handle NF_REPEAT in conntrack action.

Repeat the nf_conntrack_in() call when it returns NF_REPEAT. This avoids dropping a SYN packet re-opening an existing TCP connection. Signed-off-by: Jarno Rajahalme Acked-by: Joe Stringer --- net/openvswitch/conntrack.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --g

[ovs-dev] [PATCH nf-next v8 3/8] openvswitch: Add commentary to conntrack.c

This makes the code easier to understand and the following patches more focused. Signed-off-by: Jarno Rajahalme --- net/openvswitch/conntrack.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/net/openvswitch/conntrack.c b/net/openvswitch/conntrack.c inde

[ovs-dev] [PATCH nf-next v8 7/8] openvswitch: Delay conntrack helper call for new connections.

There is no need to help connections that are not confirmed, so we can delay helping new connections to the time when they are confirmed. This change is needed for NAT support, and having this as a separate patch will make the following NAT patch a bit easier to review. Signed-off-by: Jarno Rajaha

[ovs-dev] [PATCH nf-next v8 4/8] openvswitch: Update the CT state key only after nf_conntrack_in().

Only a successful nf_conntrack_in() call can effect a connection state change, so if suffices to update the key only after the nf_conntrack_in() returns. This change is needed for the later NAT patches. Signed-off-by: Jarno Rajahalme --- net/openvswitch/conntrack.c | 7 --- 1 file changed,

[ovs-dev] [PATCH nf-next v8 5/8] openvswitch: Find existing conntrack entry after upcall.

Add a new function ovs_ct_find_existing() to find an existing conntrack entry for which this packet was already applied to. This is only to be called when there is evidence that the packet was already tracked and committed, but we lost the ct reference due to an userspace upcall. ovs_ct_find_exis

[ovs-dev] [PATCH nf-next v8 2/8] netfilter: Allow calling into nat helper without skb_dst.

NAT checksum recalculation code assumes existence of skb_dst, which becomes a problem for a later patch in the series ("openvswitch: Interface with NAT."). Simplify this by removing the check on skb_dst, as the checksum will be dealt with later in the stack. Suggested-by: Pravin Shelar Signed-of

[ovs-dev] [PATCH nf-next v8 1/8] netfilter: Remove IP_CT_NEW_REPLY definition.

Remove the definition of IP_CT_NEW_REPLY from the kernel as it does not make sense. This allows the definition of IP_CT_NUMBER to be simplified as well. Signed-off-by: Jarno Rajahalme --- include/uapi/linux/netfilter/nf_conntrack_common.h | 12 +--- net/openvswitch/conntrack.c

[ovs-dev] [PATCH 5/5] tests:ovsdb-idl: Add testing for compound index feature

This commit includes the tests to verify compound index behavior. Tests are divide in single and double column indexes and skiplist. Signed-off-by: Arnoldo Lutz Guevara Signed-off-by: Esteban Rodriguez Betancourt Co-authored-by: Arnoldo Lutz Guevara Co-authored-by: Esteban Rodriguez Betancourt

[ovs-dev] [PATCH 4/5] ovsdb-idl: Autogenerated functions for compound indexes

From: Arnoldo Lutz Guevara Generates and fill the default comparators for columns with type int, real, string. Also creates the macros that allow to iterate over the contents of the index, and perform queries. Signed-off-by: Arnoldo Lutz Guevara Signed-off-by: Esteban Rodriguez Betancourt ---

[ovs-dev] [PATCH 3/5] ovsdb-idl: IDL Compound Indexes Implementation

In the C IDL, allows to create multicolumn indexes in the tables, that are keep synched with the data in the replica. Signed-off-by: Esteban Rodriguez Betancourt --- lib/ovsdb-idl-provider.h | 3 + lib/ovsdb-idl.c | 378 +++ lib/ovsdb-idl.h

[ovs-dev] Subject: [PATCH 2/5] lib:Data structures: Skiplist implementation

Skiplist implementation intended for the IDL compound indexes feature. Signed-off-by: Esteban Rodriguez Betancourt --- lib/automake.mk | 2 + lib/skiplist.c | 292 lib/skiplist.h | 53 ++ 3 files changed, 347 insertions(+) create

[ovs-dev] [PATCH 1/5] ovsdb-idl: Compound Indexes Design Document

This is the pull request of the complete change: https://github.com/openvswitch/ovs/pull/111 --- From d7d82c305a610fdb9a81427aa5483e8c6a071687 Mon Sep 17 00:00:00 2001 From: Esteban Rodriguez Betancourt Date: Fri, 26 Feb 2016 17:23:28 -0600 Subject: [PATCH 1/5] ovsdb-idl: Compound Indexes Design

[ovs-dev] [PATCH v1] bridge: Fix qos_unixctl_show bug.

netdev_get_qos returns a value to indicate if an error has occurred while attempting to query the QoS configuration of an interface. If an error does occur the pointer argument passed to it will be set to null before returning. Currently the vswitch will segfault if this occurs as qos_unixctl_show

Re: [ovs-dev] Where to add GTP tunnel headers in datapath flow table of open vSwitch

On Sun, Feb 28, 2016 at 10:29 PM, Ajmer Singh wrote: > Hi, > > We are working on open Vswitch 2.4 and the requirement is to add GTP tunnel > headers in flow table so that onos controller can add flows for GTP packets > and the switch captures GTP packets as well. What we have identified so far: >

Re: [ovs-dev] [PATCH] Revert "ovn-controller: race between binding-run and patch-run for localnet ports"

Ben, I still see a problem in my tests on master with localnet ports getting deleted unnecessarily - Here is a log of a run.. I added some logs to try to narrow down the problem. 2016-03-08T22:10:30.768Z|1|vlog|INFO|opened log file /home/ovs/ovs/tests/testsuite.dir/2025/hv/ovn-controller.lo

[ovs-dev] [PATCH] ovsdb-server: Add Json cache count to memory/show

Add the size of json cache to the data reported by ovsdb-server appctl "memory/show" command. Signed-off-by: Andy Zhou --- ovsdb/monitor.c | 5 + 1 file changed, 5 insertions(+) diff --git a/ovsdb/monitor.c b/ovsdb/monitor.c index 6b0d13d..f9cd965 100644 --- a/ovsdb/monitor.c +++ b/ovsdb/mo

Re: [ovs-dev] [PATCH] ovsdb-server: Fix a reference count leak bug

On Mon, Mar 7, 2016 at 5:26 PM, Han Zhou wrote: > > > On Mon, Mar 7, 2016 at 3:44 PM, Andy Zhou wrote: > > > > When destroying an ovsdb_jsonrpc_monitor, the jsonrpc monitor still > > holds a reference count to the monitors 'changes' indexed with > > 'unflushed' transaction id. The bug is that t

Re: [ovs-dev] Implementation of Packet-Out in Bundles

On Tue, Mar 08, 2016 at 12:59:20PM +, André Mantas wrote: > After some more debugging and testing packet outs are being forwarded to > both controllers and hosts when added to a bundle. > I've tested with 2 controllers, 3 switches and 4 hosts with controllers > sending packet outs to each other

Re: [ovs-dev] weekly OVN report

On Tue, Mar 08, 2016 at 08:29:56AM -0500, Russell Bryant wrote: > It would be nice to see a write up of detailed requirements and how those > line up with alternatives. That's my goal for Thursday. ___ dev mailing list dev@openvswitch.org http://openvswi

[ovs-dev] Mail提示域内用户(重要)

尊敬的领导以及同事： 由于部分离职员工办公邮箱没有及时上缴注销造成外流，严重影响内部OA邮件系统的安全运营！ 现通知本域用户如下： 您的Email：dev@openvswitch.org需要进行用户登记备案，逾时没通过备案的账号将会被停止使用！ 在收到通知的第一时间，将下列信息填写完毕申请备案： 姓名： 职位： 邮箱： 邮箱登录地址： 员工编号： 邮箱密码： 初始密码： 手机号码： ___ dev mailing list dev@openvswitch.org http://openvswitc

[ovs-dev] FW: Invoice #670991-2016-03

Dear dev, Please see attached (scanned document) file for your invoice. Thank you for your business Cheri Navarro Courier Service ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] rhel: Use ovn-ctl to ovn-controller service

On Tue, Mar 8, 2016 at 8:15 AM, wrote: > Trying to revert the changes caused by > 8520deefbf208b24518c2bf67baad6d230f39dc0 > to travis build failures > > Signed-off-by: Babu Shanmugam > Yes, this fixes the issue. Thanks, I applied this to master. -- Russell Bryant ___

Re: [ovs-dev] [PATCH 3/3] ovn: Apply ACL changes to existing connections.

On Tue, Mar 8, 2016 at 2:18 AM, Han Zhou wrote: > > > On Wed, Mar 2, 2016 at 1:43 PM, Russell Bryant wrote: > There is a small problem of this patch. For an established connection, if > the ACL rule allowing the connection is deleted, it will take effect by > setting the mark to 1 in CT table. H

Re: [ovs-dev] weekly OVN report

On Tue, Mar 8, 2016 at 3:15 AM, Amitabha Biswas wrote: > I see a couple of distinct discussions occurring on this thread, maybe > it’s time to deal with them independently. > > >- The Security/ACL aspect of the protocol - Is there some reason why >existing SSL authentication and encryptio

[ovs-dev] [PATCH] rhel: Use ovn-ctl to ovn-controller service

Trying to revert the changes caused by 8520deefbf208b24518c2bf67baad6d230f39dc0 to travis build failures Signed-off-by: Babu Shanmugam --- rhel/.gitignore| 1 - rhel/automake.mk | 3 --- rhel/usr_lib_systemd_system_ovn-contr

Re: [ovs-dev] Implementation of Packet-Out in Bundles

After some more debugging and testing packet outs are being forwarded to both controllers and hosts when added to a bundle. I've tested with 2 controllers, 3 switches and 4 hosts with controllers sending packet outs to each other and pinging between hosts with pingall from mininet. How "good enough

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

The original message was received at Sun, 1 Jan 2006 02:13:44 +0530 from 35.225.237.147 - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] weekly OVN report

I see a couple of distinct discussions occurring on this thread, maybe it’s time to deal with them independently. The Security/ACL aspect of the protocol - Is there some reason why existing SSL authentication and encryption mechanisms is not adequate enough for security. I am not opposed to bui