[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AANLkTi=ZRowUtmSR+kLLphnRtyg1uqnFZ5s+_db6P54F@mail.gmail.com>
Date: Thu, 24 Mar 2011 09:46:53 -0400
From: Mike Frysinger <vapier@...too.org>
To: Tejun Heo <tj@...nel.org>
Cc: torvalds@...l.org, Alexander van Heukelum <heukelum@...tmail.fm>,
linux-am33-list@...hat.com,
user-mode-linux-devel@...ts.sourceforge.net,
Jeff Dike <jdike@...toit.com>, linux-kernel@...r.kernel.org,
David Howells <dhowells@...hat.com>,
Mark Salter <msalter@...hat.com>,
uclinux-dist-devel@...ckfin.uclinux.org, akpm@...ux-foundation.org,
Ingo Molnar <mingo@...e.hu>,
Akira Takeuchi <takeuchi.akr@...panasonic.com>
Subject: Re: [Uclinux-dist-devel] [RFC PATCH] percpu: always align percpu
output section to PAGE_SIZE
On Thu, Mar 24, 2011 at 4:51 AM, Tejun Heo wrote:
> On Thu, Mar 24, 2011 at 09:25:23AM +0100, Tejun Heo wrote:
>> 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 <vapier@...too.org>
>>
>> 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.
Linus has already pulled it
-mike
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists