[PATCH v2 1/1] drm/amdkfd: Fix unaligned doorbell absolute offset for gfx8
Joshi, Mukul
Mukul.Joshi at amd.com
Thu Sep 28 14:30:47 UTC 2023
[AMD Official Use Only - General]
> -----Original Message-----
> From: Yadav, Arvind <Arvind.Yadav at amd.com>
> Sent: Thursday, September 28, 2023 5:54 AM
> To: Koenig, Christian <Christian.Koenig at amd.com>; Deucher, Alexander
> <Alexander.Deucher at amd.com>; Sharma, Shashank
> <Shashank.Sharma at amd.com>; Kuehling, Felix <Felix.Kuehling at amd.com>;
> Joshi, Mukul <Mukul.Joshi at amd.com>; Pan, Xinhui <Xinhui.Pan at amd.com>;
> airlied at gmail.com; daniel at ffwll.ch
> Cc: amd-gfx at lists.freedesktop.org; dri-devel at lists.freedesktop.org; linux-
> kernel at vger.kernel.org; Yadav, Arvind <Arvind.Yadav at amd.com>; Koenig,
> Christian <Christian.Koenig at amd.com>
> Subject: [PATCH v2 1/1] drm/amdkfd: Fix unaligned doorbell absolute offset
> for gfx8
>
> This patch is to adjust the absolute doorbell offset against the doorbell id
> considering the doorbell size of 32/64 bit.
>
> v2:
> - Addressed the review comment from Felix.
>
> Cc: Christian Koenig <christian.koenig at amd.com>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Signed-off-by: Shashank Sharma <shashank.sharma at amd.com>
> Signed-off-by: Arvind Yadav <Arvind.Yadav at amd.com>
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> index 0d3d538b64eb..c54c4392d26e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> @@ -407,7 +407,14 @@ static int allocate_doorbell(struct
> qcm_process_device *qpd,
>
> q->properties.doorbell_off = amdgpu_doorbell_index_on_bar(dev-
> >adev,
> qpd-
> >proc_doorbells,
> - q-
> >doorbell_id);
> + 0);
> +
It looks like amdgpu_doorbell_index_on_bar() works only for 64-bit doorbells.
Shouldn't it work for both 32-bit and 64-bit doorbells considering this is common
doorbell manager code?
Thanks,
Mukul
> + /* Adjust the absolute doorbell offset against the doorbell id
> considering
> + * the doorbell size of 32/64 bit.
> + */
> + q->properties.doorbell_off += q->doorbell_id *
> + dev->kfd->device_info.doorbell_size / 4;
> +
> return 0;
> }
>
> --
> 2.34.1
More information about the amd-gfx
mailing list