Re: [PATCH 1/2] [x86-64] align per-cpu section to configured cache bytes

Sam Ravnborg wrote:
> On Fri, Jan 27, 2006 at 02:02:42PM -0800, Zach Brown wrote:
> 
>>-  . = ALIGN(32);
>>+  . = ALIGN(CONFIG_X86_L1_CACHE_BYTES);
> 
> 
> Grepping other arch's than just x86 and x86_64 it looks like a common
> thing.
> Is this fix really only relevant for x86 + x86_64 or should it be done
> for all arch's?

I think it'd be needed if other archs had situations where C's
(load_module()'s, in particular) notion of the cacheline size differed
from vmlinux.lds.S's.  I didn't want to go screwing around with archs
that I couldn't immediately test :)

> If we do it for all archs we may as well create:
> #define PERCPU(aling) ...
> macro in asm-generic/vmlinux.lds.h

Sounds reasonable to me, should I leave that in your capable hands?

- z
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH 1/2] [x86-64] align per-cpu section to configured cache bytes
    • From: Zach Brown <[email protected]>
  • Re: [PATCH 1/2] [x86-64] align per-cpu section to configured cache bytes
    • From: Sam Ravnborg <[email protected]>

• Prev by Date: Re: [patch 1/2] rcu batch tuning
• Next by Date: Re: iommu_alloc failure and panic
• Previous by thread: Re: [PATCH 1/2] [x86-64] align per-cpu section to configured cache bytes
• Next by thread: Re: [PATCH 1/2] [x86-64] align per-cpu section to configured cache bytes
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]