[AMD Official Use Only - General] > -----Original Message----- > From: Yadav, Arvind <arvind.ya...@amd.com> > Sent: Thursday, September 28, 2023 5:54 AM > To: Koenig, Christian <christian.koe...@amd.com>; Deucher, Alexander > <alexander.deuc...@amd.com>; Sharma, Shashank > <shashank.sha...@amd.com>; Kuehling, Felix <felix.kuehl...@amd.com>; > Joshi, Mukul <mukul.jo...@amd.com>; Pan, Xinhui <xinhui....@amd.com>; > airl...@gmail.com; dan...@ffwll.ch > Cc: amd-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org; linux- > ker...@vger.kernel.org; Yadav, Arvind <arvind.ya...@amd.com>; Koenig, > Christian <christian.koe...@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.koe...@amd.com> > Cc: Alex Deucher <alexander.deuc...@amd.com> > Signed-off-by: Shashank Sharma <shashank.sha...@amd.com> > Signed-off-by: Arvind Yadav <arvind.ya...@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
