On Wed, Mar 12, 2014 at 1:36 PM, Ben Pfaff <b...@nicira.com> wrote:
> On Sun, Mar 09, 2014 at 05:48:04PM +0800, kmindg wrote:
>> We do call heap_rebuild in ofproto_run, but we do not update rule's
>> priority with latest hard_timeout and idle_timeout before heap_rebuild.
>>
>> This patch ensures that rule's priority has been updated before
>> heap_rebuild, and adds two test cases to check eviction with modified
>> hard_timeout and idle_timwout.
>>
>> Signed-off-by: kmindg <kmi...@gmail.com>
>
> Good catch.  I applied this.
>
> I left the declaration of heap_raw_change() in heap.h.  I like to be
> able to see a summary of all the available functions at the top of the
> header, even if some of the functions have inline implementations later
> in the header.

OK, I understand that now. :)
Thanks for applying this patch.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to