[PATCH 11/12] drm/amdgpu: remove maximum BO size limitation.

Michel Dänzer michel at daenzer.net
Mon Jul 3 03:14:57 UTC 2017


On 30/06/17 08:22 PM, Christian König wrote:
> From: Christian König <christian.koenig at amd.com>
> 
> We can finally remove this now.
> 
> Signed-off-by: Christian König <christian.koenig at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 11 -----------
>  1 file changed, 11 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> index 96c4493..2382785 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> @@ -58,17 +58,6 @@ int amdgpu_gem_object_create(struct amdgpu_device *adev, unsigned long size,
>  		alignment = PAGE_SIZE;
>  	}
>  
> -	if (!(initial_domain & (AMDGPU_GEM_DOMAIN_GDS | AMDGPU_GEM_DOMAIN_GWS | AMDGPU_GEM_DOMAIN_OA))) {
> -		/* Maximum bo size is the unpinned gtt size since we use the gtt to
> -		 * handle vram to system pool migrations.
> -		 */
> -		max_size = adev->mc.gtt_size - adev->gart_pin_size;
> -		if (size > max_size) {
> -			DRM_DEBUG("Allocation size %ldMb bigger than %ldMb limit\n",
> -				  size >> 20, max_size >> 20);
> -			return -ENOMEM;
> -		}
> -	}

Do we still need to limit the size of VRAM BOs?

If not, remove the now unused max_size local.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the amd-gfx mailing list