[AMD Official Use Only - General] The comment is not correct. Please ignore this one and review v2.
Best Regards, Yifan -----Original Message----- From: Zhang, Yifan <[email protected]> Sent: Thursday, August 11, 2022 8:30 PM To: [email protected] Cc: Deucher, Alexander <[email protected]>; Kuehling, Felix <[email protected]>; Joshi, Mukul <[email protected]>; Zhang, Yifan <[email protected]> Subject: [PATCH] drm/amdkfd: reserve 2 queues for sdma 6.0.1 There is only one engine in sdma 6.0.1, current code reserve 4 queues togather w/ 6.0.0 and 6.0.2, which is not necessary. Shrink it to 2. Signed-off-by: Yifan Zhang <[email protected]> --- drivers/gpu/drm/amd/amdkfd/kfd_device.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device.c b/drivers/gpu/drm/amd/amdkfd/kfd_device.c index f5853835f03a..a82ddea967f2 100644 --- a/drivers/gpu/drm/amd/amdkfd/kfd_device.c +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device.c @@ -102,13 +102,18 @@ static void kfd_device_info_set_sdma_info(struct kfd_dev *kfd) switch (sdma_version) { case IP_VERSION(6, 0, 0): - case IP_VERSION(6, 0, 1): case IP_VERSION(6, 0, 2): /* Reserve 1 for paging and 1 for gfx */ kfd->device_info.num_reserved_sdma_queues_per_engine = 2; /* BIT(0)=engine-0 queue-0; BIT(1)=engine-1 queue-0; BIT(2)=engine-0 queue-1; ... */ kfd->device_info.reserved_sdma_queues_bitmap = 0xFULL; break; + case IP_VERSION(6, 0, 1): + /* Reserve 1 for paging and 1 for gfx */ + kfd->device_info.num_reserved_sdma_queues_per_engine = 2; + /* BIT(0)=engine-0 queue-0; BIT(1)=engine-1 queue-0; BIT(2)=engine-0 queue-1; ... */ + kfd->device_info.reserved_sdma_queues_bitmap = 0x5ULL; + break; default: break; } -- 2.37.1
