[lkml]   [2013]   [Aug]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 05/16] slab: remove cachep in struct slab_rcu
    On Fri, 23 Aug 2013, JoonSoo Kim wrote:

    > I don't get it. This patch only affect to the rcu case, because it
    > change the code
    > which is in kmem_rcu_free(). It doesn't touch anything in standard case.

    In general this patchset moves struct slab to overlay struct page. The
    design of SLAB was (at least at some point in the past) to avoid struct
    page references. The freelist was kept close to struct slab so that the
    contents are in the same cache line. Moving fields to struct page will add
    another cacheline to be referenced.

    The freelist (bufctl_t) was dimensioned in such a way as to be small
    and close cache wise to struct slab. Somewhow bufctl_t grew to
    unsigned int and therefore the table became a bit large. Fundamentally
    these are indexes into the objects in page. They really could be sized
    again to just be single bytes as also explained in the comments in slab.c:

    * kmem_bufctl_t:
    * Bufctl's are used for linking objs within a slab
    * linked offsets.
    * This implementation relies on "struct page" for locating the cache &
    * slab an object belongs to.
    * This allows the bufctl structure to be small (one int), but limits
    * the number of objects a slab (not a cache) can contain when off-slab
    * bufctls are used. The limit is the size of the largest general cache
    * that does not use off-slab slabs.
    * For 32bit archs with 4 kB pages, is this 56.
    * This is not serious, as it is only for large objects, when it is unwise
    * to have too many per slab.
    * Note: This limit can be raised by introducing a general cache whose size
    * is less than 512 (PAGE_SIZE<<3), but greater than 256.

    For 56 objects the bufctl_t could really be reduced to an 8 bit integer
    which would shrink the size of the table significantly and improve speed
    by reducing cache footprint.

     \ /
      Last update: 2013-08-23 18:01    [W:0.023 / U:0.204 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site