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

2017-04-14 Thread Matt Turner
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 the contents of this buffer preserved in >> the error state (/sys/class/

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: uninitialized value on error path (rev3)

2017-04-14 Thread Patchwork
== Series Details == Series: drm/i915: uninitialized value on error path (rev3) URL : https://patchwork.freedesktop.org/series/23038/ State : success == Summary == Series 23038v3 drm/i915: uninitialized value on error path https://patchwork.freedesktop.org/api/1.0/series/23038/revisions/3/mbox

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

2017-04-14 Thread Dan Carpenter
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 error path where "ret" isn't set correctly. Signed-off-by: Dan Carpenter

Re: [Intel-gfx] [PATCH] aubdump: Don't bail if a GEM handle of 0 is passed into execbuf

2017-04-14 Thread Jason Ekstrand
On Fri, Apr 14, 2017 at 11:34 AM, Rafael Antognolli < rafael.antogno...@intel.com> wrote: > Patch is > Reviewed-by: Rafael Antognolli > Thanks! Pushed. > On Fri, Mar 24, 2017 at 04:45:01PM -0700, Jason Ekstrand wrote: > > A gem handle of 0 can be used to check for whether or not 48-bit > > ad

Re: [Intel-gfx] [PATCH] aubdump: Don't bail if a GEM handle of 0 is passed into execbuf

2017-04-14 Thread Rafael Antognolli
Patch is Reviewed-by: Rafael Antognolli On Fri, Mar 24, 2017 at 04:45:01PM -0700, Jason Ekstrand wrote: > A gem handle of 0 can be used to check for whether or not 48-bit > addressing is available. This keeps aubdump from failing on you if > you try to do the check. > --- > tools/aubdump.c | 5

Re: [Intel-gfx] [PATCH v5 17/18] drm/i915: Watchdog timeout: DRM kernel interface to set the timeout

2017-04-14 Thread Michel Thierry
On 14/04/17 09:05, Daniele Ceraolo Spurio wrote: On 24/03/17 18:30, Michel Thierry wrote: Final enablement patch for GPU hang detection using watchdog timeout. Using the gem_context_setparam ioctl, users can specify the desired timeout value in microseconds, and the driver will do the conver

[Intel-gfx] [maintainer-tools PATCH v3 2/2] dim: Curate and insert tags into patch(es)

2017-04-14 Thread Sean Paul
Launch $EDITOR when extracting tags to curate the tags immediately. Once the tags are proper, automatically add them before the committer's Signed-off-by line to all patches in the range. Changes in v2: - Append the tags before the committer's SoB (Ville) - Make launching $EDITOR contingent on -i

Re: [Intel-gfx] [PATCH 02/15] drm: Remove drm_modeset_(un)lock_crtc

2017-04-14 Thread Mike Lothian
Hi X no longer starts for me and I've bisected it back to this commit During bisect I ended up on commits where my laptop would lockup during boot Regards Mike On Tue, 4 Apr 2017 at 06:39 Daniel Vetter wrote: > On Tue, Apr 4, 2017 at 12:13 AM, kbuild test robot wrote: > > [if your patch is

Re: [Intel-gfx] [PATCH v5 17/18] drm/i915: Watchdog timeout: DRM kernel interface to set the timeout

2017-04-14 Thread Chris Wilson
On Fri, Apr 14, 2017 at 09:05:06AM -0700, Daniele Ceraolo Spurio wrote: > On 24/03/17 18:30, Michel Thierry wrote: > >+/* > >+ * Based on time out value in microseconds (us) calculate > >+ * timer count thresholds needed based on core frequency. > >+ * Watchdog can be disabled by setting it to 0. >

Re: [Intel-gfx] [PATCH v5 17/18] drm/i915: Watchdog timeout: DRM kernel interface to set the timeout

2017-04-14 Thread Daniele Ceraolo Spurio
On 24/03/17 18:30, Michel Thierry wrote: Final enablement patch for GPU hang detection using watchdog timeout. Using the gem_context_setparam ioctl, users can specify the desired timeout value in microseconds, and the driver will do the conversion to 'timestamps'. The recommended default watch

Re: [Intel-gfx] [maintainer-tools PATCH] dim: Expand drm-misc branch explanations

2017-04-14 Thread Sean Paul
On Mon, Apr 10, 2017 at 12:05:43PM -0400, Sean Paul wrote: > Add a bit more colour to the -misc branch explanations, and add a merge > timeline > similar to the chart used in drm-intel. > > Signed-off-by: Sean Paul > --- Friendly review ping. Sean > > > I've compiled the with this patch and

Re: [Intel-gfx] [PATCH 04/22] target: Make use of the new sg_map function at 16 call sites (fwd)

2017-04-14 Thread Julia Lawall
system] url: https://github.com/0day-ci/linux/commits/Logan-Gunthorpe/Introduce-common-scatterlist-map-function/20170414-142518 base: https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next :: branch date: 8 hours ago :: commit date: 8 hours ago >> d

Re: [Intel-gfx] [PATCH] drm/i915/gvt: return the actual aperture size under gvt environment

2017-04-14 Thread Li, Weinan Z
> -Original Message- > From: Joonas Lahtinen [mailto:joonas.lahti...@linux.intel.com] > Sent: Thursday, April 13, 2017 6:11 PM > To: Li, Weinan Z ; Chris Wilson wilson.co.uk> > Cc: intel-gfx@lists.freedesktop.org; intel-gvt-...@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i