Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752482Ab0LPF4u (ORCPT ); Thu, 16 Dec 2010 00:56:50 -0500 Received: from mga09.intel.com ([134.134.136.24]:27590 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752299Ab0LPF4r (ORCPT ); Thu, 16 Dec 2010 00:56:47 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.59,353,1288594800"; d="scan'208";a="584209560" Subject: Re: [patch 2/3] add new macros to make percpu readmostly section correctly align From: Shaohua Li To: Sam Ravnborg Cc: Tejun Heo , lkml , "hpa@zytor.com" , Andrew Morton , "eric.dumazet@gmail.com" In-Reply-To: <20101216054620.GA3241@merkur.ravnborg.org> References: <1291255347.12777.116.camel@sli10-conroe> <4D0243DD.3090507@gmail.com> <1292200886.2323.15.camel@sli10-conroe> <4D05EBA4.60202@gmail.com> <1292288924.2323.458.camel@sli10-conroe> <4D073FA9.9020801@gmail.com> <1292378268.2323.464.camel@sli10-conroe> <4D08CBF4.6040102@gmail.com> <1292460817.2323.465.camel@sli10-conroe> <20101216054620.GA3241@merkur.ravnborg.org> Content-Type: text/plain; charset="UTF-8" Date: Thu, 16 Dec 2010 13:56:40 +0800 Message-ID: <1292479000.2323.468.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: 1845 Lines: 49 On Thu, 2010-12-16 at 13:46 +0800, Sam Ravnborg wrote: > On Thu, Dec 16, 2010 at 08:53:37AM +0800, Shaohua Li wrote: > > On Wed, 2010-12-15 at 22:08 +0800, Tejun Heo wrote: > > > Hello, > > > > > > On 12/15/2010 02:57 AM, Shaohua Li wrote: > > > > How about this one? > > > > > > Much better. :-) > > > > > > > +#ifdef INTERNODE_CACHE_BYTES > > > > +#define INTERNODE_CACHEALIGNED \ > > > > + . = ALIGN(INTERNODE_CACHE_BYTES); > > > > +#else > > > > +#define INTERNODE_CACHEALIGNED > > > > +#endif > > > > > > Yeah, this looks good. > > > > > > > Index: linux/arch/x86/kernel/vmlinux.lds.S > > > > =================================================================== > > > > --- linux.orig/arch/x86/kernel/vmlinux.lds.S 2010-12-15 09:37:01.000000000 +0800 > > > > +++ linux/arch/x86/kernel/vmlinux.lds.S 2010-12-15 09:37:06.000000000 +0800 > > > > @@ -20,11 +20,11 @@ > > > > #define LOAD_OFFSET __START_KERNEL_map > > > > #endif > > > > > > > > -#include > > > > #include > > > > #include > > > > #include > > > > #include > > > > +#include > > > > #include > > > > > > Why do we need this chunk? > > the cache size is defined in cache.h, so I need move vmlinux.lds.h after > > cache.h > > The right fix is to move the inclusion of cache.h to asm-generic/vmlinux.lds.h. > A quick audit only found sparc that failed to guard non assembler stuff. with this, we need check every arch, at least doing a compile. I'm afraid I can't, sorry. -- 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/