[Intel-gfx] ✗ Fi.CI.BAT: failure for tests/kms_sequence: Add tests for lease ioctls

2017-09-30 Thread Patchwork
== Series Details == Series: tests/kms_sequence: Add tests for lease ioctls URL : https://patchwork.freedesktop.org/series/31232/ State : failure == Summary == IGT patchset build failed on latest successful build 09f8cb1a3758fa9868a07b83a3d3db2973d795cc igt/gem_sync: Sync before starting the

[Intel-gfx] [PATCH i-g-t] tests/kms_sequence: Add tests for lease ioctls

2017-09-30 Thread keithp
From: Keith Packard Validate that the leasing API creates leases that allow access to a subset of the available resources and that lease revocation works. Signed-off-by: Keith Packard --- tests/Makefile.sources | 1 + tests/kms_lease.c | 439 +

[Intel-gfx] ✗ Fi.CI.BAT: failure for tests/kms_sequence: Add tests for new CRTC get/queue sequence ioctls

2017-09-30 Thread Patchwork
== Series Details == Series: tests/kms_sequence: Add tests for new CRTC get/queue sequence ioctls URL : https://patchwork.freedesktop.org/series/31230/ State : failure == Summary == IGT patchset build failed on latest successful build 09f8cb1a3758fa9868a07b83a3d3db2973d795cc igt/gem_sync: Sync

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

2017-09-30 Thread keithp
From: 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 bloc

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Move core GuC functions into dedicated file

2017-09-30 Thread Sagar Arun Kamble
On 9/30/2017 11:44 PM, Michal Wajdeczko wrote: On Sat, 30 Sep 2017 19:16:42 +0200, Sagar Arun Kamble wrote: On 9/30/2017 10:36 PM, Sagar Arun Kamble wrote: On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. While here move send registers initializat

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Move core GuC functions into dedicated file

2017-09-30 Thread Michal Wajdeczko
On Sat, 30 Sep 2017 19:16:42 +0200, Sagar Arun Kamble wrote: On 9/30/2017 10:36 PM, Sagar Arun Kamble wrote: On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. While here move send registers initialization to early init as this is one time operation.

Re: [Intel-gfx] [PATCH 9/9] drm/i915: Move intel_guc_allocate_vma to guc.c

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Reviewed-by: Sagar Arun Kamble --- drivers/gpu/drm/i915/i915_guc_submission.c | 42

Re: [Intel-gfx] [PATCH 8/9] drm/i915: Move intel_guc_suspend/resume to guc.c

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. Signatures of the function will be changed in upcoming patch. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Reviewed-by: Sagar Arun Kamble --- drivers/gp

Re: [Intel-gfx] [PATCH 7/9] drm/i915: Move intel_guc_auth_huc to guc.c

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Reviewed-by: Sagar Arun Kamble --- drivers/gpu/drm/i915/intel_guc.c | 21 + drivers/gp

Re: [Intel-gfx] [PATCH 6/9] drm/i915: Move intel_guc_sample_forcewake to guc.c

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Reviewed-by: Sagar Arun Kamble nitpick: Subject prefix as drm/i915/guc --- drivers/gpu/drm/i915/intel_guc.c

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Move core GuC functions into dedicated file

2017-09-30 Thread Sagar Arun Kamble
On 9/30/2017 10:36 PM, Sagar Arun Kamble wrote: On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. While here move send registers initialization to early init as this is one time operation. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris W

Re: [Intel-gfx] [PATCH 5/9] drm/i915: Move core GuC functions into dedicated file

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC functions together. While here move send registers initialization to early init as this is one time operation. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble +void intel_guc_in

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Move uC fw helper code into dedicated files

2017-09-30 Thread Michal Wajdeczko
On Sat, 30 Sep 2017 16:43:35 +0200, Sagar Arun Kamble wrote: On 9/30/2017 3:31 PM, Sagar Arun Kamble wrote: On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: This is a prerequisite to unblock next steps. No functional changes, only s/dev_priv/i915 Signed-off-by: Michal Wajdeczko Cc: Joon

Re: [Intel-gfx] [PATCH 4/9] drm/i915: Move GuC declarations into dedicated header

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep GuC specific code in separated files. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Nitpick: Subject prefix drm/i915/guc and similarly drm/i915/huc for earlier patch? Otherwise loo

Re: [Intel-gfx] [PATCH 3/9] drm/i915: Move HuC declarations into dedicated header

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We want to keep each uC specific code in separate files. Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Reviewed-by: Sagar Arun Kamble --- drivers/gpu/drm/i915/intel_huc.h | 38 +

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Move uC fw helper code into dedicated files

2017-09-30 Thread Sagar Arun Kamble
On 9/30/2017 3:31 PM, Sagar Arun Kamble wrote: On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: This is a prerequisite to unblock next steps. No functional changes, only s/dev_priv/i915 Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Reviewed-by

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Guc code reorg

2017-09-30 Thread Sagar Arun Kamble
On 9/30/2017 4:47 PM, Michal Wajdeczko wrote: On Sat, 30 Sep 2017 10:30:16 +0200, Sagar Arun Kamble wrote: On 9/30/2017 3:26 AM, Michal Wajdeczko wrote: On Fri, 29 Sep 2017 20:07:48 +0200, Patchwork wrote: == Series Details == Series: drm/i915: Guc code reorg URL   : https://patchwo

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Guc code reorg

2017-09-30 Thread Michal Wajdeczko
On Sat, 30 Sep 2017 10:30:16 +0200, Sagar Arun Kamble wrote: On 9/30/2017 3:26 AM, Michal Wajdeczko wrote: On Fri, 29 Sep 2017 20:07:48 +0200, Patchwork wrote: == Series Details == Series: drm/i915: Guc code reorg URL : https://patchwork.freedesktop.org/series/31199/ State : warnin

Re: [Intel-gfx] [PATCH 2/9] drm/i915: Move uC fw helper code into dedicated files

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: This is a prerequisite to unblock next steps. No functional changes, only s/dev_priv/i915 Signed-off-by: Michal Wajdeczko Cc: Joonas Lahtinen Cc: Chris Wilson Cc: Sagar Arun Kamble Reviewed-by: Sagar Arun Kamble --- drivers/gpu/drm/i915/M

Re: [Intel-gfx] [PATCH 1/9] drm/i915: Drop unnecessary forward declaration

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 11:11 PM, Michal Wajdeczko wrote: We don't need it here. Signed-off-by: Michal Wajdeczko Cc: Sagar Arun Kamble Reviewed-by: Sagar Arun Kamble --- drivers/gpu/drm/i915/intel_uc.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_uc.h b/drivers/

Re: [Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Guc code reorg

2017-09-30 Thread Sagar Arun Kamble
On 9/30/2017 3:26 AM, Michal Wajdeczko wrote: On Fri, 29 Sep 2017 20:07:48 +0200, Patchwork wrote: == Series Details == Series: drm/i915: Guc code reorg URL   : https://patchwork.freedesktop.org/series/31199/ State : warning == Summary == Series 31199v1 drm/i915: Guc code reorg https://p

Re: [Intel-gfx] [PATCH v12 06/11] drm/i915/guc: Make GuC related disable/destroy functions not depend on i915.enable_guc_submission

2017-09-30 Thread Sagar Arun Kamble
On 9/29/2017 5:57 PM, Joonas Lahtinen wrote: On Thu, 2017-09-28 at 12:18 +0530, Sagar Arun Kamble wrote: During GuC load/enable, state is setup by driver that can be looked at while disabling. So remove the check for i915.enable_guc_submission parameter in those functions. Suggested-by: Chris