From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756545Ab1CXIZc (ORCPT ); Thu, 24 Mar 2011 04:25:32 -0400 Received: from mail-fx0-f46.google.com ([209.85.161.46]:36539 "EHLO mail-fx0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752618Ab1CXIZ3 (ORCPT ); Thu, 24 Mar 2011 04:25:29 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:content-transfer-encoding :in-reply-to:user-agent; b=jz64vMpia6KzSSBz8KdOM4lkfJxUYJO2q5n0bCXQ7yswM4S+KlpGmIGHWgh7Ec2UQU +R3RaiaTYPtgw1qhiw2CZoA269t1gsoZ2uoDdEbz9Bq6QD7mxz7mANpfwHe1e434+Fwm ZJyvyIxxB6Vi0ZpR+Zo3YD/bvTvD/EGT1J8a8= Date: Thu, 24 Mar 2011 09:25:23 +0100 From: Tejun Heo 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 Subject: Re: [Uclinux-dist-devel] [RFC PATCH] percpu: always align percpu output section to PAGE_SIZE Message-ID: <20110324082523.GA12038@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> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, 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 breaking > > percpu memory alignment. > > hmm, i just pushed through a fix in the Blackfin tree as we hit a boot > failure otherwise: > - PERCPU(4) > + PERCPU(PAGE_SIZE) > > > This patch removes @align parameter from PERCPU(), renames it to > > PERCPU_SECTION and makes it always align to PAGE_SIZE.  While at it, > > add PCPU_SETUP_BUG_ON() checks such that alignment problems are > > reliably detected and remove percpu alignment comment recently added > > in workqueue.c as the condition would trigger BUG way before reaching > > there. > > seems this still hasnt made it to mainline. has it stalled or > something ? feel free for the Blackfin bits: > Signed-off-by: Mike Frysinger Heh, I just forgot. I'll queue it with your Acked-by added. Thanks. -- tejun