summaryrefslogtreecommitdiffstats
path: root/drivers/cpuidle
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2014-02-10 18:00:39 +0100
committerDaniel Vetter <daniel.vetter@ffwll.ch>2014-02-12 18:53:02 +0100
commita8bb6818270c32126dba0fd2ddb139d885c5687d (patch)
tree56f0b21a168e442a3000cb517a93b84dba7ceacf /drivers/cpuidle
parentef2d633e9bcfdb73e536ca81b835dd015fe24ceb (diff)
downloadblackbird-op-linux-a8bb6818270c32126dba0fd2ddb139d885c5687d.tar.gz
blackbird-op-linux-a8bb6818270c32126dba0fd2ddb139d885c5687d.zip
drm/i915: Fix error path leak in fbdev fb allocation
In Jesse's patch to switch the fbdev framebuffer from an embedded struct to a pointer the kfree in case of an error was missed. Fix this up by using our own internal fb allocation helper directly instead of reinventing that wheel. We need a to_intel_framebuffer cast unfortunately since all the other callers of _create still look better whith using a drm_framebuffer as return pointer. v2: Add an unlocked __intel_framebuffer_create function since our dev->struct_mutex locking is too much a mess. With ppgtt we even need it to take a look at the global gtt offset of pinned objects, since the vma list might chance from underneath us. At least with the current global gtt lookup functions. Reported by Mika. Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> Cc: Jesse Barnes <jbarnes@virtuousgeek.org> Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/cpuidle')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud