Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-13 Thread Rodrigo Vivi
Rodrigo Vivi writes: > On Sat, Feb 10, 2018 at 09:43:59AM +, Chris Wilson wrote: >> Quoting Andy Lutomirski (2018-02-09 17:55:38) >> > On Fri, Feb 9, 2018 at 7:39 AM, Rodrigo Vivi >> > wrote: >> > > Rodrigo Vivi writes: >> > > So, I move the hacked scheduled to 10s and forced psr_activate

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-12 Thread Rodrigo Vivi
On Sat, Feb 10, 2018 at 09:43:59AM +, Chris Wilson wrote: > Quoting Andy Lutomirski (2018-02-09 17:55:38) > > On Fri, Feb 9, 2018 at 7:39 AM, Rodrigo Vivi wrote: > > > Rodrigo Vivi writes: > > > So, I move the hacked scheduled to 10s and forced psr_activate 10ms > > > after that and the resul

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-10 Thread Chris Wilson
Quoting Andy Lutomirski (2018-02-09 17:55:38) > On Fri, Feb 9, 2018 at 7:39 AM, Rodrigo Vivi wrote: > > Rodrigo Vivi writes: > > So, I move the hacked scheduled to 10s and forced psr_activate 10ms > > after that and the result is this: > > > > > > [ 11.757909] [drm:intel_psr_enable [i915]] *ERR

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-09 Thread Pandiyan, Dhinakaran
On Thu, 2018-02-08 at 23:39 -0800, Rodrigo Vivi wrote: > Rodrigo Vivi writes: > > > "Pandiyan, Dhinakaran" writes: > > > >> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote: > >>> Hi Andy, > >>> > >>> thanks for getting involved with PSR and sorry for not replying sooner. > >>> > >>> I fir

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-09 Thread Andy Lutomirski
On Fri, Feb 9, 2018 at 7:39 AM, Rodrigo Vivi wrote: > Rodrigo Vivi writes: > >> "Pandiyan, Dhinakaran" writes: >> >>> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote: Hi Andy, thanks for getting involved with PSR and sorry for not replying sooner. I first saw this

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-08 Thread Rodrigo Vivi
Rodrigo Vivi writes: > "Pandiyan, Dhinakaran" writes: > >> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote: >>> Hi Andy, >>> >>> thanks for getting involved with PSR and sorry for not replying sooner. >>> >>> I first saw this patch on that bugzilla entry but only now I stop to >>> really t

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-08 Thread Rodrigo Vivi
"Pandiyan, Dhinakaran" writes: > On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote: >> Hi Andy, >> >> thanks for getting involved with PSR and sorry for not replying sooner. >> >> I first saw this patch on that bugzilla entry but only now I stop to >> really think why I have written the cod

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-08 Thread Pandiyan, Dhinakaran
On Thu, 2018-02-08 at 17:01 -0800, Andy Lutomirski wrote: > > > > On Feb 8, 2018, at 4:39 PM, Pandiyan, Dhinakaran > > wrote: > > > > > >> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote: > >> Hi Andy, > >> > >> thanks for getting involved with PSR and sorry for not replying sooner.

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-08 Thread Andy Lutomirski
> On Feb 8, 2018, at 4:39 PM, Pandiyan, Dhinakaran > wrote: > > >> On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote: >> Hi Andy, >> >> thanks for getting involved with PSR and sorry for not replying sooner. >> >> I first saw this patch on that bugzilla entry but only now I stop to >>

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-08 Thread Pandiyan, Dhinakaran
On Thu, 2018-02-08 at 14:48 -0800, Rodrigo Vivi wrote: > Hi Andy, > > thanks for getting involved with PSR and sorry for not replying sooner. > > I first saw this patch on that bugzilla entry but only now I stop to > really think why I have written the code that way. > > So some clarity below.

Re: [Intel-gfx] [PATCH] drm/i915: Improve PSR activation timing

2018-02-08 Thread Rodrigo Vivi
Hi Andy, thanks for getting involved with PSR and sorry for not replying sooner. I first saw this patch on that bugzilla entry but only now I stop to really think why I have written the code that way. So some clarity below. On Mon, Feb 05, 2018 at 10:07:09PM +, Andy Lutomirski wrote: > The