[ovs-dev] [PATCH] Remove broken pip warning logs from ovsdb-server.log for ovn tests

2015-12-20 Thread Numan Siddique
Taken the fix from the commit d3292dd... (in ovn-controller-vtep.at) Signed-off-by: Numan Siddique --- tests/ovn-nbctl.at | 10 +- tests/ovn-sbctl.at | 10 +- 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/tests/ovn-nbctl.at b/tests/ovn-nbctl.at index 5358f1e..ef

[ovs-dev] [PATCH v2] datapath: stt: Fix device list management.

2015-12-20 Thread Pravin B Shelar
STT receive can accept packet on device which is not UP state. Following patch fixes this issue by introducing another list of devices which contains only devices in up state. This list can be used for searching stt devices on packet receive. Signed-off-by: Pravin B Shelar --- datapath/linux/com

Re: [ovs-dev] [PATCH] datapath: stt: Fix device list management.

2015-12-20 Thread Jesse Gross
On Sun, Dec 20, 2015 at 5:48 PM, Pravin Shelar wrote: > On Sun, Dec 20, 2015 at 10:31 AM, Jesse Gross wrote: >> One other thing that I noticed is that the STT fragment hash table >> commingles fragments that were addressed to different destination >> ports. This is unlikely to matter with the cur

Re: [ovs-dev] [PATCH] datapath: stt: Fix device list management.

2015-12-20 Thread Pravin Shelar
On Sun, Dec 20, 2015 at 10:31 AM, Jesse Gross wrote: > On Sun, Dec 20, 2015 at 1:21 AM, Pravin B Shelar wrote: >> Keep devices in UP state in stt-list. So that devices in UP state >> would be searched on a packet receive in STT. This way devices in >> DOWN state would not accept any packets. >> >

Re: [ovs-dev] [PATCH] datapath: stt: Fix error handling in stt_start().

2015-12-20 Thread Pravin Shelar
On Sun, Dec 20, 2015 at 9:07 AM, Jesse Gross wrote: > On Sun, Dec 20, 2015 at 1:21 AM, Pravin B Shelar wrote: >> The bug was reported by Joe Stringer. >> >> Signed-off-by: Pravin B Shelar > > Acked-by: Jesse Gross Thanks. I pushed it to master and branch-2.5. _

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Pravin Shelar
On Sun, Dec 20, 2015 at 12:16 PM, Joe Stringer wrote: > On 19 December 2015 at 19:19, Pravin B Shelar wrote: >> STT device is added to the device list at device create time. and >> the dev socket is initialized when dev is UP. So avoid accessing >> stt socket while searching a device. >> >> ---8<

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Pravin Shelar
On Sun, Dec 20, 2015 at 9:30 AM, Jesse Gross wrote: > On Sat, Dec 19, 2015 at 10:19 PM, Pravin B Shelar wrote: >> STT device is added to the device list at device create time. and >> the dev socket is initialized when dev is UP. So avoid accessing >> stt socket while searching a device. >> >> ---

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Joe Stringer
On 19 December 2015 at 19:19, Pravin B Shelar wrote: > STT device is added to the device list at device create time. and > the dev socket is initialized when dev is UP. So avoid accessing > stt socket while searching a device. > > ---8<--- > IP: [] nf_ip_hook+0xfd/0x180 [openvswitch] > Oops:

Re: [ovs-dev] [PATCH] datapath: stt: Fix device list management.

2015-12-20 Thread Jesse Gross
On Sun, Dec 20, 2015 at 1:21 AM, Pravin B Shelar wrote: > Keep devices in UP state in stt-list. So that devices in UP state > would be searched on a packet receive in STT. This way devices in > DOWN state would not accept any packets. > > Signed-off-by: Pravin B Shelar I think this works but it

Re: [ovs-dev] [PATCH][backport for branch-2.3] datapath: check for rx handler register

2015-12-20 Thread Jesse Gross
On Sun, Dec 20, 2015 at 3:57 AM, Wanlong Gao wrote: > Backport from the upstream commit, this commit is necessary for using > 2.3.x on RHEL with updated kernel, and absolutely more sane than the kernel > version adjudgement: > commit 572e54faff70760f4bfe4a7cffc9553e6743199e Thanks, cherry

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Jesse Gross
On Sat, Dec 19, 2015 at 10:19 PM, Pravin B Shelar wrote: > STT device is added to the device list at device create time. and > the dev socket is initialized when dev is UP. So avoid accessing > stt socket while searching a device. > > ---8<--- > IP: [] nf_ip_hook+0xfd/0x180 [openvswitch] > Oops: 0

Re: [ovs-dev] [PATCH] datapath: stt: Fix error handling in stt_start().

2015-12-20 Thread Jesse Gross
On Sun, Dec 20, 2015 at 1:21 AM, Pravin B Shelar wrote: > The bug was reported by Joe Stringer. > > Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH v2] Add Passive TCP connection to IDL

2015-12-20 Thread Ofer Ben Yacov
From: Ofer Ben-Yacov Currently the IDL does not support passive TCP connection, i.e. when the OVSDB connects to its manager. This patch enables IDL to use an already-open session (the one which was previously used for retrieving the db schema). In addition, it enables IDL to go back to "listen m

Re: [ovs-dev] [PATCH] Add Passive TCP connection to IDL

2015-12-20 Thread Russell Bryant
On 12/20/2015 08:12 AM, Ofer Ben-Yacov wrote: > Hi Russel, > > to answer your questions: > 1. limitation: the use case for passive mode is that I'm integrating IDL > to L2GW project and passive mode is prerequisite. The change that will > enable usage of the same IP and PORT (full blown server) wi

Re: [ovs-dev] [PATCH] Add Passive TCP connection to IDL

2015-12-20 Thread Ofer Ben-Yacov
Hi Russel, to answer your questions: 1. limitation: the use case for passive mode is that I'm integrating IDL to L2GW project and passive mode is prerequisite. The change that will enable usage of the same IP and PORT (full blown server) will require much more code changes, probably to Neutron als

[ovs-dev] [PATCH][backport for branch-2.3] datapath: check for rx handler register

2015-12-20 Thread Wanlong Gao
Backport from the upstream commit, this commit is necessary for using 2.3.x on RHEL with updated kernel, and absolutely more sane than the kernel version adjudgement: commit 572e54faff70760f4bfe4a7cffc9553e6743199e Author: Flavio Leitner Date: Fri Aug 28 21:52:36 2015 -03

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Pravin Shelar
On Sun, Dec 20, 2015 at 12:21 AM, Joe Stringer wrote: > On 19 December 2015 at 19:19, Pravin B Shelar wrote: >> STT device is added to the device list at device create time. and >> the dev socket is initialized when dev is UP. So avoid accessing >> stt socket while searching a device. > > Is it u

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Pravin Shelar
On Sun, Dec 20, 2015 at 12:13 AM, Joe Stringer wrote: > On 19 December 2015 at 19:19, Pravin B Shelar wrote: >> STT device is added to the device list at device create time. and >> the dev socket is initialized when dev is UP. So avoid accessing >> stt socket while searching a device. >> >> ---8<

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Joe Stringer
On 19 December 2015 at 19:19, Pravin B Shelar wrote: > STT device is added to the device list at device create time. and > the dev socket is initialized when dev is UP. So avoid accessing > stt socket while searching a device. Is it usual to accept packets while the device is down? Shouldn't we j

Re: [ovs-dev] [PATCH] datapath: stt: Do not access stt_dev socket in lookup.

2015-12-20 Thread Joe Stringer
On 19 December 2015 at 19:19, Pravin B Shelar wrote: > STT device is added to the device list at device create time. and > the dev socket is initialized when dev is UP. So avoid accessing > stt socket while searching a device. > > ---8<--- > IP: [] nf_ip_hook+0xfd/0x180 [openvswitch] > Oops: