[Intel-gfx] [PATCH 3/8] drm/i915/kbl: KBL also needs to run the SAGV code

2016-09-06 Thread Paulo Zanoni
According to BSpec, it's the "core CPUs" that need the code, which means SKL and KBL, but not BXT. I don't have a KBL to test this patch on it. Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/intel_pm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i91

Re: [Intel-gfx] [PATCH 3/8] drm/i915/kbl: KBL also needs to run the SAGV code

2016-09-07 Thread Ville Syrjälä
On Tue, Sep 06, 2016 at 09:52:14PM -0300, Paulo Zanoni wrote: > According to BSpec, it's the "core CPUs" that need the code, which > means SKL and KBL, but not BXT. > > I don't have a KBL to test this patch on it. IIRC bspec doesn't specify the sagv latency for anything but SKL, and the relevant

Re: [Intel-gfx] [PATCH 3/8] drm/i915/kbl: KBL also needs to run the SAGV code

2016-09-07 Thread Lyude
I'm not sure that kbl has this either. The kbl machine I've been working with thus-far has passed a few modesetting stress tests with the chameleon, and I don't have anything trying to control sagv stuff on it. This being said though the sagv for skylake did happen to get added right before releas

Re: [Intel-gfx] [PATCH 3/8] drm/i915/kbl: KBL also needs to run the SAGV code

2016-09-13 Thread Zanoni, Paulo R
Em Qua, 2016-09-07 às 12:17 -0400, Lyude escreveu: > I'm not sure that kbl has this either. The kbl machine I've been > working with thus-far has passed a few modesetting stress tests with > the chameleon, and I don't have anything trying to control sagv stuff > on it. > > This being said though t

Re: [Intel-gfx] [PATCH 3/8] drm/i915/kbl: KBL also needs to run the SAGV code

2016-09-14 Thread Jani Nikula
On Tue, 13 Sep 2016, "Zanoni, Paulo R" wrote: > I got confirmation from the Hardware guys that KBL does need to run the > SAGV code, and it has the same latency as SKL. Also, all SKL production > steppings need to run the SAGV code. Can you get confirmation what's the first shipped production ste

Re: [Intel-gfx] [PATCH 3/8] drm/i915/kbl: KBL also needs to run the SAGV code

2016-09-14 Thread Zanoni, Paulo R
Em Qua, 2016-09-14 às 12:59 +0300, Jani Nikula escreveu: > On Tue, 13 Sep 2016, "Zanoni, Paulo R" > wrote: > > > > I got confirmation from the Hardware guys that KBL does need to run > > the > > SAGV code, and it has the same latency as SKL. Also, all SKL > > production > > steppings need to run

Re: [Intel-gfx] [PATCH 3/8] drm/i915/kbl: KBL also needs to run the SAGV code

2016-09-15 Thread Jani Nikula
On Wed, 14 Sep 2016, "Zanoni, Paulo R" wrote: > Em Qua, 2016-09-14 às 12:59 +0300, Jani Nikula escreveu: >> On Tue, 13 Sep 2016, "Zanoni, Paulo R" >> wrote: >> > >> > I got confirmation from the Hardware guys that KBL does need to run >> > the >> > SAGV code, and it has the same latency as SKL.