Re: [Intel-gfx] [PATCH IGT v2 5/6] tests/kms_frontbuffer_tracking: Fix multidraw subtest

2017-06-30 Thread Rodrigo Vivi
cc: Paulo Zanoni... On Fri, Jun 30, 2017 at 12:12 PM, Jim Bride wrote: > The multidraw subtest was not taking whether or not the GEM buffer had > ever been in write-combining mode when checking for PSR state, so fix > that. > > Signed-off-by: Jim Bride > --- > tests/kms_frontbuffer_tracking.c |

Re: [Intel-gfx] [PATCH IGT v2 4/6] tests/kms_frontbuffer_tracking: Refactor to use IGT PSR library functions

2017-06-30 Thread Paulo Zanoni
Em Sex, 2017-06-30 às 12:12 -0700, Jim Bride escreveu: > v2: * Minor functional tweaks and bug fixes > * Rebase > > Cc: Rodrigo Vivi > Cc: Paulo Zanoni > Signed-off-by: Jim Bride This patch is not just a refactor. It changes a how the code behaves. Please split it into many patches: one pa

Re: [Intel-gfx] [PATCH IGT v2 2/6] lib: Add PSR utility functions to igt library.

2017-06-30 Thread Paulo Zanoni
Em Sex, 2017-06-30 às 12:12 -0700, Jim Bride escreveu: > Factor out some code that was replicated in three test utilities into > some new IGT library functions so that we are checking PSR status in > a consistent fashion across all of our PSR tests. > > v2: * Add sink CRC helper function > * M

[Intel-gfx] [PATCH i-g-t] lib/i915_pciids.h: Organize cnl/cfl ids.

2017-06-30 Thread Rodrigo Vivi
No functional change. When CNL patches got rebased on top of cfl the ids ended up in the middle of CFL ids. So let's clean-up this mess a bit. Also remove a spurious line. Signed-off-by: Rodrigo Vivi --- lib/i915_pciids.h | 23 +++ 1 file changed, 11 insertions(+), 12 delet

[Intel-gfx] [PATCH i-g-t 2/2] list-workarounds/cfl: Add Coffee Lake to the list of valid platorms

2017-06-30 Thread Rodrigo Vivi
Signed-off-by: Rodrigo Vivi --- scripts/list-workarounds | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/list-workarounds b/scripts/list-workarounds index 1308aa9..c236981 100755 --- a/scripts/list-workarounds +++ b/scripts/list-workarounds @@ -18,7 +18,7 @@ def find_n

[Intel-gfx] [PATCH i-g-t 1/2] list-workarounds/cnl: Add Cannonlake to the list of valid platorms

2017-06-30 Thread Rodrigo Vivi
From: Damien Lespiau Signed-off-by: Damien Lespiau --- scripts/list-workarounds | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/list-workarounds b/scripts/list-workarounds index 70c026d..1308aa9 100755 --- a/scripts/list-workarounds +++ b/scripts/list-workarounds @@

Re: [Intel-gfx] [PATCH IGT v2 4/6] tests/kms_frontbuffer_tracking: Refactor to use IGT PSR library functions

2017-06-30 Thread Paulo Zanoni
Em Sex, 2017-06-30 às 12:12 -0700, Jim Bride escreveu: > v2: * Minor functional tweaks and bug fixes > * Rebase > > Cc: Rodrigo Vivi > Cc: Paulo Zanoni > Signed-off-by: Jim Bride > --- >  tests/kms_frontbuffer_tracking.c | 119 +++ > >  1 file changed, 19

Re: [Intel-gfx] [PATCH IGT v2 6/6] tests/kms_fbcon_fbt: Refactor to use IGT PSR library functions

2017-06-30 Thread Paulo Zanoni
Em Sex, 2017-06-30 às 12:12 -0700, Jim Bride escreveu: > v2: * Minor functional tweaks and bug fixes > * Rebase > > Cc: Rodrigo Vivi > Cc: Paulo Zanoni > Signed-off-by: Jim Bride > --- >  tests/kms_fbcon_fbt.c | 54 +-- > >  1 file changed, 18

[Intel-gfx] [PATCH] intel/intel_chipset: Move IS_9XX below IS_GEN10.

2017-06-30 Thread Rodrigo Vivi
No functional change. Just organizing the code so it gets clear for future platforms. Paulo deserves credits becuase he was the one that just noticed this IS_9XX was in the wrong position after CNL patches got introduced. Cc: Paulo Zanoni Signed-off-by: Rodrigo Vivi --- intel/intel_chipset.h |

[Intel-gfx] [PATCH] drm/i915: Fix use-after-free of context during free_contexts

2017-06-30 Thread Chris Wilson
When iterating the list of contexts to free, we need to use a safe iterator as we are freeing the link as we go. Pass an extra thick brown paper bag. Fixes: 5f09a9c8ab6b ("drm/i915: Allow contexts to be unreferenced locklessly") Signed-off-by: Chris Wilson Cc: Joonas Lahtinen Cc: Tvrtko Ursulin

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Fix use-after-free of context during free_contexts

2017-06-30 Thread Patchwork
== Series Details == Series: drm/i915: Fix use-after-free of context during free_contexts URL : https://patchwork.freedesktop.org/series/26671/ State : success == Summary == Series 26671v1 drm/i915: Fix use-after-free of context during free_contexts https://patchwork.freedesktop.org/api/1.0/se

Re: [Intel-gfx] [PATCH] drm/i915: Fix use-after-free of context during free_contexts

2017-06-30 Thread Matthew Auld
On 1 July 2017 at 00:05, Chris Wilson wrote: > When iterating the list of contexts to free, we need to use a safe > iterator as we are freeing the link as we go. Pass an extra thick brown > paper bag. > > Fixes: 5f09a9c8ab6b ("drm/i915: Allow contexts to be unreferenced locklessly") > Signed-off-b

Re: [Intel-gfx] [PATCH v3 1/2] drm/i915/skl+: Check for supported plane configuration in Interlace mode

2017-06-30 Thread Mahesh Kumar
Hi, On Friday 30 June 2017 05:56 PM, Ville Syrjälä wrote: On Fri, Jun 30, 2017 at 05:40:59PM +0530, Mahesh Kumar wrote: In Gen9 platform Interlaced fetch mode doesn't support following plane configuration: - Y/Yf tiling - 90/270 rotation The rotation check seems to be missing from the cod

<    1   2