This is possible with the new force paramter in do_switch. As stated in
that patch, the goal is to get a real context stored at the time of
initialization.
References: https://bugs.freedesktop.org/show_bug.cgi?id=52429
Tested-by: Guang A Yang guang.a.y...@intel.com
Signed-off-by: Ben Widawsky
A force parameter for switch currently only has one use, the first time
we load the default context.
Slightly hand-wavy explanation: We want to get the default context
actually loaded so that the GPU has some real state to load (instead of
garbage) after a reset or resume
Therefore, the benefit
The previous state is bogus when we've gone into idle. Actually there
would be a known state of idle (ie. all units are done), but since it
differs for every platform, we can just set 0, and let the hangcheck
progress as normal.
References: https://bugs.freedesktop.org/show_bug.cgi?id=52429
On Mon, 13 Aug 2012, Mihai Moldovan io...@ionic.de wrote:
Had another look at the code and would like to apologize for the confusion...
No worries Mihai, thanks for testing!
BR,
Jani.
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
On Mon, 13 Aug 2012 22:41:08 -0700, Ben Widawsky b...@bwidawsk.net wrote:
After reset we unconditionally reinitialize lists. If the context switch
hasn't yet completed before the suspend the default context object will
end up on lists that are going to go away when we resume.
The patch
On Mon, Aug 13, 2012 at 10:35:26PM -0700, Keith Packard wrote:
Daniel Vetter daniel.vet...@ffwll.ch writes:
Cc: Keith Packard kei...@keithp.com
I tried this on top of v3.5, appears to work just fine. Thanks!
Tested-by: Keith Packard kei...@keithp.com
Merged to -fixes, thanks for
On Mon, 13 Aug 2012 22:41:11 -0700, Ben Widawsky b...@bwidawsk.net wrote:
The previous state is bogus when we've gone into idle. Actually there
would be a known state of idle (ie. all units are done), but since it
differs for every platform, we can just set 0, and let the hangcheck
progress as
On Mon, 13 Aug 2012 22:41:10 -0700, Ben Widawsky b...@bwidawsk.net wrote:
This is possible with the new force paramter in do_switch. As stated in
that patch, the goal is to get a real context stored at the time of
initialization.
References: https://bugs.freedesktop.org/show_bug.cgi?id=52429
jk Sat, Aug 11, 2012 at 12:20:19PM -0700, Ben Widawsky wrote:
On Fri, 10 Aug 2012 10:18:10 +0100
Chris Wilson ch...@chris-wilson.co.uk wrote:
When invalidating the TLBs it is documentated as requiring a post-sync
write. Failure to do so seems to result in a GPU hang.
Exposure to this
Meh, I've again forgotten about all the cc's ...
/me needs coffee
On Tue, Aug 14, 2012 at 10:28 AM, Daniel Vetter dan...@ffwll.ch wrote:
Hi Dave,
A few important fixers:
- fix various lvds backlight issues, regressed in 3.6 (Takashi Iwai)
- make the retina mbp work (ignore bogus edp bpc
On Thu, Aug 09, 2012 at 04:52:18PM +0800, Wang Xingchao wrote:
Use _PIPE macro to get correct register definition for IBX/CPT, discard
old variable i way.
Signed-off-by: Wang Xingchao xingchao.w...@intel.com
Ok, I've slurped in patches 1,24 for -next (with a bit of frobbing since
one of the
On Fri, Aug 10, 2012 at 5:52 PM, Wang, Xingchao xingchao.w...@intel.com wrote:
HI Deak,
-Original Message-
From: Imre Deak [mailto:imre.d...@gmail.com]
Sent: Friday, August 10, 2012 9:15 PM
To: Wang, Xingchao
Cc: dan...@ffwll.ch; przan...@gmail.com; intel-gfx@lists.freedesktop.org
From: Daniel Vetter daniel.vet...@ffwll.ch
They've changed it ... for no apparent reason. Meh.
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
---
drivers/char/agp/intel-agp.h| 1 +
drivers/char/agp/intel-gtt.c| 107
On Tue, 14 Aug 2012, Paulo Zanoni przan...@gmail.com wrote:
From: Daniel Vetter daniel.vet...@ffwll.ch
They've changed it ... for no apparent reason. Meh.
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
---
drivers/char/agp/intel-agp.h| 1 +
drivers/char/agp/intel-gtt.c
From: Daniel Vetter daniel.vet...@ffwll.ch
They've changed it ... for no apparent reason. Meh.
V2: remove unused 'is_hsw' field.
Signed-off-by: Daniel Vetter daniel.vet...@ffwll.ch
Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com
---
drivers/char/agp/intel-agp.h|1 +
Paulo Zanoni przan...@gmail.com writes:
+#define HSW_PTE_UNCACHED 0x
Are you sure this value should be zero? It seems pretty unlikely to me.
--
keith.pack...@intel.com
pgptzbyqd415W.pgp
Description: PGP signature
___
Intel-gfx mailing
On 2012-08-14 00:41, Chris Wilson wrote:
On Mon, 13 Aug 2012 22:41:10 -0700, Ben Widawsky b...@bwidawsk.net
wrote:
This is possible with the new force paramter in do_switch. As stated
in
that patch, the goal is to get a real context stored at the time of
initialization.
References:
On 2012-08-14 00:39, Chris Wilson wrote:
On Mon, 13 Aug 2012 22:41:11 -0700, Ben Widawsky b...@bwidawsk.net
wrote:
The previous state is bogus when we've gone into idle. Actually
there
would be a known state of idle (ie. all units are done), but since
it
differs for every platform, we can
On Tue, 14 Aug 2012 09:42:07 -0700, Ben Widawsky b...@bwidawsk.net wrote:
On 2012-08-14 00:39, Chris Wilson wrote:
On Mon, 13 Aug 2012 22:41:11 -0700, Ben Widawsky b...@bwidawsk.net
wrote:
The previous state is bogus when we've gone into idle. Actually
there
would be a known state of
Daniel, the first 3 patches I believe are all that's required to fix
https://bugs.freedesktop.org/show_bug.cgi?id=52429 The latter 2 are
things we've discussed sort of. I've managed to go back and forth
several times on whether or not we actually need them, so I'll just let
you decide if you want
Clear the cached instdone state to match what we expect from hardware
and prevent us from comparing stale values.
Actually, clearing the state is not the same as setting idle state.
There would be a known state of idle (ie. all units are done), but since
it differs for every platform, we can just
After reset we unconditionally reinitialize lists. If the context switch
hasn't yet completed before the suspend, the default context object will
end up on lists that are going to go away when we resume.
The patch forces the context switch to be synchronous before suspend
assuring that the
In order to make the HW state CCID match with what we think it should
be, we must order a switch to the default context.
The really sad thing here is that the switch can potentially fail, and
as such we have to assume contexts no longer work. There is likely room
for improvement but until we
A force parameter for switch currently only has one use, the first time
we load the default context.
Slightly hand-wavy explanation: We want to get the default context
actually loaded so that the GPU has some real state to load (instead of
garbage) after a reset or resume. The reason it's
This is possible with the new force parameter in do_switch. As stated in
that patch, the goal is to get a real context stored at the time of
initialization.
References: https://bugs.freedesktop.org/show_bug.cgi?id=52429
Tested-by: Guang A Yang guang.a.y...@intel.com
Signed-off-by: Ben Widawsky
-Original Message-
From: Imre Deak [mailto:imre.d...@gmail.com]
Sent: Tuesday, August 14, 2012 8:36 PM
To: Wang, Xingchao
Cc: dan...@ffwll.ch; przan...@gmail.com; intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH v6 3/4] drm/i915: Haswell HDMI audio
initialization
Hi Daniel/Imre,
This revised version changelog:
- add Wait for 1 vertical blank after enable audio output port
- configure pipe related transcoder instead of operate all transcoders blindly
Thanks
--xingchao
-Original Message-
From: Wang, Xingchao
Sent: Wednesday, August 15, 2012
-Original Message-
From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel Vetter
Sent: Tuesday, August 14, 2012 7:49 PM
To: Wang, Xingchao
Cc: dan...@ffwll.ch; przan...@gmail.com; intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH v6 4/4] drm/i915:
28 matches
Mail list logo