Re: [Intel-gfx] [PATCH 3/4] drm/i915: Add copyright headers to all golden context source

2014-10-24 Thread Mika Kuoppala
armin.c.re...@intel.com writes: From: Armin Reese armin.c.re...@intel.com Existing Gen6, 7, 8, and 9 golden context source files are currently generated without the copyright header. Adding them manually here. Signed-off-by: Armin Reese armin.c.re...@intel.com ---

Re: [Intel-gfx] [Beignet] Beignet crashes on vanilla 3.17.1 with IVB hardware

2014-10-24 Thread Zhigang Gong
I just checked again, and found both 3.17 and 3.17.1 should work fine on IVB with beignet. I just tested beignet on IVB with kernel 3.17.1, all unit tests passed successfully. For IVB user, no need to wait for 3.18. Don't know which application you were testing on your IVB machine. If it could be

Re: [Intel-gfx] [PATCH 2/2] drm/i915/bdw: Setup global hardware status page in execlists mode

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 04:24:46PM +0100, Thomas Daniel wrote: Write HWS_PGA address even in execlists mode as the global hardware status page is still required. This address was previously uninitialized and HWSP writes would clobber whatever buffer happened to reside at GGTT address 0.

Re: [Intel-gfx] [PATCH 01/10] drm/i915: Add automated testing support for Displayport compliance testing

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 09:58:47AM -0700, Todd Previte wrote: On 10/20/2014 10:48 AM, Paulo Zanoni wrote: 2014-10-09 12:38 GMT-03:00 Todd Previte tprev...@gmail.com: +/* Displayport compliance testing - PHY pattern testing */ +static uint8_t +intel_dp_autotest_phy_pattern(struct intel_dp

Re: [Intel-gfx] [PATCH] drm/i915: Improve reliability for Displayport link training

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 09:56:53AM -0700, Todd Previte wrote: On 10/22/2014 7:22 AM, Jani Nikula wrote: On Thu, 09 Oct 2014, Todd Previte tprev...@gmail.com wrote: Link training for Displayport can fail in many ways and at multiple different points during the training process. Previously,

Re: [Intel-gfx] [PATCH] drm/i915: Support to create uncached user mapping for a Gem object

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 04:03:56PM +0530, akash.g...@intel.com wrote: From: Akash Goel akash.g...@intel.com This patch provides support to create uncached virtual mappings for a Gem object. It intends to provide the same funtionality of 'mmap_gtt' interface without the constraints of a

Re: [Intel-gfx] [PATCH 2/2] drm/i915/bdw: Setup global hardware status page in execlists mode

2014-10-24 Thread Daniel Vetter
On Fri, Oct 24, 2014 at 08:30:56AM +, Daniel, Thomas wrote: -Original Message- From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel Vetter Sent: Friday, October 24, 2014 9:15 AM To: Daniel, Thomas Cc: intel-gfx@lists.freedesktop.org;

Re: [Intel-gfx] [PATCH v2 09/14] drm/i915: disable/re-enable PCI device around S4 freeze/thaw

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 07:23:23PM +0300, Imre Deak wrote: We already disable everything during S4 freeze, except the PCI device itself. There is no reason why we couldn't disable that too and doing so allows us to unify these handlers in the next patch with the corresponding S3 suspend/resume

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Grab gpu state through debugfs

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 08:35:24PM +0300, Mika Kuoppala wrote: because it is handy to direct bug reporters to do things like: 'grep -i suspend /sys/kernel/debug/dri/0/i915_gpu_state' Signed-off-by: Mika Kuoppala mika.kuopp...@intel.com --- drivers/gpu/drm/i915/i915_debugfs.c | 42

Re: [Intel-gfx] [PATCH] drm/i915: Support to create uncached user mapping for a Gem object

2014-10-24 Thread Chris Wilson
On Fri, Oct 24, 2014 at 10:40:10AM +0200, Daniel Vetter wrote: - Polish for Akash' patch according to Chris' review. Commit message also needs to gain the performance data. And we need to have a does PAT work test I think like Chris suggested. Done, see v2 I sent. Commit message contains

Re: [Intel-gfx] [PATCH 2/2] drm/i915/bdw: Setup global hardware status page in execlists mode

2014-10-24 Thread Chris Wilson
On Fri, Oct 24, 2014 at 10:48:43AM +0200, Daniel Vetter wrote: On Fri, Oct 24, 2014 at 08:30:56AM +, Daniel, Thomas wrote: -Original Message- From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel Vetter Sent: Friday, October 24, 2014 9:15 AM To:

Re: [Intel-gfx] [PATCH v2 09/14] drm/i915: disable/re-enable PCI device around S4 freeze/thaw

2014-10-24 Thread Imre Deak
On Fri, 2014-10-24 at 10:51 +0200, Daniel Vetter wrote: On Thu, Oct 23, 2014 at 07:23:23PM +0300, Imre Deak wrote: We already disable everything during S4 freeze, except the PCI device itself. There is no reason why we couldn't disable that too and doing so allows us to unify these handlers

Re: [Intel-gfx] [Beignet] Beignet crashes on vanilla 3.17.1 with IVB hardware

2014-10-24 Thread Zhigang Gong
Hi, Luxmark (both 2.0/2.1) works fine on my IVB machine. The back trace you provided below doesn't indicate it's a beignet related problem. It hadn't enter beignet domain and just crashed in luxmark internal. On Fri, Oct 24, 2014 at 12:04:29PM +0300, Vasily Khoruzhick wrote: Hi Zhigang,

[Intel-gfx] [PATCH] drm/i915: Move flags describing VMA mappings into the VMA

2014-10-24 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com If these flags are on the object level it will be more difficult to allow for multiple VMAs per object. v2: Simplification and cleanup after code review comments (Chris Wilson). Signed-off-by: Tvrtko Ursulin tvrtko.ursu...@intel.com Cc: Chris Wilson

[Intel-gfx] [RFC] drm/i915: Prepare for the idea there can be multiple VMA per VM.

2014-10-24 Thread Tvrtko Ursulin
From: Tvrtko Ursulin tvrtko.ursu...@intel.com With multiple VMA per object and per address space we need some sort of key to distinguish between then for which there is a new 'id' field. Old API which does not know or care about this will use a default id. I have tried to identify all places

Re: [Intel-gfx] [PATCH] drm/i915: Ignore SURFLIVE and flip counter when the GPU gets reset

2014-10-24 Thread Chris Wilson
On Wed, May 28, 2014 at 01:10:55PM +0100, Chris Wilson wrote: On Tue, May 27, 2014 at 09:33:09PM +0300, ville.syrj...@linux.intel.com wrote: diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 94ac51f..cb9dd8e 100644 ---

Re: [Intel-gfx] [PATCH] drm/i915: Ignore SURFLIVE and flip counter when the GPU gets reset

2014-10-24 Thread Ville Syrjälä
On Fri, Oct 24, 2014 at 01:40:35PM +0100, Chris Wilson wrote: On Wed, May 28, 2014 at 01:10:55PM +0100, Chris Wilson wrote: On Tue, May 27, 2014 at 09:33:09PM +0300, ville.syrj...@linux.intel.com wrote: diff --git a/drivers/gpu/drm/i915/intel_display.c

Re: [Intel-gfx] [PATCH v2 09/15] drm/i915: Ignore VBT int_crt_support on 830M

2014-10-24 Thread Jani Nikula
On Fri, 06 Jun 2014, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com My Fujitsu-Siemens Lifebook S6010 definitely has a VGA connector, but the VBT says different. Ignore the VBT for 830M since it seems such old machines would generally have a VGA

[Intel-gfx] [PATCH v4 4/5] drm/i915: only flip frontbuffer if crtc is active

2014-10-24 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk There is no point in flipping a buffer for a disabled crtc. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk --- drivers/gpu/drm/i915/intel_display.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Intel-gfx] [PATCH v4 5/5] drm/i915: remove intel_crtc_cursor_set_obj()

2014-10-24 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Merge it into the plane update_plane() callback and make other users use the update_plane() functions instead. The fb != crtc-cursor-fb was already inside intel_crtc_cursor_set_obj() so we fold intel_crtc_cursor_set_obj() inside

[Intel-gfx] [PATCH v4 2/5] drm/i915: create a prepare phase for sprite plane updates

2014-10-24 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk take out pin_fb code so the commit phase can't fail anymore. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/i915/intel_sprite.c | 63

[Intel-gfx] [PATCH v4 3/5] drm/i915: use intel_fb_obj() macros to assign gem objects

2014-10-24 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Use the macros makes the code cleaner and it also checks for a NULL fb. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/i915/intel_sprite.c | 8

[Intel-gfx] [PATCH v4 1/5] drm/i915: create a prepare step for primary planes updates

2014-10-24 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk Take out the pin_fb code so commit phase can't fail anymore. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk Reviewed-by: Ville Syrjälä ville.syrj...@linux.intel.com --- drivers/gpu/drm/i915/intel_display.c | 35

Re: [Intel-gfx] [PATCH v3 11/11] drm/i915: remove intel_pipe_set_base()

2014-10-24 Thread Gustavo Padovan
2014-10-07 Ville Syrjälä ville.syrj...@linux.intel.com: On Wed, Sep 24, 2014 at 02:20:32PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk After some refactor intel_primary_plane_setplane() does the same as intel_pipe_set_base() so we can get rid of

Re: [Intel-gfx] [PATCH v2 09/15] drm/i915: Ignore VBT int_crt_support on 830M

2014-10-24 Thread Ville Syrjälä
On Fri, Oct 24, 2014 at 04:23:22PM +0300, Jani Nikula wrote: On Fri, 06 Jun 2014, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com My Fujitsu-Siemens Lifebook S6010 definitely has a VGA connector, but the VBT says different. Ignore the VBT for 830M

[Intel-gfx] [PULL] drm-intel-next, take 3

2014-10-24 Thread Daniel Vetter
Hi Dave, Ok, new attempt, this time around with full ppgtt disabled again. drm-intel-next-2014-10-03: - first batch of skl stage 1 enabling - fixes from Rodrigo to the PSR, fbc and sink crc code - kerneldoc for the frontbuffer tracking code, runtime pm code and the basic interrupt

Re: [Intel-gfx] [PATCH v3 05/11] drm/i915: remove intel_crtc_cursor_set_obj()

2014-10-24 Thread Ville Syrjälä
On Fri, Oct 24, 2014 at 02:23:35PM +0100, Gustavo Padovan wrote: 2014-10-07 Ville Syrjälä ville.syrj...@linux.intel.com: On Wed, Sep 24, 2014 at 02:20:26PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk Merge it into the plane update_plane()

Re: [Intel-gfx] [PATCH] drm/i915: Report the actual swizzling back to userspace

2014-10-24 Thread Daniel Vetter
On Fri, Oct 24, 2014 at 12:11:11PM +0100, Chris Wilson wrote: Userspace cares about whether or not swizzling depends on the page address for its direct access into bound objects. Extend the get_tiling ioctl to report the physical swizzling value in addition to the logical swizzling value so

Re: [Intel-gfx] [PATCH] drm/i915: Move flags describing VMA mappings into the VMA

2014-10-24 Thread Daniel Vetter
On Fri, Oct 24, 2014 at 01:09:08PM +0100, Chris Wilson wrote: On Fri, Oct 24, 2014 at 12:42:33PM +0100, Tvrtko Ursulin wrote: From: Tvrtko Ursulin tvrtko.ursu...@intel.com If these flags are on the object level it will be more difficult to allow for multiple VMAs per object. v2:

[Intel-gfx] Updated drm-intel-testing

2014-10-24 Thread Daniel Vetter
Hi all, New -testing cycle with cool stuff: - suspend/resume/freeze/thaw unification from Imre - wa list improvements from MikaArun - display pll precomputation from Ander Conselvan, this removed the last -mode_set callbacks, a big step towards implementing atomic modesets - more kerneldoc for

Re: [Intel-gfx] [PATCH v4 4/5] drm/i915: only flip frontbuffer if crtc is active

2014-10-24 Thread Ville Syrjälä
On Fri, Oct 24, 2014 at 02:51:34PM +0100, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk There is no point in flipping a buffer for a disabled crtc. That thing doesn't actually flip but just signal the frontbuffer tracking code that either has just flipped or is

Re: [Intel-gfx] [PATCH] drm/i915: Report the actual swizzling back to userspace

2014-10-24 Thread Chris Wilson
On Fri, Oct 24, 2014 at 04:39:29PM +0200, Daniel Vetter wrote: On Fri, Oct 24, 2014 at 12:11:11PM +0100, Chris Wilson wrote: Userspace cares about whether or not swizzling depends on the page address for its direct access into bound objects. Extend the get_tiling ioctl to report the

Re: [Intel-gfx] [PATCH v4 5/5] drm/i915: remove intel_crtc_cursor_set_obj()

2014-10-24 Thread Ville Syrjälä
On Fri, Oct 24, 2014 at 02:51:35PM +0100, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk Merge it into the plane update_plane() callback and make other users use the update_plane() functions instead. The fb != crtc-cursor-fb was already inside

Re: [Intel-gfx] [PATCH] drm/i915: Abort command parsing for chained batches

2014-10-24 Thread Volkin, Bradley D
On Thu, Oct 23, 2014 at 08:52:59AM -0700, Volkin, Bradley D wrote: On Thu, Oct 23, 2014 at 05:31:12AM -0700, Daniel Vetter wrote: On Wed, Oct 22, 2014 at 09:04:32AM -0700, Volkin, Bradley D wrote: [snip] On Tue, Oct 21, 2014 at 08:50:33AM -0700, Daniel Vetter wrote: On Thu, Oct

[Intel-gfx] [PATCH 1/2] drm: make sure visible is set to false if fb is null

2014-10-24 Thread Gustavo Padovan
From: Gustavo Padovan gustavo.pado...@collabora.co.uk We can't let visible set true while the fb is null, some places of the code only check for visible to base its decisions. Signed-off-by: Gustavo Padovan gustavo.pado...@collabora.co.uk --- drivers/gpu/drm/drm_plane_helper.c | 5 + 1 file

Re: [Intel-gfx] [PATCH] drm/i915: properly reenable gen8 pipe IRQs

2014-10-24 Thread Jani Nikula
On Wed, 08 Oct 2014, Paulo Zanoni przan...@gmail.com wrote: From: Paulo Zanoni paulo.r.zan...@intel.com We were missing the pipe B/C vblank bits! Take a look at gen8_de_irq_postinstall for a comparison. This should fix a bunch of IGT tests. There are a few more things we could improve on

Re: [Intel-gfx] [PATCH] drm/i915: Do not export RC6p and RC6pp if they don't exist

2014-10-24 Thread Paulo Zanoni
2014-10-07 11:06 GMT-03:00 Rodrigo Vivi rodrigo.v...@intel.com: Avoid to expose RC6 and RC6pp to the platforms that doesn't support it. So powertop can be changed to show RC6p and RC6pp only on the platforms they are available. v2: Simplify by merging RC6p and RC6pp groups and respect the

Re: [Intel-gfx] [PATCH v3 07/11] drm: add helper to get crtc timings

2014-10-24 Thread Ville Syrjälä
On Wed, Sep 24, 2014 at 02:20:28PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk We need to get hdisplay and vdisplay in a few places so create a helper to make our job easier. Suggested-by: Ville Syrjälä ville.syrj...@linux.intel.com Signed-off-by:

Re: [Intel-gfx] [PATCH] drm/i915: Do not export RC6p and RC6pp if they don't exist

2014-10-24 Thread Paulo Zanoni
2014-10-01 11:47 GMT-03:00 Rodrigo Vivi rodrigo.v...@intel.com: Avoid to expose RC6 and RC6pp to the platforms that doesn't support it. So powertop can be changed to show RC6p and RC6pp only on the platforms they are available. v2: Simplify by merging RC6p and RC6pp groups and respect the

Re: [Intel-gfx] [PATCH v2] drm/i915: Audio N value computed for pixel doubled modes

2014-10-24 Thread Clint Taylor
On 09/26/2014 09:28 AM, Ville Syrjälä wrote: On Thu, Sep 25, 2014 at 09:26:36AM -0700, clinton.a.tay...@intel.com wrote: From: Clint Taylor clinton.a.tay...@intel.com HDMI audio clock config was incorrectly choosing the default for pixel doubled interlaced modes. The table was missing pixel

Re: [Intel-gfx] [PATCH v3 04/11] drm/i915: move check of intel_crtc_cursor_set_obj() out

2014-10-24 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 06:01:10PM +0300, Ville Syrjälä wrote: On Tue, Oct 07, 2014 at 05:47:52PM +0300, Ville Syrjälä wrote: On Wed, Sep 24, 2014 at 02:20:25PM -0300, Gustavo Padovan wrote: From: Gustavo Padovan gustavo.pado...@collabora.co.uk Move check inside

Re: [Intel-gfx] [PATCH i-g-t 1/2] quick-dump: Make quick dump link against libintel_tools

2014-10-24 Thread Damien Lespiau
On Tue, Oct 07, 2014 at 01:06:32PM +0300, Ville Syrjälä wrote: + $(top_srcdir)/lib/libintel_tools.la \ top_builddir Yes, caught it just before merging :) good eye though! -- Damien ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org

[Intel-gfx] [PATCH v2] drm/i915: Expose all 36-bits of TIMESTAMP register on x86_64

2014-10-24 Thread Michał Winiarski
Reading timestamp register using I915_READ64 returns incorrect value containing only 32-bits of usable timestamp on x86_64. Let's use I915_READ64_2x32 to expose missing bits and swap dwords to stay compatible with old userspace. v2: Renamed macro used for swapping dwords to be more like swab.h

[Intel-gfx] [RFC 15/25] drm/i915: Connect requests to rings at creation not submission

2014-10-24 Thread John . C . Harrison
From: John Harrison john.c.harri...@intel.com For: VIZ-4377 Signed-off-by: john.c.harri...@intel.com --- drivers/gpu/drm/i915/i915_gem.c |1 - drivers/gpu/drm/i915/intel_lrc.c|1 + drivers/gpu/drm/i915/intel_ringbuffer.c |1 + 3 files changed, 2 insertions(+), 1

Re: [Intel-gfx] [PATCH 2/4] drm/i915: Make *_crtc_mode_set work on new_config

2014-10-24 Thread Ander Conselvan de Oliveira
On 10/09/2014 12:11 PM, Daniel Vetter wrote: On Wed, Oct 08, 2014 at 06:32:21PM +0300, Ander Conselvan de Oliveira wrote: From: Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com This shouldn't change the behavior of those functions, since they are called after the new_config is

Re: [Intel-gfx] [PATCH] drm/i915: use delayed work for resume hotplug v4

2014-10-24 Thread Chris Wilson
On Thu, Oct 09, 2014 at 08:13:18AM -0700, Jesse Barnes wrote: Gets the detect code (which may take awhile) out of the resume path, speeding things up a bit. v2: use a delayed work queue instead (Daniel) v3: cancel delayed work at unload and suspend time (Jesse) v4: make delayed work comment

[Intel-gfx] [PATCH 04/10] drm/i915: Add a delay in Displayport AUX transactions for compliance testing

2014-10-24 Thread Todd Previte
The Displayport Link Layer Compliance Testing Specification 1.2 rev 1.1 specifies that repeated AUX transactions after a failure (no response / invalid response) must have a minimum delay of 400us before the resend can occur. Tests 4.2.1.1 and 4.2.1.2 are two tests that require this

Re: [Intel-gfx] significant ioremap leak in i915?

2014-10-24 Thread Breton M. Saunders
On 13/10/14 02:09, Dave Airlie wrote: So in this example 268 megabytes have been lost. This isn't RAM, its address space mapping, its not really a leak at all. It really doesn't matter whether it is ram or mapping; its a resource leak that causes the system to OOM and panic. hey I was

[Intel-gfx] [RFC] drm/i915: Remove FIXME_lrc_ctx backpointer

2014-10-24 Thread Nick Hoath
The first pass implementation of execlists required a backpointer to the context to be held in the intel_ringbuffer. However the context pointer is available higher in the call stack. Remove the backpointer from the ring buffer structure and instead pass it down through the call stack.

[Intel-gfx] [PATCH 5/8] drm/i915/skl: Move all the WM compute functions in one place

2014-10-24 Thread Damien Lespiau
The DDB allocation code managed to split in two the compute functions. Bring back skl_compute_transition_wm() and skl_compute_linetime_wm() with their little friends. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_pm.c | 44

[Intel-gfx] [PATCH 0/3] Rework of the WM flush (for the DDB allocation)

2014-10-24 Thread Damien Lespiau
Ville found that the sequencing I had to re-program the DDB wasn't quite correct and so this is an attempt to do better. This series reworks patch: [PATCH 78/89] drm/i915/skl: Flush the WM configuration of the initial SKL Stage 1 series. The core of the issue is now documented in a comment,

Re: [Intel-gfx] [RFC PATCH 2/3] drm/ipvr: drm driver for vxd392

2014-10-24 Thread David Herrmann
Hi On Thu, Oct 16, 2014 at 3:39 PM, Cheng, Yao yao.ch...@intel.com wrote: Accepted :) I will update the patch to implement the mmap interface and remove the legacy MMAP_IOCTL. BTW I didn't see a field to get mmap_offset in struct drm_gem_open, I guess something like a new

Re: [Intel-gfx] [RFC 09/21] drm/i915: Make 'i915_gem_check_olr' actually check by request not seqno

2014-10-24 Thread Daniel Vetter
On Mon, Oct 06, 2014 at 03:15:13PM +0100, john.c.harri...@intel.com wrote: From: John Harrison john.c.harri...@intel.com To thin commit message. Also I wonder whethere we should track the olr state more explicitly in the request structure instead of jumping through all these hoops. And explicit

Re: [Intel-gfx] [RFC 06/21] drm/i915: Replace last_[rwf]_seqno with last_[rwf]_req

2014-10-24 Thread Daniel Vetter
On Mon, Oct 06, 2014 at 03:15:10PM +0100, john.c.harri...@intel.com wrote: From: John Harrison john.c.harri...@intel.com For: VIZ-4377 Signed-off-by: john.c.harri...@intel.com --- drivers/gpu/drm/i915/i915_debugfs.c|6 +-- drivers/gpu/drm/i915/i915_drv.h|6 +--

[Intel-gfx] [PATCH i-g-t 5/7] configure: include the skylake quick_dump files in EXTRA_DIST

2014-10-24 Thread Thomas Wood
Signed-off-by: Thomas Wood thomas.w...@intel.com --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 9cdd202..03e856d 100644 --- a/configure.ac +++ b/configure.ac @@ -201,7 +201,7 @@ fi AM_CONDITIONAL(BUILD_TESTS, [test

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Ignore long hpds on eDP ports

2014-10-24 Thread Todd Previte
On 10/16/2014 10:46 AM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com Turning vdd on/off can generate a long hpd pulse on eDP ports. In order to handle hpd we would need to turn on vdd to perform aux transfers. This would lead to an endless cycle of vdd

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Do a dummy DPCD read before the actual read

2014-10-24 Thread Ville Syrjälä
On Thu, Oct 16, 2014 at 12:39:29PM -0700, Todd Previte wrote: On 10/16/2014 10:46 AM, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com Sometimes we seem to get utter garbage from DPCD reads. The resulting buffer is filled with the same byte, and the

Re: [Intel-gfx] [RFC 7/7] drm/i915: Modify refs to intel dp timestamps

2014-10-24 Thread Daniel Vetter
On Mon, Oct 20, 2014 at 06:20:09PM +0530, Vandana Kannan wrote: Moving timestamp values to intel_panel as part of moving all refs of PPS to intel_panel. Signed-off-by: Vandana Kannan vandana.kan...@intel.com On second though, how does the code still work before applying patch 67? Presuming

Re: [Intel-gfx] [PATCH 01/10] drm/i915: Add automated testing support for Displayport compliance testing

2014-10-24 Thread Daniel Vetter
On Thu, Oct 09, 2014 at 08:38:01AM -0700, Todd Previte wrote: Add the skeleton framework for supporting automation for Displayport compliance testing. This patch adds the necessary framework for the source device to appropriately respond to test automation requests from a sink device.

[Intel-gfx] [PATCH] drm/i915: Use vblank evade mechanism in mmio_flip

2014-10-24 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com Currently we program just DPSCNTR and DSPSTRIDE directly from the ring interrupt handler, which is fine since the hardware guarantees that those are update atomically. When we have atomic page flips we'll want to be able to

[Intel-gfx] [PATCH] drm/i915/bdw: Setup global hardware status page in execlists mode

2014-10-24 Thread Thomas Daniel
Write HWS_PGA address even in execlists mode as the global hardware status page is still required. This address was previously uninitialized and HWSP writes would clobber whatever buffer happened to reside at GGTT address 0. Issue: VIZ-2020 Signed-off-by: Thomas Daniel thomas.dan...@intel.com

Re: [Intel-gfx] [PATCH 02/10] drm/i915: Add counters in the drm_dp_aux struct for I2C NACKs and DEFERs

2014-10-24 Thread Paulo Zanoni
2014-10-09 12:38 GMT-03:00 Todd Previte tprev...@gmail.com: These counters are used for Displayort complinace testing to detect error conditions when executing certain compliance tests. Currently these are used in the EDID tests to determine if the video mode needs to be set to the

Re: [Intel-gfx] [PATCH v2 7/8] drm/i915: Create vgpu specific write MMIO to reduce traps

2014-10-24 Thread Daniel Vetter
On Wed, Oct 22, 2014 at 08:27:50PM +0800, Yu, Zhang wrote: On 10/22/2014 12:40 AM, Daniel Vetter wrote: On Thu, Oct 16, 2014 at 02:24:27PM +0800, Yu Zhang wrote: In the virtualized environment, forcewake operations are not necessory for the driver, because mmio accesses will be trapped

Re: [Intel-gfx] [PATCH 3/4] drm: add drm_mode_same_size function

2014-10-24 Thread Jesse Barnes
On Tue, 21 Oct 2014 16:49:37 +0200 Daniel Vetter dan...@ffwll.ch wrote: On Thu, Oct 09, 2014 at 12:57:44PM -0700, Jesse Barnes wrote: From: Kristian Høgsberg hoegsb...@gmail.com Like mode_equal but w/o the clock checks. Useful for checking if modes are close enough to re-use to avoid a

[Intel-gfx] [PATCH] lib: fix #define max

2014-10-24 Thread Mika Kuoppala
Regression from: commit be4710a541b517b5f8663448bffed5656d59b47b Author: Thomas Wood thomas.w...@intel.com Date: Fri Oct 10 11:20:35 2014 +0100 lib: add common min and max macros Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85218 Tested-by: Guo Jinxian jinxianx@intel.com

[Intel-gfx] [PATCH 1/3] igt/gem_evict_(alignment|everything): contend with GPU hangs

2014-10-24 Thread Chris Wilson
Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk --- lib/igt_aux.c| 69 lib/igt_aux.h| 4 +++ tests/gem_evict_alignment.c | 15 ++ tests/gem_evict_everything.c | 15 ++ 4 files changed, 103

Re: [Intel-gfx] Regression: audit: x86: drop arch from __audit_syscall_entry() interface

2014-10-24 Thread Thomas Gleixner
On Wed, 22 Oct 2014, Eric Paris wrote: That's really serious. Looking now. Indeed its serious. And it's even more serious as this masterpiece of assembly wreckage was pulled in via your tree w/o having an acked-by one of the x86 maintainers. On Wed, 2014-10-22 at 16:08 -0200, Paulo Zanoni

Re: [Intel-gfx] [PATCH 11/16] drm/i915: disable/re-enable PCI device around S4 freeze/thaw

2014-10-24 Thread Ville Syrjälä
On Wed, Sep 10, 2014 at 06:17:04PM +0300, Imre Deak wrote: We already disable everything during S4 freeze, except the PCI device itself. There is no reason why we couldn't disable that too and doing so allows us to unify these handlers in the next patch with the corresponding S3 suspend/resume

Re: [Intel-gfx] [PATCH 3/9] drm/i915: MIPI Port Ctrl related changes for dual link configuration

2014-10-24 Thread Jani Nikula
On Tue, 21 Oct 2014, Daniel Vetter dan...@ffwll.ch wrote: On Tue, Oct 21, 2014 at 12:00:30PM +0530, Singh, Gaurav K wrote: On 9/24/2014 2:57 PM, Jani Nikula wrote: On Wed, 24 Sep 2014, Gaurav K Singh gaurav.k.si...@intel.com wrote: Signed-off-by: Gaurav K Singh gaurav.k.si...@intel.com

[Intel-gfx] [PATCH 2/4] drm/i915 Add golden context support for Gen9

2014-10-24 Thread armin . c . reese
From: Armin Reese armin.c.re...@intel.com This patch includes the Gen9 batch buffer to generate a 'golden context' for that product family. Signed-off-by: Armin Reese armin.c.re...@intel.com --- drivers/gpu/drm/i915/Makefile | 3 +- drivers/gpu/drm/i915/i915_gem_render_state.c

Re: [Intel-gfx] [PATCH 05/10] drm/i915: Add debugfs interface for Displayport debug and compliance testing

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 10:58:59AM -0200, Paulo Zanoni wrote: 2014-10-23 10:50 GMT-02:00 Daniel Vetter dan...@ffwll.ch: Hi Tood, Paulo already mentioned it, so I'll just rehash quickly: I think the points from the internal pre-review first need to be address before we can dig into

Re: [Intel-gfx] [RFC PATCH 0/8] Add host i915 support for vGPU

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 01:01:28PM +0200, Gerd Hoffmann wrote: Hi, Stuf like driver load/unload, suspend/resume, runtime pm and gpu reset are already supre-fragile as-is. Every time we change something in there, a bunch of related things fall apart. With vgt we'll have even more

[Intel-gfx] [PATCH 1/2] drm/i915: Update plane parameters for cursor plane (v2)

2014-10-24 Thread Matt Roper
From: Sonika Jindal sonika.jin...@intel.com This allows the cursor plane to be updated the same way as primary and sprites, and same set_property handler is used for all of these planes. v2 (by Matt Roper): Rework to apply to latest di-nightly codebase. The switch to split check/commit

Re: [Intel-gfx] [RFC PATCH v2 2/4] drm/ipvr: drm driver for VED

2014-10-24 Thread Cheng, Yao
CC David for notifying the patch update: Add the missing v2 changelog: Take David's comment: add mmap support, remove the MMAP_IOCTL and add MMAP_OFFSET_IOCTL Take David's comment: remove postclose() and move code to preclose() Take David's comment: set NULL to set_busid

Re: [Intel-gfx] [PATCH] drm/i915: use macros to assign mmio access functions

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 03:28:24PM +0800, Yu Zhang wrote: Empty commit messages aren't good. Even for really simple refactoring please explain in 1-2 sentences the motivation for the patch, since the change itself doesn't really say that. For this one I've added something while merging.

Re: [Intel-gfx] [PATCH] drm/i915/bdw: Clean up execlist queue items in retire_work

2014-10-24 Thread Daniel Vetter
On Mon, Oct 20, 2014 at 01:29:59PM +, Daniel, Thomas wrote: -Original Message- From: Chris Wilson [mailto:ch...@chris-wilson.co.uk] Sent: Monday, October 20, 2014 2:11 PM To: Daniel, Thomas Cc: intel-gfx@lists.freedesktop.org; shuang...@linux.intel.com Subject: Re:

[Intel-gfx] [PATCH 2/5] drm/i915: factor compute_config out of __intel_mode_set

2014-10-24 Thread Jesse Barnes
This allows us to calculate the full pipe config before we do any mode setting work. Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org --- drivers/gpu/drm/i915/intel_display.c | 40 ++-- 1 file changed, 29 insertions(+), 11 deletions(-) diff --git

Re: [Intel-gfx] [PATCH] drm/i915: Ignore SURFLIVE and flip counter when the GPU gets reset

2014-10-24 Thread Daniel Vetter
On Tue, May 27, 2014 at 09:33:09PM +0300, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com During a GPU reset we need to get pending page flip cleared out since the ring contents are gone and flip will never complete on its own. This used to work until

[Intel-gfx] [PATCH v5] drm/i915: Add ppgtt create/release trace points

2014-10-24 Thread daniele . ceraolospurio
From: Daniele Ceraolo Spurio daniele.ceraolospu...@intel.com These tracepoints are useful for observing the creation and destruction of Full PPGTTs. v4: add DOC information v5: pull the DOC in drm.tmpl Signed-off-by: Daniele Ceraolo Spurio daniele.ceraolospu...@intel.com ---

[Intel-gfx] [PATCH 1/3] drm/i915: Add error_recovery option to i915_handle_error

2014-10-24 Thread Mika Kuoppala
So that we gain more fine grained control how we want to do the recovery. The aim is to grab the gpu state and skip everything related to recovery. Signed-off-by: Mika Kuoppala mika.kuopp...@intel.com --- drivers/gpu/drm/i915/i915_drv.h | 9 - drivers/gpu/drm/i915/i915_irq.c | 28

Re: [Intel-gfx] [PATCH] drm/i915: Add means to apply WA conditionally

2014-10-24 Thread Daniel Vetter
On Thu, Oct 23, 2014 at 6:07 PM, Siluvery, Arun arun.siluv...@linux.intel.com wrote: On 23/10/2014 16:51, Daniel Vetter wrote: On Thu, Oct 23, 2014 at 04:29:30PM +0100, Arun Siluvery wrote: We would want to apply some of the workarounds based on a condition to a particular platform or Gen

[Intel-gfx] [PATCH 2/3] drm/i915: Split the gpu (error) state generation

2014-10-24 Thread Mika Kuoppala
out from i915_capture_error_state so that it can be reused when we want to grab only the gpu state without any errors. Signed-off-by: Mika Kuoppala mika.kuopp...@intel.com --- drivers/gpu/drm/i915/i915_gpu_error.c | 41 +++ 1 file changed, 27 insertions(+), 14

Re: [Intel-gfx] [PATCH] drm/i915/bdw: Setup global hardware status page

2014-10-24 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) -Summary- Platform: baseline_drm_intel_nightly_pass_rate-patch_applied_pass_rate BYT: pass/total=271/271-270/271 PNV:

[Intel-gfx] [PATCH v2 6/8] drm/i915: Disable power management for i915 driver in VM

2014-10-24 Thread Yu Zhang
In XenGT, GPU power management is controlled by host i915 driver, so there is no need to provide virtualized GPU PM support. In the future it might be useful to gather VM input for freq boost, but now let's disable it simply. Signed-off-by: Yu Zhang yu.c.zh...@linux.intel.com Signed-off-by: Jike

Re: [Intel-gfx] [PATCH] tests/kms_cursor_crc: HSW/BDW only have square cursors

2014-10-24 Thread Paulo Zanoni
2014-10-14 16:32 GMT-03:00 Ville Syrjälä ville.syrj...@linux.intel.com: On Tue, Oct 14, 2014 at 02:05:42PM -0300, Paulo Zanoni wrote: From: Paulo Zanoni paulo.r.zan...@intel.com When I look at BSpec, and at cursor_size_ok() (from the Kernel's intel_display.c), I see that only 845g and i865g

Re: [Intel-gfx] [PATCH] drm/i915: Use vblank evade mechanism in

2014-10-24 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) -Summary- Platform: baseline_drm_intel_nightly_pass_rate-patch_applied_pass_rate BYT: pass/total=270/271-270/271 PNV:

[Intel-gfx] [PATCH 4/4] drm/i915/skl: Implement do_mmio_flip for SKL

2014-10-24 Thread Damien Lespiau
Nothing special to note. We mirror what is done for other platforms, but using the SKL plane registers. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_display.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git

[Intel-gfx] [PATCH 3/4] drm/i915: Add space between variable declarations and code

2014-10-24 Thread Damien Lespiau
OCD kicks in again... Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/intel_display.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 07440ad..dd071c6 100644 ---

[Intel-gfx] [PATCH 2/4] drm/i915: Introduce a vfunc for platform-specfic MMIO flip code

2014-10-24 Thread Damien Lespiau
SKL will specialize it. Signed-off-by: Damien Lespiau damien.lesp...@intel.com --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_display.c | 17 + 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h