linux-kernel - Re: [RFC/PATCH 1/3] SLAB: Add PageSlab checking to ksize()

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:	Thu, 22 May 2008 07:43:02 +0300
From:	Pekka Enberg <penberg@...helsinki.fi>
To:	Paul Mundt <lethal@...ux-sh.org>,
	Pekka Enberg <penberg@...helsinki.fi>,
	Christoph Lameter <clameter@....com>,
	linux-kernel@...r.kernel.org, mpm@...enic.com, dhowells@...hat.com
Subject: Re: [RFC/PATCH 1/3] SLAB: Add PageSlab checking to ksize()

Hi Paul,

Paul Mundt wrote:
> Shouldn't SLOB's PageSlab usage should mimic that of SLUB in this case
> instead? PG_slab doesn't buy us much if we can already sort out the size
> through compound_order(), it's the kmem_cache_alloc() and <= PAGE_SIZE
> kmalloc()'s where __GFP_COMP isn't true and where PG_slab should be set.

Well, we should really be calling ksize() in the nommu case and although 
Matt and Christoph don't seem to agree with me here, I'd much rather 
have *all* allocators set PageSlab for all the pages they return (yes, 
including pass-through ones) to get us back where we were with SLAB. We 
currently don't have any means to see whether an arbitrary page is part 
of the slab or not and I'd argue that's PageSlab is an established API 
(that makes sense).

Furthermore, if ksize() in SLOB doesn't work for kmem_cache_alloc() 
pages, I think it should be fixed as well.

			Pekka
--
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/