On 10/16/20 12:44 PM, Maarten Lankhorst wrote:
Instead of force unbinding and rebinding every time, we try to check
if our notifier seqcount is still correct when pages are bound. This
way we only rebind userptr when we need to, and prevent stalls.
Changes since v1:
- Missing mutex_unlock, rep
Let's define Maximum MST content streams up to four
generically which can be supported by modern display
controllers.
Cc: Sean Paul
Cc: Ramalingam C
Signed-off-by: Anshuman Gupta
---
include/drm/drm_hdcp.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/include/drm/
On 10/16/20 12:43 PM, Maarten Lankhorst wrote:
Instead of doing what we do currently, which will never work with
PROVE_LOCKING, do the same as AMD does, and something similar to
relocation slowpath. When all locks are dropped, we acquire the
pages for pinning. When the locks are taken, we trans
On 10/19/20 9:30 AM, Thomas Hellström (Intel) wrote:
On 10/16/20 12:43 PM, Maarten Lankhorst wrote:
Instead of doing what we do currently, which will never work with
PROVE_LOCKING, do the same as AMD does, and something similar to
relocation slowpath. When all locks are dropped, we acquire the
Op 19-10-2020 om 09:30 schreef Thomas Hellström (Intel):
>
> On 10/16/20 12:43 PM, Maarten Lankhorst wrote:
>> Instead of doing what we do currently, which will never work with
>> PROVE_LOCKING, do the same as AMD does, and something similar to
>> relocation slowpath. When all locks are dropped, we
== Series Details ==
Series: drm/hdcp: Max MST content streams
URL : https://patchwork.freedesktop.org/series/82806/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9161 -> Patchwork_18724
Summary
---
**SUCCESS**
No
In switching to using objects for our ppGTT scratch pages, care was not
taken to avoid trying to unref NULL objects on failure. And for gen6
ppGTT, it appears we forgot entirely to unwind after a partial allocation
failure.
Fixes: 89351925a477 ("drm/i915/gt: Switch to object allocations for page
== Series Details ==
Series: drm/i915/gt: Onion unwind for scratch page allocation failure
URL : https://patchwork.freedesktop.org/series/82811/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9161 -> Patchwork_18725
Summary
On 2020.10.16 17:19:19 +0200, Stefan Fritsch wrote:
> Hi,
>
> if Linux is running as a guest and the host is doing igd-pass-thorugh with
> VT-d enabled, the i915 driver does not work all that great. The most
> obvious problem is that there are dozens of 'Fault errors on pipe A'
> errrors logged
== Series Details ==
Series: drm/hdcp: Max MST content streams
URL : https://patchwork.freedesktop.org/series/82806/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9161_full -> Patchwork_18724_full
Summary
---
**FAILU
Quoting Zhenyu Wang (2020-10-19 10:19:09)
> On 2020.10.16 17:19:19 +0200, Stefan Fritsch wrote:
> > Hi,
> >
> > if Linux is running as a guest and the host is doing igd-pass-thorugh with
> > VT-d enabled, the i915 driver does not work all that great. The most
> > obvious problem is that there ar
Hi, Chris
Looks this one is still missed to push? Could you help?
Thanks
On 2020.08.19 16:32:59 +0800, Zhenyu Wang wrote:
>
> ping for committing?
>
> On 2020.08.12 10:06:31 +0800, Zhenyu Wang wrote:
> > On 2020.08.11 10:25:32 +0100, Chris Wilson wrote:
> > > The "mmio" writes into vgpu regis
If i915.ko is being used as a passthrough device, it does not know if
the host is using intel_iommu. Mixing the iommu and gfx causing a few
issues (such as scanout overfetch) which we need to workaround inside
the driver, so if we detect we are running under a hypervisor, also
assume the device acc
On 2020.10.19 10:57:38 +0100, Chris Wilson wrote:
> Quoting Zhenyu Wang (2020-10-19 10:19:09)
> > On 2020.10.16 17:19:19 +0200, Stefan Fritsch wrote:
> > > Hi,
> > >
> > > if Linux is running as a guest and the host is doing igd-pass-thorugh
> > > with
> > > VT-d enabled, the i915 driver does no
If i915.ko is being used as a passthrough device, it does not know if
the host is using intel_iommu. Mixing the iommu and gfx causing a few
issues (such as scanout overfetch) which we need to workaround inside
the driver, so if we detect we are running under a hypervisor, also
assume the device acc
Quoting Zhenyu Wang (2020-10-19 10:51:44)
> On 2020.10.19 10:57:38 +0100, Chris Wilson wrote:
> > Quoting Zhenyu Wang (2020-10-19 10:19:09)
> > > On 2020.10.16 17:19:19 +0200, Stefan Fritsch wrote:
> > > > Hi,
> > > >
> > > > if Linux is running as a guest and the host is doing igd-pass-thorugh
>
On 2020.10.19 11:20:40 +0100, Chris Wilson wrote:
> Quoting Zhenyu Wang (2020-10-19 10:51:44)
> > On 2020.10.19 10:57:38 +0100, Chris Wilson wrote:
> > > Quoting Zhenyu Wang (2020-10-19 10:19:09)
> > > > On 2020.10.16 17:19:19 +0200, Stefan Fritsch wrote:
> > > > > Hi,
> > > > >
> > > > > if Linux
Add support for multiple mst stream in hdcp port data
which will be used by RepeaterAuthStreamManage msg and
HDCP 2.2 security f/w for m' validation.
v2:
Init the hdcp port data k for HDMI/DP SST strem.
Cc: Ramalingam C
Signed-off-by: Anshuman Gupta
---
.../drm/i915/display/intel_display_types
On 19/10/2020 09:34, Chris Wilson wrote:
In switching to using objects for our ppGTT scratch pages, care was not
taken to avoid trying to unref NULL objects on failure. And for gen6
ppGTT, it appears we forgot entirely to unwind after a partial allocation
failure.
Fixes: 89351925a477 ("drm/i915/
== Series Details ==
Series: drm/i915: Force VT'd workarounds when running as a guest OS (rev2)
URL : https://patchwork.freedesktop.org/series/82814/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9162 -> Patchwork_18726
Sum
Hi Jose,
I use scripts/checkpatch.pl. And it has reported me following :
drm-tip# ./scripts/checkpatch.pl
0001-drm-i915-edp-jsl-Update-vswing-table-for-HBR-and-HBR.patch
WARNING: else is not generally useful after a break or return
#88: FILE: drivers/gpu/drm/i915/display/intel_ddi.c:1222:
+
== Series Details ==
Series: HDCP 2.2 DP MST Support (rev3)
URL : https://patchwork.freedesktop.org/series/81538/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
d159760a4a56 drm/i915/hdcp: Update CP property in update_pipe
e2cb41b29aef drm/i915/hotplug: Handle CP_IRQ for DP-MST
== Series Details ==
Series: HDCP 2.2 DP MST Support (rev3)
URL : https://patchwork.freedesktop.org/series/81538/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
-
+./drivers/gpu/drm/amd/amdgpu/../
In order to test how fast the heartbeat can respond, we measure with the
interval set to its minimum. Before we measure though, we want to be
sure we start with a fresh pulse, and so wait until any old one is
complete. During that wait though, we were continually flushing the
work, and so continual
== Series Details ==
Series: drm/i915/gt: Onion unwind for scratch page allocation failure
URL : https://patchwork.freedesktop.org/series/82811/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9161_full -> Patchwork_18725_full
On Fri, 16 Oct 2020, "Navare, Manasi" wrote:
> I really want to push back on any design changes at this point since we have
> a working solution
> and we currently dont have any special cases. Like I discussed with you all
> the commit modeset enables
> code is now generic and similar to 2p2p.
>
== Series Details ==
Series: HDCP 2.2 DP MST Support (rev3)
URL : https://patchwork.freedesktop.org/series/81538/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9162 -> Patchwork_18727
Summary
---
**FAILURE**
Serio
== Series Details ==
Series: drm/i915/selftests: Flush the old heartbeat more gently
URL : https://patchwork.freedesktop.org/series/82815/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9162 -> Patchwork_18728
Summary
--
Instead of sharing pages with breadcrumbs, give each timeline a
single page. This allows unrelated timelines not to share locks
any more during command submission.
As an additional benefit, seqno wraparound no longer requires
i915_vma_pin, which means we no longer need to worry about a
potential -
== Series Details ==
Series: drm/i915: Remove obj->mm.lock! (rev9)
URL : https://patchwork.freedesktop.org/series/82337/
State : failure
== Summary ==
CALLscripts/checksyscalls.sh
CALLscripts/atomic/check-atomics.sh
DESCEND objtool
CHK include/generated/compile.h
CC [M] d
Instead of sharing pages with breadcrumbs, give each timeline a
single page. This allows unrelated timelines not to share locks
any more during command submission.
As an additional benefit, seqno wraparound no longer requires
i915_vma_pin, which means we no longer need to worry about a
potential -
== Series Details ==
Series: drm/i915: Remove obj->mm.lock! (rev10)
URL : https://patchwork.freedesktop.org/series/82337/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
f559c8e6c1e1 drm/i915: Move cmd parser pinning to execbuffer
fd1c6790339e drm/i915: Add missing -EDEADLK handl
== Series Details ==
Series: drm/i915: Remove obj->mm.lock! (rev10)
URL : https://patchwork.freedesktop.org/series/82337/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
-
+drivers/gpu/drm/i915/gt/
== Series Details ==
Series: drm/i915: Remove obj->mm.lock! (rev10)
URL : https://patchwork.freedesktop.org/series/82337/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
./drivers/gpu/drm/i915/gem/i915_gem_shrinker.c:102: warning: Function parameter
or member 'ww' n
I think we should do both. Any log message that can be triggered 50
times per second in practice should be rate limited. Also, the rate
limiting is probably a candidate for backporting to stable kernels while
the real fix may be not.
PS: See also my other mail about "drm/i915: Detecting Vt-d w
On 17/08/2020 10:15, Allen Pais wrote:
From: Allen Pais
In preparation for unconditionally passing the
struct tasklet_struct pointer to all tasklet
callbacks, switch to using the new tasklet_setup()
and from_tasklet() to pass the tasklet pointer explicitly.
Signed-off-by: Romain Perier
Sig
Works for me and makes the fault messages disappear. Thanks.
Tested-by: Stefan Fritsch
On Mon, 19 Oct 2020, Chris Wilson wrote:
> If i915.ko is being used as a passthrough device, it does not know if
> the host is using intel_iommu. Mixing the iommu and gfx causing a few
> issues (such as scan
== Series Details ==
Series: drm/i915: Remove obj->mm.lock! (rev10)
URL : https://patchwork.freedesktop.org/series/82337/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9162 -> Patchwork_18730
Summary
---
**SUCCESS**
== Series Details ==
Series: drm/i915: Force VT'd workarounds when running as a guest OS (rev2)
URL : https://patchwork.freedesktop.org/series/82814/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9162_full -> Patchwork_18726_full
===
Hi Andrzej,
On Sat, Oct 17, 2020 at 01:34:54PM +0200, Andrzej Kre wrote:
> Hi,
>
> I know that You were involved in working on USB-C DP drivers.
> You are my last chance to resolve my issue.
>
> On my HP laptop I have Intel UHD Graphics 620.
> When I'm connecting my 4K monitor to Display Port. I
In order to test how fast the heartbeat can respond, we measure with the
interval set to its minimum. Before we measure though, we want to be
sure we start with a fresh pulse, and so wait until any old one is
complete. During that wait though, we were continually flushing the
work, and so continual
From: Ville Syrjälä
PIPECONF becamse TRANSCONF when HSW introduced the EDP transcoder.
Bigjoiner is making life even more confusing by introducing
a N:1 relationship between pipes and transcoders. In that case
we only enable/configure the transcoder corresponding to the
master pipe. Let's do some
== Series Details ==
Series: drm/i915/selftests: Flush the old heartbeat more gently (rev2)
URL : https://patchwork.freedesktop.org/series/82815/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9162 -> Patchwork_18731
Summary
On Fri, Oct 16, 2020 at 10:47:59PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Rather that try to trick LSPCON back into PCON mode from the .reset()
> hook let's just do that as a regular part of the normal modeset
> sequence, which is going to take care of the system resume case. Durin
On Fri, Oct 16, 2020 at 01:18:37PM -0700, José Roberto de Souza wrote:
> This sequence is not part of "Sequences to Initialize Display" but
> as noted in the MBus page the DBUF_CTL.Tracker_state_service needs
> to be set to 8.
>
> BSpec: 49213
> Signed-off-by: José Roberto de Souza
> ---
> drive
On Fri, Oct 16, 2020 at 10:48:00PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Rename intel_dp_sink_dpms() to intel_dp_set_power()
> so one doesn't always have to convert from the DPMS
> enum values to the actual DP D-states.
>
> Also when dealing with a branch device this has nothing
== Series Details ==
Series: drm/i915: Remove obj->mm.lock! (rev10)
URL : https://patchwork.freedesktop.org/series/82337/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9162_full -> Patchwork_18730_full
Summary
---
**
On Mon, Oct 19, 2020 at 06:27:01PM +0300, Imre Deak wrote:
> On Fri, Oct 16, 2020 at 10:47:59PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Rather that try to trick LSPCON back into PCON mode from the .reset()
> > hook let's just do that as a regular part of the normal modeset
> >
On Tue, Oct 13, 2020 at 09:11:37PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Currently we call .hpd_irq_setup() directly just before display
> resume, and follow it with another call via intel_hpd_init()
> just afterwards. Assuming the hpd pins are marked as enabled
> during the open-
On Tue, Oct 06, 2020 at 09:58:08PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> LSPCON requires HPD detection logic to be enabled when we call
> its .reset() hook during resume, to check the live state of the
> pin. To that end let's reorder the resume sequence such that
> we do HPD init
== Series Details ==
Series: drm/i915: s/pipe/transcoder/ when dealing with PIPECONF/TRANSCONF
URL : https://patchwork.freedesktop.org/series/82831/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9162 -> Patchwork_18732
Summ
On Mon, Oct 19, 2020 at 06:34:34PM +0300, Ville Syrjälä wrote:
> On Mon, Oct 19, 2020 at 06:27:01PM +0300, Imre Deak wrote:
> > On Fri, Oct 16, 2020 at 10:47:59PM +0300, Ville Syrjala wrote:
> > > From: Ville Syrjälä
> > >
> > > Rather that try to trick LSPCON back into PCON mode from the .reset(
On Mon, Oct 19, 2020 at 06:39:59PM +0300, Imre Deak wrote:
> On Tue, Oct 06, 2020 at 09:58:08PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > LSPCON requires HPD detection logic to be enabled when we call
> > its .reset() hook during resume, to check the live state of the
> > pin. T
I fixed this problem partially (not gen6 code) with patch:
http://intel-gfx-pw.fi.intel.com/series/5911/
Should I continue to fix on internal?
--CQ
> -Original Message-
> From: Intel-gfx On Behalf Of
> Chris Wilson
> Sent: Monday, October 19, 2020 1:35 AM
> To: intel-gfx@lists.freedesk
On Thu, Oct 08, 2020 at 02:45:33PM -0700, Manasi Navare wrote:
> From: Maarten Lankhorst
>
> We need to look at hw.fb for the framebuffer, and add the translation
> for the slave_plane_state. With these changes we set the correct
> rectangle on the bigjoiner slave, and don't set incorrect
> src/d
== Series Details ==
Series: drm/i915/selftests: Flush the old heartbeat more gently (rev2)
URL : https://patchwork.freedesktop.org/series/82815/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9162_full -> Patchwork_18731_full
===
On Thu, Oct 08, 2020 at 02:45:34PM -0700, Manasi Navare wrote:
> From: Maarten Lankhorst
>
> Enabling is done in a special sequence and so should plane updates
> be. Ideally the end user never notices the second pipe is used.
>
> This way ideally everything will be tear free, and updates are
> r
On Thu, Oct 15, 2020 at 09:26:45AM -0700, Navare, Manasi wrote:
> On Thu, Oct 15, 2020 at 02:52:47PM +0300, Ville Syrjälä wrote:
> > On Wed, Oct 14, 2020 at 12:04:10PM -0700, Navare, Manasi wrote:
> > > On Wed, Oct 14, 2020 at 02:26:34PM +0300, Ville Syrjälä wrote:
> > > > On Thu, Oct 08, 2020 at 0
On Thu, Oct 08, 2020 at 02:45:31PM -0700, Manasi Navare wrote:
> From: Maarten Lankhorst
>
> Unfortunately I have no way to test this, but it should be correct
> if the bios sets up bigjoiner in a sane way.
>
> Skip iterating over bigjoiner slaves, only the master has the state we
> care about.
The GPU is trashing the low pages of its reserved memory upon reset. If
we are using this memory for ringbuffers, then we will dutiful resubmit
the trashed rings after the reset causing further resets, and worse. We
must exclude this range from our own use. The value of 128KiB was found
by empirica
When allocating objects from stolen, memset() the backing store to
POISON_INUSE (0x5a) to help identify any uninitialised use of a stolen
object.
Signed-off-by: Chris Wilson
Reviewed-by: Mika Kuoppala
---
drivers/gpu/drm/i915/gem/i915_gem_stolen.c | 45 ++
1 file changed, 45
Reviewed-by: Lyude Paul
On Tue, 2020-10-13 at 21:11 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Currently we call .hpd_irq_setup() directly just before display
> resume, and follow it with another call via intel_hpd_init()
> just afterwards. Assuming the hpd pins are marked as enabled
I don't think that checkpatch.pl has all the style rules that we follow in drm.
Check the output of "✗ Fi.CI.CHECKPATCH: warning for drm/i915/edp/jsl: Update
vswing table for HBR and HBR2 (rev2)" there more style errors than that.
On Mon, 2020-10-19 at 11:08 +, Surendrakumar Upadhyay, Tejasku
== Series Details ==
Series: drm/i915: s/pipe/transcoder/ when dealing with PIPECONF/TRANSCONF
URL : https://patchwork.freedesktop.org/series/82831/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9162_full -> Patchwork_18732_full
== Series Details ==
Series: series starting with [1/2] drm/i915/gem: Poison stolen pages before use
URL : https://patchwork.freedesktop.org/series/82834/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
650f2e2f5bdd drm/i915/gem: Poison stolen pages before use
-:38: WARNING:MEMOR
== Series Details ==
Series: series starting with [1/2] drm/i915/gem: Poison stolen pages before use
URL : https://patchwork.freedesktop.org/series/82834/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separa
On Mon, 2020-10-19 at 08:31 -0700, Matt Roper wrote:
> On Fri, Oct 16, 2020 at 01:18:37PM -0700, José Roberto de Souza wrote:
> > This sequence is not part of "Sequences to Initialize Display" but
> > as noted in the MBus page the DBUF_CTL.Tracker_state_service needs
> > to be set to 8.
> >
> > BS
This sequence is not part of "Sequences to Initialize Display" but
as noted in the MBus page the DBUF_CTL.Tracker_state_service needs
to be set to 8.
BSpec: 49213
Reviewed-by: Matt Roper
Signed-off-by: José Roberto de Souza
---
drivers/gpu/drm/i915/display/intel_display_power.c | 14 +++
== Series Details ==
Series: series starting with [1/2] drm/i915/gem: Poison stolen pages before use
URL : https://patchwork.freedesktop.org/series/82834/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9163 -> Patchwork_18733
Hi Chris and Ville,
Failed to cherry-pick:
233c1ae3c83f ("drm/i915/gt: Wait for CSB entries on Tigerlake")
2c1e63bab430 ("drm/i915: Mark initial fb obj as WT on eLLC machines to avoid
rcu lockup during fbdev init")
These patches look important to 5.10. If this is the case,
please consider provid
>-Original Message-
>From: Maarten Lankhorst
>Sent: Monday, October 12, 2020 10:14 AM
>To: Ruhl, Michael J ; intel-
>g...@lists.freedesktop.org
>Subject: Re: [Intel-gfx] [PATCH 10/61] drm/i915: Disable userptr pread/pwrite
>support.
>
>Op 02-10-2020 om 22:14 schreef Ruhl, Michael J:
>>> -
Underruns happens when plane height + y offset is not a module of 4
when FBC is enabled.
Specification says that it only affects TGL display C stepping and
newer but to simply the check and as TGL is already in final costumers
hands, pre-production display stepping A and B was also included.
BSpe
== Series Details ==
Series: drm/i915/display: Program DBUF_CTL tracker state service (rev2)
URL : https://patchwork.freedesktop.org/series/82785/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9163 -> Patchwork_18734
Summar
Hi Jose,
root@tejas-System-Product-Name:/home/tejas/all-external/drm-tip# ./dim
checkpatch
a11736f058e7 (HEAD -> latest) drm/i915/edp/jsl: Update vswing table for HBR and
HBR2
-:58: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#58: FILE: drivers/gpu/drm/i915/display/intel
Yes
On Mon, 2020-10-19 at 18:13 +, Surendrakumar Upadhyay, TejaskumarX wrote:
> Hi Jose,
>
> root@tejas-System-Product-Name:/home/tejas/all-external/drm-tip# ./dim
> checkpatch
> a11736f058e7 (HEAD -> latest) drm/i915/edp/jsl: Update vswing table for HBR
> and HBR2
> -:58: CHECK:PARENTHESIS
Quoting Rodrigo Vivi (2020-10-19 18:53:10)
> Hi Chris and Ville,
>
> Failed to cherry-pick:
> 233c1ae3c83f ("drm/i915/gt: Wait for CSB entries on Tigerlake")
See https://cgit.freedesktop.org/~ickle/linux-2.6/log/?h=dif for the
dependencies.
-Chris
___
I
I just checked the CI picture and it looks much better indeed.
Only bad case being the gt_pm, which is also failing on other platforms.
So,
Acked-by: Rodrigo Vivi
On Thu, Oct 15, 2020 at 08:41:32AM +, K, SrinivasX wrote:
> Hi Hariom,
>
> With Sunil's help was able to see EHL achieving
On Thu, Oct 15, 2020 at 08:53:08AM +, Kamati Srinivas wrote:
> Removing force probe protection from JSL platform. Did
> not observe warnings, errors, flickering or any visual
> defects while doing ordinary tasks like browsing and
> editing documents in a two monitor setup.
I'm afraid we first
Quoting Rodrigo Vivi (2020-10-19 19:29:36)
>
> I just checked the CI picture and it looks much better indeed.
>
> Only bad case being the gt_pm, which is also failing on other platforms.
Not nearly in the same manner. CI is indicating that there is no RC6
entry and no power saving at all; neithe
On Mon, Oct 19, 2020 at 07:18:32PM +0100, Chris Wilson wrote:
> Quoting Rodrigo Vivi (2020-10-19 18:53:10)
> > Hi Chris and Ville,
> >
> > Failed to cherry-pick:
> > 233c1ae3c83f ("drm/i915/gt: Wait for CSB entries on Tigerlake")
>
> See https://cgit.freedesktop.org/~ickle/linux-2.6/log/?h=dif fo
== Series Details ==
Series: drm/i915/display/fbc: Implement WA 22010751166
URL : https://patchwork.freedesktop.org/series/82836/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9163 -> Patchwork_18735
Summary
---
**SU
== Series Details ==
Series: series starting with [1/2] drm/i915/gem: Poison stolen pages before use
URL : https://patchwork.freedesktop.org/series/82834/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9163_full -> Patchwork_18733_full
==
On Mon, Oct 19, 2020 at 10:56:09AM -0700, José Roberto de Souza wrote:
> Underruns happens when plane height + y offset is not a module of 4
I think you meant 'modulo' rather than 'module' here.
> when FBC is enabled.
Technically the workaround calls for us to just do our FBC disables for
non-mo
Avoid skipping what appears to be a no-op set-domain-ioctl if the cache
coherency state is inconsistent with our target domain. This also has
the utility of using the population of the pages to validate the backing
store.
The danger in skipping the first set-domain is leaving the cache
inconsisten
== Series Details ==
Series: drm/i915/display: Program DBUF_CTL tracker state service (rev2)
URL : https://patchwork.freedesktop.org/series/82785/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9163_full -> Patchwork_18734_full
==
== Series Details ==
Series: drm/i915/gem: Flush coherency domains on first set-domain-ioctl
URL : https://patchwork.freedesktop.org/series/82843/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9165 -> Patchwork_18736
Summar
From: Ville Syrjälä
We dropped the other redundant master_transcoder assignments
earlier, but this one slipped through. Get rid of it as well.
The crtc state gets fully reset before readout so there is
no point in doing this.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_
From: Ville Syrjälä
Pass the crtc+cpu_transcoder rather than the crtc state to
intel_dsc_power_domain(). This should allow us to reuse it
during readout as well.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/icl_dsi.c| 3 +-
drivers/gpu/drm/i915/display/intel_ddi.c | 3 +
From: Ville Syrjälä
Add the _if_enabled() counterpart to with_intel_display_power().
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display_power.h | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/i915/display/intel_display_power.h
b/drivers/gpu/drm
From: Ville Syrjälä
Extract the "panel transcoder" bitmask into a helper. We'll
have a couple of uses for this later.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display.c | 16 +++-
1 file changed, 11 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/
From: Ville Syrjälä
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_ddi.c | 7 +++
drivers/gpu/drm/i915/display/intel_display.c | 14 ++
2 files changed, 21 insertions(+)
diff --git a/drivers/gpu/drm/i915/display/intel_ddi.c
b/drivers/gpu/drm/i915/disp
From: Ville Syrjälä
Adjust the HSW+ transcoder state readout to just read through
all the possible transcoders for the pipe, and stuff the results
in a bitmask.
We can conveniently cross check the bitmask for invalid
combinations of enabled transcoders, and later we can easily
extend the bitmask
From: Ville Syrjälä
Read out cpu_transcoder correctly for the bigjoiner slave pipes.
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_display.c | 53
1 file changed, 53 insertions(+)
diff --git a/drivers/gpu/drm/i915/display/intel_display.c
b/drivers/g
== Series Details ==
Series: drm/i915/display/fbc: Implement WA 22010751166
URL : https://patchwork.freedesktop.org/series/82836/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_9163_full -> Patchwork_18735_full
Summary
-
== Series Details ==
Series: series starting with [1/7] drm/i915: Drop one more usless
master_transcoder assignment
URL : https://patchwork.freedesktop.org/series/82848/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
5e9c0a3f80bf drm/i915: Drop one more usless master_transcoder
== Series Details ==
Series: series starting with [1/7] drm/i915: Drop one more usless
master_transcoder assignment
URL : https://patchwork.freedesktop.org/series/82848/
State : warning
== Summary ==
$ dim sparse --fast origin/drm-tip
Sparse version: v0.6.2
Fast mode used, each commit won't b
== Series Details ==
Series: series starting with [1/7] drm/i915: Drop one more usless
master_transcoder assignment
URL : https://patchwork.freedesktop.org/series/82848/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9165 -> Patchwork_18737
On Tue, Oct 20, 2020 at 12:43:31AM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
> We dropped the other redundant master_transcoder assignments
> earlier, but this one slipped through. Get rid of it as well.
> The crtc state gets fully reset before readout so there is
> no point in doing th
On Mon, Oct 19, 2020 at 07:36:59PM +0300, Ville Syrjälä wrote:
> On Thu, Oct 08, 2020 at 02:45:31PM -0700, Manasi Navare wrote:
> > From: Maarten Lankhorst
> >
> > Unfortunately I have no way to test this, but it should be correct
> > if the bios sets up bigjoiner in a sane way.
> >
> > Skip ite
On Mon, Oct 19, 2020 at 07:30:07PM +0300, Ville Syrjälä wrote:
> On Thu, Oct 15, 2020 at 09:26:45AM -0700, Navare, Manasi wrote:
> > On Thu, Oct 15, 2020 at 02:52:47PM +0300, Ville Syrjälä wrote:
> > > On Wed, Oct 14, 2020 at 12:04:10PM -0700, Navare, Manasi wrote:
> > > > On Wed, Oct 14, 2020 at 0
1 - 100 of 113 matches
Mail list logo
