diff options
| author |   Joonsoo Kim <js1304@gmail.com> | 2012-05-17 00:13:02 +0900 |
|---|---|---|
| committer |   Pekka Enberg <penberg@kernel.org> | 2012-05-18 12:23:36 +0300 |
| commit | 02d7633fa567be7bf55a993b79d2a31b95ce2227 (patch) | |
| tree | 31df09df6c946f9b8edc909648e0a1c33dae794c /Documentation | |
| parent | slub: remove unused argument of init_kmem_cache_node() (diff) | |
| download | linux-dev-02d7633fa567be7bf55a993b79d2a31b95ce2227.tar.xz linux-dev-02d7633fa567be7bf55a993b79d2a31b95ce2227.zip | |
slub: fix a memory leak in get_partial_node()
In the case which is below,
1. acquire slab for cpu partial list
2. free object to it by remote cpu
3. page->freelist = t
then memory leak is occurred.
Change acquire_slab() not to zap freelist when it works for cpu partial list.
I think it is a sufficient solution for fixing a memory leak.
Below is output of 'slabinfo -r kmalloc-256'
when './perf stat -r 30 hackbench 50 process 4000 > /dev/null' is done.
***Vanilla***
Sizes (bytes) Slabs Debug Memory
------------------------------------------------------------------------
Object : 256 Total : 468 Sanity Checks : Off Total: 3833856
SlabObj: 256 Full : 111 Redzoning : Off Used : 2004992
SlabSiz: 8192 Partial: 302 Poisoning : Off Loss : 1828864
Loss : 0 CpuSlab: 55 Tracking : Off Lalig: 0
Align : 8 Objects: 32 Tracing : Off Lpadd: 0
***Patched***
Sizes (bytes) Slabs Debug Memory
------------------------------------------------------------------------
Object : 256 Total : 300 Sanity Checks : Off Total: 2457600
SlabObj: 256 Full : 204 Redzoning : Off Used : 2348800
SlabSiz: 8192 Partial: 33 Poisoning : Off Loss : 108800
Loss : 0 CpuSlab: 63 Tracking : Off Lalig: 0
Align : 8 Objects: 32 Tracing : Off Lpadd: 0
Total and loss number is the impact of this patch.
Cc: <stable@vger.kernel.org>
Acked-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Joonsoo Kim <js1304@gmail.com>
Signed-off-by: Pekka Enberg <penberg@kernel.org>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions