diff options
author | Marek Olšák <marek.olsak@amd.com> | 2019-01-22 15:44:54 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2019-02-01 00:33:00 -0500 |
commit | fe57085a36de5813ab63a8d178ccfb5f257f028e (patch) | |
tree | a33d93e816327e2344ca02715ee25f2bf88efd19 /drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | |
parent | a5c8e0524dbbe1107d81a1604da3d191b66ead6b (diff) |
drm/amdgpu: clean up memory/GDS/GWS/OA alignment code
- move all adjustments into one place
- specify GDS/GWS/OA alignment in basic units of the heaps
- it looks like GDS alignment was 1 instead of 4
Signed-off-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c index f4f00217546e..d21dd2f369da 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | |||
@@ -54,10 +54,6 @@ int amdgpu_gem_object_create(struct amdgpu_device *adev, unsigned long size, | |||
54 | 54 | ||
55 | memset(&bp, 0, sizeof(bp)); | 55 | memset(&bp, 0, sizeof(bp)); |
56 | *obj = NULL; | 56 | *obj = NULL; |
57 | /* At least align on page size */ | ||
58 | if (alignment < PAGE_SIZE) { | ||
59 | alignment = PAGE_SIZE; | ||
60 | } | ||
61 | 57 | ||
62 | bp.size = size; | 58 | bp.size = size; |
63 | bp.byte_align = alignment; | 59 | bp.byte_align = alignment; |
@@ -244,9 +240,6 @@ int amdgpu_gem_create_ioctl(struct drm_device *dev, void *data, | |||
244 | return -EINVAL; | 240 | return -EINVAL; |
245 | } | 241 | } |
246 | flags |= AMDGPU_GEM_CREATE_NO_CPU_ACCESS; | 242 | flags |= AMDGPU_GEM_CREATE_NO_CPU_ACCESS; |
247 | /* GDS allocations must be DW aligned */ | ||
248 | if (args->in.domains & AMDGPU_GEM_DOMAIN_GDS) | ||
249 | size = ALIGN(size, 4); | ||
250 | } | 243 | } |
251 | 244 | ||
252 | if (flags & AMDGPU_GEM_CREATE_VM_ALWAYS_VALID) { | 245 | if (flags & AMDGPU_GEM_CREATE_VM_ALWAYS_VALID) { |