mbox series

[0/3] drm/msm/a6xx: devcore dump fixes

Message ID 20221013225520.371226-1-robdclark@gmail.com (mailing list archive)
Headers show
Series drm/msm/a6xx: devcore dump fixes | expand

Message

Rob Clark Oct. 13, 2022, 10:55 p.m. UTC
From: Rob Clark <robdclark@chromium.org>

First patch fixes a recently introduced memory corruption, the remaining
two are cleanups.

Rob Clark (3):
  drm/msm/a6xx: Fix kvzalloc vs state_kcalloc usage
  drm/msm/a6xx: Skip snapshotting unused GMU buffers
  drm/msm/a6xx: Remove state objects from list before freeing

 drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 18 ++++++++++++++++--
 drivers/gpu/drm/msm/adreno/adreno_gpu.c     |  7 ++++++-
 2 files changed, 22 insertions(+), 3 deletions(-)

Comments

Chia-I Wu Oct. 14, 2022, 12:19 a.m. UTC | #1
On Thu, Oct 13, 2022 at 3:55 PM Rob Clark <robdclark@gmail.com> wrote:
>
> From: Rob Clark <robdclark@chromium.org>
>
> First patch fixes a recently introduced memory corruption, the remaining
> two are cleanups.
Series is

Reviewed-by: Chia-I Wu <olvaffe@gmail.com>

> Rob Clark (3):
>   drm/msm/a6xx: Fix kvzalloc vs state_kcalloc usage
>   drm/msm/a6xx: Skip snapshotting unused GMU buffers
>   drm/msm/a6xx: Remove state objects from list before freeing
>
>  drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 18 ++++++++++++++++--
>  drivers/gpu/drm/msm/adreno/adreno_gpu.c     |  7 ++++++-
>  2 files changed, 22 insertions(+), 3 deletions(-)
>
> --
> 2.37.3
>
Akhil P Oommen Oct. 14, 2022, 6:50 a.m. UTC | #2
On 10/14/2022 4:25 AM, Rob Clark wrote:
> From: Rob Clark <robdclark@chromium.org>
>
> First patch fixes a recently introduced memory corruption, the remaining
> two are cleanups.
>
> Rob Clark (3):
>    drm/msm/a6xx: Fix kvzalloc vs state_kcalloc usage
>    drm/msm/a6xx: Skip snapshotting unused GMU buffers
>    drm/msm/a6xx: Remove state objects from list before freeing
>
>   drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 18 ++++++++++++++++--
>   drivers/gpu/drm/msm/adreno/adreno_gpu.c     |  7 ++++++-
>   2 files changed, 22 insertions(+), 3 deletions(-)
For the entire series:

Reviewed-by: Akhil P Oommen <quic_akhilpo@quicinc.com>


-Akhil