On Tue, Jul 16, 2013 at 3:16 PM, H. Peter Anvin <h...@zytor.com> wrote: > On 07/16/2013 03:13 PM, Yinghai Lu wrote: >> >> ok, then should change >> >>> +/* No need to be aligned, but done to keep all IDTs defined the same way. >>> */ >>> +gate_desc trace_idt_table[NR_VECTORS] __page_aligned_bss; >> >> ==> >> >>> +/* Only need to be cacheline aligned, but keep all IDTs defined the same >>> way to be page aligned. */ >>> +gate_desc trace_idt_table[NR_VECTORS] __page_aligned_bss; > > It doesn't need to be cacheline aligned, either, to the best of my > knowledge. The former comment is more correct.
ok. so the old code is just for optimization to keep it cacheline aligned? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/