diff options
| author | Felix Kuehling <Felix.Kuehling@amd.com> | 2017-03-28 20:31:20 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2017-03-29 23:55:52 -0400 |
| commit | 9ceaeeafbd1177e53395946a772a2a9c4c234887 (patch) | |
| tree | a0480c7ae158e1b800f25cc3de79d2d286712f25 /drivers/gpu/drm/amd | |
| parent | 22770e5a750e905a1aaa66e20b88d080e42ddd15 (diff) | |
| download | op-kernel-dev-9ceaeeafbd1177e53395946a772a2a9c4c234887.zip op-kernel-dev-9ceaeeafbd1177e53395946a772a2a9c4c234887.tar.gz | |
drm/amdgpu: Fix Vega10 VM initialization
adev->family is not initialized yet when amdgpu_get_block_size is
called. Use adev->asic_type instead.
Minimum VM size is 512GB, not 256GB, for a single page table entry
in the root page table.
gmc_v9_0_vm_init is called after adev->vm_manager.max_pfn is
initialized. Move the minimum VM-size enforcement ahead of max_pfn
initializtion. Cast to 64-bit before the left-shift.
Signed-off-by: Felix Kuehling <Felix.Kuehling@amd.com>
Reviewed-by: Chunming Zhou <david1.zhou@amd.com>
Reviewed-by: Junwei Zhang <Jerry.Zhang@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 | ||||
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 17 |
2 files changed, 10 insertions, 13 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c index ae31744..93061a43 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c @@ -1043,10 +1043,10 @@ static bool amdgpu_check_pot_argument(int arg) static void amdgpu_get_block_size(struct amdgpu_device *adev) { /* from AI, asic starts to support multiple level VMPT */ - if (adev->family >= AMDGPU_FAMILY_AI) { + if (adev->asic_type >= CHIP_VEGA10) { if (amdgpu_vm_block_size != 9) - dev_warn(adev->dev, "Multi-VMPT limits block size to" - "one page!\n"); + dev_warn(adev->dev, + "Multi-VMPT limits block size to one page!\n"); amdgpu_vm_block_size = 9; return; } diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c index 1e4734d..df69aae 100644 --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c @@ -511,12 +511,6 @@ static int gmc_v9_0_vm_init(struct amdgpu_device *adev) * amdkfd will use VMIDs 8-15 */ adev->vm_manager.num_ids = AMDGPU_NUM_OF_VMIDS; - /* Because of four level VMPTs, vm size at least is 256GB. - 256TB is OK as well */ - if (amdgpu_vm_size < 256) { - DRM_WARN("vm size at least is 256GB!\n"); - amdgpu_vm_size = 256; - } adev->vm_manager.num_level = 3; amdgpu_vm_manager_init(adev); @@ -563,11 +557,14 @@ static int gmc_v9_0_sw_init(void *handle) if (r) return r; - /* Adjust VM size here. - * Currently default to 64GB ((16 << 20) 4k pages). - * Max GPUVM size is 48 bits. + /* Because of four level VMPTs, vm size is at least 512GB. + * The maximum size is 256TB (48bit). */ - adev->vm_manager.max_pfn = amdgpu_vm_size << 18; + if (amdgpu_vm_size < 512) { + DRM_WARN("VM size is at least 512GB!\n"); + amdgpu_vm_size = 512; + } + adev->vm_manager.max_pfn = (uint64_t)amdgpu_vm_size << 18; /* Set the internal MC address mask * This is the max address of the GPU's |
