[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Wed, 24 Feb 2016 10:09:16 -0800
From: Laura Abbott <labbott@...hat.com>
To: Paolo Bonzini <pbonzini@...hat.com>,
Laura Abbott <labbott@...oraproject.org>,
Christoph Lameter <cl@...ux.com>, Pekka Enberg <penberg@...nel.org>,
David Rientjes <rientjes@...gle.com>, Joonsoo Kim <js1304@...il.com>,
Andrew Morton <akpm@...ux-foundation.org>
Cc: linux-mm@...ck.org, linux-kernel@...r.kernel.org,
kernel-hardening@...ts.openwall.com, Kees Cook <keescook@...omium.org>
Subject: Re: [PATCHv2 1/4] slub: Drop lock at the end of free_debug_processing
On 02/24/2016 06:22 AM, Paolo Bonzini wrote:
>
>
> On 15/02/2016 19:44, Laura Abbott wrote:
>> -static inline struct kmem_cache_node *free_debug_processing(
>> +static inline int free_debug_processing(
>> struct kmem_cache *s, struct page *page,
>> void *head, void *tail, int bulk_cnt,
>> unsigned long addr, unsigned long *flags) { return NULL; }
>
> I think this has a leftover flags argument.
>
> Paolo
>
Yes, I believe Andrew folded in a patch to the mm tree.
Thanks,
Laura
>> @@ -2648,8 +2646,7 @@ static void __slab_free(struct kmem_cache *s, struct page *page,
>> stat(s, FREE_SLOWPATH);
>>
>> if (kmem_cache_debug(s) &&
>> - !(n = free_debug_processing(s, page, head, tail, cnt,
>> - addr, &flags)))
>> + !free_debug_processing(s, page, head, tail, cnt, addr))
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
