lkml.org 
[lkml]   [2013]   [Dec]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] mm/slub: fix accumulate per cpu partial cache objects
From
On Sat, Dec 28, 2013 at 3:50 AM, Li Zefan <lizefan@huawei.com> wrote:
> On 2013/12/27 17:46, Wanpeng Li wrote:
>> SLUB per cpu partial cache is a list of slab caches to accelerate objects
>> allocation. However, current codes just accumulate the objects number of
>> the first slab cache of per cpu partial cache instead of traverse the whole
>> list.
>>
>> Signed-off-by: Wanpeng Li <liwanp@linux.vnet.ibm.com>
>> ---
>> mm/slub.c | 32 +++++++++++++++++++++++---------
>> 1 files changed, 23 insertions(+), 9 deletions(-)
>>
>> diff --git a/mm/slub.c b/mm/slub.c
>> index 545a170..799bfdc 100644
>> --- a/mm/slub.c
>> +++ b/mm/slub.c
>> @@ -4280,7 +4280,7 @@ static ssize_t show_slab_objects(struct kmem_cache *s,
>> struct kmem_cache_cpu *c = per_cpu_ptr(s->cpu_slab,
>> cpu);
>> int node;
>> - struct page *page;
>> + struct page *page, *p;
>>
>> page = ACCESS_ONCE(c->page);
>> if (!page)
>> @@ -4298,8 +4298,9 @@ static ssize_t show_slab_objects(struct kmem_cache *s,
>> nodes[node] += x;
>>
>> page = ACCESS_ONCE(c->partial);
>> - if (page) {
>> - x = page->pobjects;
>> + while ((p = page)) {
>> + page = p->next;
>> + x = p->pobjects;
>> total += x;
>> nodes[node] += x;
>> }
>
> Can we apply this patch first? It was sent month ago, but Pekka was not responsive.

Applied. Wanpeng, care to resend your patch?


\
 
 \ /
  Last update: 2013-12-29 13:21    [W:0.046 / U:0.428 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site