Re: [ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-22 Thread Saloni Jain
Thanks. I have updated this in version v9. Thanks and Regards, Saloni Jain Tata Consultancy Services Mailto: saloni.j...@tcs.com Website: http://www.tcs.com Experience certainty. IT Services Business Solutions

Re: [ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-21 Thread Ben Pfaff
On Thu, Jan 14, 2016 at 05:56:02PM +0530, Saloni Jain wrote: > As per the Openflow Specification-1.4, Section: 7.4.3.1 Modify Flow Table > Message, Page:73,74: > > "When the remaining space in the flow table increases to more than > vacancy_up, and if vacancy up events are enabled, a vacancy up

Re: [ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-15 Thread Jean Tourrilhes
Saloni Jain wrote : > > If current vacancy is same as remaining space(free space), the two > statements of the specifications are contradictory. Why and how ? I prototyped the spec and I saw no trouble with those statements : https://github.com/jean2/ofsoftswitch13/commits/jean/ex

Re: [ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-14 Thread Saloni Jain
s Business Solutions Consulting -Ben Pfaff wrote: - To: Saloni Jain From: Ben Pfaff Date: 01/13/2016 10:42PM Cc: dev@openvswitch.org, saloni.jai...@gmail.com, Deepankar Gupta , Partha Datta , rishi.ba...@tcs.com Subject: Re: [ovs-dev]

Re: [ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-13 Thread Ben Pfaff
On Wed, Jan 13, 2016 at 05:44:16PM +0530, Saloni Jain wrote: > Hi Ben, > > Thanks for the review. > Regarding the test case, I want to clarify few points : > >  # OFPT_TABLE_STATUS, OFPTR_VACANCY_UP >     if test X"$1" = X"OFPTR_VACANCY_UP"; then shift; >         ovs-vsctl -- --id=@t1 create Flo

Re: [ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-13 Thread Saloni Jain
s Consulting -"dev" wrote: - To: saloni.jai...@gmail.com From: Ben Pfaff Sent by: "dev" Date: 01/13/2016 12:31AM Cc: dev@openvswitch.org, deepankar.gu...@tcs.com, partha.da...@tcs.com, Rishi Bamba Subject: Re: [ovs-dev] [PATCH v8] Implement OFPT_T

Re: [ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-12 Thread Ben Pfaff
On Tue, Jan 05, 2016 at 04:45:12PM +0530, saloni.jai...@gmail.com wrote: > From: Saloni Jain > > On change in a table state, the controller needs to be informed with > the OFPT_TABLE_STATUS message. The message is sent with reason > OFPTR_VACANCY_DOWN or OFPTR_VACANCY_UP in case of change in rema

[ovs-dev] [PATCH v8] Implement OFPT_TABLE_STATUS Message.

2016-01-05 Thread saloni . jain12
From: Saloni Jain On change in a table state, the controller needs to be informed with the OFPT_TABLE_STATUS message. The message is sent with reason OFPTR_VACANCY_DOWN or OFPTR_VACANCY_UP in case of change in remaining space eventually crossing any one of the threshold. Signed-off-by: Saloni Ja