== Series Details ==
Series: Revert "intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT"
URL : https://patchwork.freedesktop.org/series/29805/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
490b386ae36d393cdea514882a2437c6e964c13f scripts/run-tests.sh: Us
HI,
> -Original Message-
> From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Daniel Vetter
> Sent: tiistai 5. syyskuuta 2017 9.39
> To: Intel Graphics Development
> Cc: Daniel Vetter ; Vetter, Daniel
>
> Subject: [Intel-gfx] [PATCH i-g-t] Revert "intel-ci:
Hi Ville,
On 4 September 2017 at 17:37, Ville Syrjälä
wrote:
> On Thu, Aug 31, 2017 at 04:52:15PM -0300, Gabriel Krisman Bertazi wrote:
>> With this patch the new testcase igt@kms_ccs@pipe-X-invalid-ccs-offset
>> succeeds.
>
> I don't think we actually want to reject overlap. I had a patch for th
Hi Gabriel,
On 31 August 2017 at 07:18, Gabriel Krisman Bertazi
wrote:
> Two scenarios tested:
> - unaligned stripe
> - Stripe too small
'stride' in the commit message please. ;) But it is fine everywhere
through the code.
> @@ -323,7 +326,14 @@ static void generate_fb(data_t *data, struct
== Series Details ==
Series: Revert "intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT"
URL : https://patchwork.freedesktop.org/series/29805/
State : failure
== Summary ==
Test perf:
Subgroup polling:
pass -> FAIL (shard-hsw) fdo#102252
Test kms_flip:
Hi Gabriel,
On 31 August 2017 at 06:58, Gabriel Krisman Bertazi
wrote:
> + if (data->flags & TEST_BAD_CCS_HANDLE) {
> + /* Put the CCS buffer on a different BO. */
> + f.handles[0] = gem_create(data->drm_fd, size[0]);
> +
On 31 August 2017 at 06:58, Gabriel Krisman Bertazi
wrote:
> @@ -321,7 +322,13 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
> int ccs_height = ALIGN(height, 16) / 16;
> f.pitches[1] = ALIGN(ccs_width * 1, 128);
> f.modifier[1] = modifi
Hi Gabriel,
On 31 August 2017 at 06:58, Gabriel Krisman Bertazi
wrote:
> @@ -321,16 +325,19 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
> size[1] = f.pitches[1] * ALIGN(ccs_height, 32);
>
> f.handles[0] = gem_create(data->drm_fd, size[0] + size[1]);
On Mon, 2017-09-04 at 18:04 +0300, Ville Syrjälä wrote:
> On Mon, Sep 04, 2017 at 10:59:32AM +0300, Mika Kahola wrote:
> >
> > On Fri, 2017-09-01 at 16:43 +0300, Ville Syrjälä wrote:
> > >
> > > On Fri, Sep 01, 2017 at 10:51:00AM +0300, Mika Kahola wrote:
> > > >
> > > >
> > > > According to sp
== Series Details ==
Series: Fixed16.16 wrapper cleanup & wm optimization (rev7)
URL : https://patchwork.freedesktop.org/series/25692/
State : success
== Summary ==
Series 25692v7 Fixed16.16 wrapper cleanup & wm optimization
https://patchwork.freedesktop.org/api/1.0/series/25692/revisions/7/mb
On Mon, 2017-08-21 at 19:34 +0100, Matthew Auld wrote:
> In preparation for supporting huge gtt pages for the ppgtt, we introduce
> page size members for gem objects. We fill in the page sizes by
> scanning the sg table.
>
> v2: pass the sg_mask to set_pages
>
> v3: calculate the sg_mask inline
>-Original Message-
>From: Kahola, Mika
>Sent: Tuesday, September 5, 2017 2:03 PM
>To: Ville Syrjälä
>Cc: intel-gfx@lists.freedesktop.org; Shankar, Uma
>Subject: Re: [Intel-gfx] [PATCH 1/2] drm/i915/dsi: Send SHUTDOWN only for v3+
>VBT's
>
>On Mon, 2017-09-04 at 18:04 +0300, Ville Syrjä
On Tue, Sep 05, 2017 at 08:39:00AM +0200, Daniel Vetter wrote:
> This reverts commit effb78b382895c19d0946c217cee3602dcd94c50.
>
> cibuglog is for filtering unstable results, we can't simply opt to not
> test PSR.
>
> Signed-off-by: Daniel Vetter
Acked-by: Petri Latvala
___
From: Uma Shankar
This reverts commit bbdf0b2ff32aa75c7bd167569130e9391d2e6282.
Disable device ready before shutdown command was added previously to avoid a
split screen issue seen on dual link DSI panels. As of now, dual link is not
supported and will need some rework in the upstream code. For
== Series Details ==
Series: drm/i915/dsi: Fix error on DSI video mode command (rev2)
URL : https://patchwork.freedesktop.org/series/29658/
State : success
== Summary ==
Series 29658v2 drm/i915/dsi: Fix error on DSI video mode command
https://patchwork.freedesktop.org/api/1.0/series/29658/revi
== Series Details ==
Series: Fixed16.16 wrapper cleanup & wm optimization (rev7)
URL : https://patchwork.freedesktop.org/series/25692/
State : success
== Summary ==
Test perf:
Subgroup blocking:
fail -> PASS (shard-hsw) fdo#102252 +1
Test kms_setmode:
From: Tvrtko Ursulin
Exercise the new __sg_alloc_table_from_pages API (and through
it also the old sg_alloc_table_from_pages), checking that the
created table has the expected number of segments depending on
the sequence of input pages and other conditions.
v2: Move to data driven for readabilit
From: Tvrtko Ursulin
No need to sync filesystems when only listing subtest.
Extremely marginal benefit of avoid a short stall after make
followed by listing subtests.
v2: Move sync to common_init for consolidation and simplicity.
(Chris Wilson)
Signed-off-by: Tvrtko Ursulin
Cc: Chris Wils
The revert patch solved the issue
*ERROR* Video mode command 0x0041 send failed.
on my setup with APL+MIPI/DSI single link combo.
Tested-by: Mika Kahola
On Tue, 2017-09-05 at 15:14 +0530, Vidya Srinivas wrote:
> From: Uma Shankar
>
> This reverts commit bbdf0b2ff32aa75c7bd167569130e9391
== Series Details ==
Series: series starting with [1/5] lib/scatterlist: Fix offset type in
sg_alloc_table_from_pages (rev5)
URL : https://patchwork.freedesktop.org/series/28151/
State : success
== Summary ==
Series 28151v5 series starting with [1/5] lib/scatterlist: Fix offset type in
sg_al
Quoting Tvrtko Ursulin (2017-09-05 11:38:58)
> From: Tvrtko Ursulin
>
> No need to sync filesystems when only listing subtest.
>
> Extremely marginal benefit of avoid a short stall after make
> followed by listing subtests.
>
> v2: Move sync to common_init for consolidation and simplicity.
>
== Series Details ==
Series: igt_core: Skip sync when listing subtests (rev2)
URL : https://patchwork.freedesktop.org/series/27707/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
490b386ae36d393cdea514882a2437c6e964c13f scripts/run-tests.sh: Use piglit's
-
== Series Details ==
Series: drm/i915/dsi: Fix error on DSI video mode command (rev2)
URL : https://patchwork.freedesktop.org/series/29658/
State : failure
== Summary ==
Test tools_test:
Subgroup tools_test:
pass -> DMESG-WARN (shard-hsw) fdo#102543
Test perf:
On Mon, Sep 04, 2017 at 03:09:10PM -0300, Paulo Zanoni wrote:
> Em Seg, 2017-09-04 às 10:00 +0200, Daniel Vetter escreveu:
> > On Fri, Sep 01, 2017 at 04:44:38PM -0300, Paulo Zanoni wrote:
> > > Em Seg, 2017-08-14 às 11:25 +0200, Daniel Vetter escreveu:
> > > > Macros that should be C functions but
On Tue, Sep 05, 2017 at 12:30:59PM +0300, Petri Latvala wrote:
> On Tue, Sep 05, 2017 at 08:39:00AM +0200, Daniel Vetter wrote:
> > This reverts commit effb78b382895c19d0946c217cee3602dcd94c50.
> >
> > cibuglog is for filtering unstable results, we can't simply opt to not
> > test PSR.
> >
> > Si
On Mon, Sep 04, 2017 at 10:49:46AM +0200, Daniel Vetter wrote:
> On Sat, Sep 02, 2017 at 09:38:51PM -0700, Eric Anholt wrote:
> > Signed-off-by: Eric Anholt
> > ---
> >
> > This little series cleans up many compiler warnings I saw when testing
> > danvet's meson branch.
>
> On the series: Review
This is a wrapper for tools/intel_l3_parity which fails if l3_parity
is not supported in the kernel. Check support before executing test.
bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101650
Signed-off-by: Abdiel Janulgue
---
tests/tools_test.c | 4
1 file changed, 4 insertions(+)
== Series Details ==
Series: series starting with [1/5] lib/scatterlist: Fix offset type in
sg_alloc_table_from_pages (rev5)
URL : https://patchwork.freedesktop.org/series/28151/
State : success
== Summary ==
Test tools_test:
Subgroup tools_test:
pass -> DMESG-WA
Hi,
> -Original Message-
> From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Saarinen, Jani
> Sent: perjantai 1. syyskuuta 2017 12.30
> To: Ville Syrjälä ; Navare, Manasi D
>
> Cc: Sarvela, Tomi P ; Vetter, Daniel
> ; intel-gfx@lists.freedesktop.org
> Subject
On Tue, Sep 05, 2017 at 02:52:05PM +0300, Abdiel Janulgue wrote:
> This is a wrapper for tools/intel_l3_parity which fails if l3_parity
> is not supported in the kernel. Check support before executing test.
>
> bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101650
> Signed-off-by: Abdiel J
We are, the build system takes care of that.
Reviewed-by: Eric Anholt
Signed-off-by: Daniel Vetter
---
benchmarks/gem_exec_tracer.c | 2 --
benchmarks/gem_latency.c | 1 -
benchmarks/gem_syslatency.c| 2 --
lib/igt_fb.c | 1 -
lib/igt_vgem.c | 2 --
Hi all,
Ok, this time around a proper patch series with cover letter and a pile of
fixes (bunch of them thanks to Eric) thrown on top.
The motivation for this has 2 primary reasons:
- I want a build system that's fast, especially for hacking on the
library. Currently with over 300 binaries the
Also, we are _GNU_SOURCE, so simplify the conditions accordingly.
The next patch will remove _GNU_SOURCE everywhere else.
Reviewed-by: Eric Anholt
Signed-off-by: Daniel Vetter
---
lib/drmtest.c| 4 +---
lib/igt_aux.c| 4 +---
lib/igt_core.c | 4 +---
lib/ioctl_wrappers.c
We have it. Daniel Stone said the #ifdef HAVE_CONFIG_H comes from the
X11 transition to the modular build, where in the imake -> modular
build transition config.h wasn't universally available. Now we just
make this a requirement (so yeah Android better generate one too).
v2: Improve commit message
In meson I want to just set this everywhere (no reason not to), and
doing so will allow us to clean up a few things.
But that means autofoo needs to follow suit.
Reviewed-by: Eric Anholt
Signed-off-by: Daniel Vetter
---
benchmarks/Makefile.am | 2 +-
lib/Makefile.am| 1 +
lib/tests/Mak
meso will use this to run the tests for all testcases in parallel, for
great speedup!
Signed-off-by: Daniel Vetter
---
tests/igt_command_line.sh | 39 ++-
1 file changed, 26 insertions(+), 13 deletions(-)
diff --git a/tests/igt_command_line.sh b/tests/igt_com
Reviewed-by: Eric Anholt
Signed-off-by: Daniel Vetter
---
tests/gem_spin_batch.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/gem_spin_batch.c b/tests/gem_spin_batch.c
index 941aa1399ae1..09bb3cb37108 100644
--- a/tests/gem_spin_batch.c
+++ b/tests/gem_spin_batch.c
@
The igt testcase themselves contain plenty of demos, libdrm
also contains a bunch of demos, this here just bitrots.
So let's remove it.
Signed-off-by: Daniel Vetter
---
Makefile.am | 2 +-
demos/.gitignore| 1 -
demos/Android.mk| 32 --
demos/Makefile.am |
Again we want to be able to run each testcase individually.
Also, we need to make sure the target directory for the temp files
exists - meson always builds with a build-dir outside of the source
tree.
Signed-off-by: Daniel Vetter
---
assembler/test/run-test.sh | 16
1 file chan
meson runs gcc from where meson.build that includes those files is,
which means we need to add the directory ourselves.
For automake it doesn't matter, so let's just do it for simplicity.
Signed-off-by: Daniel Vetter
---
lib/uwildmat/uwildmat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Just a bit of ocd for anything non-static.
Signed-off-by: Daniel Vetter
---
lib/igt_core.c | 12 ++--
lib/igt_core.h | 2 +-
lib/igt_frame.c | 10 +-
3 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/lib/igt_core.c b/lib/igt_core.c
index ff6d19fc9bad..9f4ee68bd8
Just a bit of OCD, I like it when connections within library modules
are a bit more obvious. igt.h is ok for tests, but let's use individual
include lines for libraries consistently.
Also order standard includes before igt ones.
Signed-off-by: Daniel Vetter
---
lib/igt_alsa.c | 3 ++-
lib
gcc complains otherwise about empty ; statements ...
Signed-off-by: Daniel Vetter
---
lib/ioctl_wrappers.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
index b4d6210d5942..48750427a0c1 100644
--- a/lib/ioctl_wrappers.c
+++ b/lib/
From: Eric Anholt
These warnings are apparently new compared to the autotools build. We
can fix the things they complain about later, if we want.
Signed-off-by: Eric Anholt
Signed-off-by: Daniel Vetter
---
meson.build | 11 +++
1 file changed, 11 insertions(+)
diff --git a/meson.bui
Why?
Because it's fast.
Like really, really fast.
Some data (from a snb laptop, so rather lower-powered):
- Incremental build after $ touch lib/igt_core.c with meson: 0.6s
It notices that the symbol list of the libigt.so hasn't changed and
doesn't bother re-linking the almost 300 binaries w
Not exactly sure what's the point, but oh well.
Signed-off-by: Daniel Vetter
---
tests/kms_plane.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/kms_plane.c b/tests/kms_plane.c
index 927d5d37fece..812497500d2d 100644
--- a/tests/kms_plane.c
+++ b/tests/kms_plane.c
@
From: Eric Anholt
Signed-off-by: Eric Anholt
Signed-off-by: Daniel Vetter
---
tests/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/meson.build b/tests/meson.build
index 73833758be0e..4dd5a9c9d4c7 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -
link_whole needs that, and we need that. Reported by Chris Wilson.
Signed-off-by: Daniel Vetter
---
meson.build | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/meson.build b/meson.build
index 2b49a0db6500..0858354f1b0d 100644
--- a/meson.build
+++ b/meson.build
@@ -4,7 +4,8
From: Eric Anholt
It means that compiler errors in the .c files take you to the source
place in your editor, not a preprocessed temporary.
v2: Add the library deps, fails linking otherwise.
Signed-off-by: Eric Anholt (v1)
Signed-off-by: Daniel Vetter
---
lib/dummy.c | 0
lib/m
Signed-off-by: Daniel Vetter
---
README | 16
1 file changed, 16 insertions(+)
diff --git a/README b/README
index c3ddf7685fb4..2c2cd3637a43 100644
--- a/README
+++ b/README
@@ -159,6 +159,22 @@ The following dependencies are requires for building audio
support
libasoun
It seems like meson doesn't want you to string together targets
like make does, but wants it all in one step. So another little
shell script it is.
Signed-off-by: Daniel Vetter
---
man/defs.rst.in | 5 +
man/meson.build | 45 +
man/rst2man.sh | 1
Oversight I noticed because I have the deps for chamelium on the other
machine already.
Signed-off-by: Daniel Vetter
---
lib/meson.build | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/meson.build b/lib/meson.build
index f0672edf1830..b78e42dc78cb 100644
--- a/lib/mes
== Series Details ==
Series: igt_core: Skip sync when listing subtests (rev2)
URL : https://patchwork.freedesktop.org/series/27707/
State : failure
== Summary ==
Test gem_exec_capture:
Subgroup capture-blt:
pass -> DMESG-WARN (shard-hsw)
Test kms_flip:
Sub
Somehow my gcc has a different idea of what no-implicit-fallthrough
should look like than the one Eric used.
fixup compiler flags
Signed-off-by: Daniel Vetter
---
meson.build | 22 ++
1 file changed, 14 insertions(+), 8 deletions(-)
diff --git a/meson.build b/meson.build
in
When no action is specified on the command line, print the usage help
text and exit with failure instead of SIGABRT. Fix some typos on the
usage text.
Keep the abort() call in places where they can only be reached by
expanding the tool and forgetting to handle new parameters, with an
error message
== Series Details ==
Series: tests/tools_test: check if l3_parity is supported
URL : https://patchwork.freedesktop.org/series/29817/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
918863f8e3e8f49235fd2e4a36e11f386c06c11c intel_display_poller: Fix truncation
== Series Details ==
Series: intel_l3_parity: More helpful output in case of errors
URL : https://patchwork.freedesktop.org/series/29824/
State : warning
== Summary ==
IGT patchset tested on top of latest successful build
918863f8e3e8f49235fd2e4a36e11f386c06c11c intel_display_poller: Fix trunc
On Mon, Sep 04, 2017 at 08:51:16PM +0200, Maarten Lankhorst wrote:
> Op 04-09-17 om 17:58 schreef Ville Syrjälä:
> > On Mon, Sep 04, 2017 at 12:39:25PM +0200, Maarten Lankhorst wrote:
> >> Op 31-08-17 om 20:48 schreef Ville Syrjälä:
> >>> On Wed, Aug 30, 2017 at 09:57:03PM +0300, ville.syrj...@linu
Currently we're doing:
1. acquire lock
2. write word to hardware
3. release lock
4. repeat from 1
to load the DMC firmware. Due to the cost of acquiring/releasing a lock,
and the size of the DMC firmware, this slows down DMC loading a lot.
This patch simply acquires the lock, writes the entire f
On Tue, Sep 05, 2017 at 09:27:47AM +, Shankar, Uma wrote:
>
>
> >-Original Message-
> >From: Kahola, Mika
> >Sent: Tuesday, September 5, 2017 2:03 PM
> >To: Ville Syrjälä
> >Cc: intel-gfx@lists.freedesktop.org; Shankar, Uma
> >Subject: Re: [Intel-gfx] [PATCH 1/2] drm/i915/dsi: Send
Quoting David Weinehall (2017-09-05 14:10:50)
> Currently we're doing:
>
> 1. acquire lock
> 2. write word to hardware
> 3. release lock
> 4. repeat from 1
>
> to load the DMC firmware. Due to the cost of acquiring/releasing a lock,
> and the size of the DMC firmware, this slows down DMC loading
On Tue, Sep 05, 2017 at 02:25:36PM +0100, Chris Wilson wrote:
> Quoting David Weinehall (2017-09-05 14:10:50)
> > Currently we're doing:
> >
> > 1. acquire lock
> > 2. write word to hardware
> > 3. release lock
> > 4. repeat from 1
> >
> > to load the DMC firmware. Due to the cost of acquiring/re
On Mon, Sep 04, 2017 at 10:35:49AM +0200, Daniel Vetter wrote:
> On Fri, Sep 01, 2017 at 03:11:23PM +0100, Chris Wilson wrote:
> > If a worker requeues itself, it may switch to a different kworker pool,
> > which flush_work() considers as complete. To be strict, we then need to
> > keep flushing th
== Series Details ==
Series: drm/i915: Speed up DMC firmware loading (rev4)
URL : https://patchwork.freedesktop.org/series/29688/
State : success
== Summary ==
Series 29688v4 drm/i915: Speed up DMC firmware loading
https://patchwork.freedesktop.org/api/1.0/series/29688/revisions/4/mbox/
Test
It appears that we cannot trust scanline counters when MIPI/DSI display is
connected. In CI system this appears as flickering errors that randomly
appear in test cases. To avoid this flickering, let's just silence atomic
update failure in case with DSI panel.
Bugzilla: https://bugs.freedesktop.org
Quoting Tejun Heo (2017-09-05 14:36:28)
> On Mon, Sep 04, 2017 at 10:35:49AM +0200, Daniel Vetter wrote:
> > On Fri, Sep 01, 2017 at 03:11:23PM +0100, Chris Wilson wrote:
> > > If a worker requeues itself, it may switch to a different kworker pool,
> > > which flush_work() considers as complete. To
Currently we define any !llc machine as using snoop instead. However,
some platforms run into trouble using snoop that we would like to
disable, and to do so easily we want to be able to use the static
device_info tables.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_pci.c |
The original gen4 has an issue where writes (both render and blt) into
snoopable pages are lost. We've previously worked around this in
userspace (ddx, igt) by simply not requesting snoopable buffers, but upon
rediscovering this problem for a third time, make the kernel reject such
requests with -E
Hello,
On Tue, Sep 05, 2017 at 02:43:14PM +0100, Chris Wilson wrote:
> > Can't you use cancel[_delayed]_work_sync()?
>
> We then need a loop like:
>
> do {
> if (cancel_delayed_work_sync(wrk))
> do_work(wrk);
> else
>
Quoting Chris Wilson (2017-08-21 15:55:34)
> Quoting Michel Thierry (2017-08-18 18:23:42)
> > The corruption in CSB mmio reads we were seeing has been tracked down to
> > incorrectly touching forcewake of all domains, following an engine reset.
> > It is still a mistery why we only catched this in
== Series Details ==
Series: drm/i915/dsi: Silence atomic update failure with DSI panel
URL : https://patchwork.freedesktop.org/series/29828/
State : success
== Summary ==
Series 29828v1 drm/i915/dsi: Silence atomic update failure with DSI panel
https://patchwork.freedesktop.org/api/1.0/series
Quoting Chris Wilson (2017-09-05 14:47:34)
> Currently we define any !llc machine as using snoop instead. However,
> some platforms run into trouble using snoop that we would like to
> disable, and to do so easily we want to be able to use the static
> device_info tables.
>
> Signed-off-by: Chris
Quoting David Weinehall (2017-09-05 14:33:25)
> On Tue, Sep 05, 2017 at 02:25:36PM +0100, Chris Wilson wrote:
> > Quoting David Weinehall (2017-09-05 14:10:50)
> > > Currently we're doing:
> > >
> > > 1. acquire lock
> > > 2. write word to hardware
> > > 3. release lock
> > > 4. repeat from 1
> >
== Series Details ==
Series: series starting with [1/2] drm/i915: Move device_info.has_snoop into
the static tables
URL : https://patchwork.freedesktop.org/series/29829/
State : success
== Summary ==
Series 29829v1 series starting with [1/2] drm/i915: Move device_info.has_snoop
into the stat
On Fri, Sep 01, 2017 at 11:02:11AM +0530, Sagar Arun Kamble wrote:
> Teardown of GuC HW/SW state was not properly done in unload path.
> During unload, we can rely on intel_guc_reset_prepare being done
> as part of i915_gem_suspend for disabling GuC interfaces.
> We will have to disable GuC submiss
>-Original Message-
>From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
>Sent: Tuesday, September 5, 2017 6:46 PM
>To: Shankar, Uma
>Cc: Kahola, Mika ; Srinivas, Vidya
>; intel-gfx@lists.freedesktop.org
>Subject: Re: [Intel-gfx] [PATCH 1/2] drm/i915/dsi: Send SHUTDOWN only for v3
On Tue, Sep 05, 2017 at 02:55:31PM +, Shankar, Uma wrote:
>
>
> >-Original Message-
> >From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
> >Sent: Tuesday, September 5, 2017 6:46 PM
> >To: Shankar, Uma
> >Cc: Kahola, Mika ; Srinivas, Vidya
> >; intel-gfx@lists.freedesktop.org
== Series Details ==
Series: drm/i915: Speed up DMC firmware loading (rev4)
URL : https://patchwork.freedesktop.org/series/29688/
State : success
== Summary ==
shard-hswtotal:2255 pass:1225 dwarn:0 dfail:0 fail:15 skip:1014
time:9484s
== Logs ==
For more details see:
https://i
== Series Details ==
Series: drm/i915/dsi: Silence atomic update failure with DSI panel
URL : https://patchwork.freedesktop.org/series/29828/
State : warning
== Summary ==
Test kms_flip:
Subgroup wf_vblank-vs-modeset:
pass -> DMESG-WARN (shard-hsw)
Subgrou
On Tue, Sep 05, 2017 at 04:35:04PM +0300, Mika Kahola wrote:
> It appears that we cannot trust scanline counters when MIPI/DSI display is
> connected. In CI system this appears as flickering errors that randomly
> appear in test cases. To avoid this flickering, let's just silence atomic
> update fa
On Tue, Sep 05, 2017 at 03:39:49PM +0300, Petri Latvala wrote:
> When no action is specified on the command line, print the usage help
> text and exit with failure instead of SIGABRT. Fix some typos on the
> usage text.
>
> Keep the abort() call in places where they can only be reached by
> expand
In meson I want to just set this everywhere (no reason not to), and
doing so will allow us to clean up a few things.
But that means autofoo needs to follow suit.
v2: Rebase.
Reviewed-by: Eric Anholt
Signed-off-by: Daniel Vetter
---
benchmarks/Makefile.am | 2 +-
lib/Makefile.am| 1 +
The igt testcase themselves contain plenty of demos, libdrm
also contains a bunch of demos, this here just bitrots.
So let's remove it.
v2: Rebase.
Signed-off-by: Daniel Vetter
---
Makefile.am | 2 +-
demos/.gitignore| 1 -
demos/Android.mk| 32 --
demos/Makefi
== Series Details ==
Series: RFC: meson build system support (rev3)
URL : https://patchwork.freedesktop.org/series/29823/
State : failure
== Summary ==
IGT patchset build failed on latest successful build
918863f8e3e8f49235fd2e4a36e11f386c06c11c intel_display_poller: Fix truncation
of a test
meso will use this to run the tests for all testcases in parallel, for
great speedup!
v2: Fix bugs in the conversion. Oops, I broke the automake build.
Signed-off-by: Daniel Vetter
---
tests/igt_command_line.sh | 42 --
1 file changed, 28 insertions(+), 1
The igt testcase themselves contain plenty of demos, libdrm
also contains a bunch of demos, this here just bitrots.
So let's remove it.
v2: Rebase.
v3: Also delete demos/Makefile from configure.ac. This broke clean
builds starting with autogen.sh.
Signed-off-by: Daniel Vetter
---
Makefile.am
== Series Details ==
Series: series starting with [1/2] drm/i915: Move device_info.has_snoop into
the static tables
URL : https://patchwork.freedesktop.org/series/29829/
State : success
== Summary ==
shard-hswtotal:2255 pass:1225 dwarn:0 dfail:0 fail:15 skip:1014
time:9480s
==
On 17-09-05 18:16:23, Daniel Vetter wrote:
On Tue, Sep 05, 2017 at 03:39:49PM +0300, Petri Latvala wrote:
When no action is specified on the command line, print the usage help
text and exit with failure instead of SIGABRT. Fix some typos on the
usage text.
Keep the abort() call in places where
On Tue, Sep 05, 2017 at 02:47:34PM +0100, Chris Wilson wrote:
> Currently we define any !llc machine as using snoop instead. However,
> some platforms run into trouble using snoop that we would like to
> disable, and to do so easily we want to be able to use the static
> device_info tables.
>
> Si
Skip compressing 1 segment at the end of the frame,
avoid a pixel count mismatch nuke event when last active
pixel and dummy pixel has same color for Odd Plane
Width / Height.
v2: Use function-like macro and also use mask to clean
to make sure bit 11 is 0. (Suggested by Paulo).
Cc: Paulo Zano
Skip compressing 1 segment at the end of the frame,
avoid a pixel count mismatch nuke event when last active
pixel and dummy pixel has same color for Odd Plane
Width / Height.
For both platforms Gemini Lake and Cannon Lake.
v2: Use function-like macro and also use mask to clean
to make sure b
== Series Details ==
Series: drm/i915/cnl: WaFbcSkipSegments (rev3)
URL : https://patchwork.freedesktop.org/series/29507/
State : success
== Summary ==
Series 29507v3 drm/i915/cnl: WaFbcSkipSegments
https://patchwork.freedesktop.org/api/1.0/series/29507/revisions/3/mbox/
Test kms_cursor_legac
On Fri, Sep 01, 2017 at 07:54:34PM +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Add the missing __user to the urelocs cast to fix the following sparse
> warning:
> i915_gem_execbuffer.c:1541:47: warning: cast removes address space of
> expression
> i915_gem_execbuffer.c:
On Mon, May 22, 2017 at 09:42:01AM +0200, Daniel Vetter wrote:
> On Thu, May 18, 2017 at 10:38:37PM +0300, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Make the mode used for load detection const, and adjust all relevant
> > functions to accept a const mode.
> >
> > Signe
On Wed, 2017-08-30 at 21:52 -0700, Rodrigo Vivi wrote:
> From: "Vivi, Rodrigo"
>
> This workaround fixes a CNL PCH bug when changing
> backlight from a lower frequency to a higher frequency.
>
> During random reboot cycles, display backlight seems to
> be off/ dim for 2-3 mins.
>
> The only fun
From: Ben Widawsky
This enables the Mesa driver to advertise support for ARB_timer_query, and
thus an OpenGL version higher than 3.2.
Cc: Rodrigo Vivi
Signed-off-by: Nanley Chery
---
drivers/gpu/drm/i915/intel_uncore.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/driver
Em Ter, 2017-09-05 às 11:05 -0700, Rodrigo Vivi escreveu:
> Skip compressing 1 segment at the end of the frame,
> avoid a pixel count mismatch nuke event when last active
> pixel and dummy pixel has same color for Odd Plane
> Width / Height.
>
> For both platforms Gemini Lake and Cannon Lake.
>
>
On Tue, Sep 05, 2017 at 02:47:35PM +0100, Chris Wilson wrote:
> The original gen4 has an issue where writes (both render and blt) into
> snoopable pages are lost.
BTW is this documented somewhere, or just folk lore? I only found bspec
notes that snooped->snooped and tiled+snooped isn't allowed. Bu
On Tue, 2017-09-05 at 11:45 -0700, Nanley Chery wrote:
> From: Ben Widawsky
Do we have a signed-off by him?
Maybe go with you as author and suggested-by Ben?
>
> This enables the Mesa driver to advertise support for ARB_timer_query, and
> thus an OpenGL version higher than 3.2.
>
I tried to c
== Series Details ==
Series: drm/i915/cnl: Allow the reg_read ioctl to read the RCS TIMESTAMP
register
URL : https://patchwork.freedesktop.org/series/29839/
State : failure
== Summary ==
Series 29839v1 drm/i915/cnl: Allow the reg_read ioctl to read the RCS TIMESTAMP
register
https://patchwor
1 - 100 of 127 matches
Mail list logo
