Re: [Intel-gfx] [PATCH 02/13] drm/i915: Copy user requested buffers into the error state

2017-04-15 Thread Chris Wilson
On Fri, Apr 14, 2017 at 09:49:34PM -0700, Matt Turner wrote: > On Wed, Apr 12, 2017 at 2:43 PM, Chris Wilson > wrote: > > On Wed, Mar 29, 2017 at 04:56:24PM +0100, Chris Wilson wrote: > >> Introduce a new execobject.flag (EXEC_OBJECT_CAPTURE) that userspace may > >> use to indicate that it wants

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Copy user requested buffers into the error state (rev2)

2017-04-15 Thread Patchwork
== Series Details == Series: drm/i915: Copy user requested buffers into the error state (rev2) URL : https://patchwork.freedesktop.org/series/16311/ State : failure == Summary == Series 16311v2 drm/i915: Copy user requested buffers into the error state https://patchwork.freedesktop.org/api/1.0

[Intel-gfx] [CI] drm/i915: Copy user requested buffers into the error state

2017-04-15 Thread Chris Wilson
Introduce a new execobject.flag (EXEC_OBJECT_CAPTURE) that userspace may use to indicate that it wants the contents of this buffer preserved in the error state (/sys/class/drm/cardN/error) following a GPU hang involving this batch. Use this at your discretion, the contents of the error state. alth

Re: [Intel-gfx] [PATCH v2] drm/i915: set "ret" correctly on error paths

2017-04-15 Thread Chris Wilson
On Fri, Apr 14, 2017 at 10:54:25PM +0300, Dan Carpenter wrote: > If "crtc" is NULL, then my static checker complains that "ret" isn't > initialized on that path. It doesn't really cause a problem unless > "ret" is somehow set to -EDEADLK which is not likely. > > Chris Wilson also noticed another