Re: [Intel-gfx] [PATCH v15 5/7] vfio: ABI for mdev display dma-buf operation

2017-10-10 Thread Gerd Hoffmann
Hi, > No, the parameter wouldn't be a char, you'd use an __u32 for the > dmabuf_id.  I'm just referencing the structure of the GET_DEVICE_FD > as > an ioctl which returns an fd through the return value and takes a > single parameter.  I don't mean to imply matching the type of that > parameter.

[Intel-gfx] [GVT-g] [ANNOUNCE] 2017-Q3 release of XenGT (Intel GVT-g for Xen)

2017-10-10 Thread Xu, Terrence
Hi all, We are pleased to announce an update of Intel GVT-g for Xen. Intel GVT-g is a full GPU virtualization solution with mediated pass-through, starting from 4th generation Intel Core(TM) processors with Intel processor graphics. A virtual GPU instance is maintained for each VM, with part of

[Intel-gfx] ✗ Fi.CI.IGT: failure for igt/syncobj_wait: Avoid early close of timeline in test_wait_snapshot

2017-10-10 Thread Patchwork
== Series Details == Series: igt/syncobj_wait: Avoid early close of timeline in test_wait_snapshot URL : https://patchwork.freedesktop.org/series/31678/ State : failure == Summary == Test syncobj_wait: Subgroup wait-for-submit-snapshot: fail -> PASS (shard-h

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] tests/syncobj_wait: Replace open-coded calls to __syncobj_wait()

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [1/2] tests/syncobj_wait: Replace open-coded calls to __syncobj_wait() URL : https://patchwork.freedesktop.org/series/31672/ State : success == Summary == Test syncobj_wait: Subgroup wait-any-interrupted: fail ->

[Intel-gfx] [GVT-g] [ANNOUNCE] 2017-Q3 release of KVMGT (Intel GVT-g for KVM)

2017-10-10 Thread Xu, Terrence
Hi all, We are pleased to announce an update of Intel GVT-g for KVM. Intel GVT-g for KVM (a.k.a. KVMGT) is a full GPU virtualization solution with mediated pass-through, starting from 5th generation Intel Core(TM) processors with Intel processor graphics.  A virtual GPU instance is maintained f

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] tests/kms_setmode: Request the intiial vbl count with RELATIVE instead of ABSOLUTE

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [1/2] tests/kms_setmode: Request the intiial vbl count with RELATIVE instead of ABSOLUTE URL : https://patchwork.freedesktop.org/series/31600/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-primscrn-cur-in

Re: [Intel-gfx] [PATCH v15 4/7] drm/i915/gvt: Add opregion support

2017-10-10 Thread Du, Changbin
On Tue, Oct 10, 2017 at 05:50:04PM +0800, Tina Zhang wrote: > Windows guest driver needs vbt in opregion, to configure the setting > for display. Without opregion support, the display registers won't > be set and this blocks display model to get the correct information > of the guest display plane.

Re: [Intel-gfx] [PATCH v15 5/7] vfio: ABI for mdev display dma-buf operation

2017-10-10 Thread Alex Williamson
On Wed, 11 Oct 2017 01:46:37 + "Zhang, Tina" wrote: > > -Original Message- > > From: Alex Williamson [mailto:alex.william...@redhat.com] > > Sent: Wednesday, October 11, 2017 3:17 AM > > To: Zhang, Tina > > Cc: kra...@redhat.com; ch...@chris-wilson.co.uk; zhen...@linux.intel.com; > >

Re: [Intel-gfx] [PATCH v15 6/7] drm/i915: Introduce GEM proxy

2017-10-10 Thread Zhang, Tina
> -Original Message- > From: Joonas Lahtinen [mailto:joonas.lahti...@linux.intel.com] > Sent: Tuesday, October 10, 2017 9:02 PM > To: Zhang, Tina ; alex.william...@redhat.com; > kra...@redhat.com; ch...@chris-wilson.co.uk; zhen...@linux.intel.com; Lv, > Zhiyuan ; Wang, Zhi A ; Tian, > Kev

Re: [Intel-gfx] [PATCH v15 5/7] vfio: ABI for mdev display dma-buf operation

2017-10-10 Thread Zhang, Tina
> -Original Message- > From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Wednesday, October 11, 2017 3:17 AM > To: Zhang, Tina > Cc: kra...@redhat.com; ch...@chris-wilson.co.uk; zhen...@linux.intel.com; > Lv, Zhiyuan ; Wang, Zhi A ; > Tian, Kevin ; dan...@ffwll.ch; kwankh

[Intel-gfx] ✓ Fi.CI.IGT: success for IGT PMU support (rev11)

2017-10-10 Thread Patchwork
== Series Details == Series: IGT PMU support (rev11) URL : https://patchwork.freedesktop.org/series/28253/ State : success == Summary == Test kms_flip: Subgroup flip-vs-fences-interruptible: pass -> FAIL (shard-hsw) fdo#102946 Test drv_module_reload:

[Intel-gfx] ✗ Fi.CI.IGT: warning for tests: Drop gem_bad_length as redundant

2017-10-10 Thread Patchwork
== Series Details == Series: tests: Drop gem_bad_length as redundant URL : https://patchwork.freedesktop.org/series/31655/ State : warning == Summary == Test gem_eio: Subgroup wait: pass -> DMESG-WARN (shard-hsw) fdo#102886 +4 Test kms_universal_plane: Sub

[Intel-gfx] ✓ Fi.CI.BAT: success for Add new vblank and lease tests [v2]

2017-10-10 Thread Patchwork
== Series Details == Series: Add new vblank and lease tests [v2] URL : https://patchwork.freedesktop.org/series/31703/ State : success == Summary == IGT patchset tested on top of latest successful build 145f294d1404e86d06cd7dbfbb6e2034b764ea06 intel_aubdump: Default to 48-bit AUBs when the ge

[Intel-gfx] ✗ Fi.CI.IGT: warning for Add support for NV12

2017-10-10 Thread Patchwork
== Series Details == Series: Add support for NV12 URL : https://patchwork.freedesktop.org/series/31648/ State : warning == Summary == Test gem_eio: Subgroup wait: pass -> DMESG-WARN (shard-hsw) fdo#102886 +3 Test drv_module_reload: Subgroup basic-reload-in

[Intel-gfx] [PATCH i-g-t 0/2] Add new vblank and lease tests [v2]

2017-10-10 Thread Keith Packard
Changes since last version: * Add local definitions of new ioctls to avoid dependency on proposed libdrm bits. * Remove FIRST_PIXEL_OUT as that has been removed from the proposed kernel interface * Add tests for get_lease and list_lessees * Fix commit message on the lease test patch

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

2017-10-10 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 Signed-off-by: Keith P

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

2017-10-10 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

Re: [Intel-gfx] [PATCH i-g-t] lib/igt_gt: Allow non-default contexts to hang non-render rings

2017-10-10 Thread Chris Wilson
Quoting Vinay Belgaumkar (2017-10-11 00:59:51) > This limitation does not exist for gen8+. Oh yes it did. "It did" being the operative phrase there. Read the comment, find the commit that changed the ABI. -Chris ___ Intel-gfx mailing list Intel-gfx@lists

[Intel-gfx] ✓ Fi.CI.BAT: success for igt/drv_hangman: Convert from local recursive batch to igt_spin_t

2017-10-10 Thread Patchwork
== Series Details == Series: igt/drv_hangman: Convert from local recursive batch to igt_spin_t URL : https://patchwork.freedesktop.org/series/31694/ State : success == Summary == IGT patchset tested on top of latest successful build 145f294d1404e86d06cd7dbfbb6e2034b764ea06 intel_aubdump: Defau

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/igt_gt: Allow non-default contexts to hang non-render rings

2017-10-10 Thread Patchwork
== Series Details == Series: lib/igt_gt: Allow non-default contexts to hang non-render rings URL : https://patchwork.freedesktop.org/series/31693/ State : success == Summary == IGT patchset tested on top of latest successful build 145f294d1404e86d06cd7dbfbb6e2034b764ea06 intel_aubdump: Default

[Intel-gfx] [PATCH igt] igt/drv_hangman: Convert from local recursive batch to igt_spin_t

2017-10-10 Thread Chris Wilson
Replace the local recursive batch execbuf with the equivalent igt_spin_t. Signed-off-by: Chris Wilson --- tests/drv_missed_irq.c | 79 -- 1 file changed, 18 insertions(+), 61 deletions(-) diff --git a/tests/drv_missed_irq.c b/tests/drv_missed_irq.

[Intel-gfx] [PATCH i-g-t] lib/igt_gt: Allow non-default contexts to hang non-render rings

2017-10-10 Thread Vinay Belgaumkar
This limitation does not exist for gen8+. Cc: Michel Thierry Cc: Arkadiusz Hiler Cc: Petri Latvala Signed-off-by: Vinay Belgaumkar --- lib/igt_gt.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/igt_gt.c b/lib/igt_gt.c index b3f3b38..abf789d 100644 --- a/lib/i

[Intel-gfx] ✓ Fi.CI.BAT: success for intel_aubdump: Default to 48-bit AUBs when the gen is unknown

2017-10-10 Thread Patchwork
== Series Details == Series: intel_aubdump: Default to 48-bit AUBs when the gen is unknown URL : https://patchwork.freedesktop.org/series/31689/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test for

Re: [Intel-gfx] [PATCH i-g-t v2] intel_aubdump: Default to 48-bit AUBs when the gen is unknown

2017-10-10 Thread Kenneth Graunke
On Tuesday, October 10, 2017 4:05:17 PM PDT Jordan Justen wrote: > v2: > * Use 48-bit rather than 64-bit (Ken) > * Use 'addr_bits' rather than 'use_64bit' > > Signed-off-by: Jordan Justen Reviewed-by: Kenneth Graunke and pushed. signature.asc Description: This is a digitally signed message

[Intel-gfx] ✓ Fi.CI.BAT: success for intel_aubdump: Add device override names (from Mesa INTEL_DEVID_OVERRIDE)

2017-10-10 Thread Patchwork
== Series Details == Series: intel_aubdump: Add device override names (from Mesa INTEL_DEVID_OVERRIDE) URL : https://patchwork.freedesktop.org/series/31688/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Ad

[Intel-gfx] [PATCH i-g-t v2] intel_aubdump: Default to 48-bit AUBs when the gen is unknown

2017-10-10 Thread Jordan Justen
v2: * Use 48-bit rather than 64-bit (Ken) * Use 'addr_bits' rather than 'use_64bit' Cc: Kenneth Graunke Signed-off-by: Jordan Justen --- tools/aubdump.c | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/tools/aubdump.c b/tools/aubdump.c index f3e

[Intel-gfx] [PATCH i-g-t] intel_aubdump: Add device override names (from Mesa INTEL_DEVID_OVERRIDE)

2017-10-10 Thread Jordan Justen
This just lets you specify an override like --device=bdw for broadwell. Signed-off-by: Jordan Justen --- Resend with 'i-g-t' in subject line. tools/aubdump.c| 36 tools/intel_aubdump.in | 4 +++- 2 files changed, 35 insertions(+), 5 deletions(-) di

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v2,1/3] drm/i915: Fix eviction when the GGTT is idle but full

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [v2,1/3] drm/i915: Fix eviction when the GGTT is idle but full URL : https://patchwork.freedesktop.org/series/31682/ State : success == Summary == Series 31682v1 series starting with [v2,1/3] drm/i915: Fix eviction when the GGTT is idle but f

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Simplify intel_sanitize_enable_ppgtt

2017-10-10 Thread Patchwork
== Series Details == Series: drm/i915: Simplify intel_sanitize_enable_ppgtt URL : https://patchwork.freedesktop.org/series/31665/ State : success == Summary == Test kms_atomic_transition: Subgroup plane-all-transition: skip -> PASS (shard-hsw) Test kms_plane

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,01/12] drm/i915: Separate RPS and RC6 handling for gen6+

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [CI,01/12] drm/i915: Separate RPS and RC6 handling for gen6+ URL : https://patchwork.freedesktop.org/series/31681/ State : success == Summary == Series 31681v1 series starting with [CI,01/12] drm/i915: Separate RPS and RC6 handling for gen6+

[Intel-gfx] [PATCH v2 3/3] drm/i915/selftests: Exercise adding requests to a full GGTT

2017-10-10 Thread Chris Wilson
A bug recently encountered involved the issue where are we were submitting requests to different ppGTT, each would pin a segment of the GGTT for its logical context and ring. However, this is invisible to eviction as we do not tie the context/ring VMA to a request and so do not automatically wait u

[Intel-gfx] [PATCH v2 1/3] drm/i915: Fix eviction when the GGTT is idle but full

2017-10-10 Thread Chris Wilson
In the full-ppgtt world, we can fill the GGTT full of context objects. These context objects are currently implicitly tracked by the requests that pin them i.e. they are only unpinned when the request is completed and retired, but we do not have the link from the vma to the request (anymore). In or

[Intel-gfx] [PATCH v2 2/3] drm/i915: Wrap a timer into a i915_sw_fence

2017-10-10 Thread Chris Wilson
For some selftests, we want to issue requests but delay them going to hardware. Furthermore, we don't want those requests to block indefinitely (or else we may hang the driver and block testing) so we want to employ a timeout. So naturally we want a fence that is automatically signaled by a timer.

Re: [Intel-gfx] [PATCH] i-g-t/tests: Drop gem_seqno_wrap, gem_pin, gem_hangcheck_forcewake

2017-10-10 Thread Antonio Argenziano
On 10/10/17 13:19, Chris Wilson wrote: Quoting Antonio Argenziano (2017-10-10 18:12:17) On 10/10/17 01:55, Abdiel Janulgue wrote: This improves the GEM tests section of I-G-T to make it more suitable for CI testing Can you provide a little more details on what is the rationale behind this

[Intel-gfx] [CI 08/12] drm/i915: Name structure in dev_priv that contains RPS/RC6 state as "gt_pm"

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble Prepared substructure rps for RPS related state. autoenable_work is used for RC6 too hence it is defined outside rps structure. As we do this lot many functions are refactored to use intel_rps *rps to access rps related members. Hence renamed intel_rps_client pointer varia

[Intel-gfx] [CI 06/12] drm/i915: Name i915_runtime_pm structure in dev_priv as "runtime_pm"

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble We were using dev_priv->pm for runtime power management related state. This patch renames it to "runtime_pm" which looks more apt. v2: s/rpm/runtime_pm (Chris) Signed-off-by: Sagar Arun Kamble Cc: Imre Deak Cc: Chris Wilson Cc: Joonas Lahtinen Reviewed-by: Radoslaw S

[Intel-gfx] [CI 12/12] drm/i915: Introduce separate status variable for RC6 and LLC ring frequency setup

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble Defined new struct intel_rc6 to hold RC6 specific state and intel_ring_pstate to hold ring specific state. v2: s/intel_ring_pstate/intel_llc_pstate. Removed checks from autoenable_* functions. (Chris) Signed-off-by: Sagar Arun Kamble Cc: Imre Deak Cc: Chris Wilson Cc:

[Intel-gfx] [CI 09/12] drm/i915: Rename intel_enable_rc6 to intel_rc6_enabled

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble This function gives the status of RC6, whether disabled or if enabled then which state. intel_enable_rc6 will be used for enabling RC6 in the next patch. v2: Rebase. v3: Rebase. Signed-off-by: Sagar Arun Kamble Cc: Chris Wilson Cc: Imre Deak Cc: Joonas Lahtinen Revi

[Intel-gfx] [CI 07/12] drm/i915: Move rps.hw_lock to dev_priv and s/hw_lock/pcu_lock

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble In order to separate GT PM related functionality into new structure we are updating rps structure. hw_lock in it is used for display related PCU communication too hence move it to dev_priv. Signed-off-by: Sagar Arun Kamble Cc: Imre Deak Cc: Chris Wilson Cc: Joonas Laht

[Intel-gfx] [CI 10/12] drm/i915: Create generic function to setup LLC ring frequency table

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble Prepared intel_update_ring_freq function to setup ring frequency for applicable platforms determined by macro HAS_LLC. v2: Replaced NEEDS_RING_FREQ_UPDATE with HAS_LLC macro. (Chris) Added check while calling from intel_enable_gt_powersave. v3: s/intel_update_ring_fr

[Intel-gfx] [CI 11/12] drm/i915: Create generic functions to control RC6, RPS

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble Prepared generic functions intel_enable_rc6, intel_disable_rc6, intel_enable_rps and intel_disable_rps functions to setup RC6/RPS based on platforms. v2: Make intel_enable/disable_rc6/rps static. (Chris) v3: Added lockdep_assert_held(dev_priv->pcu_lock) in new generic fu

[Intel-gfx] [CI 05/12] drm/i915: Separate RPS and RC6 handling for CHV

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble This patch separates enable/disable of RC6 and RPS for CHV. v2: Fixed comment. Signed-off-by: Sagar Arun Kamble Cc: Imre Deak Cc: Chris Wilson Cc: Joonas Lahtinen Reviewed-by: Radoslaw Szwichtenberg Reviewed-by: Chris Wilson Link: https://patchwork.freedesktop.org

[Intel-gfx] [CI 03/12] drm/i915: Separate RPS and RC6 handling for BDW

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble This patch separates RC6 and RPS enabling for BDW. RC6/RPS Disabling are handled through gen6 functions. PM Programming guide recommends a sequence within forcewakes to configure RC6, RPS and ring frequencies in sequence. With this patch the order is still maintained. v2:

[Intel-gfx] [CI 04/12] drm/i915: Separate RPS and RC6 handling for VLV

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble This patch separates enable/disable of RC6 and RPS for VLV. v2: Removed unnecessary comments about forcewakes while enabling RC6/RPS. Added changes to output turbo control status for VLV in i915_frequency_info. Signed-off-by: Sagar Arun Kamble Cc: Imre Deak Cc: Chris W

[Intel-gfx] [CI 02/12] drm/i915: Remove superfluous IS_BDW checks and non-BDW changes from gen8_enable_rps

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble This patch removes all IS_BROADWELL checks and non-BDW changes from gen8_enable_rps as it is called only for BROADWELL. Suggested-by: Chris Wilson Signed-off-by: Sagar Arun Kamble Cc: Imre Deak Cc: Chris Wilson Cc: Joonas Lahtinen Cc: Radoslaw Szwichtenberg Reviewed

[Intel-gfx] [CI 01/12] drm/i915: Separate RPS and RC6 handling for gen6+

2017-10-10 Thread Chris Wilson
From: Sagar Arun Kamble This patch separates enable/disable of RC6 and RPS for gen6+ platforms prior to VLV. v2: Fixed checkpatch issue. (Sagar) Signed-off-by: Sagar Arun Kamble Cc: Imre Deak Cc: Chris Wilson Cc: Joonas Lahtinen Reviewed-by: Radoslaw Szwichtenberg #1 Reviewed-by: Chris Wil

[Intel-gfx] ✓ Fi.CI.BAT: success for igt/syncobj_wait: Don't close the timeline early in wait_snapshot

2017-10-10 Thread Patchwork
== Series Details == Series: igt/syncobj_wait: Don't close the timeline early in wait_snapshot URL : https://patchwork.freedesktop.org/series/31679/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test

[Intel-gfx] ✓ Fi.CI.BAT: success for Fix HDMI as dual display on CNL.

2017-10-10 Thread Patchwork
== Series Details == Series: Fix HDMI as dual display on CNL. URL : https://patchwork.freedesktop.org/series/31352/ State : success == Summary == Series 31352v1 Fix HDMI as dual display on CNL. https://patchwork.freedesktop.org/api/1.0/series/31352/revisions/1/mbox/ Test kms_pipe_crc_basic:

Re: [Intel-gfx] [RFC PATCH 00/11] Refactor HW workaround code

2017-10-10 Thread Oscar Mateo
On 10/10/2017 01:10 PM, Chris Wilson wrote: Quoting Oscar Mateo (2017-10-10 19:24:18) * Above method can be used to create a complete logical context with engine context populated by the hardware. This Logical context can be used as an Golden Context Image or template for subsequently created

Re: [Intel-gfx] [PATCH 00/12] radix-tree: split out struct radix_tree_root out to

2017-10-10 Thread Leon Romanovsky
On Mon, Oct 09, 2017 at 01:10:01AM +0900, Masahiro Yamada wrote: <...> > > By splitting out the radix_tree_root definition, > we can reduce the header file dependency. > > Reducing the header dependency will help for speeding the kernel > build, suppressing unnecessary recompile of objects during

Re: [Intel-gfx] [PATCH 00/12] radix-tree: split out struct radix_tree_root out to

2017-10-10 Thread Matthew Wilcox
On Tue, Oct 10, 2017 at 09:56:22PM +0900, Masahiro Yamada wrote: > One refactoring alone does not produce much benefits, > but making continuous efforts will disentangle the knotted threads. > Of course, this might be a pipe dream... A lot of people have had that dream, and some of those refactori

Re: [Intel-gfx] [PATCH 00/12] radix-tree: split out struct radix_tree_root out to

2017-10-10 Thread Leon Romanovsky
On Mon, Oct 09, 2017 at 02:58:58PM +0900, Masahiro Yamada wrote: > 2017-10-09 3:52 GMT+09:00 Leon Romanovsky : > > On Mon, Oct 09, 2017 at 01:10:01AM +0900, Masahiro Yamada wrote: > > > > <...> > >> > >> By splitting out the radix_tree_root definition, > >> we can reduce the header file dependency.

Re: [Intel-gfx] [PATCH 00/12] radix-tree: split out struct radix_tree_root out to

2017-10-10 Thread Matthew Wilcox
On Mon, Oct 09, 2017 at 01:10:01AM +0900, Masahiro Yamada wrote: > Reducing the header dependency will help for speeding the kernel > build, suppressing unnecessary recompile of objects during > git-bisect'ing, etc. Well, does it? You could provide measurements showing before/after time to compil

Re: [Intel-gfx] [PATCH 11/14] drm/i915: Upscale scaler max scale for NV12

2017-10-10 Thread Srinivas, Vidya
> -Original Message- > From: Kumar, Mahesh1 > Sent: Tuesday, October 10, 2017 7:35 PM > To: Srinivas, Vidya ; intel- > g...@lists.freedesktop.org > Cc: Shankar, Uma ; Konduru, Chandra > ; Kamath, Sunil ; > Maiti, Nabendu Bikash > Subject: Re: [PATCH 11/14] drm/i915: Upscale scaler max sc

Re: [Intel-gfx] [PATCH 12/14] drm/i915: Add NV12 as supported format for primary plane

2017-10-10 Thread Srinivas, Vidya
> -Original Message- > From: Kumar, Mahesh1 > Sent: Tuesday, October 10, 2017 7:40 PM > To: Srinivas, Vidya ; intel- > g...@lists.freedesktop.org > Cc: Shankar, Uma ; Konduru, Chandra > ; Kamath, Sunil ; > Maiti, Nabendu Bikash > Subject: Re: [PATCH 12/14] drm/i915: Add NV12 as supported

Re: [Intel-gfx] [PATCH 09/14] drm/i915: Set scaler mode for NV12

2017-10-10 Thread Srinivas, Vidya
> -Original Message- > From: Kumar, Mahesh1 > Sent: Tuesday, October 10, 2017 7:24 PM > To: Srinivas, Vidya ; intel- > g...@lists.freedesktop.org > Cc: Shankar, Uma ; Konduru, Chandra > ; Kamath, Sunil ; > Maiti, Nabendu Bikash > Subject: Re: [PATCH 09/14] drm/i915: Set scaler mode for N

[Intel-gfx] ✓ Fi.CI.BAT: success for igt/syncobj_wait: Avoid early close of timeline in test_wait_snapshot

2017-10-10 Thread Patchwork
== Series Details == Series: igt/syncobj_wait: Avoid early close of timeline in test_wait_snapshot URL : https://patchwork.freedesktop.org/series/31678/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a t

Re: [Intel-gfx] [PATCH i-g-t] igt/syncobj_wait: Don't close the timeline early in wait_snapshot

2017-10-10 Thread Chris Wilson
Quoting Jason Ekstrand (2017-10-10 21:02:20) > Closing the sw_sync timeline now signals any remaining fences upon it; > but test_wait_snapshot requires the fence to continue to be busy so that > the __syncobj_wait() will return with -ETIME. Fwiw, you are not going mad as it did used to work. The c

Re: [Intel-gfx] [PATCH] i-g-t/tests: Drop gem_seqno_wrap, gem_pin, gem_hangcheck_forcewake

2017-10-10 Thread Chris Wilson
Quoting Antonio Argenziano (2017-10-10 18:12:17) > > > On 10/10/17 01:55, Abdiel Janulgue wrote: > > This improves the GEM tests section of I-G-T to make it more > > suitable for CI testing > > Can you provide a little more details on what is the rationale behind > this choice. Are the tests be

Re: [Intel-gfx] [PATCH i-g-t 1/2] tests/debugfs_test: Try to light all outputs to increase chances of finding fails.

2017-10-10 Thread Chris Wilson
Quoting Maarten Lankhorst (2017-10-10 17:04:27) > Make sure read_all_entries has all outputs possible enabled, but also > add a test that runs with all outputs disabled. > > This will maximize coverage of debugfs reading, and allows the test not > to be dependent on fbcon for setup. > > Signed-of

Re: [Intel-gfx] [PATCH i-g-t] igt/syncobj_wait: Don't close the timeline early in wait_snapshot

2017-10-10 Thread Chris Wilson
Quoting Jason Ekstrand (2017-10-10 21:02:20) > Closing the sw_sync timeline now signals any remaining fences upon it; > but test_wait_snapshot requires the fence to continue to be busy so that > the __syncobj_wait() will return with -ETIME. Snap! Reviewed-by: Chris Wilson Tested-by: Chris Wilson

Re: [Intel-gfx] [RFC PATCH 00/11] Refactor HW workaround code

2017-10-10 Thread Chris Wilson
Quoting Oscar Mateo (2017-10-10 19:24:18) > * Above method can be used to create a complete logical context with > engine context populated by the hardware. This Logical context can be > used as an Golden Context Image or template for subsequently created > contexts. It's the template part we w

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: remove redundant check on has_aliasing_ppgtt

2017-10-10 Thread Patchwork
== Series Details == Series: drm/i915: remove redundant check on has_aliasing_ppgtt URL : https://patchwork.freedesktop.org/series/31661/ State : failure == Summary == Test gem_flink_race: Subgroup flink_close: fail -> PASS (shard-hsw) fdo#102655 Test perf:

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] tests/syncobj_wait: Replace open-coded calls to __syncobj_wait()

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [1/2] tests/syncobj_wait: Replace open-coded calls to __syncobj_wait() URL : https://patchwork.freedesktop.org/series/31672/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2

[Intel-gfx] [PATCH i-g-t] igt/syncobj_wait: Don't close the timeline early in wait_snapshot

2017-10-10 Thread Jason Ekstrand
Closing the sw_sync timeline now signals any remaining fences upon it; but test_wait_snapshot requires the fence to continue to be busy so that the __syncobj_wait() will return with -ETIME. --- tests/syncobj_wait.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/syn

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Parse DSI backlight/cabc ports.

2017-10-10 Thread Chauhan, Madhav
> -Original Message- > From: Nikula, Jani > Sent: Tuesday, October 10, 2017 11:27 PM > To: Chauhan, Madhav ; intel- > g...@lists.freedesktop.org > Cc: Hiremath, Shashidhar ; Shankar, Uma > > Subject: RE: [PATCH 1/2] drm/i915: Parse DSI backlight/cabc ports. > > On Tue, 10 Oct 2017, "Chauh

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] tests/kms_setmode: Request the intiial vbl count with RELATIVE instead of ABSOLUTE

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [1/2] tests/kms_setmode: Request the intiial vbl count with RELATIVE instead of ABSOLUTE URL : https://patchwork.freedesktop.org/series/31600/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393

[Intel-gfx] [PATCH igt] igt/syncobj_wait: Avoid early close of timeline in test_wait_snapshot

2017-10-10 Thread Chris Wilson
We want to keep the first timeline in an incomplete state as we replace the syncobj's fence; so we need to defer the close() until the end of the test. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103188 Signed-off-by: Chris Wilson Cc: Jason Ekstrand Cc: Dave Airlie --- tests/syncobj

[Intel-gfx] ✗ Fi.CI.BAT: warning for tests/BAT: Reduce reliance on fbcon.

2017-10-10 Thread Patchwork
== Series Details == Series: tests/BAT: Reduce reliance on fbcon. URL : https://patchwork.freedesktop.org/series/31671/ State : warning == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test for SYNCOBJ_CREATE_SIGNALED

Re: [Intel-gfx] [PATCH v15 5/7] vfio: ABI for mdev display dma-buf operation

2017-10-10 Thread Alex Williamson
On Tue, 10 Oct 2017 17:50:05 +0800 Tina Zhang wrote: > Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user mode query and > get the plan and its related information. This ioctl can be invoked with: s/plan/plane/ > 1) either flag DMABUF or REGION is set. Vendor driver returns success and >

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/guc : Removing enable_guc_loading module and Decoupling logs and ADS from submission

2017-10-10 Thread Patchwork
== Series Details == Series: drm/i915/guc : Removing enable_guc_loading module and Decoupling logs and ADS from submission URL : https://patchwork.freedesktop.org/series/31677/ State : failure == Summary == CHK include/config/kernel.release CHK include/generated/uapi/linux/version

[Intel-gfx] ✓ Fi.CI.BAT: success for IGT PMU support (rev11)

2017-10-10 Thread Patchwork
== Series Details == Series: IGT PMU support (rev11) URL : https://patchwork.freedesktop.org/series/28253/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test for SYNCOBJ_CREATE_SIGNALED with latest

[Intel-gfx] [PATCH v6 3/3] drm/i915/guc : Decouple logs and ADS from submission

2017-10-10 Thread Sujaritha Sundaresan
The Additional Data Struct (ADS) contains objects that are required by guc post FW load and are not necessarily submission-only (although that's our current only use-case). If in the future we load GuC with submission disabled to use some other GuC feature we might still end up requiring something

[Intel-gfx] [PATCH v6 2/3] drm/i915/guc : Removing i915_modparams.enable_guc_loading module parameter

2017-10-10 Thread Sujaritha Sundaresan
We currently have two module parameters that control GuC: "enable_guc_loading" and "enable_guc_submission". Whenever we need i915_modparams.enable_guc_submission=1, we also need enable_guc_loading=1. We also need enable_guc_loading=1 when we want to verify the HuC, which is every time we have a H

[Intel-gfx] [PATCH v6 0/3] drm/i915/guc : Removing enable_guc_loading module and Decoupling logs and ADS from submission

2017-10-10 Thread Sujaritha Sundaresan
The first patch simply unifies different seq_puts messages found in debugfs. Patch 2 focuses on replacing the enable_guc_loading module. Patch 3 deals with decoupling guc logs and ADS from submission. Cc: Joonas Lahtinen Cc: Michal Wajdeczko Cc: Sagar Arun Kamble Cc: Oscar Mateo Cc: Daniele

[Intel-gfx] [PATCH v6 1/3] drm/i915/guc : Unifying seq_puts messages for feature support

2017-10-10 Thread Sujaritha Sundaresan
Unifying the various seq_puts messages in debugfs to the simplest one for feature support. v2: Clarifying the commit message (Anusha) v3: Re-factoring code as per review (Michal) v4: Rebase v5: Split from following patch v6: Re-factoring code (Michal, Sagar) Clarifying commit message (Saga

Re: [Intel-gfx] [PATCH 3/3] drm/i915/dp: limit sink rates based on rate

2017-10-10 Thread Manasi Navare
On Mon, Oct 09, 2017 at 12:29:59PM +0300, Jani Nikula wrote: > Get rid of redundant intel_dp_num_rates(). We can simply look at the > rate and limit based on that. > > Cc: Manasi Navare > Signed-off-by: Jani Nikula Reviewed-by: Manasi > --- > drivers/gpu/drm/i915/intel_dp.c | 26 +++

Re: [Intel-gfx] [PATCH 2/3] drm/i915/dp: centralize max source rate conditions more

2017-10-10 Thread Manasi Navare
On Mon, Oct 09, 2017 at 12:29:58PM +0300, Jani Nikula wrote: > Turn intel_dp_source_supports_hbr2() into a simple helper to query the > pre-filled source rates array, and move the conditions about which > platforms support which rates to the single point of truth in > intel_dp_set_source_rates(). >

Re: [Intel-gfx] [RFC PATCH 00/11] Refactor HW workaround code

2017-10-10 Thread Oscar Mateo
On 10/10/2017 10:37 AM, Oscar Mateo wrote: On 10/09/2017 02:08 PM, Chris Wilson wrote: Quoting Oscar Mateo (2017-10-09 21:58:15) Currently, deciding how/where to apply new workarounds is challenging. Often, workarounds end up applied incorrectly and get lost under certain circumstances (e.

[Intel-gfx] ✓ Fi.CI.BAT: success for tests: Drop gem_bad_length as redundant

2017-10-10 Thread Patchwork
== Series Details == Series: tests: Drop gem_bad_length as redundant URL : https://patchwork.freedesktop.org/series/31655/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test for SYNCOBJ_CREATE_SIGNAL

Re: [Intel-gfx] [PATCH 1/3] drm/dp: WARN about invalid/unknown link rates and bw codes

2017-10-10 Thread Manasi Navare
On Mon, Oct 09, 2017 at 12:29:57PM +0300, Jani Nikula wrote: > Falling back to the lowest value is likely the only thing we can do, but > doing it silently seems like a bad thing to do. Catch it early and make > loud noises. > > Cc: Alex Deucher > Cc: Thierry Reding > Cc: Rob Clark > Cc: Sean P

[Intel-gfx] ✓ Fi.CI.BAT: success for Add support for NV12

2017-10-10 Thread Patchwork
== Series Details == Series: Add support for NV12 URL : https://patchwork.freedesktop.org/series/31648/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test for SYNCOBJ_CREATE_SIGNALED with latest DRM

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Parse DSI backlight/cabc ports.

2017-10-10 Thread Jani Nikula
On Tue, 10 Oct 2017, "Chauhan, Madhav" wrote: >> -Original Message- >> From: Nikula, Jani >> Sent: Tuesday, October 10, 2017 12:47 PM >> To: Chauhan, Madhav ; intel- >> g...@lists.freedesktop.org >> Cc: Hiremath, Shashidhar ; Shankar, Uma >> ; Chauhan, Madhav >> >> Subject: Re: [PATCH 1/2

[Intel-gfx] ✗ Fi.CI.BAT: warning for gtt-cache-enable

2017-10-10 Thread Patchwork
== Series Details == Series: gtt-cache-enable URL : https://patchwork.freedesktop.org/series/31669/ State : warning == Summary == Series 31669v1 gtt-cache-enable https://patchwork.freedesktop.org/api/1.0/series/31669/revisions/1/mbox/ Test chamelium: Subgroup dp-crc-fast:

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Use enum pipe for PCH transcoders

2017-10-10 Thread Patchwork
== Series Details == Series: drm/i915: Use enum pipe for PCH transcoders URL : https://patchwork.freedesktop.org/series/31654/ State : warning == Summary == Test kms_fbc_crc: Subgroup page_flip_and_mmap_gtt: pass -> SKIP (shard-hsw) shard-hswtotal:2

Re: [Intel-gfx] [RFC PATCH 00/11] Refactor HW workaround code

2017-10-10 Thread Oscar Mateo
On 10/09/2017 02:08 PM, Chris Wilson wrote: Quoting Oscar Mateo (2017-10-09 21:58:15) Currently, deciding how/where to apply new workarounds is challenging. Often, workarounds end up applied incorrectly and get lost under certain circumstances (e.g. a context switch or a GPU reset). This is a

Re: [Intel-gfx] [RFC PATCH 02/11] drm/i915: Move a bunch of workaround-related code to its own file

2017-10-10 Thread Oscar Mateo
On 10/09/2017 02:06 PM, Chris Wilson wrote: Quoting Oscar Mateo (2017-10-09 21:58:17) This has grown to be a sizable amount of code, so move it to its own file before we try to refactor anything. For the moment, we are leaving behind the WA BB code and the WAs that get applied (incorrectly) in

[Intel-gfx] [PATCH igt 1/2] tests/syncobj_wait: Replace open-coded calls to __syncobj_wait()

2017-10-10 Thread Chris Wilson
Remove the bare igt_ioctl(SYNCOBJ_WAIT) in favour of __syncobj_wait() for tidier error reporting. Signed-off-by: Chris Wilson Cc: Jason Ekstrand Cc: Dave Airlie --- tests/syncobj_wait.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/tests/syncobj_wait.c b/te

[Intel-gfx] [PATCH igt 2/2] igt/syncobj_wait: Close the sw_sync timeline after the test

2017-10-10 Thread Chris Wilson
Closing the sw_sync timeline now signals any remaining fences upon it; but test_wait_interrupted requires the fence to be busy for the __syncobj_wait() not to immediately return with -ETIME (and so be interrupted instead). Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103187 Signed-off-by

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915: Fix eviction when the GGTT is idle but full

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915: Fix eviction when the GGTT is idle but full URL : https://patchwork.freedesktop.org/series/31668/ State : failure == Summary == Series 31668v1 series starting with [1/3] drm/i915: Fix eviction when the GGTT is idle but full ht

Re: [Intel-gfx] [PATCH] i-g-t/tests: Drop gem_seqno_wrap, gem_pin, gem_hangcheck_forcewake

2017-10-10 Thread Antonio Argenziano
On 10/10/17 01:55, Abdiel Janulgue wrote: This improves the GEM tests section of I-G-T to make it more suitable for CI testing Can you provide a little more details on what is the rationale behind this choice. Are the tests being removed only because they do not fit the CI guidelines or is

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [CI,01/10] drm/i915: Extract intel_get_cagf

2017-10-10 Thread Patchwork
== Series Details == Series: series starting with [CI,01/10] drm/i915: Extract intel_get_cagf URL : https://patchwork.freedesktop.org/series/31667/ State : failure == Summary == Series 31667v1 series starting with [CI,01/10] drm/i915: Extract intel_get_cagf https://patchwork.freedesktop.org/ap

[Intel-gfx] ✓ Fi.CI.IGT: success for Adding NV12 support (rev3)

2017-10-10 Thread Patchwork
== Series Details == Series: Adding NV12 support (rev3) URL : https://patchwork.freedesktop.org/series/28103/ State : success == Summary == shard-hswtotal:2552 pass:1431 dwarn:5 dfail:0 fail:13 skip:1103 time:9650s == Logs == For more details see: https://intel-gfx-ci.01.org/t

Re: [Intel-gfx] [PATCH i-g-t v6 7/9] tests/perf_pmu: Tests for i915 PMU API

2017-10-10 Thread Chris Wilson
Quoting Tvrtko Ursulin (2017-10-10 15:17:54) > +static void > +busy_check_all(int gem_fd, const struct intel_execution_engine2 *e, > + const unsigned int num_engines) > +{ > + const struct intel_execution_engine2 *e_; > + uint64_t val[num_engines]; > + int fd[num_engi

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/uc: Firmware code reorg

2017-10-10 Thread Patchwork
== Series Details == Series: drm/i915/uc: Firmware code reorg URL : https://patchwork.freedesktop.org/series/31666/ State : failure == Summary == Series 31666v1 drm/i915/uc: Firmware code reorg https://patchwork.freedesktop.org/api/1.0/series/31666/revisions/1/mbox/ Test debugfs_test:

Re: [Intel-gfx] [PATCH i-g-t 2/2] tests/pm_backlight: Enable all possible outputs to allow tests to succeed.

2017-10-10 Thread Maarten Lankhorst
Op 10-10-17 om 18:14 schreef Ville Syrjälä: > On Tue, Oct 10, 2017 at 06:04:28PM +0200, Maarten Lankhorst wrote: >> If all outputs are disabled before this test is run, then the test >> will fail because actual_brightness stays at 0 instead of the value >> of the desired brightness. Fix this by ena

Re: [Intel-gfx] [PATCH i-g-t 2/2] tests/pm_backlight: Enable all possible outputs to allow tests to succeed.

2017-10-10 Thread Ville Syrjälä
On Tue, Oct 10, 2017 at 06:04:28PM +0200, Maarten Lankhorst wrote: > If all outputs are disabled before this test is run, then the test > will fail because actual_brightness stays at 0 instead of the value > of the desired brightness. Fix this by enabling all outputs at the > start of this test. >

[Intel-gfx] [PATCH i-g-t 2/2] tests/pm_backlight: Enable all possible outputs to allow tests to succeed.

2017-10-10 Thread Maarten Lankhorst
If all outputs are disabled before this test is run, then the test will fail because actual_brightness stays at 0 instead of the value of the desired brightness. Fix this by enabling all outputs at the start of this test. Sample failure: IGT-Version: 1.20-g01c550a27986 (x86_64) (Linux: 4.14.0-rc3-

[Intel-gfx] [PATCH i-g-t 0/2] tests/BAT: Reduce reliance on fbcon.

2017-10-10 Thread Maarten Lankhorst
https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_1235/ shows what happens in BAT if we make fbcon disable all outputs. debugfs_test.read_all_entries fails, probably in a race with fbcon. I'm hoping to solve this by running the test twice, once with all outputs enabled, other with all disabled. Thi

[Intel-gfx] [PATCH i-g-t 1/2] tests/debugfs_test: Try to light all outputs to increase chances of finding fails.

2017-10-10 Thread Maarten Lankhorst
Make sure read_all_entries has all outputs possible enabled, but also add a test that runs with all outputs disabled. This will maximize coverage of debugfs reading, and allows the test not to be dependent on fbcon for setup. Signed-off-by: Maarten Lankhorst --- tests/debugfs_test.c | 34 ++

  1   2   3   >