Re: [Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-16 Thread Daniel Vetter
On Tue, Dec 16, 2014 at 09:55:23AM +, Chris Wilson wrote: > On Tue, Dec 16, 2014 at 10:19:47AM +0100, Daniel Vetter wrote: > > Hm right, there have been patches floating around for that one ... A short > > mention in the commit message about this would be useful. > > If you notice, it was hint

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-16 Thread Chris Wilson
On Tue, Dec 16, 2014 at 10:19:47AM +0100, Daniel Vetter wrote: > On Mon, Dec 15, 2014 at 07:03:43PM +0200, Ville Syrjälä wrote: > > On Mon, Dec 15, 2014 at 04:24:48PM +, Chris Wilson wrote: > > > On Mon, Dec 15, 2014 at 10:41:52AM +0100, Daniel Vetter wrote: > > > > On Thu, Dec 11, 2014 at 08:1

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-16 Thread Daniel Vetter
On Mon, Dec 15, 2014 at 07:03:43PM +0200, Ville Syrjälä wrote: > On Mon, Dec 15, 2014 at 04:24:48PM +, Chris Wilson wrote: > > On Mon, Dec 15, 2014 at 10:41:52AM +0100, Daniel Vetter wrote: > > > On Thu, Dec 11, 2014 at 08:17:01AM +, Chris Wilson wrote: > > > > There exists a current workar

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-15 Thread Ville Syrjälä
On Mon, Dec 15, 2014 at 04:24:48PM +, Chris Wilson wrote: > On Mon, Dec 15, 2014 at 10:41:52AM +0100, Daniel Vetter wrote: > > On Thu, Dec 11, 2014 at 08:17:01AM +, Chris Wilson wrote: > > > There exists a current workaround to prevent a hang on context switch > > > should the ring go to sl

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-15 Thread Chris Wilson
On Mon, Dec 15, 2014 at 10:41:52AM +0100, Daniel Vetter wrote: > On Thu, Dec 11, 2014 at 08:17:01AM +, Chris Wilson wrote: > > There exists a current workaround to prevent a hang on context switch > > should the ring go to sleep in the middle of the restore, > > WaProgramMiArbOnOffAroundMiSetCo

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-15 Thread Daniel Vetter
On Thu, Dec 11, 2014 at 08:17:01AM +, Chris Wilson wrote: > There exists a current workaround to prevent a hang on context switch > should the ring go to sleep in the middle of the restore, > WaProgramMiArbOnOffAroundMiSetContext (applicable to all gen7+). In > spite of disabling arbitration (w

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-11 Thread shuang . he
Tested-By: PRC QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) -Summary- Platform Delta drm-intel-nightly Series Applied PNV 364/364

[Intel-gfx] [PATCH 3/3] drm/i915: Disable PMSI sleep messages on all rings around context switches

2014-12-11 Thread Chris Wilson
There exists a current workaround to prevent a hang on context switch should the ring go to sleep in the middle of the restore, WaProgramMiArbOnOffAroundMiSetContext (applicable to all gen7+). In spite of disabling arbitration (which prevents the ring from powering down during the critical section)