From a91306677c613ba7511b764b3decc9db42b24de1 Mon Sep 17 00:00:00 2001 From: James Zhu Date: Tue, 26 May 2020 09:28:41 -0400 Subject: [PATCH] ac/gpu_info: Correct Acturus cu bitmap MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The cu bitmap in amd gpu info structure is 4x4 size array, and it's usually suitable for Vega ASICs which has 4*2 SE/SH layout. But for Arcturus, SE/SH layout is changed to 8*1. To mostly reduce the impact, we make it compatible with current bitmap array as below: SE4,SH0 --> cu_bitmap[0][1] SE5,SH0 --> cu_bitmap[1][1] SE6,SH0 --> cu_bitmap[2][1] SE7,SH0 --> cu_bitmap[3][1] Signed-off-by: James Zhu Reviewed-by: Marek Olšák Part-of: --- src/amd/common/ac_gpu_info.c | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/src/amd/common/ac_gpu_info.c b/src/amd/common/ac_gpu_info.c index bdd8de374dd..f0128dcf0f0 100644 --- a/src/amd/common/ac_gpu_info.c +++ b/src/amd/common/ac_gpu_info.c @@ -619,7 +619,19 @@ bool ac_query_gpu_info(int fd, void *dev_p, info->num_good_compute_units = 0; for (i = 0; i < info->max_se; i++) { for (j = 0; j < info->max_sh_per_se; j++) { - info->cu_mask[i][j] = amdinfo->cu_bitmap[i][j]; + /* + * The cu bitmap in amd gpu info structure is + * 4x4 size array, and it's usually suitable for Vega + * ASICs which has 4*2 SE/SH layout. + * But for Arcturus, SE/SH layout is changed to 8*1. + * To mostly reduce the impact, we make it compatible + * with current bitmap array as below: + * SE4,SH0 --> cu_bitmap[0][1] + * SE5,SH0 --> cu_bitmap[1][1] + * SE6,SH0 --> cu_bitmap[2][1] + * SE7,SH0 --> cu_bitmap[3][1] + */ + info->cu_mask[i%4][j+i/4] = amdinfo->cu_bitmap[i%4][j+i/4]; info->num_good_compute_units += util_bitcount(info->cu_mask[i][j]); } -- 2.30.2