All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
@ 2016-10-26 18:22 Rodrigo Vivi
  2016-10-26 19:46 ` ✗ Fi.CI.BAT: warning for " Patchwork
                   ` (4 more replies)
  0 siblings, 5 replies; 9+ messages in thread
From: Rodrigo Vivi @ 2016-10-26 18:22 UTC (permalink / raw)
  To: intel-gfx; +Cc: Marc Herbert, Paulo Zanoni, Rodrigo Vivi

Since Broxton has same FBC block as BDW+ let's assume it also
don't have access to the stolen usable range.

FBC is currently not saving power on Broxton and I believe
the compression threshold is limited to 1x.

Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Cc: Marc Herbert <marc.herbert@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
index cbe2ebd..640db67 100644
--- a/drivers/gpu/drm/i915/intel_fbc.c
+++ b/drivers/gpu/drm/i915/intel_fbc.c
@@ -530,12 +530,11 @@ static int find_compression_threshold(struct drm_i915_private *dev_priv,
 	int ret;
 	u64 end;
 
-	/* The FBC hardware for BDW/SKL doesn't have access to the stolen
+	/* The FBC hardware for gen8+ doesn't have access to the stolen
 	 * reserved range size, so it always assumes the maximum (8mb) is used.
 	 * If we enable FBC using a CFB on that memory range we'll get FIFO
 	 * underruns, even if that range is not reserved by the BIOS. */
-	if (IS_BROADWELL(dev_priv) ||
-	    IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv))
+	if (INTEL_INFO(dev_priv)->gen <= 8)
 		end = ggtt->stolen_size - 8 * 1024 * 1024;
 	else
 		end = ggtt->stolen_usable_size;
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* ✗ Fi.CI.BAT: warning for drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
  2016-10-26 18:22 [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Rodrigo Vivi
@ 2016-10-26 19:46 ` Patchwork
  2016-10-27  5:47   ` Saarinen, Jani
  2016-10-26 20:28 ` [PATCH] " Rodrigo Vivi
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 9+ messages in thread
From: Patchwork @ 2016-10-26 19:46 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
URL   : https://patchwork.freedesktop.org/series/14426/
State : warning

== Summary ==

Series 14426v1 drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
https://patchwork.freedesktop.org/api/1.0/series/14426/revisions/1/mbox/

Test drv_module_reload_basic:
                dmesg-warn -> PASS       (fi-ilk-650)
Test kms_pipe_crc_basic:
        Subgroup suspend-read-crc-pipe-a:
                pass       -> DMESG-WARN (fi-skl-6770hq)
                dmesg-warn -> PASS       (fi-ilk-650)

fi-bdw-5557u     total:246  pass:231  dwarn:0   dfail:0   fail:0   skip:15 
fi-bsw-n3050     total:246  pass:204  dwarn:0   dfail:0   fail:0   skip:42 
fi-bxt-t5700     total:246  pass:216  dwarn:0   dfail:0   fail:0   skip:30 
fi-byt-j1900     total:246  pass:215  dwarn:0   dfail:0   fail:0   skip:31 
fi-byt-n2820     total:246  pass:211  dwarn:0   dfail:0   fail:0   skip:35 
fi-hsw-4770      total:246  pass:224  dwarn:0   dfail:0   fail:0   skip:22 
fi-hsw-4770r     total:246  pass:223  dwarn:0   dfail:0   fail:0   skip:23 
fi-ilk-650       total:246  pass:185  dwarn:0   dfail:0   fail:0   skip:61 
fi-ivb-3520m     total:246  pass:220  dwarn:0   dfail:0   fail:0   skip:26 
fi-ivb-3770      total:246  pass:220  dwarn:0   dfail:0   fail:0   skip:26 
fi-kbl-7200u     total:246  pass:222  dwarn:0   dfail:0   fail:0   skip:24 
fi-skl-6260u     total:246  pass:232  dwarn:0   dfail:0   fail:0   skip:14 
fi-skl-6700hq    total:246  pass:223  dwarn:0   dfail:0   fail:0   skip:23 
fi-skl-6700k     total:246  pass:222  dwarn:1   dfail:0   fail:0   skip:23 
fi-skl-6770hq    total:246  pass:231  dwarn:1   dfail:0   fail:0   skip:14 
fi-snb-2520m     total:246  pass:209  dwarn:0   dfail:0   fail:0   skip:37 
fi-snb-2600      total:246  pass:208  dwarn:0   dfail:0   fail:0   skip:38 

0eefa35311a3d85e6cbfda4bb326809581acc3b1 drm-intel-nightly: 2016y-10m-26d-17h-38m-29s UTC integration manifest
2412c2c drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+

Full results at https://intel-gfx-ci.01.org/CI/Patchwork_2830/

== Logs ==

For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_2830/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
  2016-10-26 18:22 [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Rodrigo Vivi
  2016-10-26 19:46 ` ✗ Fi.CI.BAT: warning for " Patchwork
@ 2016-10-26 20:28 ` Rodrigo Vivi
  2016-10-26 20:45 ` David Weinehall
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 9+ messages in thread
From: Rodrigo Vivi @ 2016-10-26 20:28 UTC (permalink / raw)
  To: intel-gfx; +Cc: Marc Herbert, Paulo Zanoni, Rodrigo Vivi

Since Broxton has same FBC block as BDW+ let's assume it also
don't have access to the stolen usable range.

FBC is currently not saving power on Broxton and I believe
the compression threshold is limited to 1x.

v2: duh! inverted check. (Thanks Marc)

Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Cc: Marc Herbert <marc.herbert@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
index cbe2ebd..214b2de 100644
--- a/drivers/gpu/drm/i915/intel_fbc.c
+++ b/drivers/gpu/drm/i915/intel_fbc.c
@@ -530,12 +530,11 @@ static int find_compression_threshold(struct drm_i915_private *dev_priv,
 	int ret;
 	u64 end;
 
-	/* The FBC hardware for BDW/SKL doesn't have access to the stolen
+	/* The FBC hardware for gen8+ doesn't have access to the stolen
 	 * reserved range size, so it always assumes the maximum (8mb) is used.
 	 * If we enable FBC using a CFB on that memory range we'll get FIFO
 	 * underruns, even if that range is not reserved by the BIOS. */
-	if (IS_BROADWELL(dev_priv) ||
-	    IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv))
+	if (INTEL_INFO(dev_priv)->gen >= 8)
 		end = ggtt->stolen_size - 8 * 1024 * 1024;
 	else
 		end = ggtt->stolen_usable_size;
-- 
1.9.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
  2016-10-26 18:22 [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Rodrigo Vivi
  2016-10-26 19:46 ` ✗ Fi.CI.BAT: warning for " Patchwork
  2016-10-26 20:28 ` [PATCH] " Rodrigo Vivi
@ 2016-10-26 20:45 ` David Weinehall
  2016-10-26 21:18   ` Vivi, Rodrigo
  2016-10-26 21:16 ` ✗ Fi.CI.BAT: warning for drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ (rev2) Patchwork
  2016-10-26 22:50 ` [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Paulo Zanoni
  4 siblings, 1 reply; 9+ messages in thread
From: David Weinehall @ 2016-10-26 20:45 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-gfx, Marc Herbert, Paulo Zanoni

On Wed, Oct 26, 2016 at 11:22:00AM -0700, Rodrigo Vivi wrote:
> Since Broxton has same FBC block as BDW+ let's assume it also
> don't have access to the stolen usable range.
> 
> FBC is currently not saving power on Broxton and I believe
> the compression threshold is limited to 1x.
> 
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Cc: Marc Herbert <marc.herbert@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> index cbe2ebd..640db67 100644
> --- a/drivers/gpu/drm/i915/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/intel_fbc.c
> @@ -530,12 +530,11 @@ static int find_compression_threshold(struct drm_i915_private *dev_priv,
>  	int ret;
>  	u64 end;
>  
> -	/* The FBC hardware for BDW/SKL doesn't have access to the stolen
> +	/* The FBC hardware for gen8+ doesn't have access to the stolen
>  	 * reserved range size, so it always assumes the maximum (8mb) is used.

Might I suggest correcting the comment to say megabyte instead of
millibit while at it?

>  	 * If we enable FBC using a CFB on that memory range we'll get FIFO
>  	 * underruns, even if that range is not reserved by the BIOS. */
> -	if (IS_BROADWELL(dev_priv) ||
> -	    IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv))
> +	if (INTEL_INFO(dev_priv)->gen <= 8)

INTEL_GEN(dev_priv)

Also, shouldn't this be >= 8? Also, remember that gen8 also includes
CherryView -- is the behaviour the same there?

>  		end = ggtt->stolen_size - 8 * 1024 * 1024;
>  	else
>  		end = ggtt->stolen_usable_size;
> -- 
> 1.9.1
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* ✗ Fi.CI.BAT: warning for drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ (rev2)
  2016-10-26 18:22 [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Rodrigo Vivi
                   ` (2 preceding siblings ...)
  2016-10-26 20:45 ` David Weinehall
@ 2016-10-26 21:16 ` Patchwork
  2016-10-26 22:50 ` [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Paulo Zanoni
  4 siblings, 0 replies; 9+ messages in thread
From: Patchwork @ 2016-10-26 21:16 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: intel-gfx

== Series Details ==

Series: drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ (rev2)
URL   : https://patchwork.freedesktop.org/series/14426/
State : warning

== Summary ==

Series 14426v2 drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
https://patchwork.freedesktop.org/api/1.0/series/14426/revisions/2/mbox/

Test kms_pipe_crc_basic:
        Subgroup bad-pipe:
                pass       -> DMESG-WARN (fi-ilk-650)
        Subgroup bad-source:
                pass       -> DMESG-WARN (fi-ilk-650)
        Subgroup hang-read-crc-pipe-a:
                pass       -> DMESG-WARN (fi-ilk-650)

fi-bdw-5557u     total:246  pass:231  dwarn:0   dfail:0   fail:0   skip:15 
fi-bsw-n3050     total:246  pass:204  dwarn:0   dfail:0   fail:0   skip:42 
fi-bxt-t5700     total:246  pass:216  dwarn:0   dfail:0   fail:0   skip:30 
fi-byt-j1900     total:246  pass:215  dwarn:0   dfail:0   fail:0   skip:31 
fi-byt-n2820     total:246  pass:211  dwarn:0   dfail:0   fail:0   skip:35 
fi-hsw-4770      total:246  pass:224  dwarn:0   dfail:0   fail:0   skip:22 
fi-hsw-4770r     total:246  pass:223  dwarn:0   dfail:0   fail:0   skip:23 
fi-ilk-650       total:246  pass:182  dwarn:3   dfail:0   fail:0   skip:61 
fi-ivb-3520m     total:246  pass:220  dwarn:0   dfail:0   fail:0   skip:26 
fi-ivb-3770      total:246  pass:220  dwarn:0   dfail:0   fail:0   skip:26 
fi-kbl-7200u     total:246  pass:222  dwarn:0   dfail:0   fail:0   skip:24 
fi-skl-6260u     total:246  pass:232  dwarn:0   dfail:0   fail:0   skip:14 
fi-skl-6700hq    total:246  pass:223  dwarn:0   dfail:0   fail:0   skip:23 
fi-skl-6700k     total:246  pass:222  dwarn:1   dfail:0   fail:0   skip:23 
fi-skl-6770hq    total:246  pass:232  dwarn:0   dfail:0   fail:0   skip:14 
fi-snb-2520m     total:246  pass:209  dwarn:0   dfail:0   fail:0   skip:37 
fi-snb-2600      total:246  pass:208  dwarn:0   dfail:0   fail:0   skip:38 

908bf09d04a2d83eadeb033f62a7f45d2c2a6aca drm-intel-nightly: 2016y-10m-26d-19h-59m-01s UTC integration manifest
4b49688 drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+

Full results at https://intel-gfx-ci.01.org/CI/Patchwork_2832/

== Logs ==

For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_2832/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
  2016-10-26 20:45 ` David Weinehall
@ 2016-10-26 21:18   ` Vivi, Rodrigo
  2016-10-26 22:08     ` David Weinehall
  0 siblings, 1 reply; 9+ messages in thread
From: Vivi, Rodrigo @ 2016-10-26 21:18 UTC (permalink / raw)
  To: tao; +Cc: intel-gfx, Herbert, Marc, Zanoni, Paulo R

On Wed, 2016-10-26 at 22:45 +0200, David Weinehall wrote:
> On Wed, Oct 26, 2016 at 11:22:00AM -0700, Rodrigo Vivi wrote:
> > Since Broxton has same FBC block as BDW+ let's assume it also
> > don't have access to the stolen usable range.
> > 
> > FBC is currently not saving power on Broxton and I believe
> > the compression threshold is limited to 1x.
> > 
> > Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> > Cc: Marc Herbert <marc.herbert@intel.com>
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > ---
> >  drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> > index cbe2ebd..640db67 100644
> > --- a/drivers/gpu/drm/i915/intel_fbc.c
> > +++ b/drivers/gpu/drm/i915/intel_fbc.c
> > @@ -530,12 +530,11 @@ static int find_compression_threshold(struct drm_i915_private *dev_priv,
> >  	int ret;
> >  	u64 end;
> >  
> > -	/* The FBC hardware for BDW/SKL doesn't have access to the stolen
> > +	/* The FBC hardware for gen8+ doesn't have access to the stolen
> >  	 * reserved range size, so it always assumes the maximum (8mb) is used.
> 
> Might I suggest correcting the comment to say megabyte instead of
> millibit while at it?

you mean s/8mb/8MB right? I can change it in a v3 for sure.

> 
> >  	 * If we enable FBC using a CFB on that memory range we'll get FIFO
> >  	 * underruns, even if that range is not reserved by the BIOS. */
> > -	if (IS_BROADWELL(dev_priv) ||
> > -	    IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv))
> > +	if (INTEL_INFO(dev_priv)->gen <= 8)
> 
> INTEL_GEN(dev_priv)

yeap, good point. will change that later .

Although for Marc the old style has more chance to get applied on his
tree for now.

Let's wait for his confirmation before the v3 anyway.

> 
> Also, shouldn't this be >= 8? Also, 

duh! sorry!
Marc already pointed me this mistake... v2 was sent already.

> remember that gen8 also includes
> CherryView -- is the behaviour the same there?

chv doesn't have fbc so I hope it never reaches this path...

> 
> >  		end = ggtt->stolen_size - 8 * 1024 * 1024;
> >  	else
> >  		end = ggtt->stolen_usable_size;
> > -- 
> > 1.9.1
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
  2016-10-26 21:18   ` Vivi, Rodrigo
@ 2016-10-26 22:08     ` David Weinehall
  0 siblings, 0 replies; 9+ messages in thread
From: David Weinehall @ 2016-10-26 22:08 UTC (permalink / raw)
  To: Vivi, Rodrigo; +Cc: intel-gfx, Herbert, Marc, Zanoni, Paulo R

On Wed, Oct 26, 2016 at 09:18:59PM +0000, Vivi, Rodrigo wrote:
> On Wed, 2016-10-26 at 22:45 +0200, David Weinehall wrote:
> > On Wed, Oct 26, 2016 at 11:22:00AM -0700, Rodrigo Vivi wrote:
> > > Since Broxton has same FBC block as BDW+ let's assume it also
> > > don't have access to the stolen usable range.
> > > 
> > > FBC is currently not saving power on Broxton and I believe
> > > the compression threshold is limited to 1x.
> > > 
> > > Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> > > Cc: Marc Herbert <marc.herbert@intel.com>
> > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > ---
> > >  drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
> > >  1 file changed, 2 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c
> > > index cbe2ebd..640db67 100644
> > > --- a/drivers/gpu/drm/i915/intel_fbc.c
> > > +++ b/drivers/gpu/drm/i915/intel_fbc.c
> > > @@ -530,12 +530,11 @@ static int find_compression_threshold(struct drm_i915_private *dev_priv,
> > >  	int ret;
> > >  	u64 end;
> > >  
> > > -	/* The FBC hardware for BDW/SKL doesn't have access to the stolen
> > > +	/* The FBC hardware for gen8+ doesn't have access to the stolen
> > >  	 * reserved range size, so it always assumes the maximum (8mb) is used.
> > 
> > Might I suggest correcting the comment to say megabyte instead of
> > millibit while at it?
> 
> you mean s/8mb/8MB right? I can change it in a v3 for sure.

Indeed.

> > 
> > >  	 * If we enable FBC using a CFB on that memory range we'll get FIFO
> > >  	 * underruns, even if that range is not reserved by the BIOS. */
> > > -	if (IS_BROADWELL(dev_priv) ||
> > > -	    IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv))
> > > +	if (INTEL_INFO(dev_priv)->gen <= 8)
> > 
> > INTEL_GEN(dev_priv)
> 
> yeap, good point. will change that later .
> 
> Although for Marc the old style has more chance to get applied on his
> tree for now.

Fair enough.

> Let's wait for his confirmation before the v3 anyway.
> 
> > 
> > Also, shouldn't this be >= 8? Also, 
> 
> duh! sorry!
> Marc already pointed me this mistake... v2 was sent already.

Yeah, I noticed just after I pressed send :)

> > remember that gen8 also includes
> > CherryView -- is the behaviour the same there?
> 
> chv doesn't have fbc so I hope it never reaches this path...

Goodie.


Kind regards, David
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
  2016-10-26 18:22 [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Rodrigo Vivi
                   ` (3 preceding siblings ...)
  2016-10-26 21:16 ` ✗ Fi.CI.BAT: warning for drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ (rev2) Patchwork
@ 2016-10-26 22:50 ` Paulo Zanoni
  4 siblings, 0 replies; 9+ messages in thread
From: Paulo Zanoni @ 2016-10-26 22:50 UTC (permalink / raw)
  To: Rodrigo Vivi, intel-gfx; +Cc: Marc Herbert

Em Qua, 2016-10-26 às 11:22 -0700, Rodrigo Vivi escreveu:
> Since Broxton has same FBC block as BDW+ let's assume it also
> don't have access to the stolen usable range.
> 
> FBC is currently not saving power on Broxton and I believe
> the compression threshold is limited to 1x.

NAK.

What is said above has nothing to do with what the code below does. The
amount of stolen memory is decided by i915_gem_init_stolen() and its
friends, not by the FBC code.

What this patch does is that it potentially reduces the amount of
usable stolen memory available to FBC by up to 8mb. That's totally not
going to help making FBC save more power. It can only do the opposite.

Also, the 1x compression threshold is the best thing.

The (unpatched) code below implements the restriction documented in
FBC_CFB_BASE, and it's not applicable to BXT (the spec explicitly says
that).

Also, we have a history of people who try to measure how much power FBC
saves, but they forget to do things such as properly setting the SATA
link power management or other stuff, leaving the system stuck in PC3
state or worse, and then they toggle FBC and conclude that FBC does not
save power. Power testing should really be done by the power management
team. 

Due to the past situations we had, I'd like to see data on the
measurements done before I can believe that the problem is really FBC.

> 
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Cc: Marc Herbert <marc.herbert@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  drivers/gpu/drm/i915/intel_fbc.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_fbc.c
> b/drivers/gpu/drm/i915/intel_fbc.c
> index cbe2ebd..640db67 100644
> --- a/drivers/gpu/drm/i915/intel_fbc.c
> +++ b/drivers/gpu/drm/i915/intel_fbc.c
> @@ -530,12 +530,11 @@ static int find_compression_threshold(struct
> drm_i915_private *dev_priv,
>  	int ret;
>  	u64 end;
>  
> -	/* The FBC hardware for BDW/SKL doesn't have access to the
> stolen
> +	/* The FBC hardware for gen8+ doesn't have access to the
> stolen
>  	 * reserved range size, so it always assumes the maximum
> (8mb) is used.
>  	 * If we enable FBC using a CFB on that memory range we'll
> get FIFO
>  	 * underruns, even if that range is not reserved by the
> BIOS. */
> -	if (IS_BROADWELL(dev_priv) ||
> -	    IS_SKYLAKE(dev_priv) || IS_KABYLAKE(dev_priv))
> +	if (INTEL_INFO(dev_priv)->gen <= 8)
>  		end = ggtt->stolen_size - 8 * 1024 * 1024;
>  	else
>  		end = ggtt->stolen_usable_size;
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: ✗ Fi.CI.BAT: warning for drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
  2016-10-26 19:46 ` ✗ Fi.CI.BAT: warning for " Patchwork
@ 2016-10-27  5:47   ` Saarinen, Jani
  0 siblings, 0 replies; 9+ messages in thread
From: Saarinen, Jani @ 2016-10-27  5:47 UTC (permalink / raw)
  To: intel-gfx, Vivi, Rodrigo

> == Series Details ==
> 
> Series: drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+
> URL   : https://patchwork.freedesktop.org/series/14426/
> State : warning
> 
> == Summary ==
> 
> Series 14426v1 drm/i915/fbc: Assume maximum 8mb of stolen is used for
> gen8+
> https://patchwork.freedesktop.org/api/1.0/series/14426/revisions/1/mbox/
> 
> Test drv_module_reload_basic:
>                 dmesg-warn -> PASS       (fi-ilk-650)
> Test kms_pipe_crc_basic:
>         Subgroup suspend-read-crc-pipe-a:
>                 pass       -> DMESG-WARN (fi-skl-6770hq)
https://bugs.freedesktop.org/show_bug.cgi?id=97929 still there.

>                 dmesg-warn -> PASS       (fi-ilk-650)
> 
> fi-bdw-5557u     total:246  pass:231  dwarn:0   dfail:0   fail:0   skip:15
> fi-bsw-n3050     total:246  pass:204  dwarn:0   dfail:0   fail:0   skip:42
> fi-bxt-t5700     total:246  pass:216  dwarn:0   dfail:0   fail:0   skip:30
> fi-byt-j1900     total:246  pass:215  dwarn:0   dfail:0   fail:0   skip:31
> fi-byt-n2820     total:246  pass:211  dwarn:0   dfail:0   fail:0   skip:35
> fi-hsw-4770      total:246  pass:224  dwarn:0   dfail:0   fail:0   skip:22
> fi-hsw-4770r     total:246  pass:223  dwarn:0   dfail:0   fail:0   skip:23
> fi-ilk-650       total:246  pass:185  dwarn:0   dfail:0   fail:0   skip:61
> fi-ivb-3520m     total:246  pass:220  dwarn:0   dfail:0   fail:0   skip:26
> fi-ivb-3770      total:246  pass:220  dwarn:0   dfail:0   fail:0   skip:26
> fi-kbl-7200u     total:246  pass:222  dwarn:0   dfail:0   fail:0   skip:24
> fi-skl-6260u     total:246  pass:232  dwarn:0   dfail:0   fail:0   skip:14
> fi-skl-6700hq    total:246  pass:223  dwarn:0   dfail:0   fail:0   skip:23
> fi-skl-6700k     total:246  pass:222  dwarn:1   dfail:0   fail:0   skip:23
> fi-skl-6770hq    total:246  pass:231  dwarn:1   dfail:0   fail:0   skip:14
> fi-snb-2520m     total:246  pass:209  dwarn:0   dfail:0   fail:0   skip:37
> fi-snb-2600      total:246  pass:208  dwarn:0   dfail:0   fail:0   skip:38
> 
> 0eefa35311a3d85e6cbfda4bb326809581acc3b1 drm-intel-nightly: 2016y-10m-
> 26d-17h-38m-29s UTC integration manifest 2412c2c drm/i915/fbc: Assume
> maximum 8mb of stolen is used for gen8+
> 
> Full results at https://intel-gfx-ci.01.org/CI/Patchwork_2830/
> 
> == Logs ==
> 
> For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_2830/


Jani Saarinen
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo


_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2016-10-27  5:47 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-26 18:22 [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Rodrigo Vivi
2016-10-26 19:46 ` ✗ Fi.CI.BAT: warning for " Patchwork
2016-10-27  5:47   ` Saarinen, Jani
2016-10-26 20:28 ` [PATCH] " Rodrigo Vivi
2016-10-26 20:45 ` David Weinehall
2016-10-26 21:18   ` Vivi, Rodrigo
2016-10-26 22:08     ` David Weinehall
2016-10-26 21:16 ` ✗ Fi.CI.BAT: warning for drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ (rev2) Patchwork
2016-10-26 22:50 ` [PATCH] drm/i915/fbc: Assume maximum 8mb of stolen is used for gen8+ Paulo Zanoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.