[AMD Official Use Only - General]
Hi Lijo,
I prefer to add a version field in header, which is used for compatible in the future, what is your idea?
When the user attempts to parse this node, they cannot accurately determine the format of the content.
Best Regards,
Kevin
-----Original Message-----
From: Lazar, Lijo <Lijo.Lazar@xxxxxxx>
Sent: Friday, October 6, 2023 1:22 PM
To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Cc: Zhang, Hawking <Hawking.Zhang@xxxxxxx>; Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Wang, Yang(Kevin) <KevinYang.Wang@xxxxxxx>; Deucher, Alexander <Alexander.Deucher@xxxxxxx>
Subject: [PATCH v2 1/4] drm/amdgpu: add pmlog structure definition
From: Alex Deucher <alexander.deucher@xxxxxxx>
Define the pmlog structures to be exposed via sysfs.
Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
Signed-off-by: Lijo Lazar <lijo.lazar@xxxxxxx>
---
drivers/gpu/drm/amd/include/kgd_pp_interface.h | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/drivers/gpu/drm/amd/include/kgd_pp_interface.h b/drivers/gpu/drm/amd/include/kgd_pp_interface.h
index e0bb6d39f0c3..9905228fd89c 100644
--- a/drivers/gpu/drm/amd/include/kgd_pp_interface.h
+++ b/drivers/gpu/drm/amd/include/kgd_pp_interface.h
@@ -980,4 +980,19 @@ struct gpu_metrics_v2_4 {
uint16_t average_soc_current;
uint16_t average_gfx_current;
};
+
+struct amdgpu_pmlog_header {
+ uint16_t structure_size;
+ uint16_t pad;
+ uint32_t mp1_ip_discovery_version;
+ uint32_t pmfw_version;
+ uint32_t pmlog_version;
+};
+
+struct amdgpu_pmlog {
+ struct amdgpu_pmlog_header common_header;
+
+ uint8_t data[];
+};
+
#endif
--
2.25.1
