Commit fdac8909 authored by Philip Yang's avatar Philip Yang Committed by Alex Deucher

drm/amdgpu: ratelimited override pte flags messages

Use ratelimited version of dev_dbg to avoid flooding dmesg log. No
functional change.
Signed-off-by: default avatarPhilip Yang <Philip.Yang@amd.com>
Reviewed-by: default avatarFelix Kuehling <Felix.Kuehling@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent e1864006
...@@ -843,14 +843,8 @@ static void amdgpu_vm_pte_update_flags(struct amdgpu_vm_update_params *params, ...@@ -843,14 +843,8 @@ static void amdgpu_vm_pte_update_flags(struct amdgpu_vm_update_params *params,
*/ */
if ((flags & AMDGPU_PTE_SYSTEM) && (adev->flags & AMD_IS_APU) && if ((flags & AMDGPU_PTE_SYSTEM) && (adev->flags & AMD_IS_APU) &&
adev->gmc.gmc_funcs->override_vm_pte_flags && adev->gmc.gmc_funcs->override_vm_pte_flags &&
num_possible_nodes() > 1) { num_possible_nodes() > 1 && !params->pages_addr)
if (!params->pages_addr) amdgpu_gmc_override_vm_pte_flags(adev, params->vm, addr, &flags);
amdgpu_gmc_override_vm_pte_flags(adev, params->vm,
addr, &flags);
else
dev_dbg(adev->dev,
"override_vm_pte_flags skipped: non-contiguous\n");
}
params->vm->update_funcs->update(params, pt, pe, addr, count, incr, params->vm->update_funcs->update(params, pt, pe, addr, count, incr,
flags); flags);
......
...@@ -1247,7 +1247,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev, ...@@ -1247,7 +1247,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev,
* the DMA address. * the DMA address.
*/ */
if (!adev->ram_is_direct_mapped) { if (!adev->ram_is_direct_mapped) {
dev_dbg(adev->dev, "RAM is not direct mapped\n"); dev_dbg_ratelimited(adev->dev, "RAM is not direct mapped\n");
return; return;
} }
...@@ -1256,7 +1256,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev, ...@@ -1256,7 +1256,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev,
*/ */
if ((*flags & AMDGPU_PTE_MTYPE_VG10_MASK) != if ((*flags & AMDGPU_PTE_MTYPE_VG10_MASK) !=
AMDGPU_PTE_MTYPE_VG10(MTYPE_NC)) { AMDGPU_PTE_MTYPE_VG10(MTYPE_NC)) {
dev_dbg(adev->dev, "MTYPE is not NC\n"); dev_dbg_ratelimited(adev->dev, "MTYPE is not NC\n");
return; return;
} }
...@@ -1267,7 +1267,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev, ...@@ -1267,7 +1267,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev,
if (adev->gmc.is_app_apu && vm->mem_id >= 0) { if (adev->gmc.is_app_apu && vm->mem_id >= 0) {
local_node = adev->gmc.mem_partitions[vm->mem_id].numa.node; local_node = adev->gmc.mem_partitions[vm->mem_id].numa.node;
} else { } else {
dev_dbg(adev->dev, "Only native mode APU is supported.\n"); dev_dbg_ratelimited(adev->dev, "Only native mode APU is supported.\n");
return; return;
} }
...@@ -1275,11 +1275,11 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev, ...@@ -1275,11 +1275,11 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev,
* page or NUMA nodes. * page or NUMA nodes.
*/ */
if (!page_is_ram(addr >> PAGE_SHIFT)) { if (!page_is_ram(addr >> PAGE_SHIFT)) {
dev_dbg(adev->dev, "Page is not RAM.\n"); dev_dbg_ratelimited(adev->dev, "Page is not RAM.\n");
return; return;
} }
nid = pfn_to_nid(addr >> PAGE_SHIFT); nid = pfn_to_nid(addr >> PAGE_SHIFT);
dev_dbg(adev->dev, "vm->mem_id=%d, local_node=%d, nid=%d\n", dev_dbg_ratelimited(adev->dev, "vm->mem_id=%d, local_node=%d, nid=%d\n",
vm->mem_id, local_node, nid); vm->mem_id, local_node, nid);
if (nid == local_node) { if (nid == local_node) {
uint64_t old_flags = *flags; uint64_t old_flags = *flags;
...@@ -1292,7 +1292,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev, ...@@ -1292,7 +1292,7 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev,
*flags = (*flags & ~AMDGPU_PTE_MTYPE_VG10_MASK) | *flags = (*flags & ~AMDGPU_PTE_MTYPE_VG10_MASK) |
AMDGPU_PTE_MTYPE_VG10(mtype_local); AMDGPU_PTE_MTYPE_VG10(mtype_local);
dev_dbg(adev->dev, "flags updated from %llx to %llx\n", dev_dbg_ratelimited(adev->dev, "flags updated from %llx to %llx\n",
old_flags, *flags); old_flags, *flags);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment