In case of fused or unused pipes, return early with a warning when reading
information
for encoder.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103206
Reported-by: Thomas Gleixner
Signed-off-by: Jani Nikula
Signed-off-by: Mika Kahola
---
drivers/gpu/drm/i915/intel_audio.c | 2 +-
1
https://patchwork.freedesktop.org/series/35365/
Judging from PW results, THRESHOLD_TOTAL 95 is still not enough for KBL-shards.
That is interesting.
/Marta
> -Original Message-
> From: Lofstedt, Marta
> Sent: Thursday, December 14, 2017 5:44 PM
> To: intel-gfx@lists.freedesktop.org
> Cc
On Thu, 14 Dec 2017 23:30:56 +0100, Srivatsa, Anusha
wrote:
-Original Message-
From: Wajdeczko, Michal
Sent: Thursday, December 14, 2017 2:18 PM
To: intel-gfx@lists.freedesktop.org; Srivatsa, Anusha
Cc: Vivi, Rodrigo
Subject: Re: [Intel-gfx] [PATCH] drm/i915/glk: Disable Guc and
On 12/14/2017 03:43 AM, Joonas Lahtinen wrote:
On Wed, 2017-12-13 at 14:59 -0800, Yaodong Li wrote:
On 12/13/2017 01:34 PM, Michal Wajdeczko wrote:
On Wed, 13 Dec 2017 19:19:06 +0100, Yaodong Li
wrote:
On 12/13/2017 01:11 AM, Joonas Lahtinen wrote:
On Tue, 2017-12-12 at 14:56 -0800, Jackie
On Thu, 2017-12-14 at 21:30 +0100, Daniel Vetter wrote:
> DK put some nice docs into the commit introducing driver private
> state, but in the git history alone it'll be lost.
>
> Also, since Ville remove the void* usage it's a good opportunity to
> give the driver private stuff some tlc on the d
On Thu, Dec 14, 2017 at 3:30 PM, Daniel Vetter wrote:
> We don't want people to accidentally stumble over there.
>
> Also rename the plane helpers to legacy plane helpers. After Ville's
> patch to make the clipping helper atomic and move it to
> drm_atomic_helper.c there's nothing left in there th
On Thu, Dec 14, 2017 at 3:30 PM, Daniel Vetter wrote:
> Complete a few missing bits, fix up the existing xcross-references and
> add a bunch more.
>
> Cc: Dave Airlie via lists.freedesktop.org
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/drm_syncobj.c | 45
> +
On Thu, Dec 14, 2017 at 3:30 PM, Daniel Vetter wrote:
> It thinks we want to document the __printf(2,0) annotion. Not sure we
> want to teach it about all possible gcc-only flags, hence why I opted
> for the cheap trick of just moving it ahead of the kerneldoc.
>
> This is only a problem for stati
On Thu, Dec 14, 2017 at 3:30 PM, Daniel Vetter wrote:
> Also some breadcrumbs for how exactly to find this. Probably should
> pass drm_connector * or at least drm_display_info * to that function
> instead. But drm_hdmi_avi_infoframe_quant_range probably also wants
> drm_connector_state (and the hd
On Thu, Dec 14, 2017 at 3:30 PM, Daniel Vetter wrote:
> DK put some nice docs into the commit introducing driver private
> state, but in the git history alone it'll be lost.
>
> Also, since Ville remove the void* usage it's a good opportunity to
> give the driver private stuff some tlc on the doc
== Series Details ==
Series: lib: Provide a library function to test nop execution
URL : https://patchwork.freedesktop.org/series/35383/
State : warning
== Summary ==
Test kms_flip:
Subgroup vblank-vs-suspend:
incomplete -> PASS (shard-hsw) fdo#103375 +2
Test kms_
== Series Details ==
Series: tests/gem_eio: Fix usage of gem_quiescent_gpu
URL : https://patchwork.freedesktop.org/series/35382/
State : failure
== Summary ==
Test gem_tiled_swapping:
Subgroup non-threaded:
dmesg-warn -> PASS (shard-hsw) fdo#104218 +1
Test kms_fli
== Series Details ==
Series: igt/tools_test: Check the tools exist before executing (rev3)
URL : https://patchwork.freedesktop.org/series/35237/
State : failure
== Summary ==
Test kms_cursor_legacy:
Subgroup flip-vs-cursor-toggle:
pass -> FAIL (shard-hsw) fd
== Series Details ==
Series: lib: Provide a library function to test nop execution
URL : https://patchwork.freedesktop.org/series/35383/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to quiesce t
== Series Details ==
Series: tests/gem_eio: Fix usage of gem_quiescent_gpu
URL : https://patchwork.freedesktop.org/series/35382/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to quiesce the GPU
== Series Details ==
Series: lib/sysfs: Handle EINTR from vfprintf()
URL : https://patchwork.freedesktop.org/series/35378/
State : failure
== Summary ==
Test gem_exec_reloc:
Subgroup basic-wc-active:
pass -> FAIL (shard-hsw)
Test kms_plane:
Subgroup
On 14/12/17 14:24, Chris Wilson wrote:
Sometimes a test wants to verify that an engine, or all of them, are
functional by executing a nop batch. Provide a common routine to submit
an empty batch then test whether the driver is wedged.
Reported-by: Antonio Argenziano
Signed-off-by: Chris Wilso
Quoting Antonio Argenziano (2017-12-14 22:01:31)
> After change "lib: Ask the kernel to quiesce the GPU" was merged,
> gem_quiescent_gpu() would not submit noop batches to all engines
> therefore the test should submit the workload and use the library
> function to wait for the workload to complete
== Series Details ==
Series: drm/i915/glk: Disable Guc and HuC on GLK
URL : https://patchwork.freedesktop.org/series/35381/
State : success
== Summary ==
Series 35381v1 drm/i915/glk: Disable Guc and HuC on GLK
https://patchwork.freedesktop.org/api/1.0/series/35381/revisions/1/mbox/
Test debug
>-Original Message-
>From: Wajdeczko, Michal
>Sent: Thursday, December 14, 2017 2:18 PM
>To: intel-gfx@lists.freedesktop.org; Srivatsa, Anusha
>
>Cc: Vivi, Rodrigo
>Subject: Re: [Intel-gfx] [PATCH] drm/i915/glk: Disable Guc and HuC on GLK
>
>On Thu, 14 Dec 2017 22:58:37 +0100, Anusha Sri
Sometimes a test wants to verify that an engine, or all of them, are
functional by executing a nop batch. Provide a common routine to submit
an empty batch then test whether the driver is wedged.
Reported-by: Antonio Argenziano
Signed-off-by: Chris Wilson
Cc: Antonio Argenziano
---
lib/i915/ge
== Series Details ==
Series: HAX (rev2)
URL : https://patchwork.freedesktop.org/series/35377/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to quiesce the GPU
with latest DRM-Tip kernel build CI
== Series Details ==
Series: bunch of drm kernel-doc patches
URL : https://patchwork.freedesktop.org/series/35375/
State : success
== Summary ==
Test kms_flip:
Subgroup vblank-vs-suspend:
incomplete -> PASS (shard-hsw) fdo#103375
Test kms_setmode:
Subgroup
On Thu, 14 Dec 2017 22:58:37 +0100, Anusha Srivatsa
wrote:
Since the firmwares are released yet to public repo,
disable them on Geminilake.
Cc: Rodrigo Vivi
Signed-off-by: Anusha Srivatsa
---
drivers/gpu/drm/i915/i915_pci.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/
On Wed, 2017-12-13 at 18:28 +, Chris Wilson wrote:
> Just printk the string, or at least do not double up on the newlines!
>
Reviewed-by: Dhinakaran Pandiyan
> Fixes: eef57324d926 ("drm/i915: setup bridge for HDMI LPE audio driver")
> Signed-off-by: Chris Wilson
> Cc: Pierre-Louis Bossart
After change "lib: Ask the kernel to quiesce the GPU" was merged,
gem_quiescent_gpu() would not submit noop batches to all engines
therefore the test should submit the workload and use the library
function to wait for the workload to complete.
Cc: Chris Wilson
Cc: Mika Kuoppala
Cc: Joonas Lahtin
== Series Details ==
Series: igt/tools_test: Check the tools exist before executing (rev3)
URL : https://patchwork.freedesktop.org/series/35237/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to q
Since the firmwares are released yet to public repo,
disable them on Geminilake.
Cc: Rodrigo Vivi
Signed-off-by: Anusha Srivatsa
---
drivers/gpu/drm/i915/i915_pci.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/i915/i915_pci.c b/drivers/gpu/drm/i915/i915_pci.c
index f
== Series Details ==
Series: lib/sysfs: Handle EINTR from vfprintf()
URL : https://patchwork.freedesktop.org/series/35378/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to quiesce the GPU
with l
---
lib/drmtest.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/drmtest.c b/lib/drmtest.c
index fd11f5f58..567d8d9b4 100644
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -165,6 +165,7 @@ void gem_quiescent_gpu(int fd)
igt_drop_caches_set(fd,
DROP_ACTIVE
== Series Details ==
Series: HAX
URL : https://patchwork.freedesktop.org/series/35377/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to quiesce the GPU
with latest DRM-Tip kernel build CI_DRM_35
== Series Details ==
Series: lib/debug: Convert igt_drop_caches_set() to use common file writers
URL : https://patchwork.freedesktop.org/series/35374/
State : warning
== Summary ==
Test kms_frontbuffer_tracking:
Subgroup fbc-1p-offscren-pri-shrfb-draw-render:
pass
Some write operations into sysfs may be slow and potentially interrupted
by a signal. So handle EINTR by repeating the vfprintf(). A partial is
reported back to the caller, as is any other error.
Signed-off-by: Chris Wilson
---
lib/igt_sysfs.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletio
== Series Details ==
Series: bunch of drm kernel-doc patches
URL : https://patchwork.freedesktop.org/series/35375/
State : success
== Summary ==
Series 35375v1 bunch of drm kernel-doc patches
https://patchwork.freedesktop.org/api/1.0/series/35375/revisions/1/mbox/
Test debugfs_test:
S
---
lib/drmtest.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/drmtest.c b/lib/drmtest.c
index fd11f5f58..016493f84 100644
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -163,6 +163,7 @@ void gem_quiescent_gpu(int fd)
{
igt_terminate_spin_batches();
+ sleep(1);
igt_
We don't want people to accidentally stumble over there.
Also rename the plane helpers to legacy plane helpers. After Ville's
patch to make the clipping helper atomic and move it to
drm_atomic_helper.c there's nothing left in there that should be
useful for modern drivers.
v2: Laurent had a few q
Complete a few missing bits, fix up the existing xcross-references and
add a bunch more.
Cc: Dave Airlie via lists.freedesktop.org
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_syncobj.c | 45 +++
include/drm/drm_syncobj.h | 32
It thinks we want to document the __printf(2,0) annotion. Not sure we
want to teach it about all possible gcc-only flags, hence why I opted
for the cheap trick of just moving it ahead of the kerneldoc.
This is only a problem for static inline functions, since for
non-inline function the kerneldoc
Also some breadcrumbs for how exactly to find this. Probably should
pass drm_connector * or at least drm_display_info * to that function
instead. But drm_hdmi_avi_infoframe_quant_range probably also wants
drm_connector_state (and the hdmi stuff moved into that), so this is a
bit more work.
Cc: Vil
Hi all,
Originally I wanted to include the atomic helper split-up here, but once
more I failed to get that done. So just a few patches to clean up
accumlated kernel-doc warnings, plus some real docs for the new best
practices for handling driver private objects. That was motivated by a
recent disc
DK put some nice docs into the commit introducing driver private
state, but in the git history alone it'll be lost.
Also, since Ville remove the void* usage it's a good opportunity to
give the driver private stuff some tlc on the doc front.
Finally try to explain why the "let's just subclass drm_
== Series Details ==
Series: lib/debug: Convert igt_drop_caches_set() to use common file writers
URL : https://patchwork.freedesktop.org/series/35374/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kerne
Hi all,
The following changes tagged drm-intel-testing-2017-12-14:
drm-intel-next-2017-12-14:
- Fix documentation build issues (Randy, Markus)
- Fix timestamp frequency calculation for perf on CNL (Lionel)
- New DMC firmware for Skylake (Anusha)
- GTT flush fixes and other GGTT write track and r
Quoting Chris Wilson (2017-12-07 09:41:26)
> As the system may use a partial vma for a GGTT mmap, access via the GGTT
> mmap is not guaranteed to be tracked by FBC's fence. The rule expressed has
> been that any access to the frontbuffer should be followed by a fb-dirty
> ioctl, so always apply and
Why open code a printf/write loop, when we already have a function
providing it?
Signed-off-by: Chris Wilson
Cc: Joonas Lahtinen
---
lib/igt_debugfs.c | 18 +-
1 file changed, 5 insertions(+), 13 deletions(-)
diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c
index 49b68dfed..1
On Thu, 2017-12-14 at 17:06 +0200, Ville Syrjälä wrote:
> On Tue, Dec 12, 2017 at 04:59:34PM -0800, Dhinakaran Pandiyan wrote:
> > Since commit 4d90f2d507ab ("drm/i915: Start tracking PSR state in crtc
> > state"), we check whether PSR can be enabled or not in
> > psr_compute_config(). Given that
On Thu, Dec 14, 2017 at 6:50 AM, Daniel Vetter wrote:
>
> Imo that's enough that I figured better not delay until Dave is back.
> Linus, can you pls apply?
Pulled.
Linus
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.f
Hi Dave, I'm back, resuming with fixes for v4.15.
BR,
Jani.
The following changes since commit 50c4c4e268a2d7a3e58ebb698ac74da0de40ae36:
Linux 4.15-rc3 (2017-12-10 17:56:26 -0800)
are available in the git repository at:
git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-fixes-2017
Hi Zhenyu,
Applied to dinq. Thanks for sending this sooner.
So, regarding your question about vacations on previous email,
I will be out only coupe days... probably 26 and 27.
My next proper vacation is only in May.
Thanks,
Rodrigo.
On Thu, Dec 14, 2017 at 03:34:34AM +, Zhenyu Wang wrote:
On Thu, 2017-12-14 at 16:37 +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: properly init lockdep class (rev2)
> URL : https://patchwork.freedesktop.org/series/34693/
> State : success
Passed the CI and merged, thanks for the patch!
Regards, Joonas
>
> == Summary ==
>
>
== Series Details ==
Series: test/kms_sysfs_edid_timing : Increase THRESHOLD_PER_CONNECTOR and
THRESHOLD_TOTAL
URL : https://patchwork.freedesktop.org/series/35365/
State : failure
== Summary ==
Test drv_suspend:
Subgroup fence-restore-tiled2untiled-hibernate:
skip
Hi Dave,
More of the same. A lot of improvements from Noralf on this one. Nothing
really big here.
Regards,
Gustavo
drm-misc-next-2017-12-14:
drm-misc-next for 4.16:
Cross-subsystem Changes:
- Documentation for amlogic dt dt-bindings
Core Changes:
- Update edid-derived drm_display_info fi
> -Original Message-
> From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel
> Vetter
> Sent: Thursday, December 14, 2017 4:25 PM
> To: Srinivas, Vidya
> Cc: intel-gfx@lists.freedesktop.org; Vetter, Daniel
> Subject: Re: [Intel-gfx] [PATCH i-g-t 4/6] i-g-t kms_plane_sc
== Series Details ==
Series: drm/i915: properly init lockdep class (rev2)
URL : https://patchwork.freedesktop.org/series/34693/
State : success
== Summary ==
Test gem_eio:
Subgroup suspend:
fail -> PASS (shard-hsw) fdo#103990
Test gem_softpin:
Subgro
== Series Details ==
Series: test/kms_sysfs_edid_timing : Increase THRESHOLD_PER_CONNECTOR and
THRESHOLD_TOTAL
URL : https://patchwork.freedesktop.org/series/35365/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib
The discussion on how to fix the issues has been stagnant for a
long time, see the Bugzilla below and:
https://patchwork.freedesktop.org/patch/170429/
After only changing display cable the result of my machine went
from always failing to hit a WARN. This is also illustrated on
CI-shards, where the
== Series Details ==
Series: drm/i915: properly init lockdep class (rev2)
URL : https://patchwork.freedesktop.org/series/34693/
State : success
== Summary ==
Test gem_tiled_swapping:
Subgroup non-threaded:
dmesg-warn -> PASS (shard-hsw) fdo#104218 +1
Test kms_curs
On Tue, Dec 12, 2017 at 04:59:34PM -0800, Dhinakaran Pandiyan wrote:
> Since commit 4d90f2d507ab ("drm/i915: Start tracking PSR state in crtc
> state"), we check whether PSR can be enabled or not in
> psr_compute_config(). Given that the psr.source_ok field is supposed to
> track this check, set th
Hi Linus&Dave,
drm-misc-fixes-2017-12-14:
drm-misc-fixes for 4.15-rc4
2 fixes for new core features, a corner case fix for the connnector_iter
fix from last week (that one is cc: stable) and 1 vc4 fix.
Imo that's enough that I figured better not delay until Dave is back.
Linus, can you pls appl
== Series Details ==
Series: drm/i915: Show engine state when hangcheck detects a stall
URL : https://patchwork.freedesktop.org/series/35354/
State : success
== Summary ==
Test gem_tiled_swapping:
Subgroup non-threaded:
dmesg-warn -> PASS (shard-hsw) fdo#104218 +1
On 2017-12-13 02:24 PM, Leo Li wrote:
On 2017-12-13 12:23 PM, Maarten Lankhorst wrote:
Op 13-12-17 om 17:19 schreef Leo Li:
Hi Daniel, Maarten,
Just digging an old thread out of the grave:
https://lists.freedesktop.org/archives/dri-devel/2017-August/150495.html
It's suppose to fix a memor
== Series Details ==
Series: HAX: fast feedback irq affinity..
URL : https://patchwork.freedesktop.org/series/35349/
State : failure
== Summary ==
Test gem_tiled_pread_basic:
incomplete -> PASS (shard-hsw) fdo#104218 +1
Test kms_cursor_crc:
Subgroup cursor-64x64-s
== Series Details ==
Series: drm/i915: properly init lockdep class (rev2)
URL : https://patchwork.freedesktop.org/series/34693/
State : success
== Summary ==
Series 34693v2 drm/i915: properly init lockdep class
https://patchwork.freedesktop.org/api/1.0/series/34693/revisions/2/mbox/
Test gem_
On Thu, 2017-12-14 at 13:29 +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: properly init lockdep class (rev2)
> URL : https://patchwork.freedesktop.org/series/34693/
> State : failure
>
> == Summary ==
>
> Series 34693v2 drm/i915: properly init lockdep class
> https://pat
== Series Details ==
Series: drm/i915: properly init lockdep class (rev2)
URL : https://patchwork.freedesktop.org/series/34693/
State : failure
== Summary ==
Series 34693v2 drm/i915: properly init lockdep class
https://patchwork.freedesktop.org/api/1.0/series/34693/revisions/2/mbox/
Test debu
On Thu, 2017-12-14 at 12:26 +, Chris Wilson wrote:
> Knowing the state of the engine when hangcheck thinks it is stalling is
> useful for both debugging hangcheck itself and the potential cause of an
> unwanted stall.
>
> Signed-off-by: Chris Wilson
> Cc: Joonas Lahtinen
> Cc: Mika Kuoppala
From: Sebastian Andrzej Siewior
The code has an ifdef and uses two functions to either init the bare
spinlock or init it and set a lock-class. It is possible to do the same
thing without an ifdef.
With this patch (in debug case) we first use the "default" lock class
which is later overwritten to
== Series Details ==
Series: drm/i915: Show engine state when hangcheck detects a stall
URL : https://patchwork.freedesktop.org/series/35354/
State : success
== Summary ==
Series 35354v1 drm/i915: Show engine state when hangcheck detects a stall
https://patchwork.freedesktop.org/api/1.0/series
On 12/14/2017 01:24 PM, Chris Wilson wrote:
Quoting Petri Latvala (2017-12-14 11:20:18)
On Thu, Dec 14, 2017 at 11:14:35AM +, Chris Wilson wrote:
Why the chdir?
For me, I was trying to standardise the test, since "../tools" bakes in
so many assumptions that simply aren't true when one in
Knowing the state of the engine when hangcheck thinks it is stalling is
useful for both debugging hangcheck itself and the potential cause of an
unwanted stall.
Signed-off-by: Chris Wilson
Cc: Joonas Lahtinen
Cc: Mika Kuoppala
---
drivers/gpu/drm/i915/intel_hangcheck.c | 7 ++-
1 file chan
== Series Details ==
Series: HAX: fast feedback irq affinity..
URL : https://patchwork.freedesktop.org/series/35349/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to quiesce the GPU
with latest
== Series Details ==
Series: igt/tools_test: Check the tools exist before executing (rev3)
URL : https://patchwork.freedesktop.org/series/35237/
State : warning
== Summary ==
IGT patchset tested on top of latest successful build
103af7286e2d1014e3d7126691fe046ab2c9f73e lib: Ask the kernel to q
---
tests/intel-ci/fast-feedback.testlist | 2 ++
tests/pm_rpm.c| 62 +++
2 files changed, 64 insertions(+)
diff --git a/tests/intel-ci/fast-feedback.testlist
b/tests/intel-ci/fast-feedback.testlist
index f71a16bcd191..e9322249dd73 100644
On Wed, 2017-12-13 at 14:59 -0800, Yaodong Li wrote:
> On 12/13/2017 01:34 PM, Michal Wajdeczko wrote:
> > On Wed, 13 Dec 2017 19:19:06 +0100, Yaodong Li
> > wrote:
> >
> > > On 12/13/2017 01:11 AM, Joonas Lahtinen wrote:
> > > > On Tue, 2017-12-12 at 14:56 -0800, Jackie Li wrote:
> > > > > Hard
As a simple fail-safe against a bad installation, check the tools exist
before testing whether they work.
v2: Check intel_l3_parity as well
v3: Hunt for tools/
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102935
Signed-off-by: Chris Wilson
Reviewed-by: Petri Latvala #v1
Reviewed-by: J
Quoting Petri Latvala (2017-12-14 11:20:18)
> On Thu, Dec 14, 2017 at 11:14:35AM +, Chris Wilson wrote:
> > >
> > > Why the chdir?
> >
> > For me, I was trying to standardise the test, since "../tools" bakes in
> > so many assumptions that simply aren't true when one invokes
> > $path/tools_t
On Thu, Dec 14, 2017 at 11:14:35AM +, Chris Wilson wrote:
> >
> > Why the chdir?
>
> For me, I was trying to standardise the test, since "../tools" bakes in
> so many assumptions that simply aren't true when one invokes
> $path/tools_test. Back to the igt_data_dir() request.
Aye, the assumpt
== Series Details ==
Series: drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set
URL : https://patchwork.freedesktop.org/series/35338/
State : success
== Summary ==
Test drv_suspend:
Subgroup fence-restore-tiled2untiled-hibernate:
skip -> FAIL
Quoting Petri Latvala (2017-12-14 11:11:47)
> On Wed, Dec 13, 2017 at 02:56:10PM +, Chris Wilson wrote:
> > As a simple fail-safe against a bad installation, check the tools exist
> > before testing whether they work.
> >
> > v2: Check intel_l3_parity as well
> >
> > Bugzilla: https://bugs.fr
On Wed, Dec 13, 2017 at 02:56:10PM +, Chris Wilson wrote:
> As a simple fail-safe against a bad installation, check the tools exist
> before testing whether they work.
>
> v2: Check intel_l3_parity as well
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102935
> Signed-off-by: Chri
Just a quick comment at the bottom.
On Wed, Dec 13, 2017 at 03:20:50PM +0530, Vidya Srinivas wrote:
> @@ -312,23 +480,40 @@ static void test_plane_scaling(data_t *d, enum pipe
> pipe)
> igt_require_f(valid_tests, "no valid crtc/connector combinations
> found\n");
> }
>
> -igt_simple_mai
On Wed, 13 Dec 2017, Lucas De Marchi wrote:
> CFL was missing from intel_early_ids[]. The PCI ID needs to be there to
> allow the memory region to be stolen, otherwise we could have RAM being
> arbitrarily overwritten if for example we keep using the UEFI framebuffer,
> depending on how BIOS has s
On Thu, Nov 23, 2017 at 12:05 AM, Anusha Srivatsa
wrote:
> From: Joseph Garvey
>
> Test that horizontal flip works with supported rotations. Includes
> a fix for the unrotated fb which was not being positioned correctly
> with portrait and landscape rectangles.
>
> v2:(from Anusha)
> - Change 180
On Thu, 14 Dec 2017, Abdiel Janulgue wrote:
> 4K modes testing by using dummy EDID data has never been working
> properly on boxes with DP++ (dual-mode) adaptors. The reason for
> this is that those modes got pruned during hdmi mode validation.
> intel_hdmi_mode_valid returns CLOCK_HIGH because
== Series Details ==
Series: drm/i915: Ignore TMDS clock limit for DP++ when EDID override is set
URL : https://patchwork.freedesktop.org/series/35338/
State : success
== Summary ==
Series 35338v1 drm/i915: Ignore TMDS clock limit for DP++ when EDID override is
set
https://patchwork.freedeskt
4K modes testing by using dummy EDID data has never been working
properly on boxes with DP++ (dual-mode) adaptors. The reason for
this is that those modes got pruned during hdmi mode validation.
intel_hdmi_mode_valid returns CLOCK_HIGH because the pixel clock
reported by the 4k mode is higher tha
Quoting Patchwork (2017-12-14 00:03:46)
> == Series Details ==
>
> Series: series starting with [CI,1/7] drm/i915/guc: Move shared data
> allocation away from submission path
> URL : https://patchwork.freedesktop.org/series/35321/
> State : warning
And pushed, thanks for the fixups and review,
87 matches
Mail list logo
