Re: [PATCH 5.10.y] drm/amd/pm: Fix negative array index read

[Sasha Levin <sashal@kernel.org>]

[ Sasha's backport helper bot ]

Hi,

✅ All tests passed successfully. No issues detected.
No action required from the submitter.

The upstream commit SHA1 provided is correct: c8c19ebf7c0b202a6a2d37a52ca112432723db5f

WARNING: Author mismatch between patch and upstream commit:
Backport author: <jianqi.ren.cn@windriver.com>
Commit author: Jesse Zhang<jesse.zhang@amd.com>

Status in newer kernel trees:
6.14.y | Present (exact SHA1)
6.13.y | Present (exact SHA1)
6.12.y | Present (exact SHA1)
6.6.y | Present (different SHA1: 4711b1347cb9)
6.1.y | Present (different SHA1: 60f4a4bc3329)
5.15.y | Present (different SHA1: d7f112ac4f8a)

Note: The patch differs from the upstream commit:
---
1:  c8c19ebf7c0b2 ! 1:  c12cb5c0d5794 drm/amd/pm: Fix negative array index read
    @@ Metadata
      ## Commit message ##
         drm/amd/pm: Fix negative array index read
     
    +    [ Upstream commit c8c19ebf7c0b202a6a2d37a52ca112432723db5f ]
    +
         Avoid using the negative values
         for clk_idex as an index into an array pptable->DpmDescriptor.
     
    @@ Commit message
         Signed-off-by: Jesse Zhang <Jesse.Zhang@amd.com>
         Reviewed-by: Tim Huang <Tim.Huang@amd.com>
         Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
    +    [Minor conflict resolved due to code context change.]
    +    Signed-off-by: Jianqi Ren <jianqi.ren.cn@windriver.com>
    +    Signed-off-by: He Zhe <zhe.he@windriver.com>
     
      ## drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c ##
     @@ drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c: static int navi10_get_current_clk_freq_by_table(struct smu_context *smu,
    @@ drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c: static int navi10_get_current_c
      	dpm_desc = &pptable->DpmDescriptor[clk_index];
      
      	/* 0 - Fine grained DPM, 1 - Discrete DPM */
    --	return dpm_desc->SnapToDiscrete == 0;
    +-	return dpm_desc->SnapToDiscrete == 0 ? true : false;
     +	return dpm_desc->SnapToDiscrete == 0 ? 1 : 0;
      }
      
      static inline bool navi10_od_feature_is_supported(struct smu_11_0_overdrive_table *od_table, enum SMU_11_0_ODFEATURE_CAP cap)
    -@@ drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c: static int navi10_emit_clk_levels(struct smu_context *smu,
    - 		if (ret)
    - 			return ret;
    - 
    --		if (!navi10_is_support_fine_grained_dpm(smu, clk_type)) {
    -+		ret = navi10_is_support_fine_grained_dpm(smu, clk_type);
    -+		if (ret < 0)
    -+			return ret;
    -+
    -+		if (!ret) {
    - 			for (i = 0; i < count; i++) {
    - 				ret = smu_v11_0_get_dpm_freq_by_index(smu,
    - 								      clk_type, i, &value);
     @@ drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c: static int navi10_print_clk_levels(struct smu_context *smu,
      		if (ret)
      			return size;
---

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-5.10.y       |  Success    |  Success   |