Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Christoph Lameter
On Fri, 8 Jun 2007, Michal Piotrowski wrote: > > Could you remove the trylock patch and see how this one fares? We may need > > both but this should avoid taking the slub_lock around any possible alloc of > > sysfs. > It's a bit tricky Hmmm... Yes that version was aginst 4-mm1 instead after the d

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Michal Piotrowski
Christoph Lameter pisze: On Fri, 8 Jun 2007, Christoph Lameter wrote: On Fri, 8 Jun 2007, Michal Piotrowski wrote: Yes, it does. Thanks! Ahhh... That leds to the discovery more sysfs problems. I need to make sure not to be holding locks while calling into sysfs. More cleanup... Could you r

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Christoph Lameter
On Fri, 8 Jun 2007, Christoph Lameter wrote: > On Fri, 8 Jun 2007, Michal Piotrowski wrote: > > > Yes, it does. Thanks! > > Ahhh... That leds to the discovery more sysfs problems. I need to make > sure not to be holding locks while calling into sysfs. More cleanup... Could you remove the trylo

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Christoph Lameter
On Fri, 8 Jun 2007, Michal Piotrowski wrote: > 0xc1081630 is in list_locations (mm/slub.c:3388). > 3383struct page *page; > 3384 > 3385if (!atomic_read(&n->nr_slabs)) > 3386continue; > 3387 > 3388spin_lock_irqs

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Michal Piotrowski
Christoph Lameter pisze: On Fri, 8 Jun 2007, Michal Piotrowski wrote: Yes, it does. Thanks! Ahhh... That leds to the discovery more sysfs problems. I need to make sure not to be holding locks while calling into sysfs. More cleanup... sysfs... I forgot about my sysfs test case #! /bin/s

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Christoph Lameter
On Fri, 8 Jun 2007, Michal Piotrowski wrote: > Yes, it does. Thanks! Ahhh... That leds to the discovery more sysfs problems. I need to make sure not to be holding locks while calling into sysfs. More cleanup... - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the bod

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Christoph Lameter
On Fri, 8 Jun 2007, Michal Piotrowski wrote: > bash shared mapping + your script in a loop > while true; do sudo ./run.sh; done > res3.txt H... Seems to be triggered from the reclaim path kmem_cache_defrag rather than the manual triggered one from the script. Taking the slub_lock on the re

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Michal Piotrowski
bash shared mapping + your script in a loop while true; do sudo ./run.sh; done > res3.txt [ 2866.154597] === [ 2866.162384] [ INFO: possible circular locking dependency detected ] [ 2866.168698] 2.6.22-rc4-mm2 #1 [ 2866.171671] ---

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Christoph Lameter
On Fri, 8 Jun 2007, Michal Piotrowski wrote: > Hi Christoph, > > On 07/06/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > > Will show up shortly at > http://ftp.kernel.org/pub/linux/kernel/people/christoph/slab-defrag/ > > I tried to apply this patchset, but without success. I tried > 2.6.22-

Re: [patch 00/12] Slab defragmentation V3

2007-06-08 Thread Christoph Lameter
On Fri, 8 Jun 2007, Michal Piotrowski wrote: > Hi Christoph, > > On 07/06/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > > Will show up shortly at > http://ftp.kernel.org/pub/linux/kernel/people/christoph/slab-defrag/ > > I tried to apply this patchset, but without success. I tried > 2.6.22-