Re: [Intel-gfx] [ANNOUNCE] 2015-Q4 release of KVMGT (Was Re: KVMGT - the implementation of ...)

2016-01-26 Thread Jike Song
Hi all, We are pleased to announce another update of Intel GVT-g for KVM. Intel GVT-g for KVM (a.k.a. KVMGT) is a full GPU virtualization solution with mediated pass-through, starting from 4th generation Intel Core(TM) processors with Intel Graphics processors. A virtual GPU instance is mainta

Re: [Intel-gfx] [Announcement] 2015-Q4 release of XenGT - a Mediated Graphics Passthrough Solution from Intel

2016-01-26 Thread Jike Song
Hi all, We are pleased to announce another update of Intel GVT-g for Xen. Intel GVT-g is a full GPU virtualization solution with mediated pass-through, starting from 4th generation Intel Core(TM) processors with Intel Graphics processors. A virtual GPU instance is maintained for each VM, with p

Re: [Intel-gfx] [PATCH v2] drm/i915: Decouple execbuf uAPI from internal implementation

2016-01-26 Thread Gong, Zhipeng
> On Fri, Jan 15, 2016 at 03:12:50PM +, Tvrtko Ursulin wrote: > > From: Tvrtko Ursulin > > + if (user_ring_id == I915_EXEC_BSD && HAS_BSD2(dev_priv)) { > > + unsigned int bsd_idx = args->flags & I915_EXEC_BSD_MASK; > > + > > + if (bsd_idx == I915_EXEC_BSD_DEFAULT) { > > +

Re: [Intel-gfx] [PATCH igt] igt/gem_softpin: Remove false dependencies on esoteric features

2016-01-26 Thread Belgaumkar, Vinay
Hi Dan, I was not dismissing Chris's patch, in fact, I was trying to see if we could keep the combined userptr/softpin tests in a separate file. Apologize if it came across as dismissive. I agree that it is better to keep the tests simple and effective, however, we thought it would be worthwhi

Re: [Intel-gfx] [RFC 00/22] Add support for GuC-based SLPC

2016-01-26 Thread O'Rourke, Tom
On Tue, Jan 26, 2016 at 09:17:51AM -0800, Jesse Barnes wrote: > On 01/26/2016 09:00 AM, Daniel Vetter wrote: > > On Tue, Jan 26, 2016 at 07:45:42AM -0800, Jesse Barnes wrote: > >> On 01/22/2016 09:00 AM, Daniel Vetter wrote: > >>> On Wed, Jan 20, 2016 at 06:26:02PM -0800, tom.orou...@intel.com wrot

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: edp resume/On time optimization. (rev10)

2016-01-26 Thread Kumar, Abhay
On 1/26/2016 3:36 AM, Ville Syrjälä wrote: On Sat, Jan 23, 2016 at 08:43:33AM -, Patchwork wrote: == Summary == Built on 8fe9e785ae04fa7c37f7935cff12d62e38054b60 drm-intel-nightly: 2016y-01m-21d-11h-02m-42s UTC integration manifest Test gem_ctx_basic: pass -> FAIL

Re: [Intel-gfx] [RFC 0/3] drm/i915: expose fifo pipe underrun counts

2016-01-26 Thread Joe Konno
On 01/26/2016 12:51 PM, Daniel Vetter wrote: > On Tue, Jan 26, 2016 at 11:36:54AM -0800, Joe Konno wrote: >> From: Joe Konno >> >> In tracking down a watermark bug, I discovered the pch and cpu underrun >> interrupt handlers would disable themselves after initial reports to prevent >> an >> inter

Re: [Intel-gfx] [PATCH 1/3] agp/intel-gtt: Don't leak the scratch page

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 07:49:54PM +0100, Daniel Vetter wrote: > Recently discovered by enabling CONFIG_DMA_API_DEBUG in our CI. By the > looks of it broken since forever. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93793 > Signed-off-by: Daniel Vetter > --- > drivers/char/agp/int

Re: [Intel-gfx] [PATCH 3/3] agp/intel-gtt: Only register fake agp driver for gen1

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 07:49:56PM +0100, Daniel Vetter wrote: > The fake agp driver for the intel graphics gart is only needed for ums > support. And we ditched that a long time ago: > > commit 03dae59c72d8ef6e005f48ba356c863e0587 > Author: Daniel Vetter > Date: Wed Jul 23 16:27:25 2014 +0

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Stop depending upon CONFIG_AGP/_INTEL

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 07:49:55PM +0100, Daniel Vetter wrote: > We need this to be able to paper over some CI fail: DMA API debuggin > complaints that we leak the gmch scratch page, but fundamentally > that's the only way to do it if there's both the intel-agp and i915 > driver using it: intel-agp

Re: [Intel-gfx] [RFC 0/3] drm/i915: expose fifo pipe underrun counts

2016-01-26 Thread Daniel Vetter
On Tue, Jan 26, 2016 at 11:36:54AM -0800, Joe Konno wrote: > From: Joe Konno > > In tracking down a watermark bug, I discovered the pch and cpu underrun > interrupt handlers would disable themselves after initial reports to prevent > an > interrupt/dmesg storm. Storms are bad, but underrun inter

Re: [Intel-gfx] [RFC 2/3] drm/i915: do not disable handler after underrun

2016-01-26 Thread Daniel Vetter
On Tue, Jan 26, 2016 at 11:36:56AM -0800, Joe Konno wrote: > From: Joe Konno > > Previously, cpu and pch underrun reporting was done on a "report once > and disable" basis to prevent interrupt/dmesg floods. > > With this change, do not disable the underrun interrupt handlers. > Instead, use thro

Re: [Intel-gfx] [PATCH v2] MAINTAINERS: Add "B:" preferred bug reporting method

2016-01-26 Thread Joe Perches
On Tue, 2016-01-26 at 15:33 +, Dave Gordon wrote: > On 26/01/16 14:42, Joe Perches wrote: > > On Tue, 2016-01-26 at 11:10 +0200, Jani Nikula wrote: > > > Different subsystems and drivers have different preferred ways of > > > receiving bug reports; mailing list or bugzillas at various > > > loc

[Intel-gfx] [RFC 3/3] drm/i915: add underrun counts to i915_display_info

2016-01-26 Thread Joe Konno
From: Joe Konno Add atomic cpu and pch underrun counters to struct intel_crtc, and increment them each time their respective interrupt handler executes. Whether dmesg underrun reporting is disabled or not, keep a count of cpu and pch underrun interrupts. Display these counts in i915_display_info

[Intel-gfx] [RFC 0/3] drm/i915: expose fifo pipe underrun counts

2016-01-26 Thread Joe Konno
From: Joe Konno In tracking down a watermark bug, I discovered the pch and cpu underrun interrupt handlers would disable themselves after initial reports to prevent an interrupt/dmesg storm. Storms are bad, but underrun interrupt handling should not cease. For my case, I need to be able to count

[Intel-gfx] [RFC 2/3] drm/i915: do not disable handler after underrun

2016-01-26 Thread Joe Konno
From: Joe Konno Previously, cpu and pch underrun reporting was done on a "report once and disable" basis to prevent interrupt/dmesg floods. With this change, do not disable the underrun interrupt handlers. Instead, use throttled (DRM_ERROR_RATELIMITED) dmesg output if dmesg underrun reporting en

[Intel-gfx] [RFC 1/3] drm/i915: get cpu, pch underrun reporting state

2016-01-26 Thread Joe Konno
From: Joe Konno There are mechanisms for "set and return previous" underrun reporting state, but no convenience functions for simply getting the underrun reporting state for a particular pipe or pch transcoder. Signed-off-by: Joe Konno --- drivers/gpu/drm/i915/intel_fifo_underrun.c | 43 ++

[Intel-gfx] [PATCH 3/3] agp/intel-gtt: Only register fake agp driver for gen1

2016-01-26 Thread Daniel Vetter
The fake agp driver for the intel graphics gart is only needed for ums support. And we ditched that a long time ago: commit 03dae59c72d8ef6e005f48ba356c863e0587 Author: Daniel Vetter Date: Wed Jul 23 16:27:25 2014 +0200 drm/i915: Ditch UMS config option With this there's no longer the

[Intel-gfx] [PATCH 2/3] drm/i915: Stop depending upon CONFIG_AGP/_INTEL

2016-01-26 Thread Daniel Vetter
We need this to be able to paper over some CI fail: DMA API debuggin complaints that we leak the gmch scratch page, but fundamentally that's the only way to do it if there's both the intel-agp and i915 driver using it: intel-agp shadow-attaches, i915 binds for real to the gpu. But when i915 unloads

[Intel-gfx] [PATCH 1/3] agp/intel-gtt: Don't leak the scratch page

2016-01-26 Thread Daniel Vetter
Recently discovered by enabling CONFIG_DMA_API_DEBUG in our CI. By the looks of it broken since forever. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93793 Signed-off-by: Daniel Vetter --- drivers/char/agp/intel-gtt.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/

Re: [Intel-gfx] [PATCH 01/25] drm/i915/fbc: wait for a vblank instead of 50ms when enabling

2016-01-26 Thread Zanoni, Paulo R
Em Ter, 2016-01-26 às 17:44 +, Rodrigo Vivi escreveu: > > > On Thu, Jan 21, 2016 at 12:03 PM Paulo Zanoni om> wrote: > > Instead of waiting for 50ms, just wait until the next vblank, since > > it's the minimum requirement. The whole infrastructure of FBC is > > based > > on vblanks, so waiti

Re: [Intel-gfx] [ANNOUNCE] xf86-video-intel 2.99.917

2016-01-26 Thread Andreas Radke
Am Wed, 6 May 2015 22:02:57 +0200 schrieb Julien Cristau : > On Sun, Dec 21, 2014 at 14:47:58 +, Chris Wilson wrote: > > > Snapshot 2.99.917 (2014-12-21) > > == > > 3 months drifted by whilst I looked elsewhere for bugs.. The > > highlight of bugs fixed here are a

[Intel-gfx] [PATCH] drm/i915: Capture revision id in error state

2016-01-26 Thread Arun Siluvery
Revision id along with device id is useful in better identification of the HW and its limitations so include this detail in error state. Signed-off-by: Arun Siluvery --- drivers/gpu/drm/i915/i915_gpu_error.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/i915_gpu_error.

Re: [Intel-gfx] [PATCH] tests/Android.mk: Make intel_residency CAIRO dependant

2016-01-26 Thread Zanoni, Paulo R
Em Ter, 2016-01-26 às 13:04 +, Derek Morton escreveu: > intel_residency has a cairo dependency through igt_fb.c. Remove it > if ANDROID_HAS_CAIRO is not defined. The patch looks good, so I pushed it so we can unbreak your build. Thanks, Paulo > > Signed-off-by: Derek Morton > --- >  tools/

Re: [Intel-gfx] [PATCH 01/25] drm/i915/fbc: wait for a vblank instead of 50ms when enabling

2016-01-26 Thread Rodrigo Vivi
On Thu, Jan 21, 2016 at 12:03 PM Paulo Zanoni wrote: > Instead of waiting for 50ms, just wait until the next vblank, since > it's the minimum requirement. The whole infrastructure of FBC is based > on vblanks, so waiting for X vblanks instead of X milliseconds sounds > like the correct way to go.

[Intel-gfx] [PATCH i-g-t] igt/gem_ringfill: Allow listing subtests in gem_ringfill

2016-01-26 Thread Gabriel Feceoru
Moved gem_quiescent_gpu() call to the run path. Signed-off-by: Gabriel Feceoru --- tests/gem_ringfill.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/gem_ringfill.c b/tests/gem_ringfill.c index 407b747..2e8c7c1 100644 --- a/tests/gem_ringfill.c +++ b/tests/gem_ringf

Re: [Intel-gfx] [RFC 00/22] Add support for GuC-based SLPC

2016-01-26 Thread Jesse Barnes
On 01/26/2016 09:00 AM, Daniel Vetter wrote: > On Tue, Jan 26, 2016 at 07:45:42AM -0800, Jesse Barnes wrote: >> On 01/22/2016 09:00 AM, Daniel Vetter wrote: >>> On Wed, Jan 20, 2016 at 06:26:02PM -0800, tom.orou...@intel.com wrote: From: Tom O'Rourke SLPC (Single Loop Power Controll

Re: [Intel-gfx] [PATCH v4] drm/i915/guc: Clean up locks in GuC

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 05:54:21PM +0100, Daniel Vetter wrote: > On Tue, Jan 26, 2016 at 12:08:37PM +, Chris Wilson wrote: > > On Tue, Jan 26, 2016 at 11:53:45AM +, Tvrtko Ursulin wrote: > > > Chris Harris has liaised with Tomi and apparently GuC firmware is > > > already on the lab SKLs fr

Re: [Intel-gfx] [PATCH] drm/i915/gen9: Probe power well 1 status on dc status query

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 05:45:31PM +0200, Mika Kuoppala wrote: > There has been cases where we read DC_STATE and get something that we > did not write there. As DMC owns power well 1, this could be that DMC > snoops DC_STATE accesses and needs to wake up power well 1 up to serve > the access. But t

Re: [Intel-gfx] [RFC 00/22] Add support for GuC-based SLPC

2016-01-26 Thread Daniel Vetter
On Tue, Jan 26, 2016 at 07:45:42AM -0800, Jesse Barnes wrote: > On 01/22/2016 09:00 AM, Daniel Vetter wrote: > > On Wed, Jan 20, 2016 at 06:26:02PM -0800, tom.orou...@intel.com wrote: > >> From: Tom O'Rourke > >> > >> SLPC (Single Loop Power Controller) is a replacement for > >> some host-based po

Re: [Intel-gfx] [RFC 3/5] drm/i915: Add support for CPU mapping to DRM_IOCTL_I915_GEM_MMAP_GTT

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 04:23:28PM +, Tvrtko Ursulin wrote: > > On 26/01/16 15:10, Chris Wilson wrote: > >On Tue, Jan 26, 2016 at 02:53:31PM +, Tvrtko Ursulin wrote: > >>From: Tvrtko Ursulin > >> > >>Signed-off-by: Tvrtko Ursulin > >>--- > >> drivers/gpu/drm/i915/i915_gem.c | 96 > >> +

Re: [Intel-gfx] [PATCH v4] drm/i915/guc: Clean up locks in GuC

2016-01-26 Thread Daniel Vetter
On Tue, Jan 26, 2016 at 12:08:37PM +, Chris Wilson wrote: > On Tue, Jan 26, 2016 at 11:53:45AM +, Tvrtko Ursulin wrote: > > Chris Harris has liaised with Tomi and apparently GuC firmware is > > already on the lab SKLs from ~two weeks ago. > > > > Since the firmware is on all machines, pres

Re: [Intel-gfx] [RFC 2/5] drm/i915: Extract code mapping errno to vm fault code

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 04:24:46PM +, Tvrtko Ursulin wrote: > > > On 26/01/16 15:18, Chris Wilson wrote: > >On Tue, Jan 26, 2016 at 02:53:30PM +, Tvrtko Ursulin wrote: > >>From: Tvrtko Ursulin > >> > >>Will be used from multiple callers in a following patch. > >> > >>Signed-off-by: Tvrtk

Re: [Intel-gfx] regression bisected: display sync drops after 7221fc33

2016-01-26 Thread dann frazier
On Tue, Jan 26, 2016 at 08:10:04AM -0700, dann frazier wrote: > On Tue, Jan 26, 2016 at 01:17:01PM +0200, Ville Syrjälä wrote: > > On Mon, Jan 25, 2016 at 09:35:26AM -0700, dann frazier wrote: > > > I've got a Lenovo T410 w/ an external display (DisplayPort<->HDMI). After > > > upgrading from a 4.2

Re: [Intel-gfx] [REGRESSION] component: add support for releasing match data

2016-01-26 Thread Daniel Vetter
On Tue, Jan 26, 2016 at 05:21:06PM +0100, Maarten Lankhorst wrote: > Op 26-01-16 om 15:48 schreef Russell King - ARM Linux: > > On Tue, Jan 26, 2016 at 03:28:34PM +0100, Maarten Lankhorst wrote: > >> Something similar to a segfault. It's trying to call 0x6b6b6b6b6b which > >> is POISON_FREE. > >> >

Re: [Intel-gfx] [RFC 3/5] drm/i915: Add support for CPU mapping to DRM_IOCTL_I915_GEM_MMAP_GTT

2016-01-26 Thread Tvrtko Ursulin
On 26/01/16 15:10, Chris Wilson wrote: On Tue, Jan 26, 2016 at 02:53:31PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gem.c | 96 ++--- include/uapi/drm/i915_drm.h | 3 ++ 2 files c

Re: [Intel-gfx] [RFC 2/5] drm/i915: Extract code mapping errno to vm fault code

2016-01-26 Thread Tvrtko Ursulin
On 26/01/16 15:18, Chris Wilson wrote: On Tue, Jan 26, 2016 at 02:53:30PM +, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Will be used from multiple callers in a following patch. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gem.c | 91 ++--

Re: [Intel-gfx] [REGRESSION] component: add support for releasing match data

2016-01-26 Thread Maarten Lankhorst
Op 26-01-16 om 15:48 schreef Russell King - ARM Linux: > On Tue, Jan 26, 2016 at 03:28:34PM +0100, Maarten Lankhorst wrote: >> Something similar to a segfault. It's trying to call 0x6b6b6b6b6b which >> is POISON_FREE. >> >> mc appears to be freed already, so calling mc->release would jump to >> inv

Re: [Intel-gfx] [maintainer-tools RFC PATCH 12/17] dim: separate handling of aliases

2016-01-26 Thread Daniel Vetter
On Tue, Jan 26, 2016 at 04:32:25PM +0200, Jani Nikula wrote: > --- > dim | 50 +++--- > 1 file changed, 35 insertions(+), 15 deletions(-) > > diff --git a/dim b/dim > index 34f2312be37a..b336c2c7d939 100755 > --- a/dim > +++ b/dim > @@ -727,7 +727,27 @@

Re: [Intel-gfx] [RFC 00/22] Add support for GuC-based SLPC

2016-01-26 Thread Jesse Barnes
On 01/22/2016 09:00 AM, Daniel Vetter wrote: > On Wed, Jan 20, 2016 at 06:26:02PM -0800, tom.orou...@intel.com wrote: >> From: Tom O'Rourke >> >> SLPC (Single Loop Power Controller) is a replacement for >> some host-based power management features. The SLPC >> implemenation runs in firmware on Gu

[Intel-gfx] [PATCH] drm/i915/gen9: Probe power well 1 status on dc status query

2016-01-26 Thread Mika Kuoppala
There has been cases where we read DC_STATE and get something that we did not write there. As DMC owns power well 1, this could be that DMC snoops DC_STATE accesses and needs to wake up power well 1 up to serve the access. But the waking up power well 1 takes time and we might end up reading during

Re: [Intel-gfx] [PATCH v2] MAINTAINERS: Add "B:" preferred bug reporting method

2016-01-26 Thread Dave Gordon
On 26/01/16 14:42, Joe Perches wrote: On Tue, 2016-01-26 at 11:10 +0200, Jani Nikula wrote: Different subsystems and drivers have different preferred ways of receiving bug reports; mailing list or bugzillas at various locations. Add "B:" entry for specifying the preference to guide bug reporters

Re: [Intel-gfx] [RFC 2/5] drm/i915: Extract code mapping errno to vm fault code

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 02:53:30PM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Will be used from multiple callers in a following patch. > > Signed-off-by: Tvrtko Ursulin > --- > drivers/gpu/drm/i915/i915_gem.c | 91 > ++--- > 1 file changed, 49

Re: [Intel-gfx] regression bisected: display sync drops after 7221fc33

2016-01-26 Thread dann frazier
On Tue, Jan 26, 2016 at 01:17:01PM +0200, Ville Syrjälä wrote: > On Mon, Jan 25, 2016 at 09:35:26AM -0700, dann frazier wrote: > > I've got a Lenovo T410 w/ an external display (DisplayPort<->HDMI). After > > upgrading from a 4.2 kernel to a 4.3, I started hitting an issue where > > the display thi

Re: [Intel-gfx] [RFC 3/5] drm/i915: Add support for CPU mapping to DRM_IOCTL_I915_GEM_MMAP_GTT

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 02:53:31PM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Signed-off-by: Tvrtko Ursulin > --- > drivers/gpu/drm/i915/i915_gem.c | 96 > ++--- > include/uapi/drm/i915_drm.h | 3 ++ > 2 files changed, 93 insertions(+), 6 d

Re: [Intel-gfx] [RFC 4/5] drm/i915: Add support for write-combined CPU mapping to DRM_IOCTL_I915_GEM_MMAP_GTT

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 02:53:32PM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin Speak of the devil. -Chris -- Chris Wilson, Intel Open Source Technology Centre ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.

Re: [Intel-gfx] [PATCH v2] MAINTAINERS: Add "B:" preferred bug reporting method

2016-01-26 Thread Jani Nikula
On Tue, 26 Jan 2016, Richard Weinberger wrote: > On Tue, Jan 26, 2016 at 3:42 PM, Joe Perches wrote: >> On Tue, 2016-01-26 at 11:10 +0200, Jani Nikula wrote: >>> Different subsystems and drivers have different preferred ways of >>> receiving bug reports; mailing list or bugzillas at various >>> l

[Intel-gfx] [RFC 3/5] drm/i915: Add support for CPU mapping to DRM_IOCTL_I915_GEM_MMAP_GTT

2016-01-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gem.c | 96 ++--- include/uapi/drm/i915_drm.h | 3 ++ 2 files changed, 93 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/

[Intel-gfx] [RFC 1/5] drm: Allow drivers setting vm_ops per vma offset node

2016-01-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin This allows drivers to implement per-object mmap(2) strategies. If not set via the drm_vma_node_set_vm_ops helper, driver default vm_ops are used preserving compatibility with the existing code base. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/drm_gem.c | 53

[Intel-gfx] [RFC 0/5] Adding CPU mmap support to DRM_IOCTL_I915_GEM_MMAP_GTT

2016-01-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin I had this code laying around from an abandoned project and decided to float it in case someone can see the usefulness of it. Assuming the approach is even remotely reasonable. Currently the driver implements two ioctls to implement mmap(2) functionality. Between the i915_ge

[Intel-gfx] [PATCH libdrm] intel: Don't ignore write_domain when using softpin

2016-01-26 Thread Michał Winiarski
Since we're short-circuting reloc handling by using softpin we need to pass additional exec2_object flags, to ensure correct synchronization and handle the gen6 case, where some objects need global gtt mapping. Cc: Chris Wilson Cc: Kristian Høgsberg Kristensen Signed-off-by: Michał Winiarski --

Re: [Intel-gfx] [PATCH v2] MAINTAINERS: Add "B:" preferred bug reporting method

2016-01-26 Thread Jani Nikula
On Tue, 26 Jan 2016, Joe Perches wrote: > On Tue, 2016-01-26 at 11:10 +0200, Jani Nikula wrote: >> Different subsystems and drivers have different preferred ways of >> receiving bug reports; mailing list or bugzillas at various >> locations. Add "B:" entry for specifying the preference to guide bu

[Intel-gfx] [RFC 5/5] drm/i915: Announce the new DRM_IOCTL_I915_GEM_MMAP_GTT capabilities

2016-01-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_dma.c | 3 +++ include/uapi/drm/i915_drm.h | 1 + 2 files changed, 4 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c index 4725e8d61d0f..b22cbe5215d2 100644 --- a/d

[Intel-gfx] [RFC 2/5] drm/i915: Extract code mapping errno to vm fault code

2016-01-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Will be used from multiple callers in a following patch. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gem.c | 91 ++--- 1 file changed, 49 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/driv

[Intel-gfx] [RFC 4/5] drm/i915: Add support for write-combined CPU mapping to DRM_IOCTL_I915_GEM_MMAP_GTT

2016-01-26 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gem.c | 36 ++-- include/uapi/drm/i915_drm.h | 3 ++- 2 files changed, 32 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915

Re: [Intel-gfx] [PATCH v2] MAINTAINERS: Add "B:" preferred bug reporting method

2016-01-26 Thread Richard Weinberger
On Tue, Jan 26, 2016 at 3:42 PM, Joe Perches wrote: > On Tue, 2016-01-26 at 11:10 +0200, Jani Nikula wrote: >> Different subsystems and drivers have different preferred ways of >> receiving bug reports; mailing list or bugzillas at various >> locations. Add "B:" entry for specifying the preference

Re: [Intel-gfx] [REGRESSION] component: add support for releasing match data

2016-01-26 Thread Russell King - ARM Linux
On Tue, Jan 26, 2016 at 03:28:34PM +0100, Maarten Lankhorst wrote: > Something similar to a segfault. It's trying to call 0x6b6b6b6b6b which > is POISON_FREE. > > mc appears to be freed already, so calling mc->release would jump to > invalid data. It seems that my devm foo wasn't quite up to scra

Re: [Intel-gfx] [PATCH igt] tools/intel_residency: use setitimer instead of {, u}alarm

2016-01-26 Thread Morton, Derek J
I verified this patch fixes the ualarm issue on android. I posted the patch [PATCH] tests/Android.mk: Make intel_residency CAIRO dependant Which adds this tool to the list that are cairo dependant in the android makefile. //Derek -Original Message- From: Intel-gfx [mailto:intel-gfx-bou

Re: [Intel-gfx] [PATCH v2] MAINTAINERS: Add "B:" preferred bug reporting method

2016-01-26 Thread Joe Perches
On Tue, 2016-01-26 at 11:10 +0200, Jani Nikula wrote: > Different subsystems and drivers have different preferred ways of > receiving bug reports; mailing list or bugzillas at various > locations. Add "B:" entry for specifying the preference to guide bug > reporters at the right location. [] > v2:

Re: [Intel-gfx] [maintainer-tools RFC PATCH 00/17] dim: function based subcommand handling

2016-01-26 Thread Jani Nikula
On Tue, 26 Jan 2016, Jani Nikula wrote: > Patches 14-17 starts abstracting subcommand cases to functions. This is > mostly mechanical work, and I didn't bother with it until I get an ack > on the approach. I mean, I didn't bother with convering *all* of the functions yet. BR, Jani. -- Jani Ni

Re: [Intel-gfx] [PATCH v2] MAINTAINERS: Add "B:" preferred bug reporting method

2016-01-26 Thread Joe Perches
On Tue, 2016-01-26 at 11:10 +0200, Jani Nikula wrote: > Different subsystems and drivers have different preferred ways of > receiving bug reports; mailing list or bugzillas at various > locations. Add "B:" entry for specifying the preference to guide bug > reporters at the right location. > > Cc:

[Intel-gfx] [maintainer-tools RFC PATCH 17/17] dim: abstract dim_push_*

2016-01-26 Thread Jani Nikula
Signed-off-by: Jani Nikula --- dim | 24 +++- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/dim b/dim index 1f51ab9292b1..d5149f01db34 100755 --- a/dim +++ b/dim @@ -373,6 +373,21 @@ function dim_push_branch update_linux_next } +function dim_push_qu

[Intel-gfx] [maintainer-tools RFC PATCH 11/17] dim: move dim_pull_request parameter check to the function

2016-01-26 Thread Jani Nikula
More abstraction is better, also guards against invalid internal use. Signed-off-by: Jani Nikula --- dim | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/dim b/dim index b9152a256a26..34f2312be37a 100755 --- a/dim +++ b/dim @@ -581,6 +581,11 @@ function dim_create_wor

[Intel-gfx] [maintainer-tools RFC PATCH 16/17] dim: abstract dim_for_each_workdirs

2016-01-26 Thread Jani Nikula
Signed-off-by: Jani Nikula --- dim | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/dim b/dim index 26ab728efb16..1f51ab9292b1 100755 --- a/dim +++ b/dim @@ -587,6 +587,18 @@ function dim_create_workdir done } +function dim_for_each_workdirs

[Intel-gfx] [maintainer-tools RFC PATCH 09/17] dim: rename update_nightly to dim_rebuild_nightly

2016-01-26 Thread Jani Nikula
Rename to match the subcommand name. Signed-off-by: Jani Nikula --- dim | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dim b/dim index 3ddbb728a044..c72f58302bae 100755 --- a/dim +++ b/dim @@ -231,7 +231,7 @@ function update_rerere_cache cd - } -function upd

[Intel-gfx] [maintainer-tools RFC PATCH 14/17] dim: abstract dim_nightly_forget

2016-01-26 Thread Jani Nikula
Signed-off-by: Jani Nikula --- dim | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/dim b/dim index 1af2163a5001..b50656cf80a0 100755 --- a/dim +++ b/dim @@ -344,6 +344,15 @@ function dim_rebuild_nightly } +function dim_nightly_forget +{ + cd $DIM_PR

[Intel-gfx] [maintainer-tools RFC PATCH 15/17] dim: abstract dim_update_branches

2016-01-26 Thread Jani Nikula
Signed-off-by: Jani Nikula --- dim | 40 +--- 1 file changed, 21 insertions(+), 19 deletions(-) diff --git a/dim b/dim index b50656cf80a0..26ab728efb16 100755 --- a/dim +++ b/dim @@ -629,6 +629,27 @@ function dim_pull_request "$addr_drm_maintai

[Intel-gfx] [maintainer-tools RFC PATCH 13/17] dim: handle subcommands through functions

2016-01-26 Thread Jani Nikula
If there's a dim_ prefixed function matching the subcommand, call it to handle the subcommand without going through the big switch case. Drop the cases which already match the convention. Once all the cases are converted to functions, the handling of subcommands becomes much cleaner. This also al

[Intel-gfx] [maintainer-tools RFC PATCH 12/17] dim: separate handling of aliases

2016-01-26 Thread Jani Nikula
--- dim | 50 +++--- 1 file changed, 35 insertions(+), 15 deletions(-) diff --git a/dim b/dim index 34f2312be37a..b336c2c7d939 100755 --- a/dim +++ b/dim @@ -727,7 +727,27 @@ function assert_branch fi } +# handle aliases case "$subcommand" i

[Intel-gfx] [maintainer-tools RFC PATCH 10/17] dim: rename magic_patch to dim_magic_patch

2016-01-26 Thread Jani Nikula
Add dim_ prefix to functions corresponding to subcommands. Signed-off-by: Jani Nikula --- dim | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dim b/dim index c72f58302bae..b9152a256a26 100755 --- a/dim +++ b/dim @@ -396,7 +396,7 @@ function dim_apply_branch eval $

[Intel-gfx] [maintainer-tools RFC PATCH 00/17] dim: function based subcommand handling

2016-01-26 Thread Jani Nikula
I propose to change the dim tool to handle subcommands via functions matching the subcommand. Each 'dim sub-command' would be handled by a function named dim_sub_command. Only functions handling subcommands would have the dim_ prefix. Patches 1-11 are mostly renames to follow the proposed naming s

[Intel-gfx] [maintainer-tools RFC PATCH 03/17] dim: rename dim_co to dim_checkout

2016-01-26 Thread Jani Nikula
Rename to match the (long) subcommand name. Move error checking to the function too. Signed-off-by: Jani Nikula --- dim | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/dim b/dim index 5a074979055d..22c19e8f5362 100755 --- a/dim +++ b/dim @@ -430,8 +4

[Intel-gfx] [maintainer-tools RFC PATCH 05/17] dim: rename dim_checkpatch to checkpatch_commit

2016-01-26 Thread Jani Nikula
Rename as there's no corresponding subcommand (the check-patch subcommand does a slightly different thing). Signed-off-by: Jani Nikula --- dim | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dim b/dim index b42362a1e56b..6a8521c8c7b9 100755 --- a/dim +++ b/dim @@ -468,7

[Intel-gfx] [maintainer-tools RFC PATCH 02/17] dim: rename dim_apply to dim_apply_branch

2016-01-26 Thread Jani Nikula
Rename to match the subcommand name. Signed-off-by: Jani Nikula --- dim | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/dim b/dim index dbb69d993c44..5a074979055d 100755 --- a/dim +++ b/dim @@ -366,7 +366,7 @@ function dim_push_branch # ensure we're on branch $1

[Intel-gfx] [maintainer-tools RFC PATCH 08/17] dim: rename create_workdir to dim_create_workdir

2016-01-26 Thread Jani Nikula
Add dim_ prefix to functions corresponding to subcommands. Signed-off-by: Jani Nikula --- dim | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dim b/dim index 0b02c9c0effe..3ddbb728a044 100755 --- a/dim +++ b/dim @@ -545,7 +545,7 @@ function prep_pull_mail prep_pul

[Intel-gfx] [maintainer-tools RFC PATCH 07/17] dim: rename setup_dim to dim_setup

2016-01-26 Thread Jani Nikula
Rename to match the subcommand. Signed-off-by: Jani Nikula --- dim | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dim b/dim index 012353112a3b..0b02c9c0effe 100755 --- a/dim +++ b/dim @@ -615,7 +615,7 @@ function dim_pull_request "$addr_drm_maintainer" }

[Intel-gfx] [maintainer-tools RFC PATCH 04/17] dim: rename dim_shell_checkpatch to shell_checkpatch

2016-01-26 Thread Jani Nikula
Drop the dim_ prefix as there's no corresponding subcommand. Signed-off-by: Jani Nikula --- dim | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dim b/dim index 22c19e8f5362..b42362a1e56b 100755 --- a/dim +++ b/dim @@ -383,7 +383,7 @@ function dim_apply_branch

[Intel-gfx] [maintainer-tools RFC PATCH 06/17] dim: rename dim_checkrange to dim_checkpatch

2016-01-26 Thread Jani Nikula
Rename to match the subcommand. Also add checkpatch subcommand (without the hyphen) to match the spelling of the tool. Signed-off-by: Jani Nikula --- dim | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dim b/dim index 6a8521c8c7b9..012353112a3b 100755 --- a/dim +++ b/di

[Intel-gfx] [maintainer-tools RFC PATCH 01/17] dim: rename dim_push to dim_push_branch

2016-01-26 Thread Jani Nikula
Rename to match the subcommand name. Move error checking to the function too. Signed-off-by: Jani Nikula --- dim | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/dim b/dim index c749cebd1187..dbb69d993c44 100755 --- a/dim +++ b/dim @@ -346,8 +346,13 @@ fun

Re: [Intel-gfx] [REGRESSION] component: add support for releasing match data

2016-01-26 Thread Maarten Lankhorst
Op 26-01-16 om 15:15 schreef Russell King - ARM Linux: > On Tue, Jan 26, 2016 at 02:42:16PM +0100, Maarten Lankhorst wrote: >> Commit ce657b1cddf1f88c56 ("component: add support for releasing match >> data") causes a general protection fault when unloading snd-hda-intel >> with the i915 module load

Re: [Intel-gfx] [REGRESSION] component: add support for releasing match data

2016-01-26 Thread Russell King - ARM Linux
On Tue, Jan 26, 2016 at 02:42:16PM +0100, Maarten Lankhorst wrote: > Commit ce657b1cddf1f88c56 ("component: add support for releasing match > data") causes a general protection fault when unloading snd-hda-intel > with the i915 module loaded on a recent skylake machine. I'm no good at interpreting

Re: [Intel-gfx] [PATCH] drm/i915: Make wa_tail_dwords flexible for future platforms.

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 01:51:19PM +, Rodrigo Vivi wrote: >On Tue, Jan 26, 2016 at 12:30 AM Chris Wilson ><[1]ch...@chris-wilson.co.uk> wrote: > > On Mon, Jan 25, 2016 at 09:17:15PM +, Chris Wilson wrote: > > On Mon, Jan 25, 2016 at 11:29:19AM -0800, Rodrigo Vivi wrote: >

[Intel-gfx] [PATCH igt 2/2] tests/pm_rpm: find an appropriate CRTC instead of hardcoding CRTC 0

2016-01-26 Thread Paulo Zanoni
BSW does not allow CRTC 0 to be used on every connector, so we need to write code to actually find a suitable CRTC. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93124 Signed-off-by: Paulo Zanoni --- tests/pm_rpm.c | 52 ++-- 1 file change

[Intel-gfx] [PATCH igt 1/2] tests/pm_rpm: remove POWER_DIR definition

2016-01-26 Thread Paulo Zanoni
Unused ever since we moved some code from pm_rpm.c to lib/. This is currently defined inside igt_aux.c. Signed-off-by: Paulo Zanoni --- tests/pm_rpm.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c index 3a5e8ba..bc81144 100644 --- a/tests/pm_rpm.c +++ b/tes

Re: [Intel-gfx] [PATCH] drm/i915: Make wa_tail_dwords flexible for future platforms.

2016-01-26 Thread Rodrigo Vivi
On Tue, Jan 26, 2016 at 12:30 AM Chris Wilson wrote: > On Mon, Jan 25, 2016 at 09:17:15PM +, Chris Wilson wrote: > > On Mon, Jan 25, 2016 at 11:29:19AM -0800, Rodrigo Vivi wrote: > > > +++ b/drivers/gpu/drm/i915/intel_lrc.c > > > @@ -764,18 +764,18 @@ intel_logical_ring_advance_and_submit(str

Re: [Intel-gfx] [PATCH igt 1/6] lib/igt_fb: make the automatic buffer sizes/strides smaller

2016-01-26 Thread Paulo Zanoni
2016-01-26 11:40 GMT-02:00 Chris Wilson : > On Tue, Jan 26, 2016 at 11:28:59AM -0200, Paulo Zanoni wrote: >> The big motivation behind this patch is that the current power-of-two >> granularity from igt_fb is way too big. There was more than one >> occasion where I had to work around this problem o

[Intel-gfx] [REGRESSION] component: add support for releasing match data

2016-01-26 Thread Maarten Lankhorst
Hey, Commit ce657b1cddf1f88c56 ("component: add support for releasing match data") causes a general protection fault when unloading snd-hda-intel with the i915 module loaded on a recent skylake machine. This breaks one of the i915 acceptance tests that performs a module unload/reload on snd-hd

[Intel-gfx] [PATCH igt] tools/intel_residency: use setitimer instead of {, u}alarm

2016-01-26 Thread Paulo Zanoni
It seems that Android doesn't have ualarm(). Let's use setitimer() instead. The tool still won't compile on Android due to igt_fb requiring Cairo, but we're supposed to solve this in another patch since our igt_fb calls don't actually require Cairo. Reported-by: Derek Morton Signed-off-by: Paulo

Re: [Intel-gfx] [PATCH igt 1/6] lib/igt_fb: make the automatic buffer sizes/strides smaller

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 11:28:59AM -0200, Paulo Zanoni wrote: > The big motivation behind this patch is that the current power-of-two > granularity from igt_fb is way too big. There was more than one > occasion where I had to work around this problem on > kms_frontbuffer_tracking, and during my las

[Intel-gfx] [PATCH igt 1/6] lib/igt_fb: make the automatic buffer sizes/strides smaller

2016-01-26 Thread Paulo Zanoni
The big motivation behind this patch is that the current power-of-two granularity from igt_fb is way too big. There was more than one occasion where I had to work around this problem on kms_frontbuffer_tracking, and during my last workaround I was requested to just make igt_fb use more minimal buff

[Intel-gfx] [PATCH igt 6/6] tests/igt_fb: rename igt_get_all_formats to igt_get_all_cairo_formats

2016-01-26 Thread Paulo Zanoni
I recently had this discussion with Daniel where I didn't want to use igt_drm_format_to_bpp() because it uses the format_desc array, and igt_fb currently assumes that all the format_desc formats have a matching valid Cairo format, so I wouldn't be able to easily add formats such as ARGB2101010. Th

[Intel-gfx] [PATCH igt 2/6] kms_frontbuffer_tracking: use igt_drm_format_to_bpp()

2016-01-26 Thread Paulo Zanoni
The only format from fb_get_bpp() not supported by igt_drm_format_to_bpp() is ARGB2101010, but we don't really use it in kms_frontbuffer_tracking, so we can do the switch. Adding ARGB2101010 to igt_fb won't be that simple since there's no equivalent Cairo format, and igt_fb users assume that all f

[Intel-gfx] [PATCH igt 3/6] kms_frontbuffer_tracking: standardize the used FB sizes

2016-01-26 Thread Paulo Zanoni
We want to make sure that both tiled and untiled buffers have the same size for the same width/height/format. This will allow better control over the failure paths exercised by our tests: when we try to flip from tiled to untiled, we'll be sure that we won't execute the error path that checks for b

[Intel-gfx] [PATCH igt 5/6] lib/igt_fb: fix igt_get_all_formats documentation

2016-01-26 Thread Paulo Zanoni
We give the callers a const pointer to a static variable that we reuse between multiple calls: they're not supposed to free it, and they don't free it today. Fix the documentation and leave the still reachable pointer instead of reworking the function and its callers. Signed-off-by: Paulo Zanoni

[Intel-gfx] [PATCH igt 4/6] lib/igt_draw: use igt_drm_format_to_bpp()

2016-01-26 Thread Paulo Zanoni
Don't reimplement the function. Signed-off-by: Paulo Zanoni --- lib/igt_draw.c | 16 +--- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/lib/igt_draw.c b/lib/igt_draw.c index f85e376..45fa10f 100644 --- a/lib/igt_draw.c +++ b/lib/igt_draw.c @@ -594,20 +594,6 @@ void ig

Re: [Intel-gfx] [PATCH 2/5] drm/i915: Cleaning up intel_dp_hpd_pulse

2016-01-26 Thread Ander Conselvan De Oliveira
On Tue, 2016-01-19 at 16:07 +0530, Shubhangi Shrivastava wrote: > Current DP detection has DPCD operations split across > intel_dp_hpd_pulse and intel_dp_detect which contains > duplicates as well. Also intel_dp_detect is called > during modes enumeration as well which will result > in multiple dpc

[Intel-gfx] [PATCH] tests/Android.mk: Make intel_residency CAIRO dependant

2016-01-26 Thread Derek Morton
intel_residency has a cairo dependency through igt_fb.c. Remove it if ANDROID_HAS_CAIRO is not defined. Signed-off-by: Derek Morton --- tools/Android.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/Android.mk b/tools/Android.mk index da4f3c0..5f64184 100644 --- a/tools/Android.mk ++

Re: [Intel-gfx] [PATCH 0/6] Pipe level color management

2016-01-26 Thread Daniel Stone
Hi, On 22 January 2016 at 15:04, Daniel Stone wrote: > On 21 January 2016 at 15:03, Lionel Landwerlin > wrote: >> Hi, >> >> This serie introduces pipe level color management through a set of properties >> attached to the CRTC. It also provides an implementation for some Intel >> platforms. >> >>

Re: [Intel-gfx] [PATCH v4] drm/i915/guc: Clean up locks in GuC

2016-01-26 Thread Chris Wilson
On Tue, Jan 26, 2016 at 11:53:45AM +, Tvrtko Ursulin wrote: > Chris Harris has liaised with Tomi and apparently GuC firmware is > already on the lab SKLs from ~two weeks ago. > > Since the firmware is on all machines, presumably we would also want > to have CI runs without the GuC enabled to e

Re: [Intel-gfx] Regression in v4.5-rc1, bisected to commit 39bfcd5235e0 ("drm/i915: more virtual south bridge detection")

2016-01-26 Thread Harald Arnesen
Bjørn Mork [2016-01-25 03:52]: > I have confirmed tha reverting this commit on top of v4.5-rc1 fixes the > problem. Confirmed. Fixes the problem with my T500 also. -- Hilsen Harald ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://list

  1   2   >