Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-08-12 Thread Wladislav Wiebe
Hi guys, we got the real root cause of the allocation issue: Subject: [PATCH 1/1] of: fdt: fix memory initialization for expanded DT Already existing property flags are filled wrong for properties created from initial FDT. This could cause problems if this DYNAMIC device-tree functions are used

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-08-12 Thread Benjamin Herrenschmidt
On Mon, 2013-08-12 at 13:06 +0200, Wladislav Wiebe wrote: Hi guys, we got the real root cause of the allocation issue: Subject: [PATCH 1/1] of: fdt: fix memory initialization for expanded DT Already existing property flags are filled wrong for properties created from initial FDT. This

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-08-06 Thread Wladislav Wiebe
Hi, On 31/07/13 19:04, Christoph Lameter wrote: On Wed, 31 Jul 2013, Wladislav Wiebe wrote: Thanks for the point, do you plan to make kmalloc_large available for extern access in a separate mainline patch? Since kmalloc_large is statically defined in slub_def.h and when including it to

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-08-06 Thread Christoph Lameter
On Tue, 6 Aug 2013, Wladislav Wiebe wrote: ok, just saw in slab/for-linus branch that those stuff is reverted again.. No that was only for the 3.11 merge by Linus. The 3.12 patches have not been put into pekkas tree. ___ Linuxppc-dev mailing list

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-08-01 Thread Wladislav Wiebe
Hi, On 31/07/13 19:34, Aaro Koskinen wrote: Hi, On Wed, Jul 31, 2013 at 01:42:31PM +0200, Wladislav Wiebe wrote: DEBUG: xxx kmalloc_slab, requested 'size' = 8388608, KMALLOC_MAX_SIZE = 4194304 [...] It seems some procfs file is trying to dump 8 MB at a single go. You need to fix that

mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-07-31 Thread Wladislav Wiebe
Hello guys, on a PPC 32-Bit board with a Linux Kernel v3.10.0 I see trouble with kmalloc_slab. Basically at system startup, something request a size of 8388608 b, but KMALLOC_MAX_SIZE has 4194304 b in our case. It points a WARNING at: .. NIP [c0099fec] kmalloc_slab+0x60/0xe8 LR [c0099fd4]

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-07-31 Thread Wladislav Wiebe
Hi Christoph, On 31/07/13 17:45, Christoph Lameter wrote: Crap you cannot do PAGE_SIZE allocations with kmalloc_large. Fails when freeing pages. Need to only do the multiple page allocs with kmalloc_large. Subject: seq_file: Use kmalloc_large for page sized allocation There is no point

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-07-31 Thread Aaro Koskinen
Hi, On Wed, Jul 31, 2013 at 01:42:31PM +0200, Wladislav Wiebe wrote: DEBUG: xxx kmalloc_slab, requested 'size' = 8388608, KMALLOC_MAX_SIZE = 4194304 [...] [ccd3be60] [c0099fd4] kmalloc_slab+0x48/0xe8 (unreliable) [ccd3be70] [c00ae650] __kmalloc+0x20/0x1b4 [ccd3be90] [c00d46f4]

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-07-31 Thread Christoph Lameter
On Wed, 31 Jul 2013, Wladislav Wiebe wrote: on a PPC 32-Bit board with a Linux Kernel v3.10.0 I see trouble with kmalloc_slab. Basically at system startup, something request a size of 8388608 b, but KMALLOC_MAX_SIZE has 4194304 b in our case. It points a WARNING at: .. NIP [c0099fec]

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-07-31 Thread Christoph Lameter
This patch will suppress the warnings by using the page allocator wrappers of the slab allocators. These are page sized allocs after all. Subject: seq_file: Use kmalloc_large for page sized allocation There is no point in using the slab allocation functions for large page order allocation. Use

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-07-31 Thread Christoph Lameter
Crap you cannot do PAGE_SIZE allocations with kmalloc_large. Fails when freeing pages. Need to only do the multiple page allocs with kmalloc_large. Subject: seq_file: Use kmalloc_large for page sized allocation There is no point in using the slab allocation functions for large page order

Re: mm/slab: ppc: ubi: kmalloc_slab WARNING / PPC + UBI driver

2013-07-31 Thread Christoph Lameter
On Wed, 31 Jul 2013, Wladislav Wiebe wrote: Thanks for the point, do you plan to make kmalloc_large available for extern access in a separate mainline patch? Since kmalloc_large is statically defined in slub_def.h and when including it to seq_file.c we have a lot of conflicting types: You