Re: [ovs-dev] [PATCH] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-12-08 Thread Shashank Shanbhag
Hi Jarno, It looks great. We tested it internally to make sure. I'm not in favor of exclusion lists, as it gets unwieldy if the number of tables of interest is pretty small (which is the case in our application). Thanks for this. Acked-by: Shashank Sha

[ovs-dev] [PATCH] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-10-18 Thread Shashank Shanbhag
Fix replace-flows and diff-flows to modify/diff flows in multiple tables. Add a --tables(-T) option that allows the user to specify a comma-separated list of table indexes to replace/diff. Signed-off-by: Shashank Shanbhag Acked-by: Romain Lenglet --- AUTHORS | 1 + NEWS

[ovs-dev] [PATCH v4] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-06-25 Thread Shashank Shanbhag
From: Shashank Shanbhag Fix replace-flows and diff-flows to modify/diff flows in multiple tables. Add a --tables(-T) option that allows the user to specify a comma-separated list of table indexes to replace/diff. Signed-off-by: Shashank Shanbhag Acked-by: Romain Lenglet --- AUTHORS

Re: [ovs-dev] [PATCH v3] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-06-23 Thread Shashank Shanbhag
Ben, Thanks for taking a look. I’ll move to using an array, and will send out the patch as soon as possible. Shashank On Jun 23, 2015, at 12:58 PM, Ben Pfaff wrote: > On Thu, Jun 18, 2015 at 02:39:07PM -0700, Shashank Shanbhag wrote: >> You also suggested that we move to using

Re: [ovs-dev] [PATCH v3] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-06-19 Thread Shashank Shanbhag
Thanks Romain. I’m actually talking about an array of *pointers* to classifiers. That comes out to 2040 bytes. === Shashank Shanbhag === On Jun 19, 2015, at 5:32 PM, Romain Lenglet wrote: > > On June 18, 2015 at 2:39:31 PM, Shashank Sh

Re: [ovs-dev] [PATCH v3] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-06-18 Thread Shashank Shanbhag
hanks, - Shashank Shanbhag, - On Thu, Jun 18, 2015 at 2:31 PM, Shashank Shanbhag < shashank.shanb...@oracle.com> wrote: > From: Shashank Shanbhag > > Fix replace-flows and diff-flows to modify/di

[ovs-dev] [PATCH v3] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-06-18 Thread Shashank Shanbhag
From: Shashank Shanbhag Fix replace-flows and diff-flows to modify/diff flows in multiple tables. Add a --tables(-T) option that allows the user to specify a comma-separated list of table indexes to replace/diff. Fix replace-flows to support OF1.3 and above. Add/change ovs-ofctl replace-flows

Re: [ovs-dev] [PATCH] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-06-05 Thread Shashank Shanbhag
idden tables aren't returned? Please correct me if I'm wrong. Shashank ----- Shashank Shanbhag, - On Fri, Jun 5, 2015 at 10:12 AM, Ben Pfaff wrote: > On Sat, May 16, 2015 a

[ovs-dev] [PATCH] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-05-16 Thread shashank . shanbhag
From: Shashank Shanbhag Fix replace-flows and diff-flows to modify/diff flows in multiple tables. Add a --tables option that allows the user to specify a comma-separated list of table indexes to replace/diff. Signed-off-by: Shashank Shanbhag Acked-by: Romain Lenglet --- AUTHORS

Re: [ovs-dev] [PATCH] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-05-14 Thread Shashank Shanbhag
Hi Ben, Thanks for your comments, and for making time to review the patch. On Thu, May 7, 2015 at 12:31 PM, Ben Pfaff wrote: > On Sun, May 03, 2015 at 05:35:43PM -0700, shashank.shanb...@gmail.com > wrote: > > From: Shashank Shanbhag > > > > Fix replace-flows and d

[ovs-dev] [PATCH] ovs-ofctl: replace-flows and diff-flows support for multiple tables

2015-05-03 Thread shashank . shanbhag
From: Shashank Shanbhag Fix replace-flows and diff-flows to modify/diff flows in multiple tables. Add a --tables option that allows the user to specify a comma-separated list of table indexes to replace/diff. Signed-off-by: Shashank Shanbhag Acked-by: Romain Lenglet --- AUTHORS