Re: [Intel-gfx] [PATCH 1/9] [v2] drm/i915/bdw: Free PPGTT struct

2014-02-24 Thread Imre Deak
On Thu, 2014-02-20 at 11:47 -0800, Ben Widawsky wrote: > GEN8 never freed the PPGTT struct. As GEN8 doesn't use full PPGTT, the > leak is small and only found on a module reload. ie. I don't think this > needs to go to stable. > > v2: The very naive, kfree in gen8 ppgtt cleanup, is subject to a do

[Intel-gfx] [PATCH 1/9] [v2] drm/i915/bdw: Free PPGTT struct

2014-02-20 Thread Ben Widawsky
GEN8 never freed the PPGTT struct. As GEN8 doesn't use full PPGTT, the leak is small and only found on a module reload. ie. I don't think this needs to go to stable. v2: The very naive, kfree in gen8 ppgtt cleanup, is subject to a double free on PPGTT initialization failure. (Spotted by Imre). Ins