Re: [Intel-gfx] [PATCH v2 04/10] drm: Add Gamma correction structure

2015-06-14 Thread Sharma, Shashank
Hi, Emil Velikov The reason behind a zero sized array is that we want to use the same variable for various color correction possible across various driver . Due to current blob implementation, it doesn’t look very efficient to have another pointer in the structure, so we are left with this opt

Re: [Intel-gfx] [PATCH 3/4] drm: Add decoding of i915 ioctls

2015-06-14 Thread Patrik Jakobsson
On Sat, Jun 13, 2015 at 1:48 AM, Dmitry V. Levin wrote: > On Thu, Jun 11, 2015 at 03:34:14PM +0200, Patrik Jakobsson wrote: >> On Thu, Jun 11, 2015 at 02:27:12AM +0300, Dmitry V. Levin wrote: >> > On Wed, Jun 10, 2015 at 02:45:24PM +0200, Patrik Jakobsson wrote: >> > > On Wed, Jun 10, 2015 at 01:3

Re: [Intel-gfx] [PATCH 2/4] drm: Add dispatcher and driver identification for DRM

2015-06-14 Thread Patrik Jakobsson
On Sat, Jun 13, 2015 at 1:41 AM, Dmitry V. Levin wrote: > On Thu, Jun 11, 2015 at 04:11:49PM +0200, Patrik Jakobsson wrote: >> On Thu, Jun 11, 2015 at 02:26:59AM +0300, Dmitry V. Levin wrote: >> > On Wed, Jun 10, 2015 at 01:52:33PM +0200, Patrik Jakobsson wrote: >> > > On Wed, Jun 10, 2015 at 01:1

Re: [Intel-gfx] [PATCH] drm/i915: Add the ddi get cdclk code for BXT.

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6564 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH] drm/i915: Delete duplicate #defines added for DCx

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6565 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH v2] drm/i915: Limit CHV max cdclk

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6567 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH] drm/i915/irq: wait a little before queuing the hotplug work function

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6568 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH 2/2] drm/i915: frontbuffer invalidate at flip schedule

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6569 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH 2/4] drm: Add dispatcher and driver identification for DRM

2015-06-14 Thread Dmitry V. Levin
On Sun, Jun 14, 2015 at 01:12:45PM +0200, Patrik Jakobsson wrote: [...] > How about adding a "void *private" field to struct tcb. That way any > syscall can store additional data across the life of the tcb. We can add a field to struct tcb, but its semantics wrt memory management should be strictl

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Drop the 64k linear scanout alignment on gen2/3

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6570 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH] drm/i915: Ignore -EIO from __i915_wait_request() during mmio flip

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6571 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH v2] drm/i915: Ignore -EIO from __i915_wait_request() during flips

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6572 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH] drm/i915: Initialize HWS page address after GPU reset

2015-06-14 Thread Daniel Vetter
On Wed, Jun 3, 2015 at 6:14 PM, Ville Syrjälä wrote: > I was going to suggest removing the same thing from the > lrc_setup_hardware_status_page(), but after another look it seems we > sometimes call .init_hw() before the context setup. Would be nice to > have a more consistent sequence for init an

Re: [Intel-gfx] [PATCH 4/4] drm/i915: don't set the FBC plane select bits on HSW+

2015-06-14 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6573 -Summary- Platform Delta drm-intel-nightly Series Applied PNV

Re: [Intel-gfx] [PATCH v2] drm/atomic: Handle vblank events in atomic ioctl correctly.

2015-06-14 Thread Daniel Vetter
On Tue, Jun 02, 2015 at 09:00:46AM +0200, Maarten Lankhorst wrote: > All users of async updates seem to clear clear crtc_state->event > correctly, so move destroying vblank event to crtc_destroy_state. > > This is better than manually free'ing it in the atomic ioctl, since > this code seems to do

Re: [Intel-gfx] Deadlock in intel_user_framebuffer_destroy()

2015-06-14 Thread Jani Nikula
On Wed, 03 Jun 2015, Chris Wilson wrote: > On Wed, Jun 03, 2015 at 03:43:32PM +0200, Lukas Wunner wrote: >> Hi, >> >> a deadlock was introduced by commit 60a5ca015ffd2aacfe5674b5a401cd2a37159e07 >> >> Author: Ville Syrjälä >> Date: Fri Jun 13 11:10:53 2014 +0300 >> >> drm/i915: Add locki

Re: [Intel-gfx] [PATCH v2 00/10] Color Manager Implementation

2015-06-14 Thread Daniel Vetter
On Tue, Jun 09, 2015 at 01:50:48PM +0100, Damien Lespiau wrote: > On Thu, Jun 04, 2015 at 07:12:31PM +0530, Kausal Malladi wrote: > > From: Kausal Malladi > > > > This patch set adds color manager implementation in drm/i915 layer. > > Color Manager is an extension in i915 driver to support color

Re: [Intel-gfx] [BUG, bisect] Re: drm/i915: WARN_ON(dev_priv->mm.busy)

2015-06-14 Thread Daniel Vetter
Can you please retest with commit 0aedb1626566efd72b369c01992ee7413c82a0c5 Author: Ville Syrjälä Date: Thu May 28 18:32:36 2015 +0300 drm/i915: Don't skip request retirement if the active list is empty Thanks, Daniel On Sun, Jun 07, 2015 at 10:56:41PM -0700, Jeremiah Mahler wrote: > all,