From 0f37a5c19b9ce2dd426ef8a279eb677019ef9f35 Mon Sep 17 00:00:00 2001 From: Ben Richard <143630488+benrichard-amd@users.noreply.github.com> Date: Wed, 16 Oct 2024 15:41:11 -0400 Subject: [PATCH] Use MI300 chip_id instead of model to detect XCD count (#448) In a previous change we started using "MI300" for gpu_model instead of the full "MI300X_A0" or "MI300X_A1", etc. The XCD detection code was using gpu_model and expecting the full name, causing the XCD count = 1. Passing chip_id fixes the issue. Signed-off-by: benrichard-amd Signed-off-by: xuchen-amd --- src/omniperf_soc/soc_base.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/omniperf_soc/soc_base.py b/src/omniperf_soc/soc_base.py index 8a4808726..db8ce9a22 100644 --- a/src/omniperf_soc/soc_base.py +++ b/src/omniperf_soc/soc_base.py @@ -187,7 +187,7 @@ def populate_mspec(self): self._mspec.chip_id = MI300_CHIP_IDS[self._mspec.chip_id] self._mspec.num_xcd = str( - total_xcds(self._mspec.gpu_model, self._mspec.compute_partition) + total_xcds(self._mspec.chip_id, self._mspec.compute_partition) ) @demarcate