[Intel-gfx] [PATCH] drm/i915: Unref context on failed eb_create
Chris Wilson
chris at chris-wilson.co.uk
Sat Apr 5 08:24:01 CEST 2014
On Fri, Apr 04, 2014 at 10:41:07PM -0700, Ben Widawsky wrote:
> I opted to do this instead of grabbing the context reference after
> eb_create since eb_create can potentially call the shrinker, and that
> makes things very complicated. This simple patch balances the ref count
> without requiring a great deal of review to make sure the shrinker path
> is safe.
>
> Theoretically (by design) the shrinker can end up destroying a context,
> which enforces the reasoning for doing the fix this way instead of
> moving the reference to later in the function.
>
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
Even the whitespace fix is correct.
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list