Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757487Ab0LJAVz (ORCPT ); Thu, 9 Dec 2010 19:21:55 -0500 Received: from mga01.intel.com ([192.55.52.88]:29389 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756247Ab0LJAVy (ORCPT ); Thu, 9 Dec 2010 19:21:54 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.59,322,1288594800"; d="scan'208";a="634903548" Subject: Re: [patch 3/3]use new macros for x86 percpu readmostly section From: Shaohua Li To: Andrew Morton Cc: lkml , "hpa@zytor.com" , "sam@ravnborg.org" , "eric.dumazet@gmail.com" , "linux-arch@vger.kernel.org" In-Reply-To: <20101209143201.6ab0a49d.akpm@linux-foundation.org> References: <1291255350.12777.117.camel@sli10-conroe> <20101209143201.6ab0a49d.akpm@linux-foundation.org> Content-Type: text/plain; charset="UTF-8" Date: Fri, 10 Dec 2010 08:21:52 +0800 Message-ID: <1291940512.2323.11.camel@sli10-conroe> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1853 Lines: 52 On Fri, 2010-12-10 at 06:32 +0800, Andrew Morton wrote: > On Thu, 02 Dec 2010 10:02:30 +0800 > Shaohua Li wrote: > > > Use the new macros to correctly align percpu readmostly data. > > > > Signed-off-by: Shaohua Li > > --- > > arch/x86/kernel/vmlinux.lds.S | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > Index: linux/arch/x86/kernel/vmlinux.lds.S > > =================================================================== > > --- linux.orig/arch/x86/kernel/vmlinux.lds.S 2010-12-02 09:48:44.000000000 +0800 > > +++ linux/arch/x86/kernel/vmlinux.lds.S 2010-12-02 09:54:24.000000000 +0800 > > @@ -226,7 +226,7 @@ SECTIONS > > * output PHDR, so the next output section - .init.text - should > > * start another segment - init. > > */ > > - PERCPU_VADDR(0, :percpu) > > + PERCPU_VADDR_CACHEALIGNED(0, :percpu, INTERNODE_CACHE_BYTES) > > #endif > > > > INIT_TEXT_SECTION(PAGE_SIZE) > > @@ -301,7 +301,7 @@ SECTIONS > > } > > > > #if !defined(CONFIG_X86_64) || !defined(CONFIG_SMP) > > - PERCPU(THREAD_SIZE) > > + PERCPU_CACHEALIGNED(THREAD_SIZE, INTERNODE_CACHE_BYTES) > > #endif > > > > . = ALIGN(PAGE_SIZE); > > > > So what is the situation now with all other architectures? Should they > be changed to correctly align their read-mostly percpu areas? > > If so, what should the arch maintainers do to achieve this? Yes, other arch should use new macro too with correct cache alignment (similar like the alignment for read-mostly section). But currently only x86 has read-mostly percpu data. Thanks, Shaohua -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/