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

Saloni Jain saloni.jain at tcs.com
Fri Jan 22 09:44:30 UTC 2016


Thanks.
I have updated this in version v9.

Thanks and Regards,
Saloni Jain
 Tata Consultancy Services
 Mailto: saloni.jain at tcs.com
 Website: http://www.tcs.com
 ____________________________________________
 Experience certainty.	IT Services
 			Business Solutions
 			Consulting
 ____________________________________________
 

-----Jean Tourrilhes <jt at labs.hpe.com> wrote: -----
To: Saloni Jain <saloni.jain at tcs.com>, Ben Pfaff <blp at ovn.org>, dev at openvswitch.org
From: Jean Tourrilhes <jt at labs.hpe.com>
Date: 01/16/2016 06:33AM
Subject: Re: [PATCH v8] Implement OFPT_TABLE_STATUS Message.

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/ext-192-vacancy

	I assume it's as simple as :
-    td->table_vacancy.vacancy = max_flows ? (count * 100) / max_flows : 0;
+    td->table_vacancy.vacancy = max_flows ? ((max_flows - count) * 100) / max_flows : 100;
	Totally untested, obviously...

	Regards,

	Jean
=====-----=====-----=====
Notice: The information contained in this e-mail
message and/or attachments to it may contain 
confidential or privileged information. If you are 
not the intended recipient, any dissemination, use, 
review, distribution, printing or copying of the 
information contained in this e-mail message 
and/or attachments to it are strictly prohibited. If 
you have received this communication in error, 
please notify us by reply e-mail or telephone and 
immediately and permanently delete the message 
and any attachments. Thank you





More information about the dev mailing list