On 02/10/2019 12:19, Chris Wilson wrote:
Since we cannot allocate underneath the vm->mutex (it is used in the
direct-reclaim paths), we need to shift the allocations off into a
mutexless worker with fence recursion prevention. To know when we need
this protection, we mark up the address spaces t
On 02/10/2019 12:19, Chris Wilson wrote:
Forgo the struct_mutex serialisation for i915_active, and interpose its
own mutex handling for active/retire.
This is a multi-layered sleight-of-hand. First, we had to ensure that no
active/retire callbacks accidentally inverted the mutex ordering rules,
On 02/10/2019 12:19, Chris Wilson wrote:
Now that we can retire without taking struct_mutex, we can do so to
handle shrinking the mmap-offset space after an allocation failure.
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
Cc: Matthew Auld
---
drivers/gpu/drm/i915/gem/i915_gem_mman.c | 17
On 02/10/2019 12:19, Chris Wilson wrote:
Requests are run from the gt and are tided into the gt runtime power
management, so pull the runtime request management under gt/
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/Makefile | 1 +
drivers/gpu/drm/i915/gem/i915_gem
Quoting Tvrtko Ursulin (2019-10-02 16:45:18)
>
> On 02/10/2019 13:48, Janusz Krzysztofik wrote:
> > If we process DROP_RESET_ACTIVE and cancel all outstanding requests by
> > forcing a GPU reset on a hardware with reset capabilities disabled or
> > not supported, we certainly end up with a termina
If we have a wedged GPU that we need to recover, but fail, add a taint
for CI to pickup and schedule a reboot.
Signed-off-by: Chris Wilson
Cc: Tvrtko Ursulin
Cc: Petri Latvala
---
drivers/gpu/drm/i915/gt/intel_reset.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/d
On 02/10/2019 12:19, Chris Wilson wrote:
With the introduction of ctx->engines[] we allow multiple logical
contexts to be used on the same engine (e.g. with virtual engines).
According to bspec, aach logical context requires a unique tag in order
for context-switching to occur correctly between
Ping on some comments from the previous round:
On 26/09/2019 14:57, Tvrtko Ursulin wrote:
On 25/09/2019 11:01, Chris Wilson wrote:
Keep track of the GEM contexts underneath i915->gem.contexts and assign
them their own lock for the purposes of list management.
Signed-off-by: Chris Wilson
Cc:
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, Thierry Reding wrote:
> > >> From: Thierry Reding
> > >>
> > >> Hi,
> > >>
== Series Details ==
Series: DC3CO Support for TGL (rev15)
URL : https://patchwork.freedesktop.org/series/64923/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6987_full -> Patchwork_14624_full
Summary
---
**FAILURE**
From: Ville Syrjälä
There are no longer any pipe<->DSI port limitations on icl+.
Populate the pipe_mask accordingly.
Cc: José Roberto de Souza
Cc: Lucas De Marchi
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/icl_dsi.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
From: Ville Syrjälä
Just set pipe_mask=~0 for the non-special cases where any pipe
will do. intel_encoder_possible_crtcs() will anyway drop out
anything that doesn't exist.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/icl_dsi.c | 4 +---
drivers/gpu/drm/i915/display/intel
From: Ville Syrjälä
We don't need to special case PCH vs. gen4 when setting up the LVDS
crtc_mask. Just claim pipes A|B|C work and
intel_encoder_possible_crtcs() will drop out any crtc that doesn't
exist.
v2: Put the special case first to match what most other encoders do
Signed-off-by: Ville S
From: Ville Syrjälä
Rename the encoder->crtc_mask to encoder->pipe_mask to better
reflect what it actually contains.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/icl_dsi.c | 2 +-
drivers/gpu/drm/i915/display/intel_crt.c | 4 ++--
drivers/gpu/drm/i915/dis
From: Ville Syrjälä
Replace the hand rolled stuff with drm_encoder_mask() when populating
possible_clones, and rename the function to
intel_encoder_possible_clones() to make it clear what it's used for.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display.c | 13 +---
From: Ville Syrjälä
To avoid accidentally breaking things in the future add a
comment explaining why we misconfigure the pipe_mask.
Also toss in a TODO for investigating a single encoder
approach as opposed to the encoder-per-pipe approach.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i91
== Series Details ==
Series: drm/i915/gem: Refactor tests on obj->ops->flags
URL : https://patchwork.freedesktop.org/series/67491/
State : failure
== Summary ==
Applying: drm/i915/gem: Refactor tests on obj->ops->flags
Using index info to reconstruct a base tree...
M drivers/gpu/drm/i915
On Wed, Oct 02, 2019 at 07:25:03PM +0300, Ville Syrjälä wrote:
From: Ville Syrjälä
There are no longer any pipe<->DSI port limitations on icl+.
Populate the pipe_mask accordingly.
Cc: José Roberto de Souza
Cc: Lucas De Marchi
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/icl
Hey,
On Wed, 2 Oct 2019, Imre Deak wrote:
> On Tue, Oct 01, 2019 at 07:35:54PM +0300, Kai Vehmanen wrote:
> > On gen10/11 platforms, driver must set the enable bit of AUD_PIN_BUF_CTL
> > as part of audio power up sequence.
>
> Good catch, seems to match bspec 21352 (and 49280 for GEN12+).
>
> B
== Series Details ==
Series: drm/i915/debugfs: Only wedge if we have reset available
URL : https://patchwork.freedesktop.org/series/67492/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6991 -> Patchwork_14628
Summary
--
== Series Details ==
Series: series starting with [1/4] drm/i915/vga: rename intel_vga_msr_write()
to intel_vga_reset_io_mem()
URL : https://patchwork.freedesktop.org/series/67493/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
2436f7c1a72f drm/i915/vga: rename intel_vga_msr_wr
On 2019-10-02 at 16:21:59 +, Patchwork wrote:
> == Series Details ==
>
> Series: DC3CO Support for TGL (rev15)
> URL : https://patchwork.freedesktop.org/series/64923/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_6987_full -> Patchwork_14624_full
> ==
On Mon, 2019-09-30 at 09:29 +0200, Peter Zijlstra 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_relea
== Series Details ==
Series: series starting with [1/4] drm/i915/vga: rename intel_vga_msr_write()
to intel_vga_reset_io_mem()
URL : https://patchwork.freedesktop.org/series/67493/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6991 -> Patchwork_14629
=
On Wed, Oct 2, 2019 at 9:25 AM Ville Syrjala
wrote:
>
> From: Ville Syrjälä
>
> Rename the encoder->crtc_mask to encoder->pipe_mask to better
> reflect what it actually contains.
>
> Signed-off-by: Ville Syrjälä
Reviewed-by: Lucas De Marchi
but... aren't they synonym? Like crtc means nothing
== Series Details ==
Series: drm/i915/selftests: Exercise potential false lite-restore (rev8)
URL : https://patchwork.freedesktop.org/series/67438/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6991 -> Patchwork_14630
Summa
If execlists's lite-restore is based on the common GEM context tag
rather than the per-intel_context LRCA, then a context switch between
two intel_contexts on the same engine derived from the same GEM context
will perform a lite-restore instead of a full context switch. We can
exploit this by poiso
On Wed, 2019-10-02 at 10:23 +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915/mg: Use tc_port instead of port parameter to MG
> registers
> URL : https://patchwork.freedesktop.org/series/67467/
> State : success
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_6986_full
== Series Details ==
Series: drm/drm_vblank: Change EINVAL by the correct errno (rev6)
URL : https://patchwork.freedesktop.org/series/51147/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6991 -> Patchwork_14631
Summary
The latest documented version of the VBT is 229, but no further data has
been added to the child device definition in block 2. Update the child
device version test to eliminate the "Expected child device config size
for VBT version XXX not known; assuming 39" debug messages from the
logs.
Bspec:
== Series Details ==
Series: drm/i915/tgl: Add the Thunderbolt PLL divider values
URL : https://patchwork.freedesktop.org/series/67498/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
8dc00b06f596 drm/i915/tgl: Add the Thunderbolt PLL divider values
-:26: CHECK:CAMELCASE: Avoid C
== Series Details ==
Series: i915/gem_exec_schedule: Replace vgem fence with sync_file
URL : https://patchwork.freedesktop.org/series/67486/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6988_full -> IGTPW_3528_full
Summary
On Wed, 2019-10-02 at 17:08 +0300, Imre Deak wrote:
> The Thunderbolt PLL divider values on TGL differ from the ICL ones,
> update the PLL parameter calculation function accordingly.
>
> Bspec: 49204
>
> Cc: Jose Souza
> Cc: Clinton A Taylor
> Cc: Lucas De Marchi
> Cc: Mika Westerberg
> Teste
== Series Details ==
Series: drm/i915/tgl: Add the Thunderbolt PLL divider values
URL : https://patchwork.freedesktop.org/series/67498/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14632
Summary
---
== Series Details ==
Series: series starting with [01/30] drm/i915/selftests: Exercise potential
false lite-restore (rev2)
URL : https://patchwork.freedesktop.org/series/67483/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
5ab21c1fa058 drm/i915/selftests: Exercise potential fa
On Wed, Oct 02, 2019 at 11:07:21PM +0300, Souza, Jose wrote:
> On Wed, 2019-10-02 at 17:08 +0300, Imre Deak wrote:
> > The Thunderbolt PLL divider values on TGL differ from the ICL ones,
> > update the PLL parameter calculation function accordingly.
> >
> > Bspec: 49204
> >
> > Cc: Jose Souza
>
== Series Details ==
Series: series starting with [01/30] drm/i915/selftests: Exercise potential
false lite-restore (rev2)
URL : https://patchwork.freedesktop.org/series/67483/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Sparse version: v0.6.0
Commit: drm/i915/selftests: Exercis
== Series Details ==
Series: drm/i915/selftests: Extract random_offset() for use with a prng
URL : https://patchwork.freedesktop.org/series/67489/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6988_full -> Patchwork_14626_full
==
== Series Details ==
Series: series starting with [01/30] drm/i915/selftests: Exercise potential
false lite-restore (rev2)
URL : https://patchwork.freedesktop.org/series/67483/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14633
=
The Thunderbolt PLL divider values on TGL differ from the ICL ones,
update the PLL parameter calculation function accordingly.
Bspec: 49204
v2:
- Remove unused refclk config. (José)
Cc: Jose Souza
Cc: Clinton A Taylor
Cc: Lucas De Marchi
Cc: Mika Westerberg
Tested-by: Mika Westerberg
Signed
== Series Details ==
Series: drm/i915/dp: remove static variable for aux last status
URL : https://patchwork.freedesktop.org/series/67499/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14634
Summary
--
tree: git://anongit.freedesktop.org/drm-intel drm-intel-next-queued
head: cf82d9ddd3b5203dea5561dfabd1216b8159af37
commit: cf82d9ddd3b5203dea5561dfabd1216b8159af37 [4/4] drm/i915/tgl: Introduce
gen12 forcewake ranges
reproduce:
# apt-get install sparse
# sparse version: v0.6.1-
== Series Details ==
Series: series starting with [1/2] drm/i915: use DRM_ERROR() instead of
drm_err()
URL : https://patchwork.freedesktop.org/series/67500/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14635
=
(switched to email. Please respond via emailed reply-to-all, not via the
bugzilla web interface).
On Tue, 01 Oct 2019 17:06:35 + bugzilla-dae...@bugzilla.kernel.org wrote:
> https://bugzilla.kernel.org/show_bug.cgi?id=205065
>
> Bug ID: 205065
>Summary: workqueue: P
On Mon, Sep 23, 2019 at 03:29:28AM -0700, Dhinakaran Pandiyan wrote:
> intel_tile_dims() computes tile height using size and width, when there
> is already a function to do just that - intel_tile_height()
>
> Cc: Ville Syrjälä
> Cc: Matt Roper
> Signed-off-by: Dhinakaran Pandiyan
Reviewed-by:
On Mon, Sep 23, 2019 at 03:29:29AM -0700, Dhinakaran Pandiyan wrote:
> Easier to read if all the alignment changes are in one place and contained
> within a function.
>
> Cc: Ville Syrjälä
> Cc: Matt Roper
> Signed-off-by: Dhinakaran Pandiyan
> ---
> drivers/gpu/drm/i915/display/intel_display.
On Mon, Sep 23, 2019 at 03:29:30AM -0700, Dhinakaran Pandiyan wrote:
> Gen-12 display decompression operates on Y-tiled compressed main surface.
> The CCS is linear
I'd mention in the commit message that we opt to to treat the
CCS as if it were a bunch of 64x1 tiles.
There are a handful of cosmet
== Series Details ==
Series: drm/i915/gt: Warn CI about an unrecoverable wedge
URL : https://patchwork.freedesktop.org/series/67502/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14636
Summary
---
*
== Series Details ==
Series: series starting with [1/6] drm/i915: Polish possible_clones setup
URL : https://patchwork.freedesktop.org/series/67504/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14637
Summ
== Series Details ==
Series: drm/i915/selftests: Exercise potential false lite-restore (rev9)
URL : https://patchwork.freedesktop.org/series/67438/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14638
Summa
+ Chris and Tvrtko
Quoting Andrew Morton (2019-10-03 00:57:21)
>
> (switched to email. Please respond via emailed reply-to-all, not via the
> bugzilla web interface).
>
> On Tue, 01 Oct 2019 17:06:35 + bugzilla-dae...@bugzilla.kernel.org wrote:
>
> > https://bugzilla.kernel.org/show_bug.cg
Quoting Joonas Lahtinen (2019-10-03 00:28:43)
> + Chris and Tvrtko
It's a trivial warning, that's already fixed. In this case by separating
out the poweroff into process context.
c7302f204490 ("drm/i915: Defer final intel_wakeref_put to process context")
-Chris
== Series Details ==
Series: drm/i915/vbt: Child device size remains unchanged through VBT 229
URL : https://patchwork.freedesktop.org/series/67510/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6992 -> Patchwork_14639
Summ
== Series Details ==
Series: drm/i915/tgl: Add the Thunderbolt PLL divider values (rev2)
URL : https://patchwork.freedesktop.org/series/67498/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
1b4f080f283b drm/i915/tgl: Add the Thunderbolt PLL divider values
-:33: CHECK:CAMELCASE:
== Series Details ==
Series: drm/i915/tgl: Add the Thunderbolt PLL divider values (rev2)
URL : https://patchwork.freedesktop.org/series/67498/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6993 -> Patchwork_14640
Summary
--
== Series Details ==
Series: drm/i915/debugfs: Only wedge if we have reset available
URL : https://patchwork.freedesktop.org/series/67492/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6991_full -> Patchwork_14628_full
Summ
== Series Details ==
Series: drm/drm_vblank: Change EINVAL by the correct errno (rev6)
URL : https://patchwork.freedesktop.org/series/51147/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6991_full -> Patchwork_14631_full
Su
From: "Paul E. McKenney"
This commit replaces the use of rcu_swap_protected() with the more
intuitively appealing rcu_replace() as a step towards removing
rcu_swap_protected().
Link:
https://lore.kernel.org/lkml/CAHk-=wiAsJLw1egFEE=z7-ggtm6wcvtyytxza1+bhqta4gg...@mail.gmail.com/
Reported-by: Li
Hi Dave, Daniel,
I hope that you enjoy XDC if you could make it this year :)
Here's the first round of fixes for drm-misc
Maxime
drm-misc-fixes-2019-10-02:
- One include fix for tilcdc
- A memory leak fix for Komeda
- Some fixes for resources cleanups with writeback
The following changes sin
== Series Details ==
Series: series starting with [1/2] drm/i915: use DRM_ERROR() instead of
drm_err()
URL : https://patchwork.freedesktop.org/series/67500/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6992_full -> Patchwork_14635_full
===
== Series Details ==
Series: drm/i915/gt: Warn CI about an unrecoverable wedge
URL : https://patchwork.freedesktop.org/series/67502/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6992_full -> Patchwork_14636_full
Summary
--
On 2019-10-02 at 23:11:36 +0530, Anshuman Gupta wrote:
> On 2019-10-02 at 16:21:59 +, Patchwork wrote:
> > == Series Details ==
> >
> > Series: DC3CO Support for TGL (rev15)
> > URL : https://patchwork.freedesktop.org/series/64923/
> > State : failure
> >
> > == Summary ==
> >
> > CI Bug L
101 - 162 of 162 matches
Mail list logo
