This was sort of annoying me:
random:~$ dmesg | tail -1 [523884.039227] [drm] Reducing the compressed framebuffer size. This may lead to less power savings than a non-reduced-size. Try to increase stolen memory size if available in BIOS. random:~$ dmesg | grep -c "Reducing the compressed" 47
This patch makes it DRM_INFO_ONCE() just like the similar message farther down in that function is pr_info_once().
Signed-off-by: Peter Jones pjones@redhat.com --- drivers/gpu/drm/i915/intel_fbc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c index b431b6733cc..88b013758da 100644 --- a/drivers/gpu/drm/i915/intel_fbc.c +++ b/drivers/gpu/drm/i915/intel_fbc.c @@ -585,7 +585,7 @@ static int intel_fbc_alloc_cfb(struct intel_crtc *crtc) if (!ret) goto err_llb; else if (ret > 1) { - DRM_INFO("Reducing the compressed framebuffer size. This may lead to less power savings than a non-reduced-size. Try to increase stolen memory size if available in BIOS.\n"); + DRM_INFO_ONCE("Reducing the compressed framebuffer size. This may lead to less power savings than a non-reduced-size. Try to increase stolen memory size if available in BIOS.\n");
}
On Fri, Jul 06, 2018 at 03:04:24PM -0400, Peter Jones wrote:
I know the feeling of seeing a message over and over again when you are already aware of that. Specially something that tells you to change BIOS configuration... for this reason I think it makes sense, so:
Acked-by: Rodrigo Vivi rodrigo.vivi@intel.com
I will just wait for CI and give a couple days to someone else to challenge. If I end up forgetting to push it please ping me...
dri-devel@lists.freedesktop.org
-
Jiri Slaby -
Peter Jones -
Rodrigo Vivi