Re: [PATCH v2 13/15] slab: use struct page for slab management

2013-10-30 Thread Joonsoo Kim
On Wed, Oct 30, 2013 at 10:42:14AM +0200, Pekka Enberg wrote: > On 10/30/2013 10:28 AM, Joonsoo Kim wrote: > >If you want an incremental patch against original patchset, > >I can do it. Please let me know what you want. > > Yes, please. Incremental is much easier to deal with if we want this > to

Re: [PATCH v2 13/15] slab: use struct page for slab management

2013-10-30 Thread Pekka Enberg
On 10/30/2013 10:28 AM, Joonsoo Kim wrote: If you want an incremental patch against original patchset, I can do it. Please let me know what you want. Yes, please. Incremental is much easier to deal with if we want this to end up in v3.13. Pekka -- To unsubscribe from this

Re: [PATCH v2 13/15] slab: use struct page for slab management

2013-10-30 Thread Joonsoo Kim
Hello, Pekka. There are two problems with this patch. One is that this makes kmemleak warning WHEN CONFIG_DEBUG_KMEMLEAK, because of false kmemleak_scan_area() call. Another is about non-existing 'struct freelist'. It is not really matter, since I just use a pointer to struct freelist. Therefore

Re: [PATCH v2 13/15] slab: use struct page for slab management

2013-10-20 Thread Christoph Lameter
On Sat, 19 Oct 2013, JoonSoo Kim wrote: > I search the history of struct page and find that the SLUB use mapping field > in past (2007 year). At that time, you inserted VM_BUG_ON(PageSlab(page)) > ('b5fab14') into page_mapping() function to find remaining use. Recently, > I never hear that this is

Re: [PATCH v2 13/15] slab: use struct page for slab management

2013-10-18 Thread JoonSoo Kim
2013/10/18 Christoph Lameter : > On Wed, 16 Oct 2013, Joonsoo Kim wrote: > >> - * see PAGE_MAPPING_ANON below. >> - */ >> + union { >> + struct address_space *mapping; /* If low bit clear, points to >> +

Re: [PATCH v2 13/15] slab: use struct page for slab management

2013-10-17 Thread Christoph Lameter
On Wed, 16 Oct 2013, Joonsoo Kim wrote: > - * see PAGE_MAPPING_ANON below. > - */ > + union { > + struct address_space *mapping; /* If low bit clear, points to > +