Re: [Intel-gfx] [PATCH 12/20] drm/i915/icl: Check for fused-off VDBOX and VEBOX instances

2018-02-17 Thread Chris Wilson
Quoting Sagar Arun Kamble (2018-02-17 08:51:44) > Earlier I had thought of calling ASSIGN_FW_DOMAINS_TABLE, > ASSIGN_*_MMIO_VFUNCS before intel_uncore_fw_domains_init > and use I915_READ here for reading the fuse. But that approach seems to > expose the vfuncs and forcewake table before fw

[Intel-gfx] [PATCH igt] tests/gem_ctx_param: Update invalid param

2018-02-17 Thread Chris Wilson
From: Antonio Argenziano Since commit: drm/i915/scheduler: Support user-defined priorities, the driver support an extra context param to set context's priority. Add tests for that interface and update invalid tests. v2: - Add arg size validation test.

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t v6] tests/perf_pmu: Verify engine busyness accuracy

2018-02-17 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-02-15 15:34:53) > From: Tvrtko Ursulin > > A subtest to verify that the engine busyness is reported with expected > accuracy on platforms where the feature is available. > > We test three patterns: 2%, 50% and 98% load per engine. > > v2:

[Intel-gfx] [PATCH igt] tests/gem_ctx_param: Update invalid param

2018-02-17 Thread Chris Wilson
From: Antonio Argenziano Since commit: drm/i915/scheduler: Support user-defined priorities, the driver support an extra context param to set context's priority. Add tests for that interface and update invalid tests. v2: - Add arg size validation test.

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] lib/igt_aux: Add function to swap int64 in array (rev2)

2018-02-17 Thread Patchwork
== Series Details == Series: series starting with [1/2] lib/igt_aux: Add function to swap int64 in array (rev2) URL : https://patchwork.freedesktop.org/series/38466/ State : failure == Summary == IGT patchset tested on top of latest successful build 1aec09098d3c3729ed152aee9ee4e6e656fa7a3f

Re: [Intel-gfx] [PATCH 0/5] Fix deadlock on runtime suspend in DRM drivers

2018-02-17 Thread Lukas Wunner
On Sun, Feb 11, 2018 at 10:38:28AM +0100, Lukas Wunner wrote: > workqueue: Allow retrieval of current task's work struct > drm: Allow determining if current task is output poll worker > drm/nouveau: Fix deadlock on runtime suspend > drm/radeon: Fix deadlock on runtime suspend >

Re: [Intel-gfx] [PATCH 12/20] drm/i915/icl: Check for fused-off VDBOX and VEBOX instances

2018-02-17 Thread Sagar Arun Kamble
On 2/13/2018 10:07 PM, Mika Kuoppala wrote: From: Oscar Mateo In Gen11, the Video Decode engines (aka VDBOX, aka VCS, aka BSD) and the Video Enhancement engines (aka VEBOX, aka VECS) could be fused off. Also, each VDBOX and VEBOX has its own power well, which only

[Intel-gfx] [PATCH igt 1/2] lib/igt_aux: Add function to swap int64 in array

2018-02-17 Thread Chris Wilson
From: Antonio Argenziano v2: Use igt_swap() Signed-off-by: Antonio Argenziano Cc: Chris Wilson Cc: Michal Winiarski Reviewed-by: Chris Wilson ---

[Intel-gfx] [PATCH igt 2/2] tests/gem_ctx_param: Update invalid param

2018-02-17 Thread Chris Wilson
From: Antonio Argenziano Since commit: drm/i915/scheduler: Support user-defined priorities, the driver support an extra context param to set context's priority. Add tests for that interface and update invalid tests. v2: - Add arg size validation test.

[Intel-gfx] [PATCH igt] lib: Remove overzealous assertion on gem_set_caching()

2018-02-17 Thread Chris Wilson
When the asserts were added for the acceptable error codes for SET_CACHING ioctl, foresight was not given to the possibility that the device may not handle the caching mode and return -ENODEV. Remove the error code assertion from the library, that is the job for the ABI tests. Signed-off-by:

Re: [Intel-gfx] [PATCH 12/20] drm/i915/icl: Check for fused-off VDBOX and VEBOX instances

2018-02-17 Thread Sagar Arun Kamble
On 2/17/2018 5:48 PM, Chris Wilson wrote: Quoting Sagar Arun Kamble (2018-02-17 12:10:32) On 2/17/2018 2:34 PM, Chris Wilson wrote: Quoting Sagar Arun Kamble (2018-02-17 08:51:44) Earlier I had thought of calling ASSIGN_FW_DOMAINS_TABLE, ASSIGN_*_MMIO_VFUNCS before

Re: [Intel-gfx] [PATCH 12/20] drm/i915/icl: Check for fused-off VDBOX and VEBOX instances

2018-02-17 Thread Chris Wilson
Quoting Sagar Arun Kamble (2018-02-17 12:10:32) > > > On 2/17/2018 2:34 PM, Chris Wilson wrote: > > Quoting Sagar Arun Kamble (2018-02-17 08:51:44) > >> Earlier I had thought of calling ASSIGN_FW_DOMAINS_TABLE, > >> ASSIGN_*_MMIO_VFUNCS before intel_uncore_fw_domains_init > >> and use I915_READ

[Intel-gfx] [PATCH igt 2/2] tests/gem_ctx_param: Update invalid param

2018-02-17 Thread Chris Wilson
From: Antonio Argenziano Since commit: drm/i915/scheduler: Support user-defined priorities, the driver support an extra context param to set context's priority. Add tests for that interface and update invalid tests. v2: - Add arg size validation test.

[Intel-gfx] [PATCH igt 1/2] lib/igt_aux: Add function to swap int64 in array

2018-02-17 Thread Chris Wilson
From: Antonio Argenziano v2: Use igt_swap() Signed-off-by: Antonio Argenziano Cc: Chris Wilson Cc: Michal Winiarski Reviewed-by: Chris Wilson ---

Re: [Intel-gfx] [PATCH 12/20] drm/i915/icl: Check for fused-off VDBOX and VEBOX instances

2018-02-17 Thread Sagar Arun Kamble
On 2/17/2018 2:34 PM, Chris Wilson wrote: Quoting Sagar Arun Kamble (2018-02-17 08:51:44) Earlier I had thought of calling ASSIGN_FW_DOMAINS_TABLE, ASSIGN_*_MMIO_VFUNCS before intel_uncore_fw_domains_init and use I915_READ here for reading the fuse. But that approach seems to expose the