| Date | Wed, 16 Oct 2013 15:27:54 +0000 | From | Christoph Lameter <> | Subject | Re: [PATCH v2 01/15] slab: correct pfmemalloc check |
| |
On Wed, 16 Oct 2013, Joonsoo Kim wrote:
> --- a/mm/slab.c > +++ b/mm/slab.c > @@ -930,7 +930,8 @@ static void *__ac_put_obj(struct kmem_cache *cachep, struct array_cache *ac, > { > if (unlikely(pfmemalloc_active)) { > /* Some pfmemalloc slabs exist, check if this is one */ > - struct page *page = virt_to_head_page(objp); > + struct slab *slabp = virt_to_slab(objp); > + struct page *page = virt_to_head_page(slabp->s_mem); > if (PageSlabPfmemalloc(page))
I hope the compiler optimizes this code correctly because virt_to_slab already does one virt_to_head_page()?
Otherwise this looks fine.
|