On Thu, Sep 03, 2015 at 07:22:18PM +0200, Michał Winiarski wrote:
> +     pts = kcalloc(pdpes * BITS_TO_LONGS(I915_PDES),
> +                   sizeof(unsigned long), GFP_TEMPORARY);

Something to remember is that kcalloc is written presuming that the size
argument (the second) is constant.

pts = kcalloc(pdpes,
              BITS_TO_LONGS(I915_PDES) * sizeof(unsigned long),
              GFP_TEMPORARY);

should be infinitesimally more efficient.

Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to