[PATCH] drm/amdkfd: Delete duplicated queue bit map reservation

Kuehling, Felix Felix.Kuehling at amd.com
Mon Oct 28 16:49:18 UTC 2019


On 2019-10-24 5:14 p.m., Zhao, Yong wrote:
> The KIQ is on the second MEC and its reservation is covered in the
> latter logic, so no need to reserve its bit twice.
>
> Change-Id: Ieee390953a60c7d43de5a9aec38803f1f583a4a9
> Signed-off-by: Yong Zhao <Yong.Zhao at amd.com>

Reviewed-by: Felix Kuehling <Felix.Kuehling at amd.com>


> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c | 8 --------
>   1 file changed, 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c
> index 8c531793fe17..d3da9dde4ee1 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c
> @@ -130,14 +130,6 @@ void amdgpu_amdkfd_device_init(struct amdgpu_device *adev)
>   				  adev->gfx.mec.queue_bitmap,
>   				  KGD_MAX_QUEUES);
>   
> -		/* remove the KIQ bit as well */
> -		if (adev->gfx.kiq.ring.sched.ready)
> -			clear_bit(amdgpu_gfx_mec_queue_to_bit(adev,
> -							  adev->gfx.kiq.ring.me - 1,
> -							  adev->gfx.kiq.ring.pipe,
> -							  adev->gfx.kiq.ring.queue),
> -				  gpu_resources.queue_bitmap);
> -
>   		/* According to linux/bitmap.h we shouldn't use bitmap_clear if
>   		 * nbits is not compile time constant
>   		 */


More information about the amd-gfx mailing list