Hi,
On 18/11/2016 19:53, ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä
By providing our own format information for the CCS formats, we should
be able to make framebuffer_check() do the right thing for the CCS
surface as well.
Note that we'll return the same format info for both Y
On Sat, Nov 19, 2016 at 12:04:32AM +, Pandiyan, Dhinakaran wrote:
> On Fri, 2016-11-18 at 09:43 +0100, Daniel Vetter wrote:
> > On Thu, Nov 17, 2016 at 06:03:46PM -0800, Dhinakaran Pandiyan wrote:
> > > drm_dp_find_vcpi_slots() returns an error when there is not enough
> > > available bandwidth
Changes:
* lib/dummyload: Add a helper to terminate recursive batch
manually.
* gem_wait: Restore sanitycheck wait on the recursive batch and
avoid using C99 locals.
* kms_flip: Don't wait on the fb - should be busy. Exit the
spin batch when flip is queued
* kms_busy: Tidy up finish_fb_busy
A lot of igt testcases need some GPU workload to make sure a race
window is big enough. Unfortunately having a fixed amount of
workload leads to spurious test failures or overly long runtimes
on some fast/slow platforms. This library contains functionality
to submit GPU workloads that should consum
More and more test-cases are using this.
Signed-off-by: Abdiel Janulgue
---
lib/igt_aux.c | 11 ---
lib/igt_aux.h | 10 ++
lib/igt_core.c | 3 ---
tests/drv_hangman.c | 1 -
4 files changed, 10 insertions(+), 15 deletions(-)
diff --git a/lib/igt_aux.c b/lib/ig
v7: Adapt to api rename
v8: Restore sanitycheck wait on the recursive batch and
avoid using C99 locals (Chris Wilson)
Cc: Chris Wilson
Cc: Daniel Vetter
Signed-off-by: Abdiel Janulgue
---
tests/gem_wait.c | 128 ---
1 file changed, 8 inse
v7: Reuse NSEC_PER_SEC defines
v8: Don't wait on the fb, it must be busy. Exit the spin batch
when flip is queued (Chris Wilson)
Cc: Chris Wilson
Cc: Daniel Vetter
Signed-off-by: Abdiel Janulgue
---
tests/kms_flip.c | 212 +++
1 file chan
v7: Adapt to api rename
v8: Tidy up finish_fb_busy (Chris Wilson)
Cc: Chris Wilson
Cc: Daniel Vetter
Signed-off-by: Abdiel Janulgue
---
tests/kms_busy.c | 81 +---
1 file changed, 6 insertions(+), 75 deletions(-)
diff --git a/tests/kms_busy.
On Fri, 18 Nov 2016, Tvrtko Ursulin wrote:
> On 18/11/2016 14:20, Jani Nikula wrote:
>> The platform flags in device info are (mostly) mutually
>> exclusive. Replace the flags with an enum. Add the platform enum also
>> for platforms that previously didn't have a flag, and give them codename
>> lo
On Fri, Nov 18, 2016 at 06:50:35PM -0800, Manasi Navare wrote:
> At the time userspace does setcrtc, we've already promised the mode
> would work. The promise is based on the theoretical capabilities of the
> link, but it's possible we can't reach this in practice. The DP spec
> describes how the l
Hi Jani,
I can also confirm that patches 1+2 (applied in top of v4.9-rc5ish) fixes it
for me (Clevo P651RA).
The previous issue where the keys broke with PRIME/monitor hotplugging is also
gone.
After system suspend/resume stuff is also still working.
Thanks!
Kind regards,
Peter
https://lekenst
On Fri, Nov 18, 2016 at 09:44:49AM -0800, Manasi Navare wrote:
> On Fri, Nov 18, 2016 at 06:21:21PM +0200, Ville Syrjälä wrote:
> > On Fri, Nov 18, 2016 at 04:35:25PM +0100, Daniel Vetter wrote:
> > > On Fri, Nov 18, 2016 at 05:28:54PM +0200, Ville Syrjälä wrote:
> > > > On Fri, Nov 18, 2016 at 03:
When unloading the module, it is expected that we have finished
executing all requests and so the signal threads should be idle. Add a
warning in case there are any residual requests in the signaler rbtrees
at that point.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/intel_breadcrumbs.c |
Hi all,
New -testing cycle with cool stuff:
Final 4.10 updates:
- fine-tune fb flushing and tracking (Chris Wilson)
- refactor state check dumper code for more conciseness (Tvrtko)
- roll out dev_priv all over the place (Tvrkto)
- finally remove __i915__ magic macro (Tvrtko)
- more gvt bugfixes (
On Mon, Nov 21, 2016 at 10:38:20AM +0100, Daniel Vetter wrote:
> On Fri, Nov 18, 2016 at 09:44:49AM -0800, Manasi Navare wrote:
> > On Fri, Nov 18, 2016 at 06:21:21PM +0200, Ville Syrjälä wrote:
> > > On Fri, Nov 18, 2016 at 04:35:25PM +0100, Daniel Vetter wrote:
> > > > On Fri, Nov 18, 2016 at 05:
On 21/11/2016 09:27, Jani Nikula wrote:
On Fri, 18 Nov 2016, Tvrtko Ursulin wrote:
On 18/11/2016 14:20, Jani Nikula wrote:
The platform flags in device info are (mostly) mutually
exclusive. Replace the flags with an enum. Add the platform enum also
for platforms that previously didn't have a
On Mon, Nov 21, 2016 at 09:42:57AM +, Chris Wilson wrote:
> On Mon, Nov 21, 2016 at 10:38:20AM +0100, Daniel Vetter wrote:
> > On Fri, Nov 18, 2016 at 09:44:49AM -0800, Manasi Navare wrote:
> > > On Fri, Nov 18, 2016 at 06:21:21PM +0200, Ville Syrjälä wrote:
> > > > On Fri, Nov 18, 2016 at 04:3
== Series Details ==
Series: drm/i915: Add a warning on shutdown if signal threads still active
URL : https://patchwork.freedesktop.org/series/15641/
State : success
== Summary ==
Series 15641v1 drm/i915: Add a warning on shutdown if signal threads still
active
https://patchwork.freedesktop.o
On 21/11/2016 09:40, Chris Wilson wrote:
When unloading the module, it is expected that we have finished
executing all requests and so the signal threads should be idle. Add a
warning in case there are any residual requests in the signaler rbtrees
at that point.
Signed-off-by: Chris Wilson
---
On 18/11/2016 11:19, Tvrtko Ursulin wrote:
From: Tvrtko Ursulin
Keep parameters relating to the pipe and crtc respectively together.
Signed-off-by: Tvrtko Ursulin
Cc: Ville Syrjälä
---
drivers/gpu/drm/i915/intel_display.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(
On Mon, Nov 21, 2016 at 10:42:37AM +, Tvrtko Ursulin wrote:
>
> On 21/11/2016 09:40, Chris Wilson wrote:
> >When unloading the module, it is expected that we have finished
> >executing all requests and so the signal threads should be idle. Add a
> >warning in case there are any residual reques
On 18/11/2016 14:19, Ville Syrjälä wrote:
On Fri, Nov 18, 2016 at 03:13:14PM +0100, Maarten Lankhorst wrote:
Op 18-11-16 om 12:40 schreef Tvrtko Ursulin:
From: Tvrtko Ursulin
It just says "plane X disabled" which does not seem very useful.
Signed-off-by: Tvrtko Ursulin
Cc: Maarten Lankhors
On 21/11/2016 10:48, Chris Wilson wrote:
On Mon, Nov 21, 2016 at 10:42:37AM +, Tvrtko Ursulin wrote:
On 21/11/2016 09:40, Chris Wilson wrote:
When unloading the module, it is expected that we have finished
executing all requests and so the signal threads should be idle. Add a
warning in c
When unloading the module, it is expected that we have finished
executing all requests and so the signal threads should be idle. Add a
warning in case there are any residual requests in the signaler rbtrees
at that point.
v2: We can also warn if there are any waiters
Signed-off-by: Chris Wilson
Op 21-11-16 om 11:50 schreef Tvrtko Ursulin:
>
> On 18/11/2016 14:19, Ville Syrjälä wrote:
>> On Fri, Nov 18, 2016 at 03:13:14PM +0100, Maarten Lankhorst wrote:
>>> Op 18-11-16 om 12:40 schreef Tvrtko Ursulin:
From: Tvrtko Ursulin
It just says "plane X disabled" which does not seem
For the context ioctls, we record a debug, but we use the
DRM_DEBUG_DRIVER for what is clearly a user, not driver, action. Use
DRM_DEBUG instead.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_gem_context.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/driver
Patches #2 and #3 are Reviewed-by: Christian König
.
The rest is Acked-by: Christian König .
Regards,
Christian.
Am 18.11.2016 um 20:52 schrieb ville.syrj...@linux.intel.com:
From: Ville Syrjälä
Second installment of my effort to remove the duplicated
depth/bpp/pixel_format from drm_framebu
On 21 November 2016 at 10:25, Abdiel Janulgue
wrote:
> v7: Adapt to api rename
> v8: Restore sanitycheck wait on the recursive batch and
> avoid using C99 locals (Chris Wilson)
>
> Cc: Chris Wilson
> Cc: Daniel Vetter
> Signed-off-by: Abdiel Janulgue
> ---
> tests/gem_wait.c | 128
> -
On ma, 2016-11-21 at 11:31 +, Chris Wilson wrote:
> For the context ioctls, we record a debug, but we use the
> DRM_DEBUG_DRIVER for what is clearly a user, not driver, action. Use
> DRM_DEBUG instead.
>
> Signed-off-by: Chris Wilson
You could phrase the commit message to be more clear; "For
== Series Details ==
Series: drm/i915: Add a warning on shutdown if signal threads still active
(rev2)
URL : https://patchwork.freedesktop.org/series/15641/
State : success
== Summary ==
Series 15641v2 drm/i915: Add a warning on shutdown if signal threads still
active
https://patchwork.freed
On Sun, 2016-11-20 at 23:20 +0530, sunil.kam...@intel.com wrote:
> From: "A.Sunil Kamath"
>
> Better to use num_scaler itself while printing scaler_info.
> This fixes a bug of printing information for the missing
> second scaler on pipe C for SKL platform.
>
> Signed-off-by: A.Sunil Kamath
Rev
From: Tvrtko Ursulin
I am told the way it is done is not safe.
Signed-off-by: Tvrtko Ursulin
Suggested-by: Maarten Lankhorst
Cc: Maarten Lankhorst
Cc: Ville Syrjälä
---
drivers/gpu/drm/i915/intel_display.c | 38 +---
1 file changed, 1 insertion(+), 37 deletio
a PT page will be released if it doesn't contain any meaningful mappings
during PPGTT page table shrinking. The PT entry in the upper level will
be set to a scratch entry.
Normally this works nicely, but in virtualization world, the PPGTT page
table is tracked by hypervisor. Releasing the PT page
On Sun, 2016-11-20 at 18:16 +, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: Use num_scalers instead of SKL_NUM_SCALERS in debugfs
> URL : https://patchwork.freedesktop.org/series/15621/
> State : success
>
> == Summary ==
>
> Series 15621v1 drm/i915: Use num_scalers instea
On 11/21/2016 01:33 PM, Tomeu Vizoso wrote:
> On 21 November 2016 at 10:25, Abdiel Janulgue
> wrote:
>> v7: Adapt to api rename
>> v8: Restore sanitycheck wait on the recursive batch and
>> avoid using C99 locals (Chris Wilson)
>>
>> Cc: Chris Wilson
>> Cc: Daniel Vetter
>> Signed-off-by:
v7: Adapt to api rename
v8: Restore sanitycheck wait on the recursive batch and
avoid using C99 locals (Chris Wilson)
v9: Explicitly quit the batch instead of timing out right away
Cc: Chris Wilson
Cc: Daniel Vetter
Cc: Tomeu Vizoso
Signed-off-by: Abdiel Janulgue
---
tests/gem_wait.c | 12
In order to prevent a race between the old callback submitting an
incomplete request and i915_gem_set_wedged() installing its nop handler,
we must ensure that the swap occurs when the machine is idle
(stop_machine).
v2: move context lost from out of BKL.
Signed-off-by: Chris Wilson
---
drivers/
On Fri, Nov 18, 2016 at 01:58:35PM +, Chris Wilson wrote:
> On Fri, Nov 18, 2016 at 03:36:45PM +0200, David Weinehall wrote:
> > On resume we unbind+bind our VMA-mappings. This patch simplifies
> > this a bit by introducing a restore_vma() helper. As a nice side-effect
> > this also makes the
On Mon, Nov 21, 2016 at 02:21:39PM +0200, David Weinehall wrote:
> On Fri, Nov 18, 2016 at 01:58:35PM +, Chris Wilson wrote:
> > On Fri, Nov 18, 2016 at 03:36:45PM +0200, David Weinehall wrote:
> > > On resume we unbind+bind our VMA-mappings. This patch simplifies
> > > this a bit by introducin
In order to prevent a race between the old callback submitting an
incomplete request and i915_gem_set_wedged() installing its nop handler,
we must ensure that the swap occurs when the machine is idle
(stop_machine).
v2: move context lost from out of BKL.
Signed-off-by: Chris Wilson
---
drivers/
Chris Wilson writes:
> On Wed, Nov 16, 2016 at 05:20:34PM +0200, Mika Kuoppala wrote:
>> Bannable property, banned status, guilty and active counts are
>> properties of i915_gem_context. Make them so.
>>
>> v2: rebase
>>
>> Cc: Chris Wilson
>> Signed-off-by: Mika Kuoppala
>
> Been hesistating
Imre Deak writes:
> On ke, 2016-11-16 at 11:33 +0200, Mika Kuoppala wrote:
>> To find out what firmware we actually loaded (from dmesg) the explicit
>> 'dmc' and 'firmware' are missing from the info printout. Add them.
>>
>> Cc: Imre Deak
>> Signed-off-by: Mika Kuoppala
>
> Reviewed-by: Imre D
On Sun, Nov 20, 2016 at 10:13:10AM +0200, Laurent Pinchart wrote:
> Hi Ville,
>
> Thank you for the patch.
>
> On Friday 18 Nov 2016 21:53:12 ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > Allow drivers to return a custom drm_format_info structure for special
> > fb layout
Hi Ville,
On Monday 21 Nov 2016 15:18:23 Ville Syrjälä wrote:
> On Sun, Nov 20, 2016 at 10:13:10AM +0200, Laurent Pinchart wrote:
> > On Friday 18 Nov 2016 21:53:12 ville.syrj...@linux.intel.com wrote:
> >> From: Ville Syrjälä
> >>
> >> Allow drivers to return a custom drm_format_info structure
On Mon, 21 Nov 2016, Tvrtko Ursulin wrote:
> On 21/11/2016 09:27, Jani Nikula wrote:
>> On Fri, 18 Nov 2016, Tvrtko Ursulin wrote:
>>> On 18/11/2016 14:20, Jani Nikula wrote:
The platform flags in device info are (mostly) mutually
exclusive. Replace the flags with an enum. Add the platf
When a user context is closed, it's file_priv backpointer is replaced by
ERR_PTR(-EBADF); be careful not to chase this invalid pointer after a
hang and a GPU reset.
Signed-off-by: Chris Wilson
Fixes: b083a0870c79 ("drm/i915: Add per client max context ban limit")
Cc: Mika Kuoppala
---
drivers/g
On Mon, Nov 21, 2016 at 08:42:13AM +, Tvrtko Ursulin wrote:
>
> Hi,
>
> On 18/11/2016 19:53, ville.syrj...@linux.intel.com wrote:
> > From: Ville Syrjälä
> >
> > By providing our own format information for the CCS formats, we should
> > be able to make framebuffer_check() do the right thing
On Mon, Nov 21, 2016 at 03:23:19PM +0200, Laurent Pinchart wrote:
> Hi Ville,
>
> On Monday 21 Nov 2016 15:18:23 Ville Syrjälä wrote:
> > On Sun, Nov 20, 2016 at 10:13:10AM +0200, Laurent Pinchart wrote:
> > > On Friday 18 Nov 2016 21:53:12 ville.syrj...@linux.intel.com wrote:
> > >> From: Ville S
Hi Ville,
On Monday 21 Nov 2016 15:31:57 Ville Syrjälä wrote:
> On Mon, Nov 21, 2016 at 03:23:19PM +0200, Laurent Pinchart wrote:
> > On Monday 21 Nov 2016 15:18:23 Ville Syrjälä wrote:
> >> On Sun, Nov 20, 2016 at 10:13:10AM +0200, Laurent Pinchart wrote:
> >>> On Friday 18 Nov 2016 21:53:12 vill
On 21/11/2016 13:27, Ville Syrjälä wrote:
On Mon, Nov 21, 2016 at 08:42:13AM +, Tvrtko Ursulin wrote:
Hi,
On 18/11/2016 19:53, ville.syrj...@linux.intel.com wrote:
From: Ville Syrjälä
By providing our own format information for the CCS formats, we should
be able to make framebuffer_che
On Fri, Nov 18, 2016 at 04:20:32PM +0200, Jani Nikula wrote:
> The platform flags in device info are (mostly) mutually
> exclusive. Replace the flags with an enum. Add the platform enum also
> for platforms that previously didn't have a flag, and give them codename
> logging in dmesg.
>
> Pineview
On Mon, Nov 21, 2016 at 03:42:34PM +0200, Laurent Pinchart wrote:
> Hi Ville,
>
> On Monday 21 Nov 2016 15:31:57 Ville Syrjälä wrote:
> > On Mon, Nov 21, 2016 at 03:23:19PM +0200, Laurent Pinchart wrote:
> > > On Monday 21 Nov 2016 15:18:23 Ville Syrjälä wrote:
> > >> On Sun, Nov 20, 2016 at 10:13
On Fri, Nov 18, 2016 at 03:31:48PM -0800, Ben Widawsky wrote:
> On 16-11-18 21:53:13, Ville Syrjälä wrote:
> >From: Ville Syrjälä
> >
> >By providing our own format information for the CCS formats, we should
> >be able to make framebuffer_check() do the right thing for the CCS
> >surface as well.
On 21 November 2016 at 10:25, Abdiel Janulgue
wrote:
> More and more test-cases are using this.
>
> Signed-off-by: Abdiel Janulgue
> ---
> lib/igt_aux.c | 11 ---
> lib/igt_aux.h | 10 ++
> lib/igt_core.c | 3 ---
> tests/drv_hangman.c | 1 -
> 4 files changed,
On 21 November 2016 at 10:25, Abdiel Janulgue
wrote:
> A lot of igt testcases need some GPU workload to make sure a race
> window is big enough. Unfortunately having a fixed amount of
> workload leads to spurious test failures or overly long runtimes
> on some fast/slow platforms. This library con
v2: Remove definition in benchmarks/gem_syslatency.c
More and more test-cases are using this.
Signed-off-by: Abdiel Janulgue
---
benchmarks/gem_syslatency.c | 4
lib/igt_aux.c | 11 ---
lib/igt_aux.h | 10 ++
lib/igt_core.c | 3 ---
On 21 November 2016 at 13:07, Abdiel Janulgue
wrote:
> v7: Adapt to api rename
> v8: Restore sanitycheck wait on the recursive batch and
> avoid using C99 locals (Chris Wilson)
> v9: Explicitly quit the batch instead of timing out right away
>
> Cc: Chris Wilson
> Cc: Daniel Vetter
> Cc: Tom
On Mon, Nov 21, 2016 at 10:44:46AM +, Tvrtko Ursulin wrote:
>
>
> On 18/11/2016 11:19, Tvrtko Ursulin wrote:
> > From: Tvrtko Ursulin
> >
> > Keep parameters relating to the pipe and crtc respectively together.
> >
> > Signed-off-by: Tvrtko Ursulin
> > Cc: Ville Syrjälä
> > ---
> > driver
Looks good to me.
Reviewed-by: Tomeu Vizoso
Cheers,
Tomeu
On 21 November 2016 at 10:25, Abdiel Janulgue
wrote:
> v7: Adapt to api rename
> v8: Tidy up finish_fb_busy (Chris Wilson)
>
> Cc: Chris Wilson
> Cc: Daniel Vetter
> Signed-off-by: Abdiel Janulgue
> ---
> tests/kms_busy.c | 81
> +
On 21 November 2016 at 10:25, Abdiel Janulgue
wrote:
> v7: Reuse NSEC_PER_SEC defines
> v8: Don't wait on the fb, it must be busy. Exit the spin batch
> when flip is queued (Chris Wilson)
>
> Cc: Chris Wilson
> Cc: Daniel Vetter
> Signed-off-by: Abdiel Janulgue
> ---
> tests/kms_flip.c | 2
== Series Details ==
Series: drm/i915: Remove plane config from intel_dump_pipe_config
URL : https://patchwork.freedesktop.org/series/15650/
State : warning
== Summary ==
Series 15650v1 drm/i915: Remove plane config from intel_dump_pipe_config
https://patchwork.freedesktop.org/api/1.0/series/1
From: Robert Foss
This subtest verifies that waiting, timing out on a wait and that counting
fences in various states works.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 66 +
1 file changed, 66 insertions
From: Robert Foss
This subtest verifies that merging two fences works in the simples possible
case.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 23 +++
1 file changed, 23 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index f
From: Robert Foss
This subtests tests that creating fences on negative timelines fail.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index 8e7764a..f9d4fe9 100644
--- a/te
From: Robert Foss
This subtest verifies that creating many timelines and merging random fences
from each timeline with eachother results in merged fences that are fully
functional.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 73
From: Robert Foss
This subtest runs a single consumer thread and multiple producer thread that
are synchronized using multiple timelines.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 139
1 file changed,
From: Robert Foss
This test verifies that stressing the kernel by creating multiple
consumer/producer threads that wait on a single timeline to be incremented
by another conumer/producer thread does not fail.
And that the order amongst the threads is maintained.
Signed-off-by: Robert Foss
Revie
From: Robert Foss
Add subtest test_sync_busy_fork which increments the timeline in a forked child
process.
Signed-off-by: Robert Foss
---
tests/sw_sync.c | 39 +++
1 file changed, 39 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index 3a67877
From: Robert Foss
This subtest verifies merging a fence with itself does not fail.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index 96dcbff..ada12
From: Robert Foss
Make sure that this test is skipped if the sw_sync feature is missing from
the host system.
Signed-off-by: Robert Foss
---
lib/sw_sync.c | 1 +
tests/sw_sync.c | 3 +++
2 files changed, 4 insertions(+)
diff --git a/lib/sw_sync.c b/lib/sw_sync.c
index 38fe670..ac793ad 10064
From: Robert Foss
This subtest verifies the access ordering of multiple consumer threads.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 103
1 file changed, 103 insertions(+)
diff --git a/tests/sw_sync.c
From: Robert Foss
This subtest verifies that the fences of a timeline are not signalled when
a timelne is closed.
Signed-off-by: Robert Foss
---
tests/sw_sync.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index b5ea692..1d2c921 10064
From: Rafael Antognolli
This test creates an already expired fence, then creates a merged fence
out of that expired one (passed twice to the merge operation), and
finally closes the merged fence. It shows that if the refcounts are
wrong on the original expired fence, it might get freed while stil
From: Robert Foss
This series implements the sw_sync test and the lib/sw_sync helper functions
for said test.
The sw_sync subtests range from very basic tests of the sw_sync functionality,
to stress testing and randomized tests.
Changes since v1:
Added "Reviewed-by: Eric Engestrom " tag
From: Robert Foss
Base functions to help testing the Sync File Framework (explicit fencing
mechanism ported from Android).
These functions allow you to create, use and destroy timelines and fences.
Signed-off-by: Robert Foss
Signed-off-by: Gustavo Padovan
Reviewed-by: Eric Engestrom
---
lib/
From: Robert Foss
Add subtest alloc_fence that verifies that it's possible to allocate a fence
on a timeline.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 16
1 file changed, 16 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
inde
From: Robert Foss
Add subtest test_timeline_closed_signaled that verifies that a signaled fence
stays signaled after its timeline has been closed.
Signed-off-by: Robert Foss
---
tests/sw_sync.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_s
From: Robert Foss
This subtest verifies that waiting on fences works properly.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 51 +++
1 file changed, 51 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
From: Robert Foss
Add subtest test_sync_merge that tests merging fences and the validity of the
resulting merged fence.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 67 +
1 file changed, 67 insertions(+)
From: Robert Foss
Add subtest test_sync_busy_fork which increments the timeline in a forked child
process, where the timeline fd has been sent through a UNIX socket.
Signed-off-by: Robert Foss
---
tests/sw_sync.c | 103
1 file changed, 1
From: Robert Foss
Add igt_require_sw_sync to provide tests to skip if sw_sync support isn't
available on the host machine.
Signed-off-by: Robert Foss
---
lib/sw_sync.c | 22 ++
lib/sw_sync.h | 1 +
2 files changed, 23 insertions(+)
diff --git a/lib/sw_sync.c b/lib/sw_sync
From: Robert Foss
Add subtest test_sync_merge_invalid that tests merging invalid fences.
Signed-off-by: Robert Foss
---
tests/sw_sync.c | 41 +
1 file changed, 41 insertions(+)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
index 562b71f..3a67877 100644
== Series Details ==
Series: drm/i915: Move the release of PT page to the upper caller
URL : https://patchwork.freedesktop.org/series/15651/
State : success
== Summary ==
Series 15651v1 drm/i915: Move the release of PT page to the upper caller
https://patchwork.freedesktop.org/api/1.0/series/1
On Mon, Nov 21, 2016 at 11:10:45AM +0100, Daniel Vetter wrote:
> On Mon, Nov 21, 2016 at 09:42:57AM +, Chris Wilson wrote:
> > On Mon, Nov 21, 2016 at 10:38:20AM +0100, Daniel Vetter wrote:
> > > On Fri, Nov 18, 2016 at 09:44:49AM -0800, Manasi Navare wrote:
> > > > On Fri, Nov 18, 2016 at 06:2
On Mon, Nov 21, 2016 at 10:29:03AM -0500, robert.f...@collabora.com wrote:
> From: Robert Foss
>
> Add initial tests for sw_sync.
>
> Signed-off-by: Robert Foss
> Signed-off-by: Gustavo Padovan
> Reviewed-by: Eric Engestrom
> ---
> lib/sw_sync.c | 12 +--
> lib/sw_sync.h
== Series Details ==
Series: drm/i915: Stop the machine as we install the wedged submit_request
handler
URL : https://patchwork.freedesktop.org/series/15652/
State : success
== Summary ==
Series 15652v1 drm/i915: Stop the machine as we install the wedged
submit_request handler
https://patchw
== Series Details ==
Series: series starting with [1/3] drm/i915: Disable hangcheck when wedged
(rev2)
URL : https://patchwork.freedesktop.org/series/15543/
State : warning
== Summary ==
Series 15543v2 Series without cover letter
https://patchwork.freedesktop.org/api/1.0/series/15543/revision
Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu:
> This patch adds variable to check for X_tiled & y_tiled planes,
> instead
> of always checking against framebuffer-modifiers.
>
> Changes:
> - Created separate patch as per Paulo's comment
> - Added x_tiled variable as well
> Changes si
== Series Details ==
Series: drm/i915: Don't deref context->file_priv ERR_PTR upon reset
URL : https://patchwork.freedesktop.org/series/15658/
State : failure
== Summary ==
Series 15658v1 drm/i915: Don't deref context->file_priv ERR_PTR upon reset
https://patchwork.freedesktop.org/api/1.0/seri
Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu:
> If IPC is enabled in BXT, display underruns are observed.
> WA: The Line Time programmed in the WM_LINETIME register should be
> half of the actual calculated Line Time.
>
> Programmed Line Time = 1/2*Calculated Line Time
>
> Signed-off-
Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu:
> IPC (Isoch Priority Control) may cause underflows.
>
> KBL WA: When IPC is enabled, watermark latency values must be
> increased
> by 4us across all levels. This brings level 0 up to 6us.
>
> Signed-off-by: Mahesh Kumar
> ---
> drivers
On Sat, 19 Nov 2016 11:20:56 +0100
David Weinehall wrote:
> On Fri, Nov 18, 2016 at 02:11:56PM -0800, Bob Paauwe wrote:
> > For a single link (channel) DSI panel we want to use a larger divider
> > and keep the clock rate down to save power when in DPI/video mode. However
> > when using a dual-li
This is the next version of the patchset for adding Chamelium support into
intel-gpu-tools. The major changes (described in more details in the actual
patches):
- Connector mappings are specified in a configuration file, no more guessing
- Some fixes for tests that were throwing false negatives
- C
The default autoresume delay is about 5 seconds. It's possible on a
system that's not very fast this might not be a long enough time, since
an asynchronous hotplug event we scheduled on the chamelium that was
intended to happen during suspend could happen before we actually manage
to suspend. So, a
Since we're going to be using lists for keeping track of EDIDs we've
allocated on the chamelium, add some generic list helpers from the
wayland project.
Signed-off-by: Lyude
Changes since v1:
- Rename list helpers to be more like those from the Linux kernel
---
lib/igt_aux.c | 27 ++
Since all of the chamelium calls are blocking, we need to be able to
make suspend/resume tests with it multi-threaded. As such, it's not the
best idea to rely on igt_system_suspend_autoresume() for skipping tests
on systems without suspend/resume support since we could accidentally
leave the thread
This adds some basic helpers for connecting to udev and watching for
sysfs hotplug events.
Signed-off-by: Lyude
---
lib/igt_kms.c | 109 ++
lib/igt_kms.h | 6
2 files changed, 115 insertions(+)
diff --git a/lib/igt_kms.c b/lib/igt_k
For the purpose of testing things such as hotplugging and bad monitors,
the ChromeOS team ended up designing a neat little device known as the
Chamelium. More information on this can be found here:
https://www.chromium.org/chromium-os/testing/chamelium
This adds support for a couple of th
Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu:
> This patch adds IPC support for platforms. This patch enables IPC
> only for BXT/KBL platform as for SKL recommendation is to keep is
> disabled.
> IPC (Isochronous Priority Control) is the hardware feature, which
> dynamically controles t
On Mon, Nov 21, 2016 at 04:48:07PM +0100, Daniel Vetter wrote:
> On Mon, Nov 21, 2016 at 11:10:45AM +0100, Daniel Vetter wrote:
> > On Mon, Nov 21, 2016 at 09:42:57AM +, Chris Wilson wrote:
> > > On Mon, Nov 21, 2016 at 10:38:20AM +0100, Daniel Vetter wrote:
> > > > On Fri, Nov 18, 2016 at 09:4
1 - 100 of 121 matches
Mail list logo
