[Intel-gfx] ✗ Fi.CI.BAT: warning for More kerneldoc cleanup

2017-01-24 Thread Patchwork
== Series Details == Series: More kerneldoc cleanup URL : https://patchwork.freedesktop.org/series/18534/ State : warning == Summary == Series 18534v1 More kerneldoc cleanup https://patchwork.freedesktop.org/api/1.0/series/18534/revisions/1/mbox/ Test gem_sync: Subgroup

Re: [Intel-gfx] [PATCH] x86/gpu: GLK uses the same GMS values as SKL

2017-01-24 Thread Ingo Molnar
* Paulo Zanoni wrote: > So don't forget to reserve its stolen memory bits. > > Cc: Ingo Molnar > Cc: H. Peter Anvin > Cc: Ander Conselvan de Oliveira > Cc: x...@kernel.org > Reviewed-by: Ander

[Intel-gfx] ✗ Fi.CI.BAT: warning for series starting with [1/2] drm/i915: ignore posting read when using vgpu

2017-01-24 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: ignore posting read when using vgpu URL : https://patchwork.freedesktop.org/series/18532/ State : warning == Summary == Series 18532v1 Series without cover letter

Re: [Intel-gfx] [PATCH 3/4] drm/i915/glk: Program pipe gamma and degamma tables

2017-01-24 Thread Ander Conselvan De Oliveira
On Tue, 2017-01-24 at 20:05 +0200, Ville Syrjälä wrote: > On Tue, Jan 24, 2017 at 03:35:54PM +0200, Ander Conselvan de Oliveira wrote: > > > > The gamma tables in Geminilake were changed. There is no split-gamma > > mode. Instead, there is a dedicated degamma table that is enabled > > whenever

Re: [Intel-gfx] [PATCH 2/2] drm/i915: ignore forcewake get/put when using vgpu

2017-01-24 Thread Chris Wilson
On Wed, Jan 25, 2017 at 02:04:56PM +0800, Weinan Li wrote: > Host maintian the hardware's forcewake state, guest don't need and also > can't control it. Although vgpu_read/write bypass forcewake_get/put in MMIO > read/write, but still have separate path called by > "intel_uncore_forcewake_get/put"

Re: [Intel-gfx] [PATCH 2/4] drm/i915/glk: Plane color correction register changes

2017-01-24 Thread Ander Conselvan De Oliveira
On Tue, 2017-01-24 at 18:16 +0200, Ville Syrjälä wrote: > On Tue, Jan 24, 2017 at 03:35:53PM +0200, Ander Conselvan de Oliveira wrote: > > > > From: Ander Conselvan De Oliveira > > > > In Geminilake, the bits for enabling pipe csc, pipe gamma and plane > >

Re: [Intel-gfx] [PATCH 1/2] drm/i915: ignore posting read when using vgpu

2017-01-24 Thread Chris Wilson
On Wed, Jan 25, 2017 at 02:04:55PM +0800, Weinan Li wrote: > No need to do posting read when vgpu actived. No. This is bloat for no gain. Almost all of those posting reads are superfluous anyway. -Chris -- Chris Wilson, Intel Open Source Technology Centre

[Intel-gfx] ✗ Fi.CI.BAT: failure for GET_PLANE2 w/ i915 implementation (rev2)

2017-01-24 Thread Patchwork
== Series Details == Series: GET_PLANE2 w/ i915 implementation (rev2) URL : https://patchwork.freedesktop.org/series/17873/ State : failure == Summary == LD drivers/usb/storage/usb-storage.o LD drivers/usb/storage/built-in.o LD drivers/acpi/built-in.o LD

[Intel-gfx] [PATCH 11/15] drm: Nuke ums vgaarb support

2017-01-24 Thread Daniel Vetter
i915, nouveau (ever since merged to upstream) and radeon all lack ums support in upstream. No point keeping the ums vgaarb support around. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_irq.c | 26 -- include/drm/drm_drv.h | 3 --- 2

[Intel-gfx] [PATCH 12/15] drm/moc: Mark legacy fields in drm_driver as such

2017-01-24 Thread Daniel Vetter
No point in documenting these, they only confuse. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 1 - include/drm/drm_drv.h | 13 - 2 files changed, 8 insertions(+), 6 deletions(-) diff --git

[Intel-gfx] [PATCH 05/15] drm/core: Use recommened kerneldoc for struct member refs

2017-01-24 Thread Daniel Vetter
I just learned that _name.member_name works and looks pretty even. It doesn't (yet) link to the member directly though, which would be really good for big structures or vfunc tables (where the per-member kerneldoc tends to be long). Also some minor drive-by polish where it makes sense, I read a

[Intel-gfx] [PATCH 09/15] drm/mga: remove device_is_agp callback

2017-01-24 Thread Daniel Vetter
It's only for a device quirk, and we might as well do that in the load callback. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/mga/mga_dma.c | 20 +++- drivers/gpu/drm/mga/mga_drv.c | 37 - 2 files changed, 19

[Intel-gfx] [PATCH 15/15] drm: Update kerneldoc for drm_crtc.[hc]

2017-01-24 Thread Daniel Vetter
After going through all the trouble of splitting out parts from drm_crtc.[hc] and then properly documenting each I've entirely forgotten to show the same TLC for CRTCs themselves! Let's make amends asap. Signed-off-by: Daniel Vetter --- Documentation/gpu/drm-kms.rst |

[Intel-gfx] [PATCH 13/15] drm/doc: Fix typos for early_unregister doc

2017-01-24 Thread Daniel Vetter
There's no late_unregister. While at it switch to the new canonical reference style. Cc: Chris Wilson Signed-off-by: Daniel Vetter --- include/drm/drm_crtc.h| 2 +- include/drm/drm_encoder.h | 2 +- include/drm/drm_plane.h | 2 +- 3

[Intel-gfx] [PATCH 03/15] drm/kms-core: Use recommened kerneldoc for struct member refs

2017-01-24 Thread Daniel Vetter
I just learned that _name.member_name works and looks pretty even. It doesn't (yet) link to the member directly though, which would be really good for big structures or vfunc tables (where the per-member kerneldoc tends to be long). Also some minor drive-by polish where it makes sense, I read a

[Intel-gfx] [PATCH 10/15] drm: remove device_is_agp callback

2017-01-24 Thread Daniel Vetter
With that the drm_pci_device_is_agp function becomes trivial, so inline that too. And while at it, move the drm_pci_agp_destroy declaration into drm-internal.h, since it's not used by drivers. Cc: Alex Deucher Cc: Ben Skeggs Signed-off-by: Daniel

[Intel-gfx] [PATCH 06/15] drm/doc: Clarify connector overview

2017-01-24 Thread Daniel Vetter
There was a bit of mix-up between initialization and registering. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_connector.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_connector.c

[Intel-gfx] [PATCH 14/15] drm: s/drm_crtc_get_hv_timings/drm_mode_get_hv_timings/

2017-01-24 Thread Daniel Vetter
The function operates on modes, not CRTCs. Also move it into drm_modes.[hc]. Spotted while reviewing CRTC docs. Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_atomic_helper.c | 2 +- drivers/gpu/drm/drm_crtc.c | 23 +--

[Intel-gfx] [PATCH 04/15] drm/gem|prime|mm: Use recommened kerneldoc for struct member refs

2017-01-24 Thread Daniel Vetter
I just learned that _name.member_name works and looks pretty even. It doesn't (yet) link to the member directly though, which would be really good for big structures or vfunc tables (where the per-member kerneldoc tends to be long). Also some minor drive-by polish where it makes sense, I read a

[Intel-gfx] [PATCH 00/15] More kerneldoc cleanup

2017-01-24 Thread Daniel Vetter
Hi all, First few patches are resends that fell through the cracks on a previous patch series. Then there's a bit of garbage collection around struct drm_driver (but I haven't yet done the full documentation run on that one). Plus at the end document drm_crtc.[hc], because I've forgotten about

[Intel-gfx] [PATCH 07/15] drm/gma500: Nuke device_is_agp callback

2017-01-24 Thread Daniel Vetter
Returning 0 for an on-chip gpu doesn't change anything at all. Cc: Patrik Jakobsson Signed-off-by: Daniel Vetter --- drivers/gpu/drm/gma500/psb_drv.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/gpu/drm/gma500/psb_drv.c

[Intel-gfx] [PATCH 02/15] drm/bridge: Use recommened kerneldoc for struct member refs

2017-01-24 Thread Daniel Vetter
I just learned that _name.member_name works and looks pretty even. It doesn't (yet) link to the member directly though, which would be really good for big structures or vfunc tables (where the per-member kerneldoc tends to be long). Also some minor drive-by polish where it makes sense, I read a

[Intel-gfx] [PATCH 08/15] drm/i810: drop device_is_agp callback

2017-01-24 Thread Daniel Vetter
Use the same trick we used for i915 when we still had ums support: Just initialize the agp support unconditionally in the driver load function. Unfortunately that means we need to export drm_agp_init again, but I think that's a lesser evil. Signed-off-by: Daniel Vetter

[Intel-gfx] [PATCH 01/15] drm/kms-helpers: Use recommened kerneldoc for struct member refs

2017-01-24 Thread Daniel Vetter
I just learned that _name.member_name works and looks pretty even. It doesn't (yet) link to the member directly though, which would be really good for big structures or vfunc tables (where the per-member kerneldoc tends to be long). Also some minor drive-by polish where it makes sense, I read a

Re: [Intel-gfx] [PATCH] drm: Don't race connector registration

2017-01-24 Thread Daniel Vetter
Hi Dave, Still waiting for your testing results on this one here ... -Daniel On Thu, Jan 12, 2017 at 05:17:21PM +0100, Daniel Vetter wrote: > On Thu, Jan 12, 2017 at 05:15:56PM +0100, Daniel Vetter wrote: > > I was under the misconception that the sysfs dev stuff can be fully > > set up, and

[Intel-gfx] [GIT PULL] GVT-g fixes for 4.10-rc6

2017-01-24 Thread Zhenyu Wang
Hi, This is gvt fixes for rc6 including shadow batch buffer re-enable which was falsely turned off during initial upstream merge, which is a security issue, Tina re-enabled and verified with Chris's patch too. Also include important ABI fix from Alex, caused by a typo but impact usage with

Re: [Intel-gfx] [PATCH v2 7/9] drm: Connector helper function to release atomic state

2017-01-24 Thread Daniel Vetter
On Tue, Jan 24, 2017 at 03:49:35PM -0800, Dhinakaran Pandiyan wrote: > Having a ->atomic_release callback is useful to release shared resources > that get allocated in compute_config(). > > Suggested-by: Daniel Vetter > Signed-off-by: Dhinakaran Pandiyan

Re: [Intel-gfx] [PATCH v2 8/9] drm/dp: Release DP MST shared link bandwidth

2017-01-24 Thread Daniel Vetter
On Tue, Jan 24, 2017 at 03:49:36PM -0800, Dhinakaran Pandiyan wrote: > Implement the ->atomic_release() callback to release the shared link > bandwidth that was originally acquired during compute_config() > > Signed-off-by: Dhinakaran Pandiyan > --- >

Re: [Intel-gfx] [PATCH v2 9/9] drm/dp: Track MST link bandwidth

2017-01-24 Thread Daniel Vetter
On Tue, Jan 24, 2017 at 03:49:37PM -0800, Dhinakaran Pandiyan wrote: > Use the added helpers to track MST link bandwidth for atomic modesets. > Link bw is acquired in the ->atomic_check() phase when CRTCs are being > enabled with drm_atomic_find_vcpi_slots() instead of drm_find_vcpi_slots(). >

[Intel-gfx] [PATCH 1/2] drm/i915: ignore posting read when using vgpu

2017-01-24 Thread Weinan Li
No need to do posting read when vgpu actived. Signed-off-by: Weinan Li --- drivers/gpu/drm/i915/i915_drv.h | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index

[Intel-gfx] [PATCH 2/2] drm/i915: ignore forcewake get/put when using vgpu

2017-01-24 Thread Weinan Li
Host maintian the hardware's forcewake state, guest don't need and also can't control it. Although vgpu_read/write bypass forcewake_get/put in MMIO read/write, but still have separate path called by "intel_uncore_forcewake_get/put" and "intel_uncore_forcewake_get/put__locked". Unnecessary MMIO

Re: [Intel-gfx] [PATCH v2 5/9] drm/dp: Introduce MST topology state

2017-01-24 Thread Daniel Vetter
On Tue, Jan 24, 2017 at 03:49:33PM -0800, Dhinakaran Pandiyan wrote: > Link bandwidth is shared between multiple display streams in DP MST > configurations. The DP MST topology manager structure maintains the > shared link bandwidth for a primary link directly connected to the GPU. For > atomic

Re: [Intel-gfx] [PATCH v2 4/9] drm: Add driver private objects to atomic state

2017-01-24 Thread Daniel Vetter
On Tue, Jan 24, 2017 at 03:49:32PM -0800, Dhinakaran Pandiyan wrote: > It is necessary to track states for objects other than connector, crtc > and plane for atomic modesets. But adding objects like DP MST link > bandwidth to drm_atomic_state would mean that a non-core object will be > modified by

Re: [Intel-gfx] [PATCH v2 2/9] drm/dp: Kill unused MST vcpi slot availability tracking

2017-01-24 Thread Daniel Vetter
On Tue, Jan 24, 2017 at 03:49:30PM -0800, Dhinakaran Pandiyan wrote: > The avail_slots member in the MST topology manager is never updated to > reflect the available vcpi slots. The check is effectively against > total_slots. So, let's make that check obvious. Secondly, since the total > vcpi time

[Intel-gfx] [PATCH 1/3] [v2] drm: Add new DRM_IOCTL_MODE_GETPLANE2

2017-01-24 Thread Ben Widawsky
Originally based off of a patch by Kristian. This new ioctl extends DRM_IOCTL_MODE_GETPLANE, by returning information about the modifiers that will work with each format. It's modified from Kristian's patch in that the modifiers and formats are setup by the driver, and then a callback is used to

Re: [Intel-gfx] [PATCH 4/4] drm: Resurrect atomic rmfb code, v2

2017-01-24 Thread Daniel Vetter
On Tue, Jan 24, 2017 at 01:44:54PM -0800, Matt Roper wrote: > On Wed, Jan 11, 2017 at 05:15:47PM +0100, Daniel Vetter wrote: > > On Thu, Dec 15, 2016 at 03:29:45PM +0100, Maarten Lankhorst wrote: > > > From: Daniel Vetter > > > > > > This was somehow lost between v3 and

[Intel-gfx] ✗ Fi.CI.BAT: warning for MAINTAINERS: update new mail list for intel gvt driver

2017-01-24 Thread Patchwork
== Series Details == Series: MAINTAINERS: update new mail list for intel gvt driver URL : https://patchwork.freedesktop.org/series/18530/ State : warning == Summary == Series 18530v1 MAINTAINERS: update new mail list for intel gvt driver

Re: [Intel-gfx] [PATCH 0/2] drm: link status property and DP link training failure handling

2017-01-24 Thread Manasi Navare
On Fri, Jan 20, 2017 at 06:27:29PM +0200, Martin Peres wrote: > On 19/01/17 13:34, Ville Syrjälä wrote: > >On Wed, Jan 18, 2017 at 11:05:18PM +0200, Martin Peres wrote: > >>On 16/12/16 15:48, Daniel Vetter wrote: > >>>On Fri, Dec 16, 2016 at 12:29:05PM +0200, Jani Nikula wrote: > The two

[Intel-gfx] [PATCH] MAINTAINERS: update new mail list for intel gvt driver

2017-01-24 Thread Zhenyu Wang
We've moved to lists.freedesktop.org from lists.01.org. Update info in MAINTAINERS. Signed-off-by: Zhenyu Wang --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 5f0420a..5bd03d5 100644 --- a/MAINTAINERS

Re: [Intel-gfx] [igvt-g-dev] [PATCH] drm/i915/gvt: Fix kmem_cache_create() name

2017-01-24 Thread Zhenyu Wang
On 2017.01.24 13:15:43 -0700, Alex Williamson wrote: > According to kmem_cache_sanity_check(), spaces are not allowed in the > name of a cache and results in a kernel oops with CONFIG_DEBUG_VM. > Convert to underscores. > > Signed-off-by: Alex Williamson > --- Will

Re: [Intel-gfx] [igvt-g-dev] [PATCH] drm/i915/gvt/kvmgt: mdev ABI is available_instances, not available_instance

2017-01-24 Thread Zhenyu Wang
On 2017.01.24 12:53:45 -0700, Alex Williamson wrote: > Per the ABI specification[1], each mdev_supported_types entry should > have an available_instances, with an "s", not available_instance. > > [1] Documentation/ABI/testing/sysfs-bus-vfio-mdev > > Signed-off-by: Alex Williamson

Re: [Intel-gfx] [PATCH] drm/i915/gvt/kvmgt: mdev ABI is available_instances, not available_instance

2017-01-24 Thread Jike Song
On 01/25/2017 03:53 AM, Alex Williamson wrote: > Per the ABI specification[1], each mdev_supported_types entry should > have an available_instances, with an "s", not available_instance. > > [1] Documentation/ABI/testing/sysfs-bus-vfio-mdev > > Signed-off-by: Alex Williamson

Re: [Intel-gfx] [PATCH v2 3/9] drm/dp: Split drm_dp_mst_allocate_vcpi

2017-01-24 Thread Dave Airlie
On 25 January 2017 at 09:49, Dhinakaran Pandiyan wrote: > drm_dp_mst_allocate_vcpi() apart from setting up the vcpi structure, > also finds if there are enough slots available. This check is a duplicate > of that implemented in drm_dp_mst_find_vcpi_slots(). Let's

Re: [Intel-gfx] [PATCH v2 1/9] drm/dp: Store drm_device in MST topology manager

2017-01-24 Thread Dave Airlie
On 25 January 2017 at 09:49, Dhinakaran Pandiyan wrote: > struct drm_dp_mst_topology_mgr currently stores a pointer to struct dev. > Changing this to instead hold a pointer to drm_device is more useful as it > gives access to DRM structures. This also makes it

[Intel-gfx] ✓ Fi.CI.BAT: success for Adding private objects to atomic state

2017-01-24 Thread Patchwork
== Series Details == Series: Adding private objects to atomic state URL : https://patchwork.freedesktop.org/series/18523/ State : success == Summary == Series 18523v1 Adding private objects to atomic state https://patchwork.freedesktop.org/api/1.0/series/18523/revisions/1/mbox/ fi-bdw-5557u

[Intel-gfx] [PATCH v2 6/9] drm/dp: Add DP MST helpers to atomically find and release vcpi slots

2017-01-24 Thread Dhinakaran Pandiyan
drm_dp_atomic_find_vcpi_slots() should be called from ->atomic_check() to check there are sufficient vcpi slots for a mode and to add that to the state. This should be followed by a call to drm_dp_mst_allocate_vcpi() in ->atomic_commit() to initialize a struct vcpi for the port.

[Intel-gfx] [PATCH v2 4/9] drm: Add driver private objects to atomic state

2017-01-24 Thread Dhinakaran Pandiyan
It is necessary to track states for objects other than connector, crtc and plane for atomic modesets. But adding objects like DP MST link bandwidth to drm_atomic_state would mean that a non-core object will be modified by the core helper functions for swapping and clearing it's state. So, lets add

[Intel-gfx] [PATCH v2 1/9] drm/dp: Store drm_device in MST topology manager

2017-01-24 Thread Dhinakaran Pandiyan
struct drm_dp_mst_topology_mgr currently stores a pointer to struct dev. Changing this to instead hold a pointer to drm_device is more useful as it gives access to DRM structures. This also makes it consistent with other DRM structures like drm_crtc, drm_connector etc. Signed-off-by: Dhinakaran

[Intel-gfx] [PATCH v2 5/9] drm/dp: Introduce MST topology state

2017-01-24 Thread Dhinakaran Pandiyan
Link bandwidth is shared between multiple display streams in DP MST configurations. The DP MST topology manager structure maintains the shared link bandwidth for a primary link directly connected to the GPU. For atomic modesetting drivers, checking if there is sufficient link bandwidth for a mode

[Intel-gfx] [PATCH v2 3/9] drm/dp: Split drm_dp_mst_allocate_vcpi

2017-01-24 Thread Dhinakaran Pandiyan
drm_dp_mst_allocate_vcpi() apart from setting up the vcpi structure, also finds if there are enough slots available. This check is a duplicate of that implemented in drm_dp_mst_find_vcpi_slots(). Let's move this check out and reuse the existing drm_dp_mst_find_vcpi_slots() function to check if

[Intel-gfx] [PATCH v2 8/9] drm/dp: Release DP MST shared link bandwidth

2017-01-24 Thread Dhinakaran Pandiyan
Implement the ->atomic_release() callback to release the shared link bandwidth that was originally acquired during compute_config() Signed-off-by: Dhinakaran Pandiyan --- drivers/gpu/drm/i915/intel_dp_mst.c | 28 1 file changed, 28

[Intel-gfx] [PATCH v2 2/9] drm/dp: Kill unused MST vcpi slot availability tracking

2017-01-24 Thread Dhinakaran Pandiyan
The avail_slots member in the MST topology manager is never updated to reflect the available vcpi slots. The check is effectively against total_slots. So, let's make that check obvious. Secondly, since the total vcpi time slots is always 63 and does not depend on the link BW, remove total_slots

[Intel-gfx] [PATCH v2 9/9] drm/dp: Track MST link bandwidth

2017-01-24 Thread Dhinakaran Pandiyan
Use the added helpers to track MST link bandwidth for atomic modesets. Link bw is acquired in the ->atomic_check() phase when CRTCs are being enabled with drm_atomic_find_vcpi_slots() instead of drm_find_vcpi_slots(). Similarly, link bw is released during ->atomic_check() with the connector helper

[Intel-gfx] [PATCH v2 7/9] drm: Connector helper function to release atomic state

2017-01-24 Thread Dhinakaran Pandiyan
Having a ->atomic_release callback is useful to release shared resources that get allocated in compute_config(). Suggested-by: Daniel Vetter Signed-off-by: Dhinakaran Pandiyan --- include/drm/drm_modeset_helper_vtables.h | 15

[Intel-gfx] [PATCH v2 0/9] Adding private objects to atomic state

2017-01-24 Thread Dhinakaran Pandiyan
Link bandwidth is shared between multiple display streams in DP MST configurations. The DP MST topology manager structure maintains the shared link bandwidth for a primary link directly connected to the GPU. For atomic modesetting drivers, checking if there is sufficient link bandwidth for a mode

[Intel-gfx] [PATCH i-g-t v2 22/33] tests/kms_properties: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_properties.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/kms_properties.c b/tests/kms_properties.c index

[Intel-gfx] [PATCH i-g-t v2 04/33] lib/igt_kms: Implement dynamic plane count support

2017-01-24 Thread Robert Foss
In upcoming drm-misc-next changes, the number of planes per pipe has been increased as more than one primary plane can be associated with a pipe. The simple fix for this would be to simply bump hardcoded value for number of frames per pipe. But a better solution would be to add support for

[Intel-gfx] [PATCH i-g-t v2 12/33] tests/kms_fence_pin_leak: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_fence_pin_leak.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_fence_pin_leak.c b/tests/kms_fence_pin_leak.c index

[Intel-gfx] [PATCH i-g-t v2 13/33] tests/kms_flip_event_leak: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_flip_event_leak.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_flip_event_leak.c b/tests/kms_flip_event_leak.c index

[Intel-gfx] [PATCH i-g-t v2 28/33] tests/kms_universal_plane: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_universal_plane.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/tests/kms_universal_plane.c

[Intel-gfx] [PATCH i-g-t v2 15/33] tests/kms_mmap_write_crc: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_mmap_write_crc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_mmap_write_crc.c b/tests/kms_mmap_write_crc.c index

[Intel-gfx] [PATCH i-g-t v2 29/33] tests/kms_vblank: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_vblank.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c index 122fbc8f..0d250653

[Intel-gfx] [PATCH i-g-t v2 19/33] tests/kms_plane: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_plane.c | 94 ++- 1 file changed, 52 insertions(+), 42 deletions(-) diff --git

[Intel-gfx] [PATCH i-g-t v2 32/33] tests/kms_plane_lowres: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_plane_lowres.c | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/tests/kms_plane_lowres.c

[Intel-gfx] [PATCH i-g-t v2 07/33] tests/kms_chv_cursor_fail: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_chv_cursor_fail.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/kms_chv_cursor_fail.c b/tests/kms_chv_cursor_fail.c

[Intel-gfx] [PATCH i-g-t v2 23/33] tests/kms_psr_sink_crc: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_psr_sink_crc.c | 52 +--- 1 file changed, 23 insertions(+), 29 deletions(-) diff --git

[Intel-gfx] [PATCH i-g-t v2 20/33] tests/kms_plane_multiple: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_plane_multiple.c | 209 - 1 file changed, 131 insertions(+), 78 deletions(-) diff --git

[Intel-gfx] [PATCH i-g-t v2 18/33] tests/kms_pipe_color: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_pipe_color.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/kms_pipe_color.c b/tests/kms_pipe_color.c index

[Intel-gfx] [PATCH i-g-t v2 09/33] tests/kms_cursor_crc: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_cursor_crc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index

[Intel-gfx] [PATCH i-g-t v2 05/33] tests/kms_atomic_transition: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/tests/kms_atomic_transition.c

[Intel-gfx] [PATCH i-g-t v2 02/33] lib/igt_kms: Avoid depencency on static plane count

2017-01-24 Thread Robert Foss
Rework kmstest_crtc and kmstest_plane structs and their usage to not depend on a static plane count. Signed-off-by: Robert Foss --- lib/igt_kms.c | 47 ++- lib/igt_kms.h | 4 ++-- 2 files changed, 32 insertions(+), 19

[Intel-gfx] [PATCH i-g-t v2 06/33] tests/kms_busy: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_busy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_busy.c b/tests/kms_busy.c index 1ae5d7fb..e6276927 100644 ---

[Intel-gfx] [PATCH i-g-t v2 00/33] lib/igt_kms: Implement dynamic plane count support

2017-01-24 Thread Robert Foss
This series implements dynamic plane count support in lib/igt_kms and modifies all of the tests that rely on a static plane count. Currently it has has gone through incomplete testing on vc4 (many tests are intel only) and intel (on intel-ci). This series can be found in the branch

[Intel-gfx] [PATCH i-g-t v2 01/33] lib/igt_kms: Add index property to kmstest_plane struct

2017-01-24 Thread Robert Foss
Add an index property which helps accessing the corresponding igt_plane_t structure through the igt_*_get_plane() functions. Signed-off-by: Robert Foss --- lib/igt_kms.c| 12 ++-- lib/igt_kms.h| 1 + tests/kms_plane_lowres.c | 2 +- 3

[Intel-gfx] [PATCH i-g-t v2 17/33] tests/kms_panel_fitting: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_panel_fitting.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/kms_panel_fitting.c

[Intel-gfx] [PATCH i-g-t v2 27/33] tests/kms_sink_crc_basic: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_sink_crc_basic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_sink_crc_basic.c b/tests/kms_sink_crc_basic.c index

[Intel-gfx] [PATCH i-g-t v2 24/33] tests/kms_pwrite_crc: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_pwrite_crc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_pwrite_crc.c b/tests/kms_pwrite_crc.c index

[Intel-gfx] [PATCH i-g-t v2 30/33] tests/prime_mmap_kms: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/prime_mmap_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/prime_mmap_kms.c b/tests/prime_mmap_kms.c index

[Intel-gfx] [PATCH i-g-t v2 21/33] tests/kms_plane_scaling: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_plane_scaling.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/kms_plane_scaling.c

[Intel-gfx] [PATCH i-g-t v2 33/33] lib/igt_kms: Remove code obsoleted by dyn n_planes implementation

2017-01-24 Thread Robert Foss
Due to the dyn n_planes implementation some attributes and functions were made obsolete and cand be removed. However to keep all of the tests building the obsolete code is removed after all of the tests have been made compatible with the dyn n_planes changes. Signed-off-by: Robert Foss

[Intel-gfx] [PATCH i-g-t v2 26/33] tests/kms_rotation_crc: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_rotation_crc.c | 63 1 file changed, 32 insertions(+), 31 deletions(-) diff --git

[Intel-gfx] [PATCH i-g-t v2 25/33] tests/kms_rmfb: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_rmfb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_rmfb.c b/tests/kms_rmfb.c index 17a3065a..5753d74c 100644 ---

[Intel-gfx] [PATCH i-g-t v2 16/33] tests/kms_mmio_vs_cs_flip: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_mmio_vs_cs_flip.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/kms_mmio_vs_cs_flip.c b/tests/kms_mmio_vs_cs_flip.c

[Intel-gfx] [PATCH i-g-t v2 31/33] tests/kms_ccs: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_ccs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c index 047a3e87..11acda89 100644 ---

[Intel-gfx] [PATCH i-g-t v2 10/33] tests/kms_cursor_legacy: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_cursor_legacy.c | 32 ++-- 1 file changed, 18 insertions(+), 14 deletions(-) diff --git a/tests/kms_cursor_legacy.c

[Intel-gfx] [PATCH i-g-t v2 14/33] tests/kms_legacy_colorkey: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_legacy_colorkey.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/kms_legacy_colorkey.c b/tests/kms_legacy_colorkey.c

[Intel-gfx] [PATCH i-g-t v2 03/33] lib/igt_kms: Rename kmstest properties nplanes and plane

2017-01-24 Thread Robert Foss
Rename these properties to have them use the same naming convention as the igt_*_t structs. Signed-off-by: Robert Foss --- lib/igt_kms.c| 16 +--- lib/igt_kms.h| 4 ++-- tests/kms_plane_lowres.c | 10 +- 3 files changed, 16

[Intel-gfx] [PATCH i-g-t v2 11/33] tests/kms_fbc_crc: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_fbc_crc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tests/kms_fbc_crc.c b/tests/kms_fbc_crc.c index

[Intel-gfx] [PATCH i-g-t v2 08/33] tests/kms_crtc_background_color: Add support for dynamic number of planes

2017-01-24 Thread Robert Foss
Add changes reflecting the new support for dynamic number of planes per pipe. Signed-off-by: Robert Foss --- tests/kms_crtc_background_color.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/kms_crtc_background_color.c

[Intel-gfx] DP Aux interfaces inquiry

2017-01-24 Thread Mario.Limonciello
Hi, Recently Synaptics collaborated with Dell on a plugin [1] for fwupd that allows flashing Synaptics MST hubs using the DP aux interface to manipulate the DPCD [2]. Currently the plugin hardcodes the max number of DP aux devices to look for to 3 (as that's what we've seen so far on HW), but

Re: [Intel-gfx] [PATCH 4/4] drm: Resurrect atomic rmfb code, v2

2017-01-24 Thread Matt Roper
On Wed, Jan 11, 2017 at 05:15:47PM +0100, Daniel Vetter wrote: > On Thu, Dec 15, 2016 at 03:29:45PM +0100, Maarten Lankhorst wrote: > > From: Daniel Vetter > > > > This was somehow lost between v3 and the merged version in Maarten's > > patch merged as: > > > > commit

Re: [Intel-gfx] [PATCH 1/1] drm/i915: Copy user requested buffers into the error state

2017-01-24 Thread Chris Wilson
On Tue, Jan 24, 2017 at 11:44:26AM -0800, Ben Widawsky wrote: > >+for (j = 0; j < ee->user_bo_count; j++) > >+print_error_obj(m, dev_priv->engine[i], > >+"user", ee->user_bo[j]); > >+ > > We'll need a way to be able to figure out

Re: [Intel-gfx] [PATCH 1/1] drm/i915: Copy user requested buffers into the error state

2017-01-24 Thread Chris Wilson
On Tue, Jan 24, 2017 at 11:44:26AM -0800, Ben Widawsky wrote: > On 17-01-24 13:16:56, Mika Kuoppala wrote: > >+for (j = 0; j < ee->user_bo_count; j++) > >+print_error_obj(m, dev_priv->engine[i], > >+"user", ee->user_bo[j]); > >+ >

[Intel-gfx] [PATCH] drm/i915/gvt: Fix kmem_cache_create() name

2017-01-24 Thread Alex Williamson
According to kmem_cache_sanity_check(), spaces are not allowed in the name of a cache and results in a kernel oops with CONFIG_DEBUG_VM. Convert to underscores. Signed-off-by: Alex Williamson --- drivers/gpu/drm/i915/gvt/execlist.c |2 +- 1 file changed, 1

[Intel-gfx] ✓ Fi.CI.BAT: success for x86/gpu: GLK uses the same GMS values as SKL

2017-01-24 Thread Patchwork
== Series Details == Series: x86/gpu: GLK uses the same GMS values as SKL URL : https://patchwork.freedesktop.org/series/18505/ State : success == Summary == Series 18505v1 x86/gpu: GLK uses the same GMS values as SKL https://patchwork.freedesktop.org/api/1.0/series/18505/revisions/1/mbox/

[Intel-gfx] [PATCH] drm/i915/gvt/kvmgt: mdev ABI is available_instances, not available_instance

2017-01-24 Thread Alex Williamson
Per the ABI specification[1], each mdev_supported_types entry should have an available_instances, with an "s", not available_instance. [1] Documentation/ABI/testing/sysfs-bus-vfio-mdev Signed-off-by: Alex Williamson --- This should really be fixed before initial

Re: [Intel-gfx] [PATCH 1/1] drm/i915: Copy user requested buffers into the error state

2017-01-24 Thread Ben Widawsky
On 17-01-24 13:16:56, Mika Kuoppala wrote: From: Chris Wilson Introduce a new execobject.flag (EXEC_OBJECT_CAPTURE) that userspace may use to indicate that it wants the contents of this buffer preserved in the error state (/sys/class/drm/cardN/error) following a GPU

[Intel-gfx] [PATCH] x86/gpu: GLK uses the same GMS values as SKL

2017-01-24 Thread Paulo Zanoni
So don't forget to reserve its stolen memory bits. Cc: Ingo Molnar Cc: H. Peter Anvin Cc: Ander Conselvan de Oliveira Cc: x...@kernel.org Reviewed-by: Ander Conselvan de Oliveira

Re: [Intel-gfx] [PATCH 3/4] drm/i915/glk: Program pipe gamma and degamma tables

2017-01-24 Thread Ville Syrjälä
On Tue, Jan 24, 2017 at 03:35:54PM +0200, Ander Conselvan de Oliveira wrote: > The gamma tables in Geminilake were changed. There is no split-gamma > mode. Instead, there is a dedicated degamma table that is enabled > whenever pipe CSC is enabled. > > The dedicated gamma table has 16 bit

Re: [Intel-gfx] [PATCH i-g-t v4] tools: Add intel_dp_compliance for DisplayPort 1.2 compliance automation

2017-01-24 Thread Manasi Navare
On Tue, Jan 24, 2017 at 11:49:48AM +0200, Petri Latvala wrote: > > Just one more nitpick (in two parts) below. With that, you can > consider this now to be > > Reviewed-by: Petri Latvala > > This patch can be pushed when the kernel changes that contain "must be > kept

  1   2   >