From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 24 Mar 2011 09:51:12 +0100 From: Tejun Heo Message-ID: <20110324085112.GB12038@htj.dyndns.org> References: <4CC69B22.2050708@kernel.org> <20101025224111.7798.38960.stgit@warthog.procyon.org.uk> <12101.1288088542@redhat.com> <4CC6C62F.1000804@kernel.org> <4CC6C941.2080003@kernel.org> <4CC6CD4D.3010501@kernel.org> <4CC6E05A.8040508@kernel.org> <20110324082523.GA12038@htj.dyndns.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: QUOTED-PRINTABLE In-Reply-To: <20110324082523.GA12038@htj.dyndns.org> Sender: linux-kernel-owner@vger.kernel.org Subject: Re: [Uclinux-dist-devel] [RFC PATCH] percpu: always align percpu output section to PAGE_SIZE To: Mike Frysinger Cc: torvalds@osdl.org, Alexander van Heukelum , linux-am33-list@redhat.com, user-mode-linux-devel@lists.sourceforge.net, Jeff Dike , linux-kernel@vger.kernel.org, David Howells , Mark Salter , uclinux-dist-devel@blackfin.uclinux.org, akpm@linux-foundation.org, Ingo Molnar , Akira Takeuchi List-ID: On Thu, Mar 24, 2011 at 09:25:23AM +0100, Tejun Heo wrote: > Hello, >=20 > On Thu, Mar 24, 2011 at 02:46:01AM -0400, Mike Frysinger wrote: > > On Tue, Oct 26, 2010 at 10:06, Tejun Heo wrote: > > > The linker script macros PERCPU_VADDR() and PERCPU() are used to > > > define this output section and the latter takes @align parameter. > > > Several architectures are using @align smaller than PAGE_SIZE bre= aking > > > percpu memory alignment. > >=20 > > hmm, i just pushed through a fix in the Blackfin tree as we hit a b= oot > > failure otherwise: > > - PERCPU(4) > > + PERCPU(PAGE_SIZE) > >=20 > > > This patch removes @align parameter from PERCPU(), renames it to > > > PERCPU_SECTION and makes it always align to PAGE_SIZE. =A0While a= t it, > > > add PCPU_SETUP_BUG_ON() checks such that alignment problems are > > > reliably detected and remove percpu alignment comment recently ad= ded > > > in workqueue.c as the condition would trigger BUG way before reac= hing > > > there. > >=20 > > seems this still hasnt made it to mainline. has it stalled or > > something ? feel free for the Blackfin bits: > > Signed-off-by: Mike Frysinger >=20 > Heh, I just forgot. I'll queue it with your Acked-by added. BTW, you're gonna push out the blackfin change to mainline soon, right? I'll queue the patch for the next merge window once the blackfin fix hits mainline. Thanks. --=20 tejun -- 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/