linux-kernel - Re: [GIT PULL v2] Early SLAB fixes for 2.6.31

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 15 Jun 2009 12:20:04 +0200
From:	Heiko Carstens <heiko.carstens@...ibm.com>
To:	Pekka Enberg <penberg@...helsinki.fi>
Cc:	Nick Piggin <npiggin@...e.de>, torvalds@...ux-foundation.org,
	linux-kernel@...r.kernel.org, akpm@...ux-foundation.org,
	cl@...ux-foundation.org, kamezawa.hiroyu@...fujitsu.com,
	lizf@...fujitsu.com, mingo@...e.hu, yinghai@...nel.org,
	benh@...nel.crashing.org
Subject: Re: [GIT PULL v2] Early SLAB fixes for 2.6.31

> > ---
> >  mm/slub.c |   17 ++++++++++++++---
> >  1 file changed, 14 insertions(+), 3 deletions(-)
> > 
> > Index: linux-2.6/mm/slub.c
> > ===================================================================
> > --- linux-2.6.orig/mm/slub.c
> > +++ linux-2.6/mm/slub.c
> > @@ -2596,6 +2596,7 @@ static noinline struct kmem_cache *dma_k
> >  	struct kmem_cache *s;
> >  	char *text;
> >  	size_t realsize;
> > +	unsigned long slabflags;
> >  
> >  	s = kmalloc_caches_dma[index];
> >  	if (s)
> > @@ -2617,9 +2618,18 @@ static noinline struct kmem_cache *dma_k
> >  			 (unsigned int)realsize);
> >  	s = kmalloc(kmem_size, flags & ~SLUB_DMA);
> >  
> > +	/*
> > +	 * Must defer sysfs creation to a workqueue because we don't know
> > +	 * what context we are called from. Before sysfs comes up, we don't
> > +	 * need to do anything because slab_sysfs_init will start by
> > +	 * adding all existing slabs to sysfs.
> > +	 */
> > +	slabflags = SLAB_CACHE_DMA;
> > +	if (slab_state >= SYSFS)
> > +		slabflags |= __SYSFS_ADD_DEFERRED;
> > +
> >  	if (!s || !text || !kmem_cache_open(s, flags, text,
> > -			realsize, ARCH_KMALLOC_MINALIGN,
> > -			SLAB_CACHE_DMA|__SYSFS_ADD_DEFERRED, NULL)) {
> > +			realsize, ARCH_KMALLOC_MINALIGN, slabflags, NULL)) {
> >  		kfree(s);
> >  		kfree(text);
> >  		goto unlock_out;
> > @@ -2628,7 +2638,8 @@ static noinline struct kmem_cache *dma_k
> >  	list_add(&s->list, &slab_caches);
> >  	kmalloc_caches_dma[index] = s;
> >  
> > -	schedule_work(&sysfs_add_work);
> > +	if (slab_state >= SYSFS)
> > +		schedule_work(&sysfs_add_work);
> >  
> >  unlock_out:
> >  	up_write(&slub_lock);
> 
> Looks good to me. Heiko, does it fix your case?

Yes, works fine. Thanks!

Tested-by: Heiko Carstens <heiko.carstens@...ibm.com>
--
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/