On Fri, Oct 18, 2013 at 1:33 PM, Pravin Shelar <[email protected]> wrote:
> On Fri, Oct 18, 2013 at 1:21 PM, Jesse Gross <[email protected]> wrote:
>> On Fri, Oct 18, 2013 at 1:06 PM, Pravin B Shelar <[email protected]> wrote:
>>> +       u8 tcp_flags;                   /* Union of seen TCP flags. */
>>> +} ____cacheline_aligned;
>>
>> It probably doesn't matter much but I think we could make this
>> ____cacheline_aligned_in_smp.
>>
> ____cacheline_aligned_in_smp also assigns data section, so I can not
> use it for struct definition.

There are two versions - the one with the double underscores sets the
data section but the one here is the same as what you have but
conditional on CONFIG_SMP.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to