Re: [Intel-gfx] Fast User Switching does not seem to work

2014-10-08 Thread Felix Miata
constantine composed on 2014-10-09 00:38 (UTC): > I am encountering a peculiar problem with Fast User Switching or Kernel > Mode Setting and I do not know how to troubleshoot it. > When I run two (or more) Virtual Terminals switching between them > (Ctrl+Alt+F2-3, etc.) takes 2+ seconds. (That is

[Intel-gfx] Fast User Switching does not seem to work

2014-10-08 Thread constantine
Hello all! I am encountering a peculiar problem with Fast User Switching or Kernel Mode Setting and I do not know how to troubleshoot it. When I run two (or more) Virtual Terminals switching between them (Ctrl+Alt+F2-3, etc.) takes 2+ seconds. (That is either simply initiated with startx from tty

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

2014-10-08 Thread Paulo Zanoni
2014-10-08 5:25 GMT-03:00 Jani Nikula : > On Wed, 08 Oct 2014, Paulo Zanoni wrote: >> From: Paulo Zanoni >> >> 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 cou

[Intel-gfx] [PATCH 1/2] Use = rather than == in test.

2014-10-08 Thread Adam Sampson
The POSIX standard only requires test to support "="; "==" works in bash but not in dash. Other comparisons in configure.ac use "=" already. Signed-off-by: Adam Sampson --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) This is the same patch that I submitted to Bugzilla:

[Intel-gfx] [PATCH 2/2] Don't use += to append to a shell variable.

2014-10-08 Thread Adam Sampson
POSIX only requires "=" to be supported; "+=" works in bash but not in dash. Signed-off-by: Adam Sampson --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 76487c0..9cdd202 100644 --- a/configure.ac +++ b/configure.ac @@ -203,7

[Intel-gfx] [PATCH 3/4] drm/i915: Convert shared dpll reference count to a crtc mask

2014-10-08 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira This will be used in a follow up patch to properly release shared DPLLs without relying on the shared_dpll field in pipe_config. Signed-off-by: Ander Conselvan de Oliveira --- drivers/gpu/drm/i915/i915_debugfs.c | 4 +-- drivers/gpu/drm/i915/i915_drv.h

[Intel-gfx] [PATCH 1/4] drm/i915: Replace some loop through encoders with intel_pipe_has_type()

2014-10-08 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira In the ironlake mode set code, there was two instances of a loop through encoders to find out if one of them has INTEL_OUTPUT_LVDS type. Simplify the code by deleting some lines and use intel_pipe_has_type() instead. Signed-off-by: Ander Conselvan de Oliveira

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

2014-10-08 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira This shouldn't change the behavior of those functions, since they are called after the new_config is made effective and that points to the current config. In a follow up patch, the mode set sequence will be changed so this is called before disabling crtcs, and in

[Intel-gfx] [PATCH 4/4] drm/i915: Compute clocks and choose DPLLs before disabling crtcs

2014-10-08 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira It is possible for a mode set to fail if there aren't shared DPLLS that match the new configuration requirement or other errors in clock computation. Since that step was executed after disabling crtcs, in the failure case the hardware configuration is changed and

[Intel-gfx] [PATCH 0/4] [RFC] Stage shared dpll configs

2014-10-08 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira Hi, This series changes the mode set sequence so that the clock and PLL logic that was done in the *_crtc_mode_set() hooks is done before disabling crtcs. This avoids having to restore the old configuration in the case of failure, since the hardware was never to

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

2014-10-08 Thread Jesse Barnes
On Wed, 8 Oct 2014 07:43:34 +0100 Chris Wilson wrote: > On Tue, Oct 07, 2014 at 01:25:23PM -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) > > > > Signed-off-b

Re: [Intel-gfx] [RFC 13/38] drm/i915: Make gen6_write_pdes gen6_map_page_tables

2014-10-08 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 06:11:09PM +0100, Michel Thierry wrote: > From: Ben Widawsky > > Split out single mappings which will help with upcoming work. Also while > here, rename the function because it is a better description - but this > function is going away soon. > > Signed-off-by: Ben Widawsky

Re: [Intel-gfx] [RFC 09/38] drm/i915: s/pd/pdpe, s/pt/pde

2014-10-08 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 06:11:05PM +0100, Michel Thierry wrote: > From: Ben Widawsky > > The actual correct way to think about this with the new style of page > table data structures is as the actual entry that is being indexed into > the array. "pd", and "pt" aren't representative of what the ope

Re: [Intel-gfx] [RFC 06/38] drm/i915: fix gtt_total_entries()

2014-10-08 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 06:11:02PM +0100, Michel Thierry wrote: > From: Ben Widawsky > > It's useful to have it not as a macro for some upcoming work. Generally > since we try to avoid macros anyway, I think it doesn't hurt to put this > as its own patch. > > Signed-off-by: Ben Widawsky > Signed-

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

2014-10-08 Thread Jani Nikula
On Wed, 08 Oct 2014, Paulo Zanoni wrote: > From: Paulo Zanoni > > 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 this code, but this > should be the

Re: [Intel-gfx] [RFC 05/38] drm/i915: Split out aliasing binds

2014-10-08 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 06:11:01PM +0100, Michel Thierry wrote: > From: Ben Widawsky > > This patch finishes off actually separating the aliasing and global > finds. Prior to this, all global binds would be aliased. Now if aliasing > binds are required, they must be explicitly asked for. So far,

Re: [Intel-gfx] [RFC 04/38] drm/i915: Make pin global flags explicit

2014-10-08 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 06:11:00PM +0100, Michel Thierry wrote: > From: Ben Widawsky > > The driver currently lets callers pin global, and then tries to do > things correctly inside the function. Doing so has two downsides: > 1. It's not possible to exclusively pin to a global, or an aliasing > ad

Re: [Intel-gfx] [RFC 01/38] drm/i915: Add some extra guards in evict_vm

2014-10-08 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 06:10:57PM +0100, Michel Thierry wrote: > From: Ben Widawsky > > Signed-off-by: Ben Widawsky > Signed-off-by: Michel Thierry > --- > drivers/gpu/drm/i915/i915_gem_evict.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/i915/i915_gem_evict.c >

Re: [Intel-gfx] [PULL] topic/vblank-rework, take 2

2014-10-08 Thread Thierry Reding
On Fri, Sep 12, 2014 at 09:48:59AM +0200, Daniel Vetter wrote: [...] > Ville Syrjälä (16): > drm: Always reject drm_vblank_get() after drm_vblank_off() This particular commit seems to be wreaking havoc. Inki already reported this at some point and Andrzej and Joonyoung tried to fix this on E

Re: [Intel-gfx] [PATCH] drm: Implement O_NONBLOCK support on /dev/dri/cardN

2014-10-08 Thread Daniel Vetter
On Wed, Oct 08, 2014 at 10:45:48AM +0100, Chris Wilson wrote: > On Wed, Oct 08, 2014 at 11:16:38AM +0200, Daniel Vetter wrote: > > On Tue, Oct 7, 2014 at 3:13 PM, Chris Wilson > > wrote: > > > The implmentation is simple in the extreme: we only want to wait for > > > events if the device was open

[Intel-gfx] [PATCH 1/2] drm/i915: Report the current number of bytes freed during oom

2014-10-08 Thread Chris Wilson
The shrinker reports the number of pages freed, but we try to log the number of bytes - which leads to some nonsense values being reportedly freed during oom. Reported-by: Maarten Lankhorst Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/i915_gem.c | 8 1 file changed, 4 insertion

[Intel-gfx] [PATCH 2/2] drm/i915: Suppress no action noise from oom shrinker

2014-10-08 Thread Chris Wilson
If we are not able to free anything (the shrinker leaves nothing on the global object lists), do not log anything. This is useful when other subsystems are being stress-tested for their oom behaviour and i915.ko is shouting into the logs about doing nothing. Reported-by: Dave Jones Signed-off-by:

Re: [Intel-gfx] [PATCH] drm: Implement O_NONBLOCK support on /dev/dri/cardN

2014-10-08 Thread Chris Wilson
On Wed, Oct 08, 2014 at 11:16:38AM +0200, Daniel Vetter wrote: > On Tue, Oct 7, 2014 at 3:13 PM, Chris Wilson wrote: > > The implmentation is simple in the extreme: we only want to wait for > > events if the device was opened in blocking mode, otherwise we grab what > > is available and report an

Re: [Intel-gfx] [PATCH] drm/i915/hdmi: Compute port_clock for 27.027 pixel replicated modes

2014-10-08 Thread Daniel Vetter
On Tue, Oct 07, 2014 at 11:29:30AM +0300, Ville Syrjälä wrote: > We do it automagically but we make the decision based on the primary > plane pixel format which is a bit silly considering there are sprites > and whatnot to consider. Maybe we should just always go for deep color > whenever possible?

[Intel-gfx] [PATCH i-g-t] quick_dump: Drop common_display.txt from VLV/CHV

2014-10-08 Thread Damien Lespiau
Those registers are big core registers and weren't really relevant for VLV/CHV. Drop them from the dump then. Suggested-by: Ville Syrjälä Signed-off-by: Damien Lespiau --- tools/quick_dump/cherryview | 1 - tools/quick_dump/valleyview | 1 - 2 files changed, 2 deletions(-) diff --git a/tools/q

Re: [Intel-gfx] [PATCH i-g-t 1/3] quick_dump: Move base_display.txt to indivual platforms

2014-10-08 Thread Damien Lespiau
On Tue, Oct 07, 2014 at 01:26:37PM +0300, Ville Syrjälä wrote: > On Mon, Oct 06, 2014 at 06:56:43PM +0100, Damien Lespiau wrote: > > SKL will have a whole separate display regs file, so merge > > base_display.txt into each platform file. > > Please drop it from vlv/chv. It's not appropriate for th

Re: [Intel-gfx] [PATCH] drm: Implement O_NONBLOCK support on /dev/dri/cardN

2014-10-08 Thread Daniel Vetter
On Tue, Oct 7, 2014 at 3:13 PM, Chris Wilson wrote: > The implmentation is simple in the extreme: we only want to wait for > events if the device was opened in blocking mode, otherwise we grab what > is available and report an error if there was none. > > Signed-off-by: Chris Wilson > Cc: dri-de.

Re: [Intel-gfx] [RFC 00/38] PPGTT dynamic page allocations

2014-10-08 Thread Chris Wilson
On Tue, Oct 07, 2014 at 06:10:56PM +0100, Michel Thierry wrote: > This is based on the first 55 patches of Ben's 48b addressing work, taking > into consideration the latest changes in (mainly aliasing) ppgtt rules. > > Because of these changes in the tree, the first 17 patches of the original > se