On Tue, Oct 29, 2013 at 11:15 AM, Pravin Shelar <pshe...@nicira.com> wrote:
> On Tue, Oct 29, 2013 at 10:24 AM, Andy Zhou <az...@nicira.com> wrote:
>> This is not a review, but it would be nice to add some code comments in
>> proper data structure layout, so that the same logic can be followed in the
>> future.
>>
> ok, I will add general guideline in datapath.h as follows:
>
> /*
>  * data structure members should be arranged according to frequency of access
>  * in ovs fast-path. So that most frequently accessed members are in
> first cache-line
>  * for a datapath object.
>  */

that may be fine as general rule, but I don't see how it helps to
avoid cache misses.
Do you have performance number before and after?
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to