On Wed, Nov 22, 2023 at 2:32 AM Alex Deucher <alexander.deucher@xxxxxxx> wrote:
>
> Return AMDGPU_BO_INVALID_OFFSET if the AGP aperture is disabled.
> There is no reason to check further if the aperture is disabled.
>
> Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
Ping?
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c
> index 5f71414190e9..0f85f906791b 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c
> @@ -181,6 +181,9 @@ uint64_t amdgpu_gmc_agp_addr(struct ttm_buffer_object *bo)
> {
> struct amdgpu_device *adev = amdgpu_ttm_adev(bo->bdev);
>
> + if (!adev->gmc.agp_size)
> + return AMDGPU_BO_INVALID_OFFSET;
> +
> if (bo->ttm->num_pages != 1 || bo->ttm->caching == ttm_cached)
> return AMDGPU_BO_INVALID_OFFSET;
>
> --
> 2.42.0
>
