[Intel-gfx] [PATCH] drm/i915: Show FBC worker status in debugfs

2017-12-20 Thread Chris Wilson
Include the pending update from the FBC worker in i915_fbc_status. Signed-off-by: Chris Wilson Cc: Paulo Zanoni --- drivers/gpu/drm/i915/i915_debugfs.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/i915_pciids.h: synchronize with kernel header (rev3)

2017-12-20 Thread Patchwork
== Series Details == Series: lib/i915_pciids.h: synchronize with kernel header (rev3) URL : https://patchwork.freedesktop.org/series/35121/ State : success == Summary == IGT patchset tested on top of latest successful build 851c417b6b7a4ea28af67cfad116c8b7dc1d263e igt/kms_frontbuffer_tracking:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Show FBC worker status in debugfs

2017-12-20 Thread Patchwork
== Series Details == Series: drm/i915: Show FBC worker status in debugfs URL : https://patchwork.freedesktop.org/series/35646/ State : success == Summary == Series 35646v1 drm/i915: Show FBC worker status in debugfs https://patchwork.freedesktop.org/api/1.0/series/35646/revisions/1/mbox/ Test

Re: [Intel-gfx] [PATCH i-g-t] lib/i915_pciids.h: synchronize with kernel header

2017-12-20 Thread Rodrigo Vivi
On Wed, Dec 20, 2017 at 08:06:24PM +, De Marchi, Lucas wrote: > On Tue, 2017-12-19 at 15:28 -0800, Rodrigo Vivi wrote: > > On Tue, Dec 19, 2017 at 10:07:30PM +, Chris Wilson wrote: > > > Quoting Rodrigo Vivi (2017-12-19 21:59:43) > > > > On Fri, Dec 08, 2017 at 10:06:46PM +, Lucas De Ma

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/psr: Don't name status or debug registers like control registers.

2017-12-20 Thread Patchwork
== Series Details == Series: drm/i915/psr: Don't name status or debug registers like control registers. URL : https://patchwork.freedesktop.org/series/35644/ State : success == Summary == Test kms_flip: Subgroup plain-flip-ts-check: pass -> FAIL (shard-hsw)

Re: [Intel-gfx] [PATCH] drm/i915/psr: Don't name status or debug registers like control registers.

2017-12-20 Thread Rodrigo Vivi
On Wed, Dec 20, 2017 at 08:10:21PM +, Dhinakaran Pandiyan wrote: > Avoids some typo pitfalls. > > Cc: Chris Wilson > Cc: Rodrigo Vivi > Signed-off-by: Dhinakaran Pandiyan Reviewed-by: Rodrigo Vivi > --- > drivers/gpu/drm/i915/i915_debugfs.c | 4 ++-- > drivers/gpu/drm/i915/i915_reg.h

[Intel-gfx] ✗ Fi.CI.IGT: warning for lib/i915_pciids.h: synchronize with kernel header (rev3)

2017-12-20 Thread Patchwork
== Series Details == Series: lib/i915_pciids.h: synchronize with kernel header (rev3) URL : https://patchwork.freedesktop.org/series/35121/ State : warning == Summary == Test kms_flip: Subgroup vblank-vs-suspend-interruptible: pass -> DMESG-WARN (shard-snb) fdo#10

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Show FBC worker status in debugfs

2017-12-20 Thread Patchwork
== Series Details == Series: drm/i915: Show FBC worker status in debugfs URL : https://patchwork.freedesktop.org/series/35646/ State : warning == Summary == Test gem_softpin: Subgroup noreloc-s4: fail -> SKIP (shard-snb) fdo#103375 Subgroup noreloc-s

Re: [Intel-gfx] [PATCH i-g-t] lib/i915_pciids.h: synchronize with kernel header

2017-12-20 Thread Rodrigo Vivi
On Wed, Dec 20, 2017 at 08:55:24PM +, Lucas De Marchi wrote: > Synchronize with kernel header as of > c99d7832dcd7 ("drm/i915/cfl: Adding more Coffee Lake PCI IDs.") > > Signed-off-by: Lucas De Marchi Reviewed-by: Rodrigo Vivi > --- > lib/i915_pciids.h | 28 ++-- >

Re: [Intel-gfx] [PATCH i-g-t] lib/i915_pciids.h: synchronize with kernel header

2017-12-20 Thread Rodrigo Vivi
On Wed, Dec 20, 2017 at 11:21:28PM +, Rodrigo Vivi wrote: > On Wed, Dec 20, 2017 at 08:55:24PM +, Lucas De Marchi wrote: > > Synchronize with kernel header as of > > c99d7832dcd7 ("drm/i915/cfl: Adding more Coffee Lake PCI IDs.") > > > > Signed-off-by: Lucas De Marchi > > Reviewed-by: Ro

Re: [Intel-gfx] [PATCH i-g-t] scripts/trace.pl: Fix context id recycling

2017-12-20 Thread John Harrison
Looks good. Or at least, it looks to be the same as it was originally. Reviewed-by:  John Harrison On 12/20/2017 1:38 AM, Tvrtko Ursulin wrote: From: Tvrtko Ursulin Last refactoring introduced a copy and paste error, fix it. Signed-off-by: Tvrtko Ursulin Reported-by: John Harrison Cc: Jo

Re: [Intel-gfx] [PATCH i-g-t] scripts/trace.pl: Optimize event parsing and processing

2017-12-20 Thread John Harrison
On 12/20/2017 1:54 AM, Tvrtko Ursulin wrote: What was the effect of all this on your big traces? I am only testing with a smaller one which goes from ~3.3s to ~2.2s. On a larger trace it might be non-linear gains due to double sort avoidance, unless there will be some other effects to cancel th

Re: [Intel-gfx] [PATCH] i915: Reject CCS modifiers for pipe C on Geminilake

2017-12-20 Thread Rodrigo Vivi
On Wed, Dec 20, 2017 at 03:43:14PM +, Ben Widawsky wrote: > On 17-12-19 22:24:10, Gabriel Krisman Bertazi wrote: > > Current code advertises (on the modifiers blob property) support for CCS > > modifier for pipe C on GLK, only to reject it later when validating the > > request before the atomic

Re: [Intel-gfx] [-next PATCH 2/4] treewide: Use DEVICE_ATTR_RW

2017-12-20 Thread Zhang Rui
On Tue, 2017-12-19 at 10:15 -0800, Joe Perches wrote: > Convert DEVICE_ATTR uses to DEVICE_ATTR_RW where possible. > > Done with perl script: > > $ git grep -w --name-only DEVICE_ATTR | \ >   xargs perl -i -e 'local $/; while (<>) { > s/\bDEVICE_ATTR\s*\(\s*(\w+)\s*,\s*\(?(\s*S_IRUGO\s*\|\s*S_IWU

Re: [Intel-gfx] [-next PATCH 3/4] treewide: Use DEVICE_ATTR_RO

2017-12-20 Thread Zhang Rui
On Tue, 2017-12-19 at 10:15 -0800, Joe Perches wrote: > Convert DEVICE_ATTR uses to DEVICE_ATTR_RO where possible. > > Done with perl script: > > $ git grep -w --name-only DEVICE_ATTR | \ >   xargs perl -i -e 'local $/; while (<>) { > s/\bDEVICE_ATTR\s*\(\s*(\w+)\s*,\s*\(?(?:\s*S_IRUGO\s*|\s*0444

[Intel-gfx] [PATCH 2/2] drm/i915/guc : GEM_BUG_ON for GuC reset

2017-12-20 Thread Sujaritha Sundaresan
Including GEM_BUG_ON for GuC reset function in intel_uncore. Signed-off-by: Sujaritha Sundaresan Cc: Chris Wilson Cc: Michal Wajdeczko Cc: Sagar Arun Kamble --- drivers/gpu/drm/i915/intel_uncore.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel

[Intel-gfx] [PATCH 1/2] drm/i915/guc : Decoupling ADS and logs from submission

2017-12-20 Thread Sujaritha Sundaresan
The Additional Data Struct (ADS) contains objects that are required by GuC post FW load and are not necessarily submission-only. Even with submission disabled we may require something inside the ADS, so it makes more sense for them to be always created. Similarly, we still want to access GuC logs

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915/guc : Decoupling ADS and logs from submission

2017-12-20 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/guc : Decoupling ADS and logs from submission URL : https://patchwork.freedesktop.org/series/35650/ State : success == Summary == Series 35650v1 series starting with [1/2] drm/i915/guc : Decoupling ADS and logs from submission

[Intel-gfx] [PATCH] drm/i915/dmc: DMC 1.07 for Cannonlake

2017-12-20 Thread Anusha Srivatsa
There is a new version of DMC available for KBL. The release notes mentions: 1. Fix for the issue where DC_STATE was getting enabled even when disabled by driver causing data corruption Cc: Rodrigo Vivi Signed-off-by: Anusha Srivatsa --- drivers/gpu/drm/i915/intel_csr.c | 4 ++-- 1 file change

[Intel-gfx] [PATCH] CNL DMC v1.07

2017-12-20 Thread Anusha Srivatsa
Adding Pull Request to the cover letter. This is to trigger the CI systems and not the official Pull Request(which will be sent after ack from CI). The following changes since commit 2451bb228f6401ca3f2ecd498bf3bab613c532a7: linux-firmware: liquidio: add v1.7.0 vswitch firmware (2017-12-06 22:0

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dmc: DMC 1.07 for Cannonlake

2017-12-20 Thread Patchwork
== Series Details == Series: drm/i915/dmc: DMC 1.07 for Cannonlake URL : https://patchwork.freedesktop.org/series/35651/ State : success == Summary == Series 35651v1 drm/i915/dmc: DMC 1.07 for Cannonlake https://patchwork.freedesktop.org/api/1.0/series/35651/revisions/1/mbox/ Test debugfs_tes

Re: [Intel-gfx] [PATCH] drm/syncobj: Stop reusing the same struct file for all syncobj -> fd

2017-12-20 Thread Dave Airlie
> @@ -494,12 +473,11 @@ static int drm_syncobj_fd_to_handle(struct drm_file > *file_private, > spin_unlock(&file_private->syncobj_table_lock); > idr_preload_end(); > > - if (ret < 0) { > - fput(syncobj->file); > - return ret; > - } > -

[Intel-gfx] [PATCH i-g-t] syncobj: add test for handle->fd + close twice.

2017-12-20 Thread Dave Airlie
From: Dave Airlie This demos a bug found in the vulkan CTS that causes VFS: Close: file count is 0 in dmesg, and ls /proc/self/fd to oops. Signed-off-by: Dave Airlie --- tests/syncobj_basic.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/tests/syncobj_basic.c b/tests

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/guc : Decoupling ADS and logs from submission

2017-12-20 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/guc : Decoupling ADS and logs from submission URL : https://patchwork.freedesktop.org/series/35650/ State : success == Summary == Test kms_cursor_crc: Subgroup cursor-64x64-suspend: skip -> PASS

[Intel-gfx] [PULL] gvt-fixes for 4.15

2017-12-20 Thread Zhenyu Wang
Hi, Please pull one fix for 4.15 that correct default pipe enable for virtual display in a previous commit from Xiaolin. thanks -- The following changes since commit 11474e9091cf2002e948647fd9f63a7f027e488a: drm/i915/gvt: set max priority for gvt context (2017-12-06 11:38:21 +0800) are avail

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/dmc: DMC 1.07 for Cannonlake

2017-12-20 Thread Patchwork
== Series Details == Series: drm/i915/dmc: DMC 1.07 for Cannonlake URL : https://patchwork.freedesktop.org/series/35651/ State : success == Summary == Test kms_cursor_crc: Subgroup cursor-64x64-suspend: skip -> PASS (shard-snb) fdo#102365 Test gem_tiled_swap

[Intel-gfx] [PATCH i-g-t 2/2] tests/kms_lease: add tests for lease ioctls [v3]

2017-12-20 Thread Keith Packard
Validate that the leasing API creates leases that allow access to a subset of the available resources and that lease revocation works. v2: from Dave Airlie * Update ioctl numbers to latest proposed values. * Fix commit message * Add tests for get_lease and list_lessees v3: adapt to final ker

[Intel-gfx] [PATCH i-g-t 1/2] tests/kms_sequence: Add tests for new CRTC get/queue sequence ioctls [v2]

2017-12-20 Thread Keith Packard
These ioctls replace drmWaitVBlank and add ns time resolution and 64-bit sequence numbers to comply with the Vulkan API specifications. The tests were derived from the existing kms_vblank tests with the 'wait' variant elided as the new API doesn't provide a mechanism for blocking in the kernel. v

[Intel-gfx] [PATCH i-g-t 0/2] tests: Add tests for new sequence API and leases

2017-12-20 Thread Keith Packard
Both of these APIs are now in the kernel; it would be nice to have the tests integrated into the test suite. This includes an updated version of the kms_lease patch (v3) which adapts to the final kernel API changes. ___ Intel-gfx mailing list Intel-gfx@

[Intel-gfx] [PATCH] drm/dp: Power cycle display if LINK_ADDRESS fails.

2017-12-20 Thread Dhinakaran Pandiyan
Occasionally there are LINK_ADDRESS sideband messages timing out with the Lenovo MST dock + Dell MST monitor(w/ in-built branch) setup I have. These failures lead to the display not coming up on boot. Power cycling the port corresponding to the MST monitor's branch device and resending the message

Re: [Intel-gfx] [PATCH] drm/dp: Power cycle display if LINK_ADDRESS fails.

2017-12-20 Thread Jani Nikula
On Wed, 20 Dec 2017, Dhinakaran Pandiyan wrote: > Occasionally there are LINK_ADDRESS sideband messages timing out with the > Lenovo MST dock + Dell MST monitor(w/ in-built branch) setup I have. These > failures lead to the display not coming up on boot. Power cycling the port > corresponding to t

[Intel-gfx] ✓ Fi.CI.BAT: success for tests: Add tests for new sequence API and leases

2017-12-20 Thread Patchwork
== Series Details == Series: tests: Add tests for new sequence API and leases URL : https://patchwork.freedesktop.org/series/35657/ State : success == Summary == IGT patchset tested on top of latest successful build 382865bf1204241cdb609fbf8953a1de781368f7 lib/i915_pciids.h: synchronize with

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/dp: Power cycle display if LINK_ADDRESS fails.

2017-12-20 Thread Patchwork
== Series Details == Series: drm/dp: Power cycle display if LINK_ADDRESS fails. URL : https://patchwork.freedesktop.org/series/35659/ State : success == Summary == Series 35659v1 drm/dp: Power cycle display if LINK_ADDRESS fails. https://patchwork.freedesktop.org/api/1.0/series/35659/revisions

[Intel-gfx] ✗ Fi.CI.IGT: failure for tests: Add tests for new sequence API and leases

2017-12-20 Thread Patchwork
== Series Details == Series: tests: Add tests for new sequence API and leases URL : https://patchwork.freedesktop.org/series/35657/ State : failure == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-blt: pass -> FAIL (shard-

Re: [Intel-gfx] [PATCH] drm/syncobj: Stop reusing the same struct file for all syncobj -> fd

2017-12-20 Thread Chris Wilson
Quoting Dave Airlie (2017-12-21 02:42:56) > > @@ -494,12 +473,11 @@ static int drm_syncobj_fd_to_handle(struct drm_file > > *file_private, > > spin_unlock(&file_private->syncobj_table_lock); > > idr_preload_end(); > > > > - if (ret < 0) { > > - fput(syncobj->fil

<    1   2