Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-12 Thread Joonsoo Kim
On Wed, Sep 11, 2013 at 02:22:25PM +, Christoph Lameter wrote: > On Wed, 11 Sep 2013, Joonsoo Kim wrote: > > > Anyway, could you review my previous patchset, that is, 'overload struct > > slab > > over struct page to reduce memory usage'? I'm not sure whether your answer > > is > > ack or

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-12 Thread Joonsoo Kim
On Wed, Sep 11, 2013 at 02:22:25PM +, Christoph Lameter wrote: On Wed, 11 Sep 2013, Joonsoo Kim wrote: Anyway, could you review my previous patchset, that is, 'overload struct slab over struct page to reduce memory usage'? I'm not sure whether your answer is ack or not. I

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-11 Thread Christoph Lameter
On Wed, 11 Sep 2013, Joonsoo Kim wrote: > Anyway, could you review my previous patchset, that is, 'overload struct slab > over struct page to reduce memory usage'? I'm not sure whether your answer is > ack or not. I scanned over it before but I was not able to see if it was correct on first

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-11 Thread Christoph Lameter
On Wed, 11 Sep 2013, Joonsoo Kim wrote: Anyway, could you review my previous patchset, that is, 'overload struct slab over struct page to reduce memory usage'? I'm not sure whether your answer is ack or not. I scanned over it before but I was not able to see if it was correct on first glance.

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-10 Thread Joonsoo Kim
On Tue, Sep 10, 2013 at 09:25:05PM +, Christoph Lameter wrote: > On Tue, 10 Sep 2013, Joonsoo Kim wrote: > > > On Mon, Sep 09, 2013 at 02:44:03PM +, Christoph Lameter wrote: > > > On Mon, 9 Sep 2013, Joonsoo Kim wrote: > > > > > > > 32 byte is not minimum object size, minimum *kmalloc*

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-10 Thread Christoph Lameter
On Tue, 10 Sep 2013, Joonsoo Kim wrote: > On Mon, Sep 09, 2013 at 02:44:03PM +, Christoph Lameter wrote: > > On Mon, 9 Sep 2013, Joonsoo Kim wrote: > > > > > 32 byte is not minimum object size, minimum *kmalloc* object size > > > in default configuration. There are some slabs that their

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-10 Thread Christoph Lameter
On Tue, 10 Sep 2013, Joonsoo Kim wrote: On Mon, Sep 09, 2013 at 02:44:03PM +, Christoph Lameter wrote: On Mon, 9 Sep 2013, Joonsoo Kim wrote: 32 byte is not minimum object size, minimum *kmalloc* object size in default configuration. There are some slabs that their object size is

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-10 Thread Joonsoo Kim
On Tue, Sep 10, 2013 at 09:25:05PM +, Christoph Lameter wrote: On Tue, 10 Sep 2013, Joonsoo Kim wrote: On Mon, Sep 09, 2013 at 02:44:03PM +, Christoph Lameter wrote: On Mon, 9 Sep 2013, Joonsoo Kim wrote: 32 byte is not minimum object size, minimum *kmalloc* object size

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-09 Thread Joonsoo Kim
On Mon, Sep 09, 2013 at 02:44:03PM +, Christoph Lameter wrote: > On Mon, 9 Sep 2013, Joonsoo Kim wrote: > > > 32 byte is not minimum object size, minimum *kmalloc* object size > > in default configuration. There are some slabs that their object size is > > less than 32 byte. If we have a 8

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-09 Thread Christoph Lameter
On Mon, 9 Sep 2013, Joonsoo Kim wrote: > 32 byte is not minimum object size, minimum *kmalloc* object size > in default configuration. There are some slabs that their object size is > less than 32 byte. If we have a 8 byte sized kmem_cache, it has 512 objects > in 4K page. As far as I can recall

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-09 Thread Christoph Lameter
On Mon, 9 Sep 2013, Joonsoo Kim wrote: 32 byte is not minimum object size, minimum *kmalloc* object size in default configuration. There are some slabs that their object size is less than 32 byte. If we have a 8 byte sized kmem_cache, it has 512 objects in 4K page. As far as I can recall

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-09 Thread Joonsoo Kim
On Mon, Sep 09, 2013 at 02:44:03PM +, Christoph Lameter wrote: On Mon, 9 Sep 2013, Joonsoo Kim wrote: 32 byte is not minimum object size, minimum *kmalloc* object size in default configuration. There are some slabs that their object size is less than 32 byte. If we have a 8 byte sized

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-08 Thread Joonsoo Kim
On Fri, Sep 06, 2013 at 03:58:18PM +, Christoph Lameter wrote: > On Fri, 6 Sep 2013, Joonsoo Kim wrote: > > > Currently, the freelist of a slab consist of unsigned int sized indexes. > > Most of slabs have less number of objects than 256, since restriction > > for page order is at most 1 in

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-08 Thread Joonsoo Kim
On Fri, Sep 06, 2013 at 03:58:18PM +, Christoph Lameter wrote: On Fri, 6 Sep 2013, Joonsoo Kim wrote: Currently, the freelist of a slab consist of unsigned int sized indexes. Most of slabs have less number of objects than 256, since restriction for page order is at most 1 in default

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-06 Thread Christoph Lameter
On Fri, 6 Sep 2013, Joonsoo Kim wrote: > Currently, the freelist of a slab consist of unsigned int sized indexes. > Most of slabs have less number of objects than 256, since restriction > for page order is at most 1 in default configuration. For example, > consider a slab consisting of 32 byte

Re: [REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-06 Thread Christoph Lameter
On Fri, 6 Sep 2013, Joonsoo Kim wrote: Currently, the freelist of a slab consist of unsigned int sized indexes. Most of slabs have less number of objects than 256, since restriction for page order is at most 1 in default configuration. For example, consider a slab consisting of 32 byte sized

[REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-05 Thread Joonsoo Kim
Currently, the freelist of a slab consist of unsigned int sized indexes. Most of slabs have less number of objects than 256, since restriction for page order is at most 1 in default configuration. For example, consider a slab consisting of 32 byte sized objects on two continous pages. In this

[REPOST PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-05 Thread Joonsoo Kim
Currently, the freelist of a slab consist of unsigned int sized indexes. Most of slabs have less number of objects than 256, since restriction for page order is at most 1 in default configuration. For example, consider a slab consisting of 32 byte sized objects on two continous pages. In this

[PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-02 Thread Joonsoo Kim
Currently, the freelist of a slab consist of unsigned int sized indexes. Most of slabs have less number of objects than 256, since restriction for page order is at most 1 in default configuration. For example, consider a slab consisting of 32 byte sized objects on two continous pages. In this

[PATCH 3/4] slab: introduce byte sized index for the freelist of a slab

2013-09-02 Thread Joonsoo Kim
Currently, the freelist of a slab consist of unsigned int sized indexes. Most of slabs have less number of objects than 256, since restriction for page order is at most 1 in default configuration. For example, consider a slab consisting of 32 byte sized objects on two continous pages. In this