linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
@ 2010-09-12 21:47 Linus Walleij
  2010-09-13  5:55 ` Baruch Siach
  2010-09-13 13:58 ` Christoph Lameter
  0 siblings, 2 replies; 7+ messages in thread
From: Linus Walleij @ 2010-09-12 21:47 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Christoph,

commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
"slub: Remove static kmem_cache_cpu array for boot"
made my ARM 926 UP system hang at boot, if I revert this (have
to back out a lot of SLUB patches for that) it comes back up. (Same if I
switch to SLAB.)

Any hints on how I can get patch around this in a nice way?

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
  2010-09-12 21:47 Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe Linus Walleij
@ 2010-09-13  5:55 ` Baruch Siach
  2010-09-13  8:41   ` Mark Brown
  2010-09-13 13:58 ` Christoph Lameter
  1 sibling, 1 reply; 7+ messages in thread
From: Baruch Siach @ 2010-09-13  5:55 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Linus,

On Sun, Sep 12, 2010 at 11:47:39PM +0200, Linus Walleij wrote:
> commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
> "slub: Remove static kmem_cache_cpu array for boot"

In which tree have you found this commit? I cant find 
e18d65f0500b95d8724b17d8ea9f1116cf390bbe in Linus' (Torvalds) tree up to -rc4.

baruch

> made my ARM 926 UP system hang at boot, if I revert this (have
> to back out a lot of SLUB patches for that) it comes back up. (Same if I
> switch to SLAB.)
> 
> Any hints on how I can get patch around this in a nice way?

-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
  2010-09-13  5:55 ` Baruch Siach
@ 2010-09-13  8:41   ` Mark Brown
  0 siblings, 0 replies; 7+ messages in thread
From: Mark Brown @ 2010-09-13  8:41 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Sep 13, 2010 at 07:55:49AM +0200, Baruch Siach wrote:
> Hi Linus,

> On Sun, Sep 12, 2010 at 11:47:39PM +0200, Linus Walleij wrote:
> > commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
> > "slub: Remove static kmem_cache_cpu array for boot"

> In which tree have you found this commit? I cant find 
> e18d65f0500b95d8724b17d8ea9f1116cf390bbe in Linus' (Torvalds) tree up to -rc4.

It's in -next via some means.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
  2010-09-12 21:47 Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe Linus Walleij
  2010-09-13  5:55 ` Baruch Siach
@ 2010-09-13 13:58 ` Christoph Lameter
  2010-09-14  5:49   ` Pekka Enberg
  1 sibling, 1 reply; 7+ messages in thread
From: Christoph Lameter @ 2010-09-13 13:58 UTC (permalink / raw)
  To: linux-arm-kernel

On Sun, 12 Sep 2010, Linus Walleij wrote:

> commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
> "slub: Remove static kmem_cache_cpu array for boot"
> made my ARM 926 UP system hang at boot, if I revert this (have
> to back out a lot of SLUB patches for that) it comes back up. (Same if I
> switch to SLAB.)
>
> Any hints on how I can get patch around this in a nice way?

This is UP right? Any more details on this one? Does ARM have
earlyprintk?

Tejun also has some patches that make the percpu allocator work on UP so
that we do not need the hack (you must have the hack which is commit
c23e5ad729ac43f5e2fbc080f94cf61f07b4d45f as well to make UP work right!).

I am not sure if they are already in -next though.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
  2010-09-13 13:58 ` Christoph Lameter
@ 2010-09-14  5:49   ` Pekka Enberg
  2010-09-20  7:42     ` Pekka Enberg
  0 siblings, 1 reply; 7+ messages in thread
From: Pekka Enberg @ 2010-09-14  5:49 UTC (permalink / raw)
  To: linux-arm-kernel

On 9/13/10 4:58 PM, Christoph Lameter wrote:
> On Sun, 12 Sep 2010, Linus Walleij wrote:
>
>> commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
>> "slub: Remove static kmem_cache_cpu array for boot"
>> made my ARM 926 UP system hang at boot, if I revert this (have
>> to back out a lot of SLUB patches for that) it comes back up. (Same if I
>> switch to SLAB.)
>>
>> Any hints on how I can get patch around this in a nice way?
>
> This is UP right? Any more details on this one? Does ARM have
> earlyprintk?
>
> Tejun also has some patches that make the percpu allocator work on UP so
> that we do not need the hack (you must have the hack which is commit
> c23e5ad729ac43f5e2fbc080f94cf61f07b4d45f as well to make UP work right!).
>
> I am not sure if they are already in -next though.

Yes, they should be in linux-next. I haven't pulled the changes to 
slab.git, though. Linus, do things work in linux-next if you revert 
commit 5249d039500f05a5ab379286b1d23ab9b04d3f2c ("Slub: UP bandaid")?

			Pekka

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
  2010-09-14  5:49   ` Pekka Enberg
@ 2010-09-20  7:42     ` Pekka Enberg
  2010-09-20 13:12       ` Linus Walleij
  0 siblings, 1 reply; 7+ messages in thread
From: Pekka Enberg @ 2010-09-20  7:42 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Sep 14, 2010 at 8:49 AM, Pekka Enberg <penberg@cs.helsinki.fi> wrote:
> On 9/13/10 4:58 PM, Christoph Lameter wrote:
>>
>> On Sun, 12 Sep 2010, Linus Walleij wrote:
>>
>>> commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
>>> "slub: Remove static kmem_cache_cpu array for boot"
>>> made my ARM 926 UP system hang at boot, if I revert this (have
>>> to back out a lot of SLUB patches for that) it comes back up. (Same if I
>>> switch to SLAB.)
>>>
>>> Any hints on how I can get patch around this in a nice way?
>>
>> This is UP right? Any more details on this one? Does ARM have
>> earlyprintk?
>>
>> Tejun also has some patches that make the percpu allocator work on UP so
>> that we do not need the hack (you must have the hack which is commit
>> c23e5ad729ac43f5e2fbc080f94cf61f07b4d45f as well to make UP work right!).
>>
>> I am not sure if they are already in -next though.
>
> Yes, they should be in linux-next. I haven't pulled the changes to slab.git,
> though. Linus, do things work in linux-next if you revert commit
> 5249d039500f05a5ab379286b1d23ab9b04d3f2c ("Slub: UP bandaid")?

Linus, care to retest linux-next on your configuration? The above
commit has now been replaced with proper UP per-cpu allocator support
by Tejun Heo.

                        Pekka

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe
  2010-09-20  7:42     ` Pekka Enberg
@ 2010-09-20 13:12       ` Linus Walleij
  0 siblings, 0 replies; 7+ messages in thread
From: Linus Walleij @ 2010-09-20 13:12 UTC (permalink / raw)
  To: linux-arm-kernel

2010/9/20 Pekka Enberg <penberg@kernel.org>:

> Linus, care to retest linux-next on your configuration? The above
> commit has now been replaced with proper UP per-cpu allocator support
> by Tejun Heo.

For various administrative reasons I have lost contact with the hardware,
hang on and I'll retest as soon as I can!

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2010-09-20 13:12 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-12 21:47 Regression on UP ARM from commit e18d65f0500b95d8724b17d8ea9f1116cf390bbe Linus Walleij
2010-09-13  5:55 ` Baruch Siach
2010-09-13  8:41   ` Mark Brown
2010-09-13 13:58 ` Christoph Lameter
2010-09-14  5:49   ` Pekka Enberg
2010-09-20  7:42     ` Pekka Enberg
2010-09-20 13:12       ` Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).