mbox series

[0/1,SRU,N] FPS of glxgear with fullscreen is too low on MTL platform

Message ID 20240704074755.48885-1-acelan.kao@canonical.com
Headers show
Series FPS of glxgear with fullscreen is too low on MTL platform | expand

Message

AceLan Kao July 4, 2024, 7:47 a.m. UTC
From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>

BugLink: https://bugs.launchpad.net/bugs/2069380

[Impact]
Run `glxgears -fullscreen` and the FPS can't stay at 60 FPS
298 frames in 5.0 seconds = 59.465 FPS
241 frames in 5.0 seconds = 48.193 FPS
261 frames in 5.0 seconds = 52.030 FPS
268 frames in 5.0 seconds = 53.596 FPS

[Fix]
Bisect the kernel and found a commit in v6.9-rc1 fixes the issue
c08c364102d0 drm/i915: Bypass LMEMBAR/GTTMMADR for MTL stolen memory access

[Test]
1. Run `glxgears -fullscreen` on MTL platforms
2. The FPS should be always around 60

[Where problems could occur]
It's a workaround only affects MTL platforms, and there is no
"fixed commit" for this patch after v6.9-rc1. So, it should be safe and
essential for MTL platforms.

Ville Syrjälä (1):
  drm/i915: Bypass LMEMBAR/GTTMMADR for MTL stolen memory access

 drivers/gpu/drm/i915/gem/i915_gem_stolen.c |  6 +++++-
 drivers/gpu/drm/i915/gt/intel_ggtt.c       | 10 +++++++++-
 drivers/gpu/drm/i915/i915_reg.h            |  3 +++
 drivers/gpu/drm/i915/i915_utils.c          | 17 +++++++++++++++++
 drivers/gpu/drm/i915/i915_utils.h          |  2 ++
 5 files changed, 36 insertions(+), 2 deletions(-)

Comments

Andrei Gherzan July 4, 2024, 8:56 a.m. UTC | #1
On 24/07/04 03:47pm, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/2069380
> 
> [Impact]
> Run `glxgears -fullscreen` and the FPS can't stay at 60 FPS
> 298 frames in 5.0 seconds = 59.465 FPS
> 241 frames in 5.0 seconds = 48.193 FPS
> 261 frames in 5.0 seconds = 52.030 FPS
> 268 frames in 5.0 seconds = 53.596 FPS
> 
> [Fix]
> Bisect the kernel and found a commit in v6.9-rc1 fixes the issue
> c08c364102d0 drm/i915: Bypass LMEMBAR/GTTMMADR for MTL stolen memory access
> 
> [Test]
> 1. Run `glxgears -fullscreen` on MTL platforms
> 2. The FPS should be always around 60
> 
> [Where problems could occur]
> It's a workaround only affects MTL platforms, and there is no
> "fixed commit" for this patch after v6.9-rc1. So, it should be safe and
> essential for MTL platforms.
> 
> Ville Syrjälä (1):
>   drm/i915: Bypass LMEMBAR/GTTMMADR for MTL stolen memory access
> 
>  drivers/gpu/drm/i915/gem/i915_gem_stolen.c |  6 +++++-
>  drivers/gpu/drm/i915/gt/intel_ggtt.c       | 10 +++++++++-
>  drivers/gpu/drm/i915/i915_reg.h            |  3 +++
>  drivers/gpu/drm/i915/i915_utils.c          | 17 +++++++++++++++++
>  drivers/gpu/drm/i915/i915_utils.h          |  2 ++
>  5 files changed, 36 insertions(+), 2 deletions(-)

Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Paolo Pisati July 4, 2024, 9 a.m. UTC | #2
On Thu, Jul 04, 2024 at 03:47:54PM +0800, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/2069380

Acked-by: Paolo Pisati <paolo.pisati@canonical.com>
Stefan Bader July 4, 2024, 7:04 p.m. UTC | #3
On 04.07.24 09:47, AceLan Kao wrote:
> From: "Chia-Lin Kao (AceLan)" <acelan.kao@canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/2069380
> 
> [Impact]
> Run `glxgears -fullscreen` and the FPS can't stay at 60 FPS
> 298 frames in 5.0 seconds = 59.465 FPS
> 241 frames in 5.0 seconds = 48.193 FPS
> 261 frames in 5.0 seconds = 52.030 FPS
> 268 frames in 5.0 seconds = 53.596 FPS
> 
> [Fix]
> Bisect the kernel and found a commit in v6.9-rc1 fixes the issue
> c08c364102d0 drm/i915: Bypass LMEMBAR/GTTMMADR for MTL stolen memory access
> 
> [Test]
> 1. Run `glxgears -fullscreen` on MTL platforms
> 2. The FPS should be always around 60
> 
> [Where problems could occur]
> It's a workaround only affects MTL platforms, and there is no
> "fixed commit" for this patch after v6.9-rc1. So, it should be safe and
> essential for MTL platforms.
> 
> Ville Syrjälä (1):
>    drm/i915: Bypass LMEMBAR/GTTMMADR for MTL stolen memory access
> 
>   drivers/gpu/drm/i915/gem/i915_gem_stolen.c |  6 +++++-
>   drivers/gpu/drm/i915/gt/intel_ggtt.c       | 10 +++++++++-
>   drivers/gpu/drm/i915/i915_reg.h            |  3 +++
>   drivers/gpu/drm/i915/i915_utils.c          | 17 +++++++++++++++++
>   drivers/gpu/drm/i915/i915_utils.h          |  2 ++
>   5 files changed, 36 insertions(+), 2 deletions(-)
> 

Applied to noble:linux/master-next. Thanks.

-Stefan