Re: [PATCH] [v2] drm/i915: Fix another another use-after-free in do_switch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Aug 09, 2014 at 01:15:16PM -0700, Ben Widawsky wrote:
> See the following for many more details.
> 
> commit acc240d41ea1ab9c488a79219fb313b5b46265ae
> Author: Daniel Vetter <daniel.vetter@xxxxxxxx>
> Date:   Thu Dec 5 15:42:34 2013 +0100
> 
>     drm/i915: Fix use-after-free in do_switch
> 
> In this case, the issue is only for full PPGTT:
> do_switch
>   context_unref
>     ppgtt_release
>       i915_gpu_idle
> 	switch_to_default
> 	from changes to default context
> 
> This could be backported to the pre do_switch cleanup I did in this
> series. However, it's much cleaner and more obvious as a patch on top,
> so I'd really like to do this as a post cleanup patch.
> 
> v2: There was a bug in the original patch where the ring->last_context
> was set too early. I am not sure how this wasn't being hit when I sent
> this previously. Perhaps I tested the wrong patch previously.
> 
> Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx>

Ok, I convinced myself that the you are fixing the bug you describe and
don't seem to be introducing a new one, so

Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris

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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux