Hi Ben,

> > This commit enables the eviction mechanism on the basis of
> > importance as
> > per the openflow specification 1.4.
> > 
> >     ovs-ofctl -O OpenFlow14 mod-table <switch> <table> evict
> > -Enable eviction on <table> of <switch>. Eviction adds a mechanism
> >  enabling the switch to automatically eliminate entries of lower
> >  importance to make space for newer entries.If want to enable
> >  eviction
> >  on all tables, user can set the <table> as 'ALL'.
> > 
> >     ovs-ofctl -O OpenFlow14 mod-table <switch> <table> noevict
> > -Disable eviction on <table> of <switch>.
> > 
> >     ovs-ofctl -O OpenFlow14 dump-tables-desc <switch>
> > -This command provides a way to list the current configuration
> >  (eviction for importance) of the tables on a <switch>, which is
> >  set
> >  using the mod-table command.
> > 
> > Signed-off-by: Saloni Jain <saloni.j...@tcs.com>
> > Signed-off-by: Hiteshi Kalra <hiteshi.ka...@tcs.com>
> 
> I don't understand why OFPTC11_TABLE_MISS_MASK has something to do
> with
> enabling or disabling eviction.  Please explain.

Agreed, the same is not required. As part of initial patch submission we had 
this part of code but later on we removed the same
while working/investigating on vacancy events.
(FYI: we have freeze the code for vacancy events feature and after review of 
the current patch we will be submitting the same as some portion of code is 
dependent on the current patch)

I will send the revised patch for "ovs-ofctl: Implementation of eviction on the 
basis of Importance" today i.e 11.02.2015.
Thank you for the review.


Rishi

Thank You
Regards
Rishi Bamba
=====-----=====-----=====
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


_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to