[PATCH] drm/amd/amdgpu: remove unnecessary RAS context field

Clements, John John.Clements at amd.com
Fri Aug 13 12:57:58 UTC 2021


[Public]

Reviewed-by: John Clements <john.clements at amd.com>

From: Li, Candice <Candice.Li at amd.com>
Sent: Friday, August 13, 2021 7:46 PM
To: amd-gfx at lists.freedesktop.org
Cc: Clements, John <John.Clements at amd.com>; Li, Candice <Candice.Li at amd.com>
Subject: [PATCH] drm/amd/amdgpu: remove unnecessary RAS context field


[Public]

Delete ras_if->name in the RAS ctx structure and remove related lines.

Signed-off-by: Candice Li candice.li at amd.com<mailto:candice.li at amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c   | 1 -
drivers/gpu/drm/amd/amdgpu/amdgpu_hdp.c   | 1 -
drivers/gpu/drm/amd/amdgpu/amdgpu_mmhub.c | 1 -
drivers/gpu/drm/amd/amdgpu/amdgpu_nbio.c  | 1 -
drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c   | 4 +---
drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h   | 5 +++--
drivers/gpu/drm/amd/amdgpu/amdgpu_sdma.c  | 1 -
drivers/gpu/drm/amd/amdgpu/amdgpu_umc.c   | 1 -
drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c  | 1 -
drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c    | 4 ++--
10 files changed, 6 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
index 9cfef56b2aee..5beaa7c1bd11 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
@@ -615,7 +615,6 @@ int amdgpu_gfx_ras_late_init(struct amdgpu_device *adev)
                            adev->gfx.ras_if->block = AMDGPU_RAS_BLOCK__GFX;
                            adev->gfx.ras_if->type = AMDGPU_RAS_ERROR__MULTI_UNCORRECTABLE;
                            adev->gfx.ras_if->sub_block_index = 0;
-                           strcpy(adev->gfx.ras_if->name, "gfx");
             }
             fs_info.head = ih_info.head = *adev->gfx.ras_if;
             r = amdgpu_ras_late_init(adev, adev->gfx.ras_if,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_hdp.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_hdp.c
index 1d50d534d77c..a766e1aad2b9 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_hdp.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_hdp.c
@@ -41,7 +41,6 @@ int amdgpu_hdp_ras_late_init(struct amdgpu_device *adev)
                            adev->hdp.ras_if->block = AMDGPU_RAS_BLOCK__HDP;
                            adev->hdp.ras_if->type = AMDGPU_RAS_ERROR__MULTI_UNCORRECTABLE;
                            adev->hdp.ras_if->sub_block_index = 0;
-                           strcpy(adev->hdp.ras_if->name, "hdp");
             }
             ih_info.head = fs_info.head = *adev->hdp.ras_if;
             r = amdgpu_ras_late_init(adev, adev->hdp.ras_if,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mmhub.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_mmhub.c
index ead3dc572ec5..24297dc51434 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_mmhub.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_mmhub.c
@@ -41,7 +41,6 @@ int amdgpu_mmhub_ras_late_init(struct amdgpu_device *adev)
                            adev->mmhub.ras_if->block = AMDGPU_RAS_BLOCK__MMHUB;
                            adev->mmhub.ras_if->type = AMDGPU_RAS_ERROR__MULTI_UNCORRECTABLE;
                            adev->mmhub.ras_if->sub_block_index = 0;
-                           strcpy(adev->mmhub.ras_if->name, "mmhub");
             }
             ih_info.head = fs_info.head = *adev->mmhub.ras_if;
             r = amdgpu_ras_late_init(adev, adev->mmhub.ras_if,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_nbio.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_nbio.c
index 6201a5f4b4fa..6afb02fef8cf 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_nbio.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_nbio.c
@@ -39,7 +39,6 @@ int amdgpu_nbio_ras_late_init(struct amdgpu_device *adev)
                            adev->nbio.ras_if->block = AMDGPU_RAS_BLOCK__PCIE_BIF;
                            adev->nbio.ras_if->type = AMDGPU_RAS_ERROR__MULTI_UNCORRECTABLE;
                            adev->nbio.ras_if->sub_block_index = 0;
-                           strcpy(adev->nbio.ras_if->name, "pcie_bif");
             }
             ih_info.head = fs_info.head = *adev->nbio.ras_if;
             r = amdgpu_ras_late_init(adev, adev->nbio.ras_if,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
index 3811b6b6a192..96a8fd0ca1df 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
@@ -64,7 +64,6 @@ const char *ras_block_string[] = {
};

 #define ras_err_str(i) (ras_error_string[ffs(i)])
-#define ras_block_str(i) (ras_block_string[i])

 #define RAS_DEFAULT_FLAGS (AMDGPU_RAS_FLAG_INIT_BY_VBIOS)

@@ -530,7 +529,7 @@ static inline void put_obj(struct ras_manager *obj)
             if (obj && (--obj->use == 0))
                            list_del(&obj->node);
             if (obj && (obj->use < 0))
-                           DRM_ERROR("RAS ERROR: Unbalance obj(%s) use\n", obj->head.name);
+                           DRM_ERROR("RAS ERROR: Unbalance obj(%s) use\n", ras_block_str(obj->head.block));
}

 /* make one obj and return it. */
@@ -793,7 +792,6 @@ static int amdgpu_ras_enable_all_features(struct amdgpu_device *adev,
                                          .type = default_ras_type,
                                          .sub_block_index = 0,
                            };
-                           strcpy(head.name, ras_block_str(i));
                            if (bypass) {
                                          /*
                                           * bypass psp. vbios enable ras for us.
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
index 471ffe885fdf..abc5710898e8 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.h
@@ -53,6 +53,9 @@ enum amdgpu_ras_block {
             AMDGPU_RAS_BLOCK__LAST
};

+extern const char *ras_block_string[];
+
+#define ras_block_str(i) (ras_block_string[i])
#define AMDGPU_RAS_BLOCK_COUNT AMDGPU_RAS_BLOCK__LAST
#define AMDGPU_RAS_BLOCK_MASK   ((1ULL << AMDGPU_RAS_BLOCK_COUNT) - 1)

@@ -306,8 +309,6 @@ struct ras_common_if {
             enum amdgpu_ras_block block;
             enum amdgpu_ras_error_type type;
             uint32_t sub_block_index;
-             /* block name */
-             char name[32];
};

 struct amdgpu_ras {
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_sdma.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_sdma.c
index de91d29c9d96..65debb65a5df 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_sdma.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_sdma.c
@@ -105,7 +105,6 @@ int amdgpu_sdma_ras_late_init(struct amdgpu_device *adev,
                            adev->sdma.ras_if->block = AMDGPU_RAS_BLOCK__SDMA;
                            adev->sdma.ras_if->type = AMDGPU_RAS_ERROR__MULTI_UNCORRECTABLE;
                            adev->sdma.ras_if->sub_block_index = 0;
-                           strcpy(adev->sdma.ras_if->name, "sdma");
             }
             fs_info.head = ih_info->head = *adev->sdma.ras_if;

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_umc.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_umc.c
index 0c7c56a91b25..a90029ee9733 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_umc.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_umc.c
@@ -41,7 +41,6 @@ int amdgpu_umc_ras_late_init(struct amdgpu_device *adev)
                            adev->umc.ras_if->block = AMDGPU_RAS_BLOCK__UMC;
                            adev->umc.ras_if->type = AMDGPU_RAS_ERROR__MULTI_UNCORRECTABLE;
                            adev->umc.ras_if->sub_block_index = 0;
-                           strcpy(adev->umc.ras_if->name, "umc");
             }
             ih_info.head = fs_info.head = *adev->umc.ras_if;

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c
index 258cf86b32f6..2e47bc446700 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_xgmi.c
@@ -663,7 +663,6 @@ static int amdgpu_xgmi_ras_late_init(struct amdgpu_device *adev)
                            adev->gmc.xgmi.ras_if->block = AMDGPU_RAS_BLOCK__XGMI_WAFL;
                            adev->gmc.xgmi.ras_if->type = AMDGPU_RAS_ERROR__MULTI_UNCORRECTABLE;
                            adev->gmc.xgmi.ras_if->sub_block_index = 0;
-                           strcpy(adev->gmc.xgmi.ras_if->name, "xgmi_wafl");
             }
             ih_info.head = fs_info.head = *adev->gmc.xgmi.ras_if;
             r = amdgpu_ras_late_init(adev, adev->gmc.xgmi.ras_if,
diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c b/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c
index cef929746739..1c94a14fc18d 100644
--- a/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c
+++ b/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c
@@ -372,13 +372,13 @@ static void nbio_v7_4_handle_ras_controller_intr_no_bifring(struct amdgpu_device
                                                                                     "errors detected in %s block, "
                                                                                     "no user action is needed.\n",
                                                                                     obj->err_data.ce_count,
-                                                                                     adev->nbio.ras_if->name);
+                                                                                    ras_block_str(adev->nbio.ras_if->block));

                                           if (err_data.ue_count)
                                                         dev_info(adev->dev, "%ld uncorrectable hardware "
                                                                                     "errors detected in %s block\n",
                                                                                     obj->err_data.ue_count,
-                                                                                     adev->nbio.ras_if->name);
+                                                                                    ras_block_str(adev->nbio.ras_if->block));
                            }

                             dev_info(adev->dev, "RAS controller interrupt triggered "
--
2.17.1




Thanks,
Candice

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20210813/da42b4d9/attachment-0001.htm>


More information about the amd-gfx mailing list