[Intel-gfx] [lkp-robot] [lib/rbtree, drm/mm] 3e6e51217d: WARNING:at_lib/stackdepot.c:#depot_save_stack

2017-12-06 Thread kernel test robot
FYI, we noticed the following commit (built with gcc-7): commit: 3e6e51217dd14dcda10d4bc9a38b1440e2d42c14 ("lib/rbtree,drm/mm: Add rbtree_replace_node_cached()") git://anongit.freedesktop.org/drm-intel topic/core-for-CI in testcase: trinity with following parameters: runtime: 300s tes

[Intel-gfx] ✓ Fi.CI.BAT: success for overlay: fix debugfs path when debugfs mounted on path '/debug'

2017-12-06 Thread Patchwork
== Series Details == Series: overlay: fix debugfs path when debugfs mounted on path '/debug' URL : https://patchwork.freedesktop.org/series/35012/ State : success == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 meson: build a full de

Re: [Intel-gfx] [PATCH v4 0/9] drm/i915: Implement HDCP

2017-12-06 Thread Ramalingam C
Sean, Could you please share the level of functional testing is done on this code? like with Receiver/Repeaters and port type tested (DP/HDMI/DP over USB TypeC ?) Whether Compliance test is attempted on this code? Thanks -Ram On Thursday 07 December 2017 05:30 AM, Sean Paul wrote: Welcome

Re: [Intel-gfx] [PATCH v4 5/9] drm/i915: Add HDCP framework + base implementation

2017-12-06 Thread Ramalingam C
As v3 implementation removes the mode set from the path of HDCP state change, IMO that would have been preferred until we have the ville's changes mentioned by daniel. Now once again modeset is brought back :( And have we already thought about holding the modeset lock for 5+ Sec for HDCP a

Re: [Intel-gfx] [PATCH v4 4/9] drm: Add some HDCP related #defines

2017-12-06 Thread Ramalingam C
Looks Good to me. Reviewed-by: Ramalingam C -Ram On Thursday 07 December 2017 05:30 AM, Sean Paul wrote: In preparation for implementing HDCP in i915, add some HDCP related register offsets and defines. The dpcd register offsets will go in drm_dp_helper.h whereas the ddc offsets along with ge

[Intel-gfx] [PATCH i-g-t] overlay: fix debugfs path when debugfs mounted on path '/debug'

2017-12-06 Thread changbin . du
From: Changbin Du It mistakenly set debugfs root path to "/debug/dri", so correct it. Signed-off-by: Changbin Du --- overlay/debugfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/overlay/debugfs.c b/overlay/debugfs.c index 9f3e5cc..5516949 100644 --- a/overlay/debugfs.c

Re: [Intel-gfx] [PATCH v4 8/9] drm/i915: Implement HDCP for HDMI

2017-12-06 Thread Ramalingam C
On Thursday 07 December 2017 05:30 AM, Sean Paul wrote: This patch adds HDCP support for HDMI connectors by implementing the intel_hdcp_shim. Nothing too special, just a bunch of DDC reads/writes. Changes in v2: - Rebased on drm-intel-next Changes in v3: - Initialize new worker Changes in v4:

Re: [Intel-gfx] [PATCH v4 9/9] drm/i915: Implement HDCP for DisplayPort

2017-12-06 Thread Ramalingam C
Thanks for handling the reauth req from downstream. you might want to fix the missed single occurance of "//" On Thursday 07 December 2017 05:30 AM, Sean Paul wrote: This patch adds HDCP support for DisplayPort connectors by implementing the intel_hdcp_shim. Most of this is straightforward re

[Intel-gfx] ✗ Fi.CI.BAT: failure for mmio save restore refine in vgpu switch

2017-12-06 Thread Patchwork
== Series Details == Series: mmio save restore refine in vgpu switch URL : https://patchwork.freedesktop.org/series/35011/ State : failure == Summary == CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrelease.h CHK i

[Intel-gfx] [PATCH 2/3] drm/i915/gvt: refine mocs save restore policy

2017-12-06 Thread Weinan Li
Save and restore the mocs regs of one VM in GVT-g burning too much CPU utilization. Add LRI command scan to monitor the change of mocs registers, save the state in vreg, and use delta update policy to restore them. It can obviously reduce the MMIO r/w count, and improve the performance of context s

[Intel-gfx] [PATCH 3/3] drm/i915/gvt: load host render mocs once in mocs switch

2017-12-06 Thread Weinan Li
Load host render mocs registers once for delta update of mocs switch, it reduces mmio read times obviously, then brings performance improvement during multi-vms switch. Signed-off-by: Weinan Li --- drivers/gpu/drm/i915/gvt/render.c | 51 --- 1 file changed, 42

[Intel-gfx] [PATCH 0/3] mmio save restore refine in vgpu switch

2017-12-06 Thread Weinan Li
Merge switch_mmio_to_vgpu and switch_mmio_to_host, use delta update for mocs save restore, deal host mocs value as fixed, it won't be changed after initialization. These can save vgpu switch time to reduce CPU utilization and improve GPU performance in GVT-g with multi-VMs. Weinan Li (3): drm/i9

[Intel-gfx] [PATCH 1/3] drm/i915/gvt: optimize for vGPU mmio switch

2017-12-06 Thread Weinan Li
now mmio switch between vGPUs need to switch to host first then to expected vGPU, it waste one time mmio save/restore. r/w mmio usually is time-consuming, and there are so many mocs registers need to save/restore during vGPU switch. Combine the switch_to_host and switch_to_vgpu can reduce 1 time mm

[Intel-gfx] ✗ Fi.CI.IGT: warning for series starting with [1/4] lib: copy intel_aub.h from libdrm

2017-12-06 Thread Patchwork
== Series Details == Series: series starting with [1/4] lib: copy intel_aub.h from libdrm URL : https://patchwork.freedesktop.org/series/34999/ State : warning == Summary == Test pm_rc6_residency: Subgroup rc6-accuracy: pass -> SKIP (shard-snb) Test gem_soft

[Intel-gfx] ✓ Fi.CI.IGT: success for igt/perf_pmu: Tweak wait_for_rc6, yet again

2017-12-06 Thread Patchwork
== Series Details == Series: igt/perf_pmu: Tweak wait_for_rc6, yet again URL : https://patchwork.freedesktop.org/series/34998/ State : success == Summary == Test kms_cursor_crc: Subgroup cursor-64x64-suspend: pass -> SKIP (shard-snb) fdo#102365 Test pm_rpm:

Re: [Intel-gfx] [RFC 0/4] GPU/CPU timestamps correlation for relating OA samples with system events

2017-12-06 Thread Robert Bragg
On Thu, Dec 7, 2017 at 12:48 AM, Robert Bragg wrote: > > at least from what I wrote back then it looks like I was seeing a drift of > a few milliseconds per second on SKL. I vaguely recall it being much worse > given the frequency constants we had for Haswell. > Sorry I didn't actually re-read m

Re: [Intel-gfx] [PATCH v3 04/11] drm/exynos: Use drm_fb_helper_lastclose() and _poll_changed()

2017-12-06 Thread Inki Dae
2017년 12월 06일 03:24에 Noralf Trønnes 이(가) 쓴 글: > This driver can use drm_fb_helper_lastclose() as its .lastclose callback. > It can also use drm_fb_helper_output_poll_changed() as its > .output_poll_changed callback. > > Cc: Inki Dae > Cc: Joonyoung Shim > Cc: Seung-Woo Kim > Cc: Kyungmin Park

Re: [Intel-gfx] [RFC 0/4] GPU/CPU timestamps correlation for relating OA samples with system events

2017-12-06 Thread Robert Bragg
On Wed, Nov 15, 2017 at 12:13 PM, Sagar Arun Kamble < sagar.a.kam...@intel.com> wrote: > We can compute system time corresponding to GPU timestamp by taking a > reference point (CPU monotonic time, GPU timestamp) and then adding > delta time computed using timecounter/cyclecounter support in kerne

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [RFC,1/5] igt_dummyload: Wrap function parameters into struct

2017-12-06 Thread Patchwork
== Series Details == Series: series starting with [RFC,1/5] igt_dummyload: Wrap function parameters into struct URL : https://patchwork.freedesktop.org/series/34997/ State : failure == Summary == Test kms_flip: Subgroup wf_vblank: pass -> DMESG-WARN (shard-hsw)

[Intel-gfx] [PATCH v2 1/2] CONTRIBUTING: Fix spelling mistake and line length

2017-12-06 Thread Sean Paul
Noticed while I was reading it. Makes for a good first contribution, I guess. Changes in v2: - None Reviewed-by: Petri Latvala Signed-off-by: Sean Paul --- CONTRIBUTING | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/CONTRIBUTING b/CONTRIBUTING index 561c5dd8..ca2ed

[Intel-gfx] [PATCH v2 2/2] kms_content_protection: Add Content Protection test

2017-12-06 Thread Sean Paul
Pretty simple test: - initializes the output - clears the content protection property - verifies that it clears - sets the content protection property to desired - verifies that it transitions to enabled Does this for both legacy and atomic. Changes in v2: - Don't check for i915 gen - Skip test i

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Implement HDCP (rev4)

2017-12-06 Thread Sean Paul
On Wed, Dec 6, 2017 at 7:08 PM, Patchwork wrote: > == Series Details == > > Series: drm/i915: Implement HDCP (rev4) > URL : https://patchwork.freedesktop.org/series/34671/ > State : failure > Ah, son of a beesting! I'll rebase on -tip and resend once people have time to provide feedback. Sean

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Implement HDCP (rev4)

2017-12-06 Thread Patchwork
== Series Details == Series: drm/i915: Implement HDCP (rev4) URL : https://patchwork.freedesktop.org/series/34671/ State : failure == Summary == Applying: drm: Fix link-status kerneldoc line lengths error: Failed to merge in the changes. Using index info to reconstruct a base tree... M d

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/4] lib: copy intel_aub.h from libdrm

2017-12-06 Thread Patchwork
== Series Details == Series: series starting with [1/4] lib: copy intel_aub.h from libdrm URL : https://patchwork.freedesktop.org/series/34999/ State : success == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 meson: build a full depen

[Intel-gfx] [PATCH v4 9/9] drm/i915: Implement HDCP for DisplayPort

2017-12-06 Thread Sean Paul
This patch adds HDCP support for DisplayPort connectors by implementing the intel_hdcp_shim. Most of this is straightforward read/write from/to DPCD registers. One thing worth pointing out is the Aksv output bit. It wasn't easily separable like it's HDMI counterpart, so it's crammed in with the re

[Intel-gfx] [PATCH v4 8/9] drm/i915: Implement HDCP for HDMI

2017-12-06 Thread Sean Paul
This patch adds HDCP support for HDMI connectors by implementing the intel_hdcp_shim. Nothing too special, just a bunch of DDC reads/writes. Changes in v2: - Rebased on drm-intel-next Changes in v3: - Initialize new worker Changes in v4: - Remove SKL_ prefix from most register names (Daniel) - Wr

[Intel-gfx] [PATCH v4 4/9] drm: Add some HDCP related #defines

2017-12-06 Thread Sean Paul
In preparation for implementing HDCP in i915, add some HDCP related register offsets and defines. The dpcd register offsets will go in drm_dp_helper.h whereas the ddc offsets along with generic HDCP stuff will get stuffed in drm_hdcp.h, which is new. Changes in v2: - drm_hdcp.h gets MIT license (D

[Intel-gfx] [PATCH v4 5/9] drm/i915: Add HDCP framework + base implementation

2017-12-06 Thread Sean Paul
This patch adds the framework required to add HDCP support to intel connectors. It implements Aksv loading from fuse, and parts 1/2/3 of the HDCP authentication scheme. Note that without shim implementations, this does not actually implement HDCP. That will come in subsequent patches. Changes in

[Intel-gfx] [PATCH v4 6/9] drm/i915: Make use of indexed write GMBUS feature

2017-12-06 Thread Sean Paul
This patch enables the indexed write feature of the GMBUS to concatenate 2 consecutive messages into one. The criteria for an indexed write is that both messages are writes, the first is length == 1, and the second is length > 0. The first message is sent out by the GMBUS as the slave command, and

[Intel-gfx] [PATCH v4 7/9] drm/i915: Add function to output Aksv over GMBUS

2017-12-06 Thread Sean Paul
Once the Aksv is available in the PCH, we need to get it on the wire to the receiver via DDC. The hardware doesn't allow us to read the value directly, so we need to tell GMBUS to source the Aksv internally and send it to the right offset on the receiver. The way we do this is to initiate an index

[Intel-gfx] [PATCH v4 2/9] drm/i915: Add more control to wait_for routines

2017-12-06 Thread Sean Paul
This patch adds a little more control to a couple wait_for routines such that we can avoid open-coding read/wait/timeout patterns which: - need the value of the register after the wait_for - run arbitrary operation for the read portion This patch also chooses the correct sleep function (based on

[Intel-gfx] [PATCH v4 3/9] drm: Add Content Protection property

2017-12-06 Thread Sean Paul
This patch adds a new optional connector property to allow userspace to enable protection over the content it is displaying. This will typically be implemented by the driver using HDCP. The property is a tri-state with the following values: - OFF: Self explanatory, no content protection - DESIRED:

[Intel-gfx] [PATCH v4 0/9] drm/i915: Implement HDCP

2017-12-06 Thread Sean Paul
Welcome to version 4 of the patchset. I think we're nearing the finish line (hopefully) now. This set addresses the review feedback from v3. I applied some R-b's from v3 review, and converted others to Cc since other changes were made to the patch, and I didn't want to speak for reviewers. Thanks

[Intel-gfx] [PATCH v4 1/9] drm: Fix link-status kerneldoc line lengths

2017-12-06 Thread Sean Paul
I'm adding some stuff below it and it's killing my editor's vibe. Changes in v2: - Added to the series Changes in v3: - None Changes in v4: - None Cc: Manasi Navare Acked-by: Daniel Vetter Signed-off-by: Sean Paul --- drivers/gpu/drm/drm_connector.c | 9 + 1 file changed, 5 insertions

Re: [Intel-gfx] [PATCH 4/5] drm/i915: Introduce a non-blocking power domain for vblank interrupts

2017-12-06 Thread Rodrigo Vivi
On Wed, Dec 06, 2017 at 10:47:40PM +, Dhinakaran Pandiyan wrote: > When DC states are enabled and PSR is active, the hardware enters DC5/DC6 > states resulting in frame counter resets. The frame counter resets mess > up the vblank counting logic. So in order to disable DC states when > vblank i

[Intel-gfx] ✓ Fi.CI.BAT: success for igt/perf_pmu: Tweak wait_for_rc6, yet again

2017-12-06 Thread Patchwork
== Series Details == Series: igt/perf_pmu: Tweak wait_for_rc6, yet again URL : https://patchwork.freedesktop.org/series/34998/ State : success == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 meson: build a full dependency for lib_ig

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [RFC,1/5] igt_dummyload: Wrap function parameters into struct

2017-12-06 Thread Patchwork
== Series Details == Series: series starting with [RFC,1/5] igt_dummyload: Wrap function parameters into struct URL : https://patchwork.freedesktop.org/series/34997/ State : success == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 me

Re: [Intel-gfx] [RFC i-g-t 4/5] igt_dummyload: Add preemptible parameter to spin batch

2017-12-06 Thread Chris Wilson
Quoting Antonio Argenziano (2017-12-06 23:03:45) > void emit_recursive_batch(igt_spin_t *spin, int fd, igt_spin_opt_t opts); > diff --git a/lib/igt_gt.c b/lib/igt_gt.c > index 9ec3b0f6..48d40e61 100644 > --- a/lib/igt_gt.c > +++ b/lib/igt_gt.c > @@ -293,7 +293,8 @@ igt_hang_t igt_hang_ctx(int fd,

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/5] drm/vblank: Do not update vblank counts if vblanks are already disabled.

2017-12-06 Thread Patchwork
== Series Details == Series: series starting with [1/5] drm/vblank: Do not update vblank counts if vblanks are already disabled. URL : https://patchwork.freedesktop.org/series/34996/ State : failure == Summary == Series 34996v1 series starting with [1/5] drm/vblank: Do not update vblank coun

[Intel-gfx] [PATCH i-g-t 2/4] tools/intel_aubdump: Set addr_bits before write_header

2017-12-06 Thread Scott D Phillips
write_header() uses addr_bits, so do the initialization earlier. Also set the gen to a non-zero value in case of unknown device, for use by a later patch. Signed-off-by: Scott D Phillips Reviewed-by: Jordan Justen --- tools/aubdump.c | 14 +- 1 file changed, 9 insertions(+), 5 delet

[Intel-gfx] [PATCH igt] igt/perf_pmu: Tweak wait_for_rc6, yet again

2017-12-06 Thread Chris Wilson
Still CI remains obstinate that RC6 is not smoothly incrementing during the sample period. Tweak the wait_for_rc6() to first wait for the initial Evaluation Interval before polling. Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin --- tests/perf_pmu.c | 15 +++ tests/pm_rc6_re

[Intel-gfx] [PATCH i-g-t v2 4/4] tools/intel_aubdump: Add ability to simulate execlist submission

2017-12-06 Thread Scott D Phillips
Newer devices do not have the legacy ring buffer submission model, so aub files generated using that model cannot be handled by some internal tools. The execlist submission modeled by this change is pretty simplistic, using GGTT only and synchronizing after every batch. v2: - Move addr_bits init i

[Intel-gfx] [PATCH i-g-t 3/4] lib/intel_aub: Add new MEM_TRACE commands

2017-12-06 Thread Scott D Phillips
The memtrace aub commands are similar to the existing ones, but different. Notably memtrace has commands for register write and poll. Signed-off-by: Scott D Phillips --- lib/intel_aub.h | 26 ++ 1 file changed, 26 insertions(+) diff --git a/lib/intel_aub.h b/lib/intel_au

[Intel-gfx] [PATCH i-g-t 1/4] lib: copy intel_aub.h from libdrm

2017-12-06 Thread Scott D Phillips
No functionality related to aub is provided by libdrm aside from intel_aub.h which somewhat defines the file format. Move the header into this project to ease aub-related development. Signed-off-by: Scott D Phillips --- lib/Makefile.am | 1 - lib/{stubs/drm => }/intel_aub.h | 0

[Intel-gfx] [RFC i-g-t 3/5] lib/igt_hang_ctx: Use dummyload batch to hang ctx.

2017-12-06 Thread Antonio Argenziano
To hang a context we were effectively reimplementing a spinning batch and never stopping it. This patch reuses the recursive batch from igt_dummyload to hang a context. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_dummyload.c | 4 +++- lib/igt_dummyload.h | 2 ++ lib/igt_gt.

[Intel-gfx] [RFC i-g-t 2/5] igt_hang_ctx: Wrap parameters into struct

2017-12-06 Thread Antonio Argenziano
This patch wraps the parameters for the function used to hang a context (igt_hang_ctx) into a struct to cleanup the interface and make it easier to add more parameters. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_dummyload.c | 25 - lib/igt_gt.c

[Intel-gfx] [RFC i-g-t 5/5] igt_hang_ctx: Add preemptible parameter

2017-12-06 Thread Antonio Argenziano
This patch adds a parameter to make the hanging context pre-emptible. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_gt.c| 4 ++-- lib/igt_gt.h| 1 + tests/drv_hangman.c | 2 +- tests/gem_reset_stats.c | 2 +- tests/gem_softpin.c | 2 +- 5 files c

[Intel-gfx] [RFC i-g-t 4/5] igt_dummyload: Add preemptible parameter to spin batch

2017-12-06 Thread Antonio Argenziano
This patch adds a parameter that allows to make the spinning batch pre-emptible by adding an arbitration point to the spinning loop. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_dummyload.c | 4 ++-- lib/igt_dummyload.h | 1 + lib/igt_gt.c | 3 ++-

[Intel-gfx] [RFC i-g-t 1/5] igt_dummyload: Wrap function parameters into struct

2017-12-06 Thread Antonio Argenziano
The intent of this patch is to clean-up the interface of the spinning batch workload by grouping the parameters used to spawn it into a struct. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_dummyload.c | 19 +++ lib/igt_dummyload.h | 16 -

[Intel-gfx] [PATCH 4/5] drm/i915: Introduce a non-blocking power domain for vblank interrupts

2017-12-06 Thread Dhinakaran Pandiyan
When DC states are enabled and PSR is active, the hardware enters DC5/DC6 states resulting in frame counter resets. The frame counter resets mess up the vblank counting logic. So in order to disable DC states when vblank interrupts are required and to disallow DC states when vblanks interrupts are

[Intel-gfx] [PATCH 1/5] drm/vblank: Do not update vblank counts if vblanks are already disabled.

2017-12-06 Thread Dhinakaran Pandiyan
Updating the vblank counts requires register reads and these reads may not return meaningful values after the vblank interrupts are disabled as the device may go to low power state. An additional change would be to allow the driver to save the vblank counts before entering a low power state, but th

[Intel-gfx] [PATCH 5/5] drm/i915: Use the vblank power domain disallow or disable DC states.

2017-12-06 Thread Dhinakaran Pandiyan
Disable DC states before enabling vblank interrupts and conversely enable DC states after disabling. Since the frame counter may have got reset between disabling and enabling, use drm_crtc_vblank_restore() to compute the missed vblanks. Signed-off-by: Dhinakaran Pandiyan --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH 3/5] drm/i915: Use an atomic_t array to track power domain use count.

2017-12-06 Thread Dhinakaran Pandiyan
Convert the power_domains->domain_use_count array that tracks per-domain use count to atomic_t type. This is needed to be able to read/write the use counts outside of the power domain mutex. Signed-off-by: Dhinakaran Pandiyan --- drivers/gpu/drm/i915/i915_debugfs.c | 2 +- drivers/gpu/drm/i

[Intel-gfx] [PATCH 2/5] drm/vblank: Restoring vblank counts after device runtime PM events.

2017-12-06 Thread Dhinakaran Pandiyan
The HW frame counter can get reset when devices enters low power states and this messes up any following vblank count updates. So, compute the missed vblank interrupts for that low power state duration using time stamps. This is similar to _crtc_vblank_on() except that it doesn't enable vblank inte

[Intel-gfx] ✗ Fi.CI.BAT: failure for Revert "drm/i915: Display WA #1133 WaFbcSkipSegments:cnl, glk" (rev2)

2017-12-06 Thread Patchwork
== Series Details == Series: Revert "drm/i915: Display WA #1133 WaFbcSkipSegments:cnl, glk" (rev2) URL : https://patchwork.freedesktop.org/series/33969/ State : failure == Summary == Applying: Revert "drm/i915: Display WA #1133 WaFbcSkipSegments:cnl, glk" error: Failed to merge in the changes.

[Intel-gfx] [PATCH] Revert "drm/i915: Display WA #1133 WaFbcSkipSegments:cnl, glk"

2017-12-06 Thread Rodrigo Vivi
From: Radhakrishna Sripada This reverts commit 8f067837c4b713ce2e69be95af7b2a5eb3bd7de8. HSD says "WA withdrawn. It was causing corruption with some images. WA is not strictly necessary since this bug just causes loss of FBC compression with some sizes and images, but doesn't break anything." F

[Intel-gfx] [PULL] drm-misc-fixes

2017-12-06 Thread Daniel Vetter
Hi Dave, drm-misc-fixes-2017-12-06: Just the connector_iter corner-case regression fix. Enjoy your time off next week! Cheers, Daniel The following changes since commit ae64f9bd1d3621b5e60d7363bc20afb46aede215: Linux 4.15-rc2 (2017-12-03 11:01:47 -0500) are available in the Git repository

[Intel-gfx] [PATCH] drm/i915: Use copy_from_user() in fence copying

2017-12-06 Thread Kees Cook
There's no good reason to separate the access_ok() from the copy, especially since the access_ok() size is hard-coded instead of using sizeof(). Instead, just use copy_from_user() directly. Fixes: cf6e7bac6357 ("drm/i915: Add support for drm syncobjs") Cc: Jason Ekstrand Cc: Chris Wilson Signed-

Re: [Intel-gfx] [RFC 0/4] GPU/CPU timestamps correlation for relating OA samples with system events

2017-12-06 Thread Lionel Landwerlin
I've put together some trival IGT tests : https://github.com/djdeath/intel-gpu-tools/commits/wip/djdeath/cpu-timestamps With a few changes which I pointed in the review : https://github.com/djdeath/linux/commit/d0e4cf4d3f464491b4ffe97d112284d1ce73656d Put together it seems to work relatively we

Re: [Intel-gfx] [RFC 3/4] drm/i915/perf: Extract raw GPU timestamps from OA reports

2017-12-06 Thread Lionel Landwerlin
On 15/11/17 12:13, Sagar Arun Kamble wrote: From: Sourab Gupta The OA reports contain the least significant 32 bits of the gpu timestamp. This patch enables retrieval of the timestamp field from OA reports, to forward as 64 bit raw gpu timestamps in the perf samples. v2: Rebase w.r.t new timec

[Intel-gfx] ✓ Fi.CI.IGT: success for make stolen resource centric (rev5)

2017-12-06 Thread Patchwork
== Series Details == Series: make stolen resource centric (rev5) URL : https://patchwork.freedesktop.org/series/34256/ State : success == Summary == Warning: bzip CI_DRM_3466/shard-snb2/results25.json.bz2 wasn't in correct JSON format Test perf: Subgroup oa-exponents:

Re: [Intel-gfx] [PATCH] e1000e: Taint a HW lockup

2017-12-06 Thread Jeff Kirsher
On Wed, 2017-12-06 at 10:47 +0100, Daniel Vetter wrote: > On Tue, Dec 5, 2017 at 7:05 PM, Chris Wilson k> wrote: > > Quoting Chris Wilson (2017-12-05 18:00:00) > > > When we see an e1000e HW lockup in CI, it is typically fatal with > > > the > > > hang repeating until the host is forcibly rebooted

[Intel-gfx] ✗ Fi.CI.BAT: failure for lib/igt_sysfs: Let igt_sysfs_read|write return -errno

2017-12-06 Thread Patchwork
== Series Details == Series: lib/igt_sysfs: Let igt_sysfs_read|write return -errno URL : https://patchwork.freedesktop.org/series/34989/ State : failure == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 meson: build a full dependency f

Re: [Intel-gfx] [PATCH i-g-t] lib/igt_sysfs: Let igt_sysfs_read|write return -errno

2017-12-06 Thread Chris Wilson
Quoting Michal Wajdeczko (2017-12-06 19:00:22) > In some cases debugfs or sysfs may return errors that we > want to check. Return -errno from helper functions to make > asserts easier. > > Signed-off-by: Michal Wajdeczko > Cc: Chris Wilson > Cc: Joonas Lahtinen > --- > lib/igt_sysfs.c | 18 +++

[Intel-gfx] [PATCH i-g-t] lib/igt_sysfs: Let igt_sysfs_read|write return -errno

2017-12-06 Thread Michal Wajdeczko
In some cases debugfs or sysfs may return errors that we want to check. Return -errno from helper functions to make asserts easier. Signed-off-by: Michal Wajdeczko Cc: Chris Wilson Cc: Joonas Lahtinen --- lib/igt_sysfs.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-)

[Intel-gfx] ✗ Fi.CI.IGT: warning for series starting with [1/2] lib: Export kmsg()

2017-12-06 Thread Patchwork
== Series Details == Series: series starting with [1/2] lib: Export kmsg() URL : https://patchwork.freedesktop.org/series/34974/ State : warning == Summary == Warning: bzip CI_DRM_3466/shard-snb2/results25.json.bz2 wasn't in correct JSON format Test drv_module_reload: Subgroup basic-n

Re: [Intel-gfx] [PATCH 01/10] x86/early-quirks: Extend Intel graphics stolen memory placement to 64bit

2017-12-06 Thread Chris Wilson
Quoting Matthew Auld (2017-12-06 18:17:21) > From: Joonas Lahtinen > > To give upcoming SKU BIOSes more flexibility in placing the Intel > graphics stolen memory, make all variables storing the placement or size > compatible with full 64 bit range. Also by exporting the stolen region > as a resou

Re: [Intel-gfx] [PATCH 10/10] drm/i915: prefer stolen_usable_size for the range sanity check

2017-12-06 Thread Chris Wilson
Quoting Matthew Auld (2017-12-06 18:17:30) > In i915_pages_create_for_stolen it probably makes more sense to check if > the range overflows the stolen_usable_size, since the size of dsm will also > include the reserved portion which we can't touch. > > Signed-off-by: Matthew Auld > Cc: Joonas Lah

Re: [Intel-gfx] [PATCH 09/10] drm/i915: prefer resource_size_t for everything stolen

2017-12-06 Thread Chris Wilson
Quoting Matthew Auld (2017-12-06 18:17:29) > Keeps things consistent now that we make use of struct resource. This > should keep us covered in case we ever get huge amounts of stolen > memory. > > v2: bunch of missing conversions (Chris) > > Signed-off-by: Matthew Auld > Cc: Joonas Lahtinen > C

[Intel-gfx] ✓ Fi.CI.BAT: success for make stolen resource centric (rev5)

2017-12-06 Thread Patchwork
== Series Details == Series: make stolen resource centric (rev5) URL : https://patchwork.freedesktop.org/series/34256/ State : success == Summary == Series 34256v5 make stolen resource centric https://patchwork.freedesktop.org/api/1.0/series/34256/revisions/5/mbox/ Test gem_mmap_gtt:

[Intel-gfx] ✗ Fi.CI.IGT: warning for lib/rendercopy: Refactoring rendercopy libraries

2017-12-06 Thread Patchwork
== Series Details == Series: lib/rendercopy: Refactoring rendercopy libraries URL : https://patchwork.freedesktop.org/series/34972/ State : warning == Summary == Warning: bzip CI_DRM_3466/shard-snb2/results25.json.bz2 wasn't in correct JSON format Test kms_flip: Subgroup vblank-vs-sus

Re: [Intel-gfx] [PATCH 08/10] drm/i915: give stolen_usable_size a more suitable home

2017-12-06 Thread Chris Wilson
Quoting Matthew Auld (2017-12-06 18:17:28) > Kick it out of i915_ggtt and keep it grouped with dsm and dsm_reserved, > where it makes the most sense. > > Signed-off-by: Matthew Auld > Cc: Joonas Lahtinen > Cc: Chris Wilson > Cc: Paulo Zanoni > --- > drivers/gpu/drm/i915/i915_drv.h| 12

Re: [Intel-gfx] [PATCH 05/10] drm/i915: make dsm struct resource centric

2017-12-06 Thread Chris Wilson
Quoting Matthew Auld (2017-12-06 18:17:25) > Now that we are using struct resource to track the stolen region, it is > more convenient if we track dsm in a resource as well. > > v2: check range_overflow when writing to 32b registers (Chris) > pepper in some comments (Chris) > v3: refit i915_st

Re: [Intel-gfx] [PATCH i-g-t v8] tests/kms_frontbuffer_tracking: Including DRRS test coverage

2017-12-06 Thread Paulo Zanoni
Em Qua, 2017-12-06 às 20:43 +0530, Lohith BS escreveu: > Dynamic Refresh Rate Switch(DRRS) is used to switch the panel's > refresh rate to the lowest vrefresh supported by panel, when frame is > not flipped for more than a Sec. > > In kernel, DRRS uses the front buffer tracking infrastructure. > H

[Intel-gfx] [PATCH 04/10] drm/i915: nuke the duplicated stolen discovery

2017-12-06 Thread Matthew Auld
We duplicate the stolen discovery code in early-quirks and in i915, however now that the stolen region is exported as a resource from early-quirks we can nuke the duplication. v2: check overflows_type Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Paulo Zanoni Reviewed-b

[Intel-gfx] [PATCH 10/10] drm/i915: prefer stolen_usable_size for the range sanity check

2017-12-06 Thread Matthew Auld
In i915_pages_create_for_stolen it probably makes more sense to check if the range overflows the stolen_usable_size, since the size of dsm will also include the reserved portion which we can't touch. Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Paulo Zanoni --- drivers

[Intel-gfx] [PATCH 03/10] x86/early-quirks: reverse the if ladders

2017-12-06 Thread Matthew Auld
Makes things a little easier to follow. Suggested-by: Ville Syrjälä Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Ville Syrjälä Cc: Chris Wilson Cc: Paulo Zanoni Cc: Thomas Gleixner Cc: Ingo Molnar Cc: H. Peter Anvin Cc: x...@kernel.org Cc: linux-ker...@vger.kernel.org Reviewed-by:

[Intel-gfx] [PATCH 09/10] drm/i915: prefer resource_size_t for everything stolen

2017-12-06 Thread Matthew Auld
Keeps things consistent now that we make use of struct resource. This should keep us covered in case we ever get huge amounts of stolen memory. v2: bunch of missing conversions (Chris) Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Paulo Zanoni --- drivers/char/agp/inte

[Intel-gfx] [PATCH 05/10] drm/i915: make dsm struct resource centric

2017-12-06 Thread Matthew Auld
Now that we are using struct resource to track the stolen region, it is more convenient if we track dsm in a resource as well. v2: check range_overflow when writing to 32b registers (Chris) pepper in some comments (Chris) v3: refit i915_stolen_to_dma() v4: kill ggtt->stolen_size Signed-off-by

[Intel-gfx] [PATCH 06/10] drm/i915: make reserved struct resource centric

2017-12-06 Thread Matthew Auld
Now that we are using struct resource to track the stolen region, it is more convenient if we track the reserved portion of that region in a resource as well. v2: s/<= end + 1/< end/ (Chris) v3: prefer DEFINE_RES_MEM Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Paulo Za

[Intel-gfx] [PATCH 08/10] drm/i915: give stolen_usable_size a more suitable home

2017-12-06 Thread Matthew Auld
Kick it out of i915_ggtt and keep it grouped with dsm and dsm_reserved, where it makes the most sense. Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Paulo Zanoni --- drivers/gpu/drm/i915/i915_drv.h| 12 +++- drivers/gpu/drm/i915/i915_gem_gtt.h| 10 --

[Intel-gfx] [PATCH 07/10] drm/i915: make mappable struct resource centric

2017-12-06 Thread Matthew Auld
Now that we are using struct resource to track the stolen region, it is more convenient if we track the mappable region in a resource as well. v2: prefer iomap and gmadr naming scheme prefer DEFINE_RES_MEM Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Paulo Zanoni R

[Intel-gfx] [PATCH 01/10] x86/early-quirks: Extend Intel graphics stolen memory placement to 64bit

2017-12-06 Thread Matthew Auld
From: Joonas Lahtinen To give upcoming SKU BIOSes more flexibility in placing the Intel graphics stolen memory, make all variables storing the placement or size compatible with full 64 bit range. Also by exporting the stolen region as a resource, we can then nuke the duplicated stolen discovery i

[Intel-gfx] [PATCH 00/10] make stolen resource centric

2017-12-06 Thread Matthew Auld
Continuation of Paulo' stolen series[1], addressing the feedback from Joonas and Chris. [1] https://patchwork.freedesktop.org/series/30923/ Joonas Lahtinen (1): x86/early-quirks: Extend Intel graphics stolen memory placement to 64bit Matthew Auld (9): x86/early-quirks: replace the magica

[Intel-gfx] [PATCH 02/10] x86/early-quirks: replace the magical increment start values

2017-12-06 Thread Matthew Auld
Replace the magical +2, +9 etc. with +MB, which is far easier to read. Suggested-by: Ville Syrjälä Signed-off-by: Matthew Auld Cc: Joonas Lahtinen Cc: Ville Syrjälä Cc: Chris Wilson Cc: Paulo Zanoni Cc: Thomas Gleixner Cc: Ingo Molnar Cc: H. Peter Anvin Cc: x...@kernel.org Cc: linux-ker..

Re: [Intel-gfx] HDCP as a Kconfig option

2017-12-06 Thread A. Wilcox
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 04/12/17 15:24, Daniel Vetter wrote: > Hi, > > I understand there there's concerns about the content protection > stuff, but please note: > > - The patches under discussion enforce nothing, they only allow you > to enable HDCP if you chose to d

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Prevent machine hang from Broxton's vtd w/a and error capture (rev2)

2017-12-06 Thread Patchwork
== Series Details == Series: drm/i915: Prevent machine hang from Broxton's vtd w/a and error capture (rev2) URL : https://patchwork.freedesktop.org/series/34969/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render:

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] lib: Export kmsg()

2017-12-06 Thread Patchwork
== Series Details == Series: series starting with [1/2] lib: Export kmsg() URL : https://patchwork.freedesktop.org/series/34974/ State : success == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 meson: build a full dependency for lib_

Re: [Intel-gfx] [PATCH v2] drm/i915: Prevent machine hang from Broxton's vtd w/a and error capture

2017-12-06 Thread Bloomfield, Jon
> -Original Message- > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf > Of Bloomfield, Jon > Sent: Wednesday, December 6, 2017 9:01 AM > To: Chris Wilson ; intel-gfx@lists.freedesktop.org > Cc: Daniel Vetter > Subject: Re: [Intel-gfx] [PATCH v2] drm/i915: Preven

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/rendercopy: Refactoring rendercopy libraries

2017-12-06 Thread Patchwork
== Series Details == Series: lib/rendercopy: Refactoring rendercopy libraries URL : https://patchwork.freedesktop.org/series/34972/ State : success == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 meson: build a full dependency for l

Re: [Intel-gfx] [PATCH igt 2/2] igt/debugfs_tests: Record which file is being opened in kmsg

2017-12-06 Thread Chris Wilson
Quoting Tvrtko Ursulin (2017-12-06 17:04:40) > > On 06/12/2017 17:02, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2017-12-06 16:51:24) > >> > >> On 06/12/2017 16:38, Chris Wilson wrote: > >>> When tracking down the cause of a particular kernel warning, knowing > >>> which file it is associated

Re: [Intel-gfx] [PATCH igt 2/2] igt/debugfs_tests: Record which file is being opened in kmsg

2017-12-06 Thread Tvrtko Ursulin
On 06/12/2017 17:02, Chris Wilson wrote: Quoting Tvrtko Ursulin (2017-12-06 16:51:24) On 06/12/2017 16:38, Chris Wilson wrote: When tracking down the cause of a particular kernel warning, knowing which file it is associated with can be a big clue. So write the filename into the kernel message

Re: [Intel-gfx] [PATCH igt 2/2] igt/debugfs_tests: Record which file is being opened in kmsg

2017-12-06 Thread Chris Wilson
Quoting Tvrtko Ursulin (2017-12-06 16:51:24) > > On 06/12/2017 16:38, Chris Wilson wrote: > > When tracking down the cause of a particular kernel warning, knowing > > which file it is associated with can be a big clue. So write the > > filename into the kernel message log prior to opening it. > >

Re: [Intel-gfx] [PATCH v2] drm/i915: Prevent machine hang from Broxton's vtd w/a and error capture

2017-12-06 Thread Bloomfield, Jon
> -Original Message- > From: Chris Wilson [mailto:ch...@chris-wilson.co.uk] > Sent: Wednesday, December 6, 2017 7:38 AM > To: intel-gfx@lists.freedesktop.org > Cc: Chris Wilson ; Bloomfield, Jon > ; Harrison, John C ; > Ursulin, Tvrtko ; Joonas Lahtinen > ; Daniel Vetter > Subject: [PATCH

Re: [Intel-gfx] [PATCH igt 2/2] igt/debugfs_tests: Record which file is being opened in kmsg

2017-12-06 Thread Tvrtko Ursulin
On 06/12/2017 16:38, Chris Wilson wrote: When tracking down the cause of a particular kernel warning, knowing which file it is associated with can be a big clue. So write the filename into the kernel message log prior to opening it. Signed-off-by: Chris Wilson --- tests/debugfs_test.c | 3 ++

[Intel-gfx] [PATCH igt 2/2] igt/debugfs_tests: Record which file is being opened in kmsg

2017-12-06 Thread Chris Wilson
When tracking down the cause of a particular kernel warning, knowing which file it is associated with can be a big clue. So write the filename into the kernel message log prior to opening it. Signed-off-by: Chris Wilson --- tests/debugfs_test.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletio

[Intel-gfx] [PATCH igt 1/2] lib: Export kmsg()

2017-12-06 Thread Chris Wilson
Export the kmsg() function for use by tests to write into the kernel message log, useful for tests to inline their progress with kernel error messages. Signed-off-by: Chris Wilson --- lib/igt_core.c | 20 ++-- lib/igt_core.h | 17 + 2 files changed, 23 insertions(

[Intel-gfx] ✗ Fi.CI.BAT: warning for tests/kms_frontbuffer_tracking: Idleness DRRS coverage (rev3)

2017-12-06 Thread Patchwork
== Series Details == Series: tests/kms_frontbuffer_tracking: Idleness DRRS coverage (rev3) URL : https://patchwork.freedesktop.org/series/32888/ State : warning == Summary == IGT patchset tested on top of latest successful build 1db12466cb5ad8483cd469753d2e312a62d717b7 meson: build a full depe

Re: [Intel-gfx] ✗ Fi.CI.IGT: warning for series starting with [v4,1/7] drm/i915/huc: Move firmware selection to init_early

2017-12-06 Thread Chris Wilson
Quoting Patchwork (2017-12-06 16:09:04) > == Series Details == > > Series: series starting with [v4,1/7] drm/i915/huc: Move firmware selection > to init_early > URL : https://patchwork.freedesktop.org/series/34968/ > State : warning > > == Summary == > > Test kms_frontbuffer_tracking: >

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Prevent machine hang from Broxton's vtd w/a and error capture (rev2)

2017-12-06 Thread Patchwork
== Series Details == Series: drm/i915: Prevent machine hang from Broxton's vtd w/a and error capture (rev2) URL : https://patchwork.freedesktop.org/series/34969/ State : success == Summary == Series 34969v2 drm/i915: Prevent machine hang from Broxton's vtd w/a and error capture https://patch

  1   2   >