[Intel-gfx] ✓ Fi.CI.IGT: success for scripts/trace.pl: Optimize event parsing and processing

== Series Details == Series: scripts/trace.pl: Optimize event parsing and processing URL : https://patchwork.freedesktop.org/series/35569/ State : success == Summary == Test kms_cursor_crc: Subgroup cursor-256x85-sliding: skip -> PASS (shard-hsw) Test

Re: [Intel-gfx] [PATCH] drm/i915: Show if we consider the engine is idle in the GPU error state

Quoting Chris Wilson (2017-12-19 13:14:19) > Useful for verifying our bookkeeper when we encounter is knowing whether +encounter "no progress" > we think the engine is idle at the time of the GPU hang. ___ Intel-gfx mailing list

Re: [Intel-gfx] [PATCH igt 2/2] igt/kms_frontbuffer_tracking: Access via GGTT is not guaranteed to be tracked

On Thu, Dec 14, 2017 at 08:09:21PM +, Chris Wilson wrote: > Quoting Chris Wilson (2017-12-07 09:41:26) > > As the system may use a partial vma for a GGTT mmap, access via the GGTT > > mmap is not guaranteed to be tracked by FBC's fence. The rule expressed has > > been that any access to the

Re: [Intel-gfx] [PATCH i-g-t 1/3] lib/igt_kms: Drop all stale events on first commit.

Quoting Chris Wilson (2017-12-07 16:12:34) > Quoting Maarten Lankhorst (2017-12-07 15:57:09) > > Op 07-12-17 om 16:50 schreef Chris Wilson: > > > Quoting Maarten Lankhorst (2017-12-07 15:42:54) > > >> Op 07-12-17 om 16:03 schreef Chris Wilson: > > >>> Quoting Maarten Lankhorst (2017-12-07

[Intel-gfx] [PATCH] drm/i915: Show if we consider the engine is idle in the GPU error state

Useful for verifying our bookkeeper when we encounter is knowing whether we think the engine is idle at the time of the GPU hang. References: https://bugs.freedesktop.org/show_bug.cgi?id=104305 Signed-off-by: Chris Wilson Cc: Tvrtko Ursulin

Re: [Intel-gfx] [PATCH igt] igt/perf_pmu: Speed up frequency measurement

Quoting Tvrtko Ursulin (2017-12-19 12:57:54) > > On 15/12/2017 21:05, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2017-12-15 18:24:42) > >> > >> Ah I see.. only for the spin batch. Why not then gem_sync or maybe we > >> should add igt_spin_batch_free_sync? > > > > gem_quiescent_gpu goes one

[Intel-gfx] [PATCH] drm/i915: Remove pointer indirection for hangcheck_state local

Use the local on-stack struct directly rather than hide it behind a pointer. This should be both clearer for the reader and the compiler (we rely on the compiler seeing through the functions to spot uninitialized uses of the local). Signed-off-by: Chris Wilson Cc: Mika

Re: [Intel-gfx] [PATCH igt] igt/perf_pmu: Speed up frequency measurement

On 15/12/2017 21:05, Chris Wilson wrote: Quoting Tvrtko Ursulin (2017-12-15 18:24:42) On 15/12/2017 17:05, Chris Wilson wrote: Use the normal batch_duration_ns and display the sampled frequency: Frequency: min=100, max=750, boost=750 MHz Min frequency: requested 100.0, actual

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v2,1/3] drm/i915: Add pretty printer for device info flags

== Series Details == Series: series starting with [v2,1/3] drm/i915: Add pretty printer for device info flags URL : https://patchwork.freedesktop.org/series/35574/ State : success == Summary == Series 35574v1 series starting with [v2,1/3] drm/i915: Add pretty printer for device info flags

[Intel-gfx] ✓ Fi.CI.IGT: success for igt/gem_linear_blits: Compute GTT size using 4G limit (rev2)

== Series Details == Series: igt/gem_linear_blits: Compute GTT size using 4G limit (rev2) URL : https://patchwork.freedesktop.org/series/30216/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render: fail ->

[Intel-gfx] [PATCH i-g-t v4] overlay: parse tracepoints from sysfs to figure out fields' location

With changes going to drm-tip, the tracepoints field locations are going to change. This change introduces a tracepoint parser (using a peg parser) which lets us figure out field positions on the fly. v2: Fix automake build (Lionel) v3: Make overlay build conditional on peg (Petri) Make

Re: [Intel-gfx] [PATCH] drm/i915: Add a strong mb to resetting the has-CS-interrupt bit

Quoting Tvrtko Ursulin (2017-12-19 12:26:38) > > On 19/12/2017 09:01, Chris Wilson wrote: > > After a reset, the state of the CSB registers are scrubbed and not valid > > until a powercontext is reloaded. We only know when a powercontext has > > been reloaded once we see a CS-interrupt, before

Re: [Intel-gfx] [PATCH i-g-t v3] overlay: parse tracepoints from sysfs to figure out fields' location

On Tue, Dec 19, 2017 at 12:00:58PM +, Lionel Landwerlin wrote: > With changes going to drm-tip, the tracepoints field locations are > going to change. This change introduces a tracepoint parser (using a > peg parser) which lets us figure out field positions on the fly. > > v2: Fix automake

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Fix indentation for intel_ddi_clk_select

== Series Details == Series: drm/i915: Fix indentation for intel_ddi_clk_select URL : https://patchwork.freedesktop.org/series/35570/ State : success == Summary == Series 35570v1 drm/i915: Fix indentation for intel_ddi_clk_select

[Intel-gfx] [PATCH igt v2] igt/pm_rps: Always allocate spin[0]

Avoid having to test for spin[0] existing by starting the load-loop with it allocated. v2: Preallocate the spin[1] as well for high load. References: https://bugs.freedesktop.org/show_bug.cgi?id=104060 Signed-off-by: Chris Wilson --- tests/pm_rps.c | 27

Re: [Intel-gfx] [RFC] drm/i915: Extend CREATE_CONTEXT to allow inheritance ala clone()

Quoting Michał Winiarski (2017-12-19 12:15:39) > On Fri, Dec 15, 2017 at 01:20:17PM +, Chris Wilson wrote: > > A context encompasses the driver's view of process related state, and > > encapsulates the logical GPU state where available. Each context is > > currently equivalent to a process in

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

On Tue, Dec 19, 2017 at 12:07:00PM +, Chris Wilson wrote: > The vk cts test: > dEQP-VK.api.external.semaphore.opaque_fd.export_multiple_times_temporary > > triggers a lot of > VFS: Close: file count is 0 > > Dave pointed out that clearing the syncobj->file from > drm_syncobj_file_release()

Re: [Intel-gfx] [PATCH] drm/i915: Add a strong mb to resetting the has-CS-interrupt bit

On 19/12/2017 09:01, Chris Wilson wrote: After a reset, the state of the CSB registers are scrubbed and not valid until a powercontext is reloaded. We only know when a powercontext has been reloaded once we see a CS-interrupt, before then we must ignore the CSB registers within the

[Intel-gfx] [PATCH] drm/i915: Grab power domain in skl_pipe_wm_get_hw_state()

This should get rid of unclaimed register debug warnings, if it still happens we should put this in a intel_crtc->active check.. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104172 Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_pm.c

Re: [Intel-gfx] [RFC] drm/i915: Extend CREATE_CONTEXT to allow inheritance ala clone()

On Fri, Dec 15, 2017 at 01:20:17PM +, Chris Wilson wrote: > A context encompasses the driver's view of process related state, and > encapsulates the logical GPU state where available. Each context is > currently equivalent to a process in CPU terms. Like with processes, > sometimes the user

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

The vk cts test: dEQP-VK.api.external.semaphore.opaque_fd.export_multiple_times_temporary triggers a lot of VFS: Close: file count is 0 Dave pointed out that clearing the syncobj->file from drm_syncobj_file_release() was sufficient to silence the test, but that opens a can of worm since we

Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for igt/pm_rps: Always allocate spin[0]

On Mon, Dec 11, 2017 at 04:18:40PM +, Patchwork wrote: > == Series Details == > > Series: igt/pm_rps: Always allocate spin[0] > URL : https://patchwork.freedesktop.org/series/35176/ > State : failure > > == Summary == > > Test gem_tiled_swapping: > Subgroup non-threaded: >

[Intel-gfx] [PATCH i-g-t v3] overlay: parse tracepoints from sysfs to figure out fields' location

With changes going to drm-tip, the tracepoints field locations are going to change. This change introduces a tracepoint parser (using a peg parser) which lets us figure out field positions on the fly. v2: Fix automake build (Lionel) v3: Make overlay build conditional on peg (Petri) Make

Re: [Intel-gfx] [PATCH] drm/i915: Fix indentation for intel_ddi_clk_select

Em Ter, 2017-12-19 às 11:26 +, Chris Wilson escreveu: > drivers/gpu/drm/i915/intel_ddi.c:2098 intel_ddi_clk_select() warn: > inconsistent indenting > > References: 8edcda1266f9 ("drm/i915: Protect DDI port to DPLL map > from theoretical race.") > Signed-off-by: Chris Wilson

[Intel-gfx] [PATCH v2 3/3] drm/i915: Add pretty printer for modparams

We dump modparams in few places (debugfs, gpu_error) using different functions. Lets add reusable function to avoid code duplication. add/remove: 1/0 grow/shrink: 0/2 up/down: 1096/-2339 (-1243) Function old new delta i915_params_dump

[Intel-gfx] [PATCH v2 2/3] drm/i915: Convert intel_device_info_dump into pretty printer

Convert intel_device_info_dump into pretty printer to be consistent with the rest of the driver code. Suggested-by: Chris Wilson Signed-off-by: Michal Wajdeczko Cc: Chris Wilson --- drivers/gpu/drm/i915/i915_drv.c

[Intel-gfx] [PATCH v2 1/3] drm/i915: Add pretty printer for device info flags

We dump device flags in few places (init_early, debugfs, gpu_error) using different functions. Lets add reusable function to avoid code duplication. add/remove: 1/0 grow/shrink: 0/3 up/down: 1296/-3572 (-2276) Function old new delta

[Intel-gfx] ✓ Fi.CI.BAT: success for scripts/trace.pl: Optimize event parsing and processing

== Series Details == Series: scripts/trace.pl: Optimize event parsing and processing URL : https://patchwork.freedesktop.org/series/35569/ State : success == Summary == IGT patchset tested on top of latest successful build d3d04e5f74f05850f595c729fb0c00eb7dd9c688 scripts/trace.pl: Fix ring

[Intel-gfx] [PATCH] drm/i915: Fix indentation for intel_ddi_clk_select

drivers/gpu/drm/i915/intel_ddi.c:2098 intel_ddi_clk_select() warn: inconsistent indenting References: 8edcda1266f9 ("drm/i915: Protect DDI port to DPLL map from theoretical race.") Signed-off-by: Chris Wilson Cc: Paulo Zanoni Cc: Ville

[Intel-gfx] ✗ Fi.CI.BAT: failure for mm/vmalloc: Replace opencoded 4-level page walkers (rev3)

== Series Details == Series: mm/vmalloc: Replace opencoded 4-level page walkers (rev3) URL : https://patchwork.freedesktop.org/series/13822/ State : failure == Summary == Applying: mm/vmalloc: Replace opencoded 4-level page walkers Using index info to reconstruct a base tree... M

[Intel-gfx] ✓ Fi.CI.BAT: success for igt/gem_linear_blits: Compute GTT size using 4G limit (rev2)

== Series Details == Series: igt/gem_linear_blits: Compute GTT size using 4G limit (rev2) URL : https://patchwork.freedesktop.org/series/30216/ State : success == Summary == IGT patchset tested on top of latest successful build d3d04e5f74f05850f595c729fb0c00eb7dd9c688 scripts/trace.pl: Fix

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

From: Tvrtko Ursulin A couple of small optimizations which altogether bring around 30% improvement in my testing. 1. Do less string processing on tracepoints names and push more of the check into the if-ladder. 2. Pull out common db key and ctx processing and cache

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Add a strong mb to resetting the has-CS-interrupt bit

== Series Details == Series: drm/i915: Add a strong mb to resetting the has-CS-interrupt bit URL : https://patchwork.freedesktop.org/series/35566/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-primscrn-cur-indfb-draw-pwrite: skip ->

[Intel-gfx] [PATCH v2] mm/vmalloc: Replace opencoded 4-level page walkers

Rather than open-code the intricacies of walking the 4-level page tables, use the generic page table walker apply_to_page_range() instead. The important change is that it now cleans up after an unsuccessful insertion and propagates the correct error. The current failure may lead to a WARN if we

Re: [Intel-gfx] [PATCH igt v2] igt/gem_linear_blits: Compute GTT size using 4G limit

On Fri, Dec 15, 2017 at 12:22:01PM +, Chris Wilson wrote: > Quoting Chris Wilson (2017-09-13 11:39:14) > > Both gem_linear_blits and gem_tiled_blit do not request the full 48b > > GTT layout for their objects, restricting themselves to 4G. The > > underlying test that they trigger eviction is

Re: [Intel-gfx] [PATCH i-g-t v2 1/1] meson: Install .testlist files and README from tests/intel-ci

On Tue, Dec 12, 2017 at 01:07:56PM +0200, Petri Latvala wrote: > v2: README too, and a better variable name (Ville) > > Signed-off-by: Petri Latvala > Cc: Daniel Vetter > Cc: Ville Syrjälä Reviewed-by: Daniel

Re: [Intel-gfx] [PATCH 4/5] drm/i915: Write AVI infoframes for Parade LSPCON

On Mon, Dec 18, 2017 at 08:15:30PM +0100, Maarten Lankhorst wrote: > Op 14-11-17 om 16:17 schreef Shashank Sharma: > > Different LSPCON vendors specify their custom methods to pass > > AVI infoframes to the LSPCON chip, so does Parade tech. > > > > This patch adds functions to arrange and write

Re: [Intel-gfx] [PATCH i-g-t v2] overlay: parse tracepoints from sysfs to figure out fields' location

On Tue, Dec 19, 2017 at 01:06:18AM +, Lionel Landwerlin wrote: > With changes going to drm-tip, the tracepoints field locations are > going to change. This change introduces a tracepoint parser (using a > peg parser) which lets us figure out field positions on the fly. > > v2: Fix automake

Re: [Intel-gfx] [PATCH 4/5] drm/atomic: document how to handle driver private objects

On Mon, Dec 18, 2017 at 06:13:46PM +0200, Laurent Pinchart wrote: > Hi Daniel, > > Thank you for the patch. > > On Thursday, 14 December 2017 22:30:53 EET Daniel Vetter wrote: > > DK put some nice docs into the commit introducing driver private > > state, but in the git history alone it'll be

Re: [Intel-gfx] [PATCH 4/5] drm/atomic: document how to handle driver private objects

On Mon, Dec 18, 2017 at 06:09:20PM +0200, Laurent Pinchart wrote: > Hi Alex, > > On Friday, 15 December 2017 03:57:48 EET Alex Deucher wrote: > > On Thu, Dec 14, 2017 at 3:30 PM, Daniel Vetter wrote: > > > DK put some nice docs into the commit introducing driver private > > > state, but in the

Re: [Intel-gfx] [PATCH] kthread: finer-grained lockdep/cross-release completion

On Mon, Dec 18, 2017 at 09:42:13AM -0800, Linus Torvalds wrote: > On Sun, Dec 17, 2017 at 11:11 PM, Daniel Vetter wrote: > > > > This didn't seem to have made it into -rc4. Anything needed to get it > > going? > > Do you actually see the problem in -rc4? > > Because we ended up

[Intel-gfx] ✗ Fi.CI.IGT: warning for scripts/trace.pl: Fix ring ignore

== Series Details == Series: scripts/trace.pl: Fix ring ignore URL : https://patchwork.freedesktop.org/series/35565/ State : warning == Summary == Warning: bzip IGTPW_699/shard-kbl7/results14.json.bz2 wasn't in correct JSON format Test kms_frontbuffer_tracking: Subgroup

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Add a strong mb to resetting the has-CS-interrupt bit

== Series Details == Series: drm/i915: Add a strong mb to resetting the has-CS-interrupt bit URL : https://patchwork.freedesktop.org/series/35566/ State : success == Summary == Series 35566v1 drm/i915: Add a strong mb to resetting the has-CS-interrupt bit

Re: [Intel-gfx] [PATCH i-g-t v2] overlay: parse tracepoints from sysfs to figure out fields' location

Quoting Lionel Landwerlin (2017-12-19 01:06:18) > With changes going to drm-tip, the tracepoints field locations are > going to change. This change introduces a tracepoint parser (using a > peg parser) which lets us figure out field positions on the fly. > > v2: Fix automake build (Lionel) > >

Re: [Intel-gfx] [PATCH i-g-t] scripts/trace.pl: Fix ring ignore

Quoting Tvrtko Ursulin (2017-12-19 08:46:52) > From: Tvrtko Ursulin > > Breakage from previous refactoring. > > Signed-off-by: Tvrtko Ursulin > Reported-by: John Harrison > Cc: John Harrison

[Intel-gfx] ✓ Fi.CI.BAT: success for scripts/trace.pl: Fix ring ignore

== Series Details == Series: scripts/trace.pl: Fix ring ignore URL : https://patchwork.freedesktop.org/series/35565/ State : success == Summary == IGT patchset tested on top of latest successful build cfb711c046c22c3881d2334f0b43ec4eb0b9a5fc scripts/trace.pl: Auto-detect tracepoint field

[Intel-gfx] [PATCH] drm/i915: Add a strong mb to resetting the has-CS-interrupt bit

After a reset, the state of the CSB registers are scrubbed and not valid until a powercontext is reloaded. We only know when a powercontext has been reloaded once we see a CS-interrupt, before then we must ignore the CSB registers within the execlists_submission_tasklet. However, glk is

[Intel-gfx] [PATCH i-g-t] scripts/trace.pl: Fix ring ignore

From: Tvrtko Ursulin Breakage from previous refactoring. Signed-off-by: Tvrtko Ursulin Reported-by: John Harrison Cc: John Harrison --- scripts/trace.pl | 2 +- 1 file changed, 1

[Intel-gfx] ✗ Fi.CI.BAT: failure for overlay: parse tracepoints from sysfs to figure out fields' location (rev2)

== Series Details == Series: overlay: parse tracepoints from sysfs to figure out fields' location (rev2) URL : https://patchwork.freedesktop.org/series/35545/ State : failure == Summary == IGT patchset build failed on latest successful build cfb711c046c22c3881d2334f0b43ec4eb0b9a5fc

[Intel-gfx] ✗ Fi.CI.BAT: failure for overlay: parse tracepoints from sysfs to figure out fields' location (rev2)

== Series Details == Series: overlay: parse tracepoints from sysfs to figure out fields' location (rev2) URL : https://patchwork.freedesktop.org/series/35545/ State : failure == Summary == IGT patchset build failed on latest successful build cfb711c046c22c3881d2334f0b43ec4eb0b9a5fc

Re: [Intel-gfx] [PATCH v2] drm/i915: Generalize definition for crtc mask

On Mon, 2017-12-18 at 16:05 -0800, Rodrigo Vivi wrote: > On Wed, Dec 13, 2017 at 09:25:16AM +, Mika Kahola wrote: > > > > crtc_mask is defined explicitly defined for a certain number of > > pipes per > > platform. Let's generalize this in a way that crtc_mask dependens > > only on > > the

<    1   2