Re: [ovs-dev] Running openvswitch in Android

2013-06-12 Thread Jing Ai
ovs-vsctl hang for bridge related operations usually indicates that ovs-vswitchd daemon is not running. Can you check it? Also, ovs-vsctl show cmd works since it only interacts with ovsdb & ovsdb-server. Best, Jing On Wed, Jun 12, 2013 at 4:26 PM, muddin wrote: > Hello, > > > We have compi

Re: [ovs-dev] [PATCH] ofp-actions: make sure table_id in goto_table instruction is larger than the current table_id

2013-06-05 Thread Jing Ai
Thanks, Ben! Best, Jing On Wed, Jun 5, 2013 at 1:38 PM, Ben Pfaff wrote: > On Wed, Jun 05, 2013 at 01:18:09PM -0700, Jing Ai wrote: > > Found a bug that OVS allows goto_table_id to be smaller than (or equal > to) the current table id where the flow resides. It potentiall

Re: [ovs-dev] [PATCH v2]: ofp-actions: check table_id in goto_table instruction is larger than the current table_id

2013-06-04 Thread Jing Ai
Tue, Jun 04, 2013 at 01:20:57PM -0700, Jing Ai wrote: > > No problem. I just pinged the thread to get attention. Thanks for your > > review. Here I attached the patch regenerated by "git format-patch". > > > > Signed-off-by: Jing Ai > > Unfortunately it was

Re: [ovs-dev] [PATCH v2]: ofp-actions: check table_id in goto_table instruction is larger than the current table_id

2013-06-04 Thread Jing Ai
No problem. I just pinged the thread to get attention. Thanks for your review. Here I attached the patch regenerated by "git format-patch". Signed-off-by: Jing Ai --- AUTHORS | 3 ++- lib/ofp-actions.c | 5 + lib/ofp-actions.h | 1 + lib/ofp-util.c

Re: [ovs-dev] [PATCH v2]: ofp-actions: check table_id in goto_table instruction is larger than the current table_id

2013-06-04 Thread Jing Ai
Anyone reviewing it? Thanks! Best, Jing On Fri, May 31, 2013 at 5:46 PM, Jing Ai wrote: > --- Resent the patch since the last one was messed up in format > --- Added and changed AUTHOR information > > Found a bug that OVS does not return any error message when the > goto_tabl

[ovs-dev] [PATCH v2]: ofp-actions: check table_id in goto_table instruction is larger than the current table_id

2013-05-31 Thread Jing Ai
ned-off-by: Jing Ai --- AUTHORS | 3 ++- lib/ofp-actions.c | 5 + lib/ofp-actions.h | 1 + lib/ofp-util.c| 8 +--- tests/ofp-actions.at | 4 utilities/ovs-ofctl.c | 17 ++--- 6 files changed, 31 insertions(+), 7 deletions(-) diff --git a

[ovs-dev] [PATCH]: ofp-actions: check table_id in goto_table instruction is larger than the current table_id

2013-05-31 Thread Jing Ai
0, n_bytes=0, tcp,in_port=1,dl_src=00:06:07:08:09:0a,dl_dst=00:01:02:03:04:05,nw_src=192.168.0.1,nw_dst=192.168.0.2,nw_tos=0,nw_ecn=0,tp_src=1234,tp_dst=80,actions=goto_table:3 Best,Jing Signed-off-by: Jing Ai lib/ofp-actions.c | 5 + lib/ofp-actions.h | 1 + lib/ofp-util.c

Re: [ovs-dev] [PATCH v2 1/3] ovs-vswitchd: An option to wait for userspace > flow restore to complete. (Gurucharan Shetty)

2013-05-30 Thread Jing Ai
Date: Thu, 30 May 2013 08:12:16 -0700 Subject: Re: [ovs-dev] [PATCH v2 1/3] ovs-vswitchd: An option to wait for userspace > flow restore to complete. (Gurucharan Shetty) From: shet...@nicira.com To: ai_jing2...@hotmail.com CC: dev@openvswitch.org On Wed, May 29, 2013 at 7:18 PM, Jing Ai wr

Re: [ovs-dev] [PATCH v2 1/3] ovs-vswitchd: An option to wait for userspace > flow restore to complete. (Gurucharan Shetty)

2013-05-29 Thread Jing Ai
Date: Wed, 29 May 2013 13:46:05 -0700 Subject: Re: [ovs-dev] [PATCH v2 1/3] ovs-vswitchd: An option to wait for userspace > flow restore to complete. (Gurucharan Shetty) From: shet...@nicira.com To: ai_jing2...@hotmail.com CC: dev@openvswitch.org On Wed, May 29, 2013 at 1:23 PM, Jing Ai wr

Re: [ovs-dev] [PATCH v2 1/3] ovs-vswitchd: An option to wait for userspace > flow restore to complete. (Gurucharan Shetty)

2013-05-29 Thread Jing Ai
> While upgrading openvswitch, it helps to restore openflow flows before > starting packet processing. Typically we want to restart openvswitch, > add the openflow flows and then start packet processing. > > To do this, we look for the other_config:flow-restore-wait column > in the Open_vSwitch t

Re: [ovs-dev] [PATCH] ofproto-dpif: handle_flow_miss_common() without > facets. (Ethan Jackson)

2013-05-29 Thread Jing Ai
> Message: 3 > Date: Wed, 29 May 2013 10:21:28 -0700 > From: Ethan Jackson > Subject: [ovs-dev] [PATCH] ofproto-dpif: handle_flow_miss_common() > without facets. > To: dev@openvswitch.org > Message-ID: <1369848088-47098-1-git-send-email-et...@nicira.com> > > handle_flow_miss_common() is in

Re: [ovs-dev] dev Digest, Vol 46, Issue 129

2013-05-22 Thread Jing Ai
> On Mon, May 20, 2013 at 06:53:14AM +, Gurucharan Shetty wrote: > > This commit provides an option to enable or disable packet processing > > coming from the datapath. > > > > This option is useful during Open vSwitch upgrades. Typically we want > > to restart openvswitch, add the openflow

Re: [ovs-dev] [ovs-discuss] How to specify the OF version in OVS?

2013-01-09 Thread Jing Ai
gt; To: b...@nicira.com; dev@openvswitch.org > CC: ai_jing2...@hotmail.com; b...@openvswitch.org > Subject: Re: [ovs-discuss] How to specify the OF version in OVS? > > On Thu, Jan 03, 2013 at 09:05:21AM -0800, Ben Pfaff wrote: > > On Thu, Jan 03, 2013 at 06:52:30AM +, Jing Ai

Re: [ovs-dev] [PATCH] ofp-actions: Allow write_metadata instruction before goto_table instruction.

2013-01-03 Thread Jing Ai
Re: [ovs-dev] [PATCH] ofp-actions: Allow write_metadata instruction > before goto_table instruction. > > On Thu, Jan 03, 2013 at 06:41:37AM +, Jing Ai wrote: > > In the current codes, when running "ovs-ofctl add-flow br0 > conditions>,actions=write_metadata:<64-

[ovs-dev] [PATCH] ofp-actions: Allow write_metadata instruction before goto_table instruction.

2013-01-02 Thread Jing Ai
ofctl: Incorrect instruction orderingAccording to OpenFlow spec 1.2, only goto_table instruction could go after write_metadata instruction. This patch intends to address the above issue. Signed-off-by: Jing Ai lib/ofp-actions.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/l