[Intel-gfx] [PATCH] drm/i915/execlists: Assign virtual_engine->uncore from first sibling

2019-10-08 Thread Chris Wilson
Copy across the engine->uncore shortcut to the virtual_engine from its first physical engine, similar to the handling of the engine->gt backpointer. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/intel_lrc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/gt/int

[Intel-gfx] [PATCH] drm/i915/selftests: Assign the engine->uncore shortcut

2019-10-08 Thread Chris Wilson
Set up the engine->uncore shortcut on mock_engine creation. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/mock_engine.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/gt/mock_engine.c b/drivers/gpu/drm/i915/gt/mock_engine.c index 5d43cbc3f345..3d88397c0dbb 10

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for DC3CO Support for TGL test with DC3CO IGT (rev2)

2019-10-08 Thread Imre Deak
On Mon, Oct 07, 2019 at 06:37:48PM +, Patchwork wrote: > == Series Details == > > Series: DC3CO Support for TGL test with DC3CO IGT (rev2) > URL : https://patchwork.freedesktop.org/series/67525/ > State : failure Thanks for the patches merged to -dinq. For the failure see below. > > == Su

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/execlists: Assign virtual_engine->uncore from first sibling

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Assign virtual_engine->uncore from first sibling URL : https://patchwork.freedesktop.org/series/67719/ State : success == Summary == CI Bug Log - changes from CI_DRM_7030 -> Patchwork_14698 S

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for DC3CO Support for TGL test with DC3CO IGT (rev2)

2019-10-08 Thread Gupta, Anshuman
Thanks Imre for merging the patches 😊 -Original Message- From: Imre Deak Sent: Tuesday, October 8, 2019 1:59 PM To: intel-gfx@lists.freedesktop.org; Gupta, Anshuman ; Vudum, Lakshminarayana Subject: Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for DC3CO Support for TGL test with DC3CO IGT (

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for DC3CO Support for TGL test with DC3CO IGT (rev2)

2019-10-08 Thread Vudum, Lakshminarayana
CI bug log filter is updated for Bug 108686 to associate dmesg-fail as well. Lakshmi. -Original Message- From: Gupta, Anshuman Sent: Tuesday, October 8, 2019 11:40 AM To: Deak, Imre ; intel-gfx@lists.freedesktop.org; Vudum, Lakshminarayana Subject: RE: [Intel-gfx] ✗ Fi.CI.IGT: failure

Re: [Intel-gfx] [PATCH 1/4] dma-buf: change DMA-buf locking convention

2019-10-08 Thread Daniel Vetter
On Thu, Aug 29, 2019 at 04:29:14PM +0200, Christian König wrote: > This patch is a stripped down version of the locking changes > necessary to support dynamic DMA-buf handling. > > For compatibility we cache the DMA-buf mapping as soon as > exporter/importer disagree on the dynamic handling. Nee

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Assign the engine->uncore shortcut

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/selftests: Assign the engine->uncore shortcut URL : https://patchwork.freedesktop.org/series/67720/ State : success == Summary == CI Bug Log - changes from CI_DRM_7030 -> Patchwork_14699 Summary ---

Re: [Intel-gfx] [PATCH 1/4] dma-buf: change DMA-buf locking convention

2019-10-08 Thread Daniel Vetter
On Wed, Oct 02, 2019 at 08:37:50AM +, Koenig, Christian wrote: > Hi Daniel, > > once more a ping on this. Any more comments or can we get it comitted? Sorry got a bit smashed past weeks, but should be resurrected now back from xdc. -Daniel > > Thanks, > Christian. > > Am 24.09.19 um 11:50 s

Re: [Intel-gfx] [PATCH v3 02/21] drm/i915: introduce intel_memory_region

2019-10-08 Thread Chris Wilson
Quoting Matthew Auld (2019-10-04 18:04:33) > +int > +i915_gem_object_get_pages_buddy(struct drm_i915_gem_object *obj) > +{ > + struct intel_memory_region *mem = obj->mm.region; > + struct list_head *blocks = &obj->mm.blocks; > + unsigned int flags = I915_ALLOC_MIN_PAGE_SIZE; > +

Re: [Intel-gfx] [PATCH v3 03/21] drm/i915/region: support contiguous allocations

2019-10-08 Thread Chris Wilson
Quoting Matthew Auld (2019-10-04 18:04:34) > @@ -98,10 +103,12 @@ i915_gem_object_get_pages_buddy(struct > drm_i915_gem_object *obj) > } > > void i915_gem_object_init_memory_region(struct drm_i915_gem_object *obj, > - struct intel_memory_region *mem) > +

Re: [Intel-gfx] [PATCH v3 04/21] drm/i915/region: support volatile objects

2019-10-08 Thread Chris Wilson
Quoting Matthew Auld (2019-10-04 18:04:35) > diff --git a/drivers/gpu/drm/i915/intel_memory_region.h > b/drivers/gpu/drm/i915/intel_memory_region.h > index 29b86ca17dd9..323270a1ef67 100644 > --- a/drivers/gpu/drm/i915/intel_memory_region.h > +++ b/drivers/gpu/drm/i915/intel_memory_region.h > @@ -

Re: [Intel-gfx] [PATCH v3 05/21] drm/i915: Add memory region information to device_info

2019-10-08 Thread Chris Wilson
Quoting Matthew Auld (2019-10-04 18:04:36) > From: Abdiel Janulgue > > Exposes available regions for the platform. Shared memory will > always be available. > > Signed-off-by: Abdiel Janulgue > Signed-off-by: Matthew Auld > --- > drivers/gpu/drm/i915/i915_drv.h | 2 ++ > drivers/gpu/

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Assign the engine->uncore shortcut

2019-10-08 Thread Tvrtko Ursulin
On 08/10/2019 08:11, Chris Wilson wrote: Set up the engine->uncore shortcut on mock_engine creation. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/mock_engine.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/gt/mock_engine.c b/drivers/gpu/drm/i915/gt/moc

Re: [Intel-gfx] [PATCH] drm/i915/execlists: Assign virtual_engine->uncore from first sibling

2019-10-08 Thread Tvrtko Ursulin
On 08/10/2019 08:03, Chris Wilson wrote: Copy across the engine->uncore shortcut to the virtual_engine from its first physical engine, similar to the handling of the engine->gt backpointer. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/intel_lrc.c | 1 + 1 file changed, 1 insertio

Re: [Intel-gfx] [PATCH] drm/i915: customize DPCD brightness control for specific panel

2019-10-08 Thread Jani Nikula
On Mon, 07 Oct 2019, Adam Jackson wrote: > On Mon, 2019-10-07 at 12:08 +0300, Jani Nikula wrote: > >> The problem with the EDID quirks is that exposing the quirks sticks out >> like a sore thumb. Thus far all of it has been contained in drm_edid.c >> and they affect how the EDID gets parsed, for a

Re: [Intel-gfx] [PATCH 2/4] drm/ttm: use the parent resv for ghost objects v2

2019-10-08 Thread Daniel Vetter
On Thu, Aug 29, 2019 at 04:29:15PM +0200, Christian König wrote: > This way we can even pipeline imported BO evictions. > > v2: Limit this to only cases when the parent object uses a separate > reservation object as well. This fixes another OOM problem. > > Signed-off-by: Christian König Si

[Intel-gfx] [PATCH] drm/i915: Describe structure member in documentation

2019-10-08 Thread Anna Karas
Add description of wakeref member of intel_shared_dpll structure to documentation. Cc: Lucas De Marchi Cc: Vivek Kasireddy Signed-off-by: Anna Karas --- drivers/gpu/drm/i915/display/intel_dpll_mgr.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_dpl

Re: [Intel-gfx] [PATCH v2 00/21] drm/dp: Various helper improvements and cleanups

2019-10-08 Thread Daniel Vetter
On Wed, Oct 02, 2019 at 06:14:19PM +0200, Thierry Reding wrote: > On Mon, Sep 23, 2019 at 04:52:02PM +0200, Thierry Reding wrote: > > On Mon, Sep 23, 2019 at 04:52:50PM +0300, Jani Nikula wrote: > > > On Fri, 20 Sep 2019, Thierry Reding wrote: > > > > On Mon, Sep 02, 2019 at 01:31:00PM +0200, Thie

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Describe structure member in documentation (rev2)

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915: Describe structure member in documentation (rev2) URL : https://patchwork.freedesktop.org/series/67102/ State : success == Summary == CI Bug Log - changes from CI_DRM_7032 -> Patchwork_14700 Summary --

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/execlists: Assign virtual_engine->uncore from first sibling

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Assign virtual_engine->uncore from first sibling URL : https://patchwork.freedesktop.org/series/67719/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7030_full -> Patchwork_14698_full =

[Intel-gfx] [PATCH] drm/i915/gt: Flush submission tasklet before waiting/retiring

2019-10-08 Thread Chris Wilson
A common bane of ours is arbitrary delays in ksoftirqd processing our submission tasklet. Give the submission tasklet a kick before we wait to avoid those delays eating into a tight timeout. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/intel_engine.h | 3 +- drivers/gpu/drm/i915

[Intel-gfx] [PATCH i-g-t] i915/gem_eio: Reduce amount of incidental work after the delayed hang

2019-10-08 Thread Chris Wilson
Since we time how long it takes for the waiter to be woken upon injecting the hang, we want to avoid as much distractions as possible along the critical path. Signed-off-by: Chris Wilson --- tests/i915/gem_eio.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a

Re: [Intel-gfx] [PATCH v3 1/2] drm/i915: Move SAGV block time to dev_priv

2019-10-08 Thread Ville Syrjälä
On Fri, Oct 04, 2019 at 03:14:48PM -0700, James Ausmus wrote: > In prep for newer platforms having more complicated ways to determine > the SAGV block time, move the variable to dev_priv, and extract the > setting to an initial setup function. While we're at it, update the if > ladder to follow the

Re: [Intel-gfx] [PATCH v3 02/21] drm/i915: introduce intel_memory_region

2019-10-08 Thread Matthew Auld
On 08/10/2019 09:59, Chris Wilson wrote: Quoting Matthew Auld (2019-10-04 18:04:33) +int +i915_gem_object_get_pages_buddy(struct drm_i915_gem_object *obj) +{ + struct intel_memory_region *mem = obj->mm.region; + struct list_head *blocks = &obj->mm.blocks; + unsigned int flags =

Re: [Intel-gfx] [PATCH v3 02/21] drm/i915: introduce intel_memory_region

2019-10-08 Thread Chris Wilson
Quoting Matthew Auld (2019-10-08 12:57:50) > On 08/10/2019 09:59, Chris Wilson wrote: > > Quoting Matthew Auld (2019-10-04 18:04:33) > >> +int > >> +i915_gem_object_get_pages_buddy(struct drm_i915_gem_object *obj) > >> +{ > >> + struct intel_memory_region *mem = obj->mm.region; > >> + s

Re: [Intel-gfx] [PATCH 03/11] drm/i915: keep power domains init/remove calls at the same level

2019-10-08 Thread Jani Nikula
On Mon, 07 Oct 2019, Jani Nikula wrote: > Move intel_power_domains_init_hw() call one level higher, to be on the > same level as all the other intel_power_domains_*() calls in the > probe/remove paths. > > This also moves the power domain hw init earlier in the sequence, along > with the dependent

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Flush submission tasklet before waiting/retiring

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/gt: Flush submission tasklet before waiting/retiring URL : https://patchwork.freedesktop.org/series/67732/ State : success == Summary == CI Bug Log - changes from CI_DRM_7032 -> Patchwork_14701 Summary

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/selftests: Assign the engine->uncore shortcut

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/selftests: Assign the engine->uncore shortcut URL : https://patchwork.freedesktop.org/series/67720/ State : success == Summary == CI Bug Log - changes from CI_DRM_7030_full -> Patchwork_14699_full Summa

[Intel-gfx] [CI] drm/i915/perf: drop list of streams

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin At some point in time there was the idea that we could have multiple stream from the same piece of HW but that never materialized and given the hard time we already have making everything work with the submission side, there is no real point having this list of 1 element a

Re: [Intel-gfx] [CI] drm/i915/perf: drop list of streams

2019-10-08 Thread Lionel Landwerlin
On 08/10/2019 17:01, Chris Wilson wrote: From: Lionel Landwerlin At some point in time there was the idea that we could have multiple stream from the same piece of HW but that never materialized and given the hard time we already have making everything work with the submission side, there is no

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/perf: drop list of streams

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/perf: drop list of streams URL : https://patchwork.freedesktop.org/series/67734/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.6.0 Commit: drm/i915/perf: drop list of streams +drivers/gpu/drm/i915/i915_perf.c:1408:15: warning

[Intel-gfx] [PATCH] drm/i915/selftests: Assign the intel_runtime_pm pointer for mock_uncore

2019-10-08 Thread Chris Wilson
Couple up our mock_uncore to know about the fake global device and its runtime powermanagement. Signed-off-by: Chris Wilson Cc: Matthew Auld --- drivers/gpu/drm/i915/gt/mock_engine.c| 1 + drivers/gpu/drm/i915/selftests/mock_gem_device.c | 2 +- drivers/gpu/drm/i915/selftests/mock_u

Re: [Intel-gfx] [PATCH] drm/i915/gt: Flush submission tasklet before waiting/retiring

2019-10-08 Thread Summers, Stuart
On Tue, 2019-10-08 at 11:56 +0100, Chris Wilson wrote: > A common bane of ours is arbitrary delays in ksoftirqd processing our > submission tasklet. Give the submission tasklet a kick before we wait > to > avoid those delays eating into a tight timeout. > > Signed-off-by: Chris Wilson > --- > dr

Re: [Intel-gfx] [PATCH v8 1/4] drm/panel: Add helper for reading DT rotation

2019-10-08 Thread Sean Paul
On Mon, Oct 07, 2019 at 03:12:00PM -0700, dbasehore . wrote: > On Mon, Oct 7, 2019 at 9:38 AM Sean Paul wrote: > > > > On Wed, Sep 25, 2019 at 03:58:30PM -0700, Derek Basehore wrote: > > > This adds a helper function for reading the rotation (panel > > > orientation) from the device tree. > > > >

Re: [Intel-gfx] [PATCH] drm/i915/gt: Flush submission tasklet before waiting/retiring

2019-10-08 Thread Chris Wilson
Quoting Summers, Stuart (2019-10-08 15:52:15) > On Tue, 2019-10-08 at 11:56 +0100, Chris Wilson wrote: > > A common bane of ours is arbitrary delays in ksoftirqd processing our > > submission tasklet. Give the submission tasklet a kick before we wait > > to > > avoid those delays eating into a tigh

Re: [Intel-gfx] [PATCH] drm/i915/gt: Flush submission tasklet before waiting/retiring

2019-10-08 Thread Chris Wilson
Quoting Summers, Stuart (2019-10-08 15:52:15) > On Tue, 2019-10-08 at 11:56 +0100, Chris Wilson wrote: > > A common bane of ours is arbitrary delays in ksoftirqd processing our > > submission tasklet. Give the submission tasklet a kick before we wait > > to > > avoid those delays eating into a tigh

Re: [Intel-gfx] [v8, 2/4] drm/panel: set display info in panel attach

2019-10-08 Thread Sean Paul
/snip > > > > > diff --git a/include/drm/drm_panel.h b/include/drm/drm_panel.h > > > > > index d16158deacdc..f3587a54b8ac 100644 > > > > > --- a/include/drm/drm_panel.h > > > > > +++ b/include/drm/drm_panel.h > > > > > @@ -141,6 +141,56 @@ struct drm_panel { > > > > >*/ > > > > > con

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Describe structure member in documentation (rev2)

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915: Describe structure member in documentation (rev2) URL : https://patchwork.freedesktop.org/series/67102/ State : success == Summary == CI Bug Log - changes from CI_DRM_7032_full -> Patchwork_14700_full

Re: [Intel-gfx] [PATCH] drm/i915/gt: Flush submission tasklet before waiting/retiring

2019-10-08 Thread Summers, Stuart
On Tue, 2019-10-08 at 15:56 +0100, Chris Wilson wrote: > Quoting Summers, Stuart (2019-10-08 15:52:15) > > On Tue, 2019-10-08 at 11:56 +0100, Chris Wilson wrote: > > > A common bane of ours is arbitrary delays in ksoftirqd processing > > > our > > > submission tasklet. Give the submission tasklet a

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/perf: drop list of streams

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/perf: drop list of streams URL : https://patchwork.freedesktop.org/series/67734/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7033 -> Patchwork_14702 Summary --- **FAILURE**

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Assign the intel_runtime_pm pointer for mock_uncore

2019-10-08 Thread Matthew Auld
On Tue, 8 Oct 2019 at 15:50, Chris Wilson wrote: > > Couple up our mock_uncore to know about the fake global device and its > runtime powermanagement. > > Signed-off-by: Chris Wilson > Cc: Matthew Auld Thanks, Reviewed-by: Matthew Auld ___ Intel-gfx

Re: [Intel-gfx] [PATCH 2/2] drm/i915/tgl: Restrict availables engines to rcs0 by default

2019-10-08 Thread Summers, Stuart
On Tue, 2019-10-01 at 14:54 +0100, Chris Wilson wrote: > CI is still unstable whenever we enable more than one engine, and we > have not yet found a better hack than restricting it to using just > rcs0. > > However, to allow testing to continue on the other engines by > developers, we allow the av

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Use a modparam to restrict exposed engines

2019-10-08 Thread Summers, Stuart
On Tue, 2019-10-01 at 14:54 +0100, Chris Wilson wrote: > Allow the user to restrict the available set of engines via a module > parameter. > > Signed-off-by: Chris Wilson > Cc: Stuart Summers > Cc: Andi Shyti > Cc: Mika Kuoppala > Cc: Tvrtko Ursulin > Cc: Joonas Lahtinen > Cc: Martin Peres

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/selftests: Assign the intel_runtime_pm pointer for mock_uncore

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/selftests: Assign the intel_runtime_pm pointer for mock_uncore URL : https://patchwork.freedesktop.org/series/67736/ State : success == Summary == CI Bug Log - changes from CI_DRM_7033 -> Patchwork_14703

[Intel-gfx] [PATCH 1/3] drm/i915: introduce intel_memory_region

2019-10-08 Thread Matthew Auld
Support memory regions, as defined by a given (start, end), and allow creating GEM objects which are backed by said region. The immediate goal here is to have something to represent our device memory, but later on we also want to represent every memory domain with a region, so stolen, shmem, and of

[Intel-gfx] [PATCH 2/3] drm/i915/region: support contiguous allocations

2019-10-08 Thread Matthew Auld
Some kernel internal objects may need to be allocated as a contiguous block, also thinking ahead the various kernel io_mapping interfaces seem to expect it, although this is purely a limitation in the kernel API...so perhaps something to be improved. Signed-off-by: Matthew Auld Cc: Joonas Lahtine

[Intel-gfx] [PATCH 3/3] drm/i915/region: support volatile objects

2019-10-08 Thread Matthew Auld
Volatile objects are marked as DONTNEED while pinned, therefore once unpinned the backing store can be discarded. This is limited to kernel internal objects. Signed-off-by: Matthew Auld Signed-off-by: CQ Tang Cc: Joonas Lahtinen Cc: Abdiel Janulgue Reviewed-by: Chris Wilson --- drivers/gpu/d

[Intel-gfx] [PATCH 0/3] intel_memory_region bits

2019-10-08 Thread Matthew Auld
First block of patches from the 'LMEM basics' series. Matthew Auld (3): drm/i915: introduce intel_memory_region drm/i915/region: support contiguous allocations drm/i915/region: support volatile objects drivers/gpu/drm/i915/Makefile | 2 + drivers/gpu/drm/i915/gem/i915_gem

Re: [Intel-gfx] [PATCH 1/3] drm/i915: introduce intel_memory_region

2019-10-08 Thread Chris Wilson
Quoting Matthew Auld (2019-10-08 17:01:14) > +static void close_objects(struct intel_memory_region *mem, > + struct list_head *objects) > +{ > + struct drm_i915_private *i915 = mem->i915; > + struct drm_i915_gem_object *obj, *on; > + > + list_for_each_entry

Re: [Intel-gfx] [PATCH 2/3] drm/i915/region: support contiguous allocations

2019-10-08 Thread Chris Wilson
Quoting Matthew Auld (2019-10-08 17:01:15) > Some kernel internal objects may need to be allocated as a contiguous > block, also thinking ahead the various kernel io_mapping interfaces seem > to expect it, although this is purely a limitation in the kernel > API...so perhaps something to be improve

[Intel-gfx] [PATCH] drm/i915: Select DPLL's via mask

2019-10-08 Thread Matt Roper
This slightly simplifies the EHL DPLL4 handling and also gives us more flexibility in the future in case we need to skip the use of specific PLL's (e.g., due to hardware workarounds and such). Cc: Lucas De Marchi Signed-off-by: Matt Roper --- drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 42 +

[Intel-gfx] [PATCH 4/9] drm/i915: Expose C8 on VLV/CHV sprite planes

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä VLV/CHV sprite planes also support the C8 format. Let's expose that. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_sprite.c | 6 ++ drivers/gpu/drm/i915/i915_reg.h | 1 + 2 files changed, 7 insertions(+) diff --git a/drivers/gpu/drm/i9

[Intel-gfx] [PATCH 8/9] drm/i915: Do not enable HDR mode when color keying is active

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä The spec says that color keying and HDR mode are mutually exclusive. So let's not enable HDR mode when color keying is active. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_atomic_plane.c | 5 + drivers/gpu/drm/i915/display/intel_display.c

[Intel-gfx] [PATCH 5/9] drm/i915: Add 10bpc formats with alpha for icl+

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä ICL+ again supports alpha blending with 10bpc pixel formats. Expose them. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_sprite.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_sprite.c b/drivers/gpu/drm/i9

[Intel-gfx] [PATCH 9/9] drm/i915: Eliminate redundancy in intel_primary_plane_create()

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä Lots of redundant assignments inside intel_primary_plane_create(). Get rid of them. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_display.c | 43 +++- 1 file changed, 14 insertions(+), 29 deletions(-) diff --git a/drivers/gpu/drm/i915/

[Intel-gfx] [PATCH 6/9] drm/i915: Sort format arrays consistently

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä Let's try to keep the pixel format arrays somewhat sorted: 1. RGB before YUV 2. smaller bpp before larger bpp 3. X before A 4. RGB before BGR Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_display.c | 2 +- drivers/gpu/drm/i915/display/intel_sprite.c

[Intel-gfx] [PATCH 1/9] drm/i915: Expose 10:10:10 XRGB formats on SNB-BDW sprites

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä SNB-BDW support 10:10:10 formats on the sprite planes. Let's expose them. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_sprite.c | 16 1 file changed, 16 insertions(+) diff --git a/drivers/gpu/drm/i915/display/intel_sprite.c b/driver

[Intel-gfx] [PATCH 2/9] drm/i915: Expose alpha formats on VLV/CHV primary planes

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä Currently we expose VLV/CHV alpha blending only on the sprite planes, but the primary planes can do it as well. Let's flip it on. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_display.c | 57 +++- drivers/gpu/drm/i915/i915_reg.h

[Intel-gfx] [PATCH 3/9] drm/i915: Add missing 10bpc formats for pipe B sprites on CHV

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä CHV pipe B sprites gained support for the 10bpc X/ARGB pixel formats. On VLV and CHV pipe A/C these are only supported by the the primary plane. Add the require bits to expose the new formats. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_sprite.c | 33

[Intel-gfx] [PATCH 7/9] drm/i915: Reject ckey+fp16 on skl+

2019-10-08 Thread Ville Syrjala
From: Ville Syrjälä According to the spec color keying is not supported with fp16 pixel formats on skl+. Reject that combo. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_sprite.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/drivers/gpu/drm/i915/

Re: [Intel-gfx] [PATCH] drm/i915: Select DPLL's via mask

2019-10-08 Thread Lucas De Marchi
On Tue, Oct 08, 2019 at 09:12:52AM -0700, Matt Roper wrote: This slightly simplifies the EHL DPLL4 handling and also gives us more flexibility in the future in case we need to skip the use of specific PLL's (e.g., due to hardware workarounds and such). Cc: Lucas De Marchi Signed-off-by: Matt Ro

Re: [Intel-gfx] [PATCH] drm/i915: Select DPLL's via mask

2019-10-08 Thread Ville Syrjälä
On Tue, Oct 08, 2019 at 09:12:52AM -0700, Matt Roper wrote: > This slightly simplifies the EHL DPLL4 handling and also gives us more > flexibility in the future in case we need to skip the use of specific > PLL's (e.g., due to hardware workarounds and such). > > Cc: Lucas De Marchi > Signed-off-b

Re: [Intel-gfx] [PATCH -next] treewide: remove unused argument in lock_release()

2019-10-08 Thread Daniel Vetter
On Thu, Sep 19, 2019 at 12:09:40PM -0400, Qian Cai wrote: > Since the commit b4adfe8e05f1 ("locking/lockdep: Remove unused argument > in __lock_release"), @nested is no longer used in lock_release(), so > remove it from all lock_release() calls and friends. > > Signed-off-by: Qian Cai Ack on the

Re: [Intel-gfx] [PATCH] drm/i915: Select DPLL's via mask

2019-10-08 Thread Lucas De Marchi
On Tue, Oct 08, 2019 at 07:28:45PM +0300, Ville Syrjälä wrote: On Tue, Oct 08, 2019 at 09:12:52AM -0700, Matt Roper wrote: This slightly simplifies the EHL DPLL4 handling and also gives us more flexibility in the future in case we need to skip the use of specific PLL's (e.g., due to hardware wor

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gt: Flush submission tasklet before waiting/retiring

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/gt: Flush submission tasklet before waiting/retiring URL : https://patchwork.freedesktop.org/series/67732/ State : success == Summary == CI Bug Log - changes from CI_DRM_7032_full -> Patchwork_14701_full

Re: [Intel-gfx] [PATCH 07/24] drm/i915: Introduce intel_atomic_get_plane_state_after_check()

2019-10-08 Thread Ville Syrjälä
On Fri, Oct 04, 2019 at 01:34:57PM +0200, Maarten Lankhorst wrote: > Use this in all the places where we try to acquire planes after the planes > atomic_check(). > > In case of intel_modeset_all_pipes() this is not yet done after atomic_check, > but seems like it will be in the future. To add some

[Intel-gfx] [CI v3 1/2] drm/i915: Move SAGV block time to dev_priv

2019-10-08 Thread James Ausmus
In prep for newer platforms having more complicated ways to determine the SAGV block time, move the variable to dev_priv, and extract the setting to an initial setup function. While we're at it, update the if ladder to follow the new gen -> old gen order preference, and warn on any non-specified ge

[Intel-gfx] [CI v3 2/2] drm/i915/tgl: Read SAGV block time from PCODE

2019-10-08 Thread James Ausmus
Starting from TGL, we now need to read the SAGV block time via a PCODE mailbox, rather than having a static value. BSpec: 49326 v2: Fix up pcode val data type (Ville), tighten variable scope (Ville) Cc: Ville Syrjälä Cc: Stanislav Lisovskiy Cc: Lucas De Marchi Signed-off-by: James Ausmus Rev

[Intel-gfx] [CI] drm/i915: Select DPLL's via mask

2019-10-08 Thread Matt Roper
This slightly simplifies the EHL DPLL4 handling and also gives us more flexibility in the future in case we need to skip the use of specific PLL's (e.g., due to hardware workarounds and such). v2: - Replace GENMASK() with or'd BIT()'s to make the specific DPLLs more explicit. (Ville) - s/uns

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for intel_memory_region bits

2019-10-08 Thread Patchwork
== Series Details == Series: intel_memory_region bits URL : https://patchwork.freedesktop.org/series/67738/ State : warning == Summary == $ dim checkpatch origin/drm-tip 46562367c0c6 drm/i915: introduce intel_memory_region -:60: WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does

Re: [Intel-gfx] [PATCH 12/24] drm/i915: Split plane hw and uapi state

2019-10-08 Thread Ville Syrjälä
On Fri, Oct 04, 2019 at 01:35:02PM +0200, Maarten Lankhorst wrote: > Splitting plane state is easier than splitting crtc_state, > before plane check we copy the drm properties to hw so we can > do the same in bigjoiner later on. > > We copy the state after we did all the modeset handling, but fort

Re: [Intel-gfx] [PATCH 14/24] drm/i915/dp: Allow big joiner modes in intel_dp_mode_valid(), v2.

2019-10-08 Thread Ville Syrjälä
On Fri, Oct 04, 2019 at 01:35:04PM +0200, Maarten Lankhorst wrote: > Small changes to intel_dp_mode_valid(), allow listing modes that > can only be supported in the bigjoiner configuration, which is > not supported yet. > > eDP does not support bigjoiner, so do not expose bigjoiner only > modes on

[Intel-gfx] ✓ Fi.CI.BAT: success for intel_memory_region bits

2019-10-08 Thread Patchwork
== Series Details == Series: intel_memory_region bits URL : https://patchwork.freedesktop.org/series/67738/ State : success == Summary == CI Bug Log - changes from CI_DRM_7035 -> Patchwork_14704 Summary --- **SUCCESS** No regressi

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/9] drm/i915: Expose 10:10:10 XRGB formats on SNB-BDW sprites

2019-10-08 Thread Patchwork
== Series Details == Series: series starting with [1/9] drm/i915: Expose 10:10:10 XRGB formats on SNB-BDW sprites URL : https://patchwork.freedesktop.org/series/67741/ State : success == Summary == CI Bug Log - changes from CI_DRM_7035 -> Patchwork_14705 ==

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,v3,1/2] drm/i915: Move SAGV block time to dev_priv

2019-10-08 Thread Patchwork
== Series Details == Series: series starting with [CI,v3,1/2] drm/i915: Move SAGV block time to dev_priv URL : https://patchwork.freedesktop.org/series/67743/ State : warning == Summary == $ dim checkpatch origin/drm-tip b44bcf0ca073 drm/i915: Move SAGV block time to dev_priv -:61: WARNING:UN

[Intel-gfx] [PATCH] drm/i915/gt: Give engine->kernel_context distinct timeline lock classes

2019-10-08 Thread Chris Wilson
Assign a separate lockclass to the perma-pinned timelines of the kernel_context, such that we can use them from within the user timelines should we ever need to inject GPU operations to fixup faults during request construction. Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin Cc: Matthew Auld ---

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,v3,1/2] drm/i915: Move SAGV block time to dev_priv

2019-10-08 Thread Patchwork
== Series Details == Series: series starting with [CI,v3,1/2] drm/i915: Move SAGV block time to dev_priv URL : https://patchwork.freedesktop.org/series/67743/ State : success == Summary == CI Bug Log - changes from CI_DRM_7035 -> Patchwork_14706 ===

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Select DPLL's via mask (rev2)

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915: Select DPLL's via mask (rev2) URL : https://patchwork.freedesktop.org/series/67740/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7035 -> Patchwork_14707 Summary --- **FAILURE**

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/selftests: Assign the intel_runtime_pm pointer for mock_uncore

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/selftests: Assign the intel_runtime_pm pointer for mock_uncore URL : https://patchwork.freedesktop.org/series/67736/ State : success == Summary == CI Bug Log - changes from CI_DRM_7033_full -> Patchwork_14703_full ==

Re: [Intel-gfx] [PATCH 15/24] drm/i915: Try to make bigjoiner work in atomic check, v2.

2019-10-08 Thread Ville Syrjälä
On Fri, Oct 04, 2019 at 01:35:05PM +0200, Maarten Lankhorst wrote: > When the clock is higher than the dotclock, try with 2 pipes enabled. > If we can enable 2, then we will go into big joiner mode, and steal > the adjacent crtc. > > This only links the crtc's in software, no hardware or plane > p

Re: [Intel-gfx] [PATCH -next] treewide: remove unused argument in lock_release()

2019-10-08 Thread Peter Zijlstra
On Tue, Oct 08, 2019 at 06:33:51PM +0200, Daniel Vetter wrote: > On Thu, Sep 19, 2019 at 12:09:40PM -0400, Qian Cai wrote: > > Since the commit b4adfe8e05f1 ("locking/lockdep: Remove unused argument > > in __lock_release"), @nested is no longer used in lock_release(), so > > remove it from all lock

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gt: Give engine->kernel_context distinct timeline lock classes

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915/gt: Give engine->kernel_context distinct timeline lock classes URL : https://patchwork.freedesktop.org/series/67748/ State : success == Summary == CI Bug Log - changes from CI_DRM_7035 -> Patchwork_14708

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Select DPLL's via mask (rev3)

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915: Select DPLL's via mask (rev3) URL : https://patchwork.freedesktop.org/series/67740/ State : success == Summary == CI Bug Log - changes from CI_DRM_7035 -> Patchwork_14709 Summary --- **SUCCESS**

Re: [Intel-gfx] [PATCH] drm/i915/gt: Give engine->kernel_context distinct timeline lock classes

2019-10-08 Thread Matthew Auld
On Tue, 8 Oct 2019 at 19:59, Chris Wilson wrote: > > Assign a separate lockclass to the perma-pinned timelines of the > kernel_context, such that we can use them from within the user timelines > should we ever need to inject GPU operations to fixup faults during > request construction. > > Signed-

[Intel-gfx] [PATCH] drm/i915: Catch GTT fault errors for gen11+ planes

2019-10-08 Thread Matt Roper
Gen11+ has more hardware planes than gen9 so we need to test additional pipe interrupt register bits to recognize any GTT faults that happen on these extra planes. Bspec: 50335 Signed-off-by: Matt Roper --- drivers/gpu/drm/i915/i915_irq.c | 4 +++- drivers/gpu/drm/i915/i915_reg.h | 8 2

Re: [Intel-gfx] ✓ Fi.CI.BAT: success for intel_memory_region bits

2019-10-08 Thread Chris Wilson
Quoting Patchwork (2019-10-08 19:17:44) > == Series Details == > > Series: intel_memory_region bits > URL : https://patchwork.freedesktop.org/series/67738/ > State : success > > == Summary == > > CI Bug Log - changes from CI_DRM_7035 -> Patchwork_14704 > ===

[Intel-gfx] Reconfigurable OA queries

2019-10-08 Thread Chris Wilson
This is Lionel's work to enable OA for Vulkan, greatly bastardised on top of the struct_mutex removal. It claims to be doing the right thing... -Chris ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listi

[Intel-gfx] [PATCH 4/9] drm/i915: add support for perf configuration queries

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin Listing configurations at the moment is supported only through sysfs. This might cause issues for applications wanting to list configurations from a container where sysfs isn't available. This change adds a way to query the number of configurations and their content throu

[Intel-gfx] [PATCH 9/9] drm/i915/perf: allow holding preemption on filtered ctx

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin We would like to make use of perf in Vulkan. The Vulkan API is much lower level than OpenGL, with applications directly exposed to the concept of command buffers (pretty much equivalent to our batch buffers). In Vulkan, queries are always limited in scope to a command buff

[Intel-gfx] [PATCH 3/9] drm/i915/perf: allow for CS OA configs to be created lazily

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin Here we introduce a mechanism by which the execbuf part of the i915 driver will be able to request that a batch buffer containing the programming for a particular OA config be created. We'll execute these OA configuration buffers right before executing a set of userspace

[Intel-gfx] [PATCH 6/9] drm/i915/perf: execute OA configuration from command stream

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin We haven't run into issues with programming the global OA/NOA registers configuration from CPU so far, but HW engineers actually recommend doing this from the command streamer. On TGL in particular one of the clock domain in which some of that programming goes might not be

[Intel-gfx] [PATCH 5/9] drm/i915/perf: implement active wait for noa configurations

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin NOA configuration take some amount of time to apply. That amount of time depends on the size of the GT. There is no documented time for this. For example, past experimentations with powergating configuration changes seem to indicate a 60~70us delay. We go with 500us as def

[Intel-gfx] [PATCH 2/9] drm/i915/perf: introduce a versioning of the i915-perf uapi

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin Reporting this version will help application figure out what level of the support the running kernel provides. v2: Add i915_perf_ioctl_version() (Chris) Signed-off-by: Lionel Landwerlin Reviewed-by: Chris Wilson --- drivers/gpu/drm/i915/i915_getparam.c | 4 driv

[Intel-gfx] [PATCH 1/9] drm/i915/perf: store the associated engine of a stream

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin We'll use this information later to verify that a client trying to reconfigure the stream does so on the right engine. Signed-off-by: Lionel Landwerlin --- drivers/gpu/drm/i915/i915_perf.c | 29 +++--- drivers/gpu/drm/i915/i915_perf_types.h |

[Intel-gfx] [PATCH 8/9] drm/i915: add a new perf configuration execbuf parameter

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin We want the ability to dispatch a set of command buffer to the hardware, each with a different OA configuration. To achieve this, we reuse a couple of fields from the execbuf2 struct (I CAN HAZ execbuf3?) to notify what OA configuration should be used for a batch buffer. T

[Intel-gfx] [PATCH 7/9] drm/i915: introduce a mechanism to extend execbuf2

2019-10-08 Thread Chris Wilson
From: Lionel Landwerlin We're planning to use this for a couple of new feature where we need to provide additional parameters to execbuf. v2: Check for invalid flags in execbuffer2 (Lionel) v3: Rename I915_EXEC_EXT -> I915_EXEC_USE_EXTENSIONS (Chris) Signed-off-by: Lionel Landwerlin Reviewed-

[Intel-gfx] [PATCH] drm/i915/tgl: Enable DDI/Port G

2019-10-08 Thread Khaled Almahallawy
In TGL there we are missing the initialization of port G. Do the same as for other ports. Signed-off-by: Khaled Almahallawy --- drivers/gpu/drm/i915/display/intel_bios.c | 4 drivers/gpu/drm/i915/display/intel_display.c | 6 ++ drivers/gpu/drm/i915/display/intel_display.h | 1 + d

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Catch GTT fault errors for gen11+ planes

2019-10-08 Thread Patchwork
== Series Details == Series: drm/i915: Catch GTT fault errors for gen11+ planes URL : https://patchwork.freedesktop.org/series/67752/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7036 -> Patchwork_14710 Summary ---

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/9] drm/i915/perf: store the associated engine of a stream

2019-10-08 Thread Patchwork
== Series Details == Series: series starting with [1/9] drm/i915/perf: store the associated engine of a stream URL : https://patchwork.freedesktop.org/series/67754/ State : warning == Summary == $ dim checkpatch origin/drm-tip f9a4f9fef858 drm/i915/perf: store the associated engine of a strea

  1   2   >