[Intel-gfx] [PATCH 09/11] drm/i915: fixup execbuf failure path
Chris Wilson
chris at chris-wilson.co.uk
Fri Jan 15 14:08:59 CET 2010
On Fri, 15 Jan 2010 13:24:16 +0100, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> Objects were unconditionally unreferenced in the error path.
> Fix this up like with the object pinning.
>
[snip]
> - for (i = 0; i < args->buffer_count; i++) {
> + for (i = 0; i < referenced; i++) {
> if (object_list[i]) {
> obj_priv = object_list[i]->driver_private;
> obj_priv->in_execbuffer = false;
If this is truly an issue, then why not take advantage of the
if(object_list[i]) ? The simplest fix here would be to just
if (!object_list) break;
instead.
-ickle
--
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx
mailing list