<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<p><br>
</p>
<div class="moz-cite-prefix">On 2023-10-12 12:39, David Francis
wrote:<br>
</div>
<blockquote type="cite" cite="mid:20231012163941.553980-1-David.Francis@amd.com">
<pre class="moz-quote-pre" wrap="">On NUMA systems, local memory gets the local mtype, set by an
override callback. If EXT_COHERENT is set, memory will be set as
MTYPE_UC by default, with local memory MTYPE_CC.
Add an option in the override function for this case, and
add a check to ensure it is not used on UNCACHED memory.
Signed-off-by: David Francis <a class="moz-txt-link-rfc2396E" href="mailto:David.Francis@amd.com"><David.Francis@amd.com></a>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 13 +++++----
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h | 8 +++++-
drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +-
drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 33 +++++++++++++++--------
drivers/gpu/drm/amd/amdkfd/kfd_svm.c | 6 ++---
5 files changed, 41 insertions(+), 21 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index 46d27c8ffff7..189341944bf1 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
@@ -761,6 +761,7 @@ static void amdgpu_vm_tlb_seq_cb(struct dma_fence *fence,
* @immediate: immediate submission in a page fault
* @unlocked: unlocked invalidation during MM callback
* @flush_tlb: trigger tlb invalidation after update completed
+ * @allow_override: change MTYPE for local NUMA nodes
* @resv: fences we need to sync to
* @start: start of mapped range
* @last: last mapped entry
@@ -777,7 +778,7 @@ static void amdgpu_vm_tlb_seq_cb(struct dma_fence *fence,
* 0 for success, negative erro code for failure.
*/
int amdgpu_vm_update_range(struct amdgpu_device *adev, struct amdgpu_vm *vm,
- bool immediate, bool unlocked, bool flush_tlb,
+ bool immediate, bool unlocked, bool flush_tlb, bool allow_override,
struct dma_resv *resv, uint64_t start, uint64_t last,
uint64_t flags, uint64_t offset, uint64_t vram_base,
struct ttm_resource *res, dma_addr_t *pages_addr,
@@ -815,6 +816,7 @@ int amdgpu_vm_update_range(struct amdgpu_device *adev, struct amdgpu_vm *vm,
params.immediate = immediate;
params.pages_addr = pages_addr;
params.unlocked = unlocked;
+ params.allow_override = allow_override;
/* Implicitly sync to command submissions in the same VM before
* unmapping. Sync to moving fences before mapping.
@@ -1062,6 +1064,7 @@ int amdgpu_vm_bo_update(struct amdgpu_device *adev, struct amdgpu_bo_va *bo_va,
trace_amdgpu_vm_bo_update(mapping);
r = amdgpu_vm_update_range(adev, vm, false, false, flush_tlb,
+ !(bo->flags & AMDGPU_GEM_CREATE_UNCACHED),
resv, mapping->start, mapping->last,
update_flags, mapping->offset,
vram_base, mem, pages_addr,
@@ -1257,8 +1260,8 @@ int amdgpu_vm_clear_freed(struct amdgpu_device *adev,
mapping->start < AMDGPU_GMC_HOLE_START)
init_pte_value = AMDGPU_PTE_DEFAULT_ATC;
- r = amdgpu_vm_update_range(adev, vm, false, false, true, resv,
- mapping->start, mapping->last,
+ r = amdgpu_vm_update_range(adev, vm, false, false, true, false,
+ resv, mapping->start, mapping->last,
init_pte_value, 0, 0, NULL, NULL,
&f);
amdgpu_vm_free_mapping(adev, vm, mapping, f);
@@ -2546,8 +2549,8 @@ bool amdgpu_vm_handle_fault(struct amdgpu_device *adev, u32 pasid,
goto error_unlock;
}
- r = amdgpu_vm_update_range(adev, vm, true, false, false, NULL, addr,
- addr, flags, value, 0, NULL, NULL, NULL);
+ r = amdgpu_vm_update_range(adev, vm, true, false, false, false,
+ NULL, addr, addr, flags, value, 0, NULL, NULL, NULL);
if (r)
goto error_unlock;
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
index 6e71978db13f..9ea8b5b644e3 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h
@@ -244,6 +244,12 @@ struct amdgpu_vm_update_params {
* @table_freed: return true if page table is freed when updating
*/
bool table_freed;
+
+ /**
+ * @allow_override: true for memory that is not uncached: allows MTYPE
+ * to be overridden for NUMA local memory.
+ */
+ bool allow_override;
};
struct amdgpu_vm_update_funcs {
@@ -436,7 +442,7 @@ int amdgpu_vm_handle_moved(struct amdgpu_device *adev,
void amdgpu_vm_bo_base_init(struct amdgpu_vm_bo_base *base,
struct amdgpu_vm *vm, struct amdgpu_bo *bo);
int amdgpu_vm_update_range(struct amdgpu_device *adev, struct amdgpu_vm *vm,
- bool immediate, bool unlocked, bool flush_tlb,
+ bool immediate, bool unlocked, bool flush_tlb, bool allow_override,
struct dma_resv *resv, uint64_t start, uint64_t last,
uint64_t flags, uint64_t offset, uint64_t vram_base,
struct ttm_resource *res, dma_addr_t *pages_addr,
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c
index 9b025fd17b84..80273809c93f 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c
@@ -842,7 +842,7 @@ static void amdgpu_vm_pte_update_flags(struct amdgpu_vm_update_params *params,
* to be on the same NUMA node.
*/
if ((flags & AMDGPU_PTE_SYSTEM) && (adev->flags & AMD_IS_APU) &&
- adev->gmc.gmc_funcs->override_vm_pte_flags &&
+ params->allow_override && adev->gmc.gmc_funcs->override_vm_pte_flags &&
num_possible_nodes() > 1 && !params->pages_addr)
amdgpu_gmc_override_vm_pte_flags(adev, params->vm, addr, &flags);
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
index 3a1050344b59..1456f76b7fb2 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -1251,12 +1251,15 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev,
return;
}
- /* Only override mappings with MTYPE_NC, which is the safe default for
- * cacheable memory.
+ /* MTYPE_NC is the same default and can be overridden.
+ * MTYPE_UC will be present if the memory is external-coherent
+ * and can also be overridden.
*/
if ((*flags & AMDGPU_PTE_MTYPE_VG10_MASK) !=
- AMDGPU_PTE_MTYPE_VG10(MTYPE_NC)) {
- dev_dbg_ratelimited(adev->dev, "MTYPE is not NC\n");
+ AMDGPU_PTE_MTYPE_VG10(MTYPE_NC) &&
+ (*flags & AMDGPU_PTE_MTYPE_VG10_MASK) !=
+ AMDGPU_PTE_MTYPE_VG10(MTYPE_UC)) {
+ dev_dbg_ratelimited(adev->dev, "MTYPE is not NC or UC\n");
return;
}
@@ -1283,15 +1286,23 @@ static void gmc_v9_0_override_vm_pte_flags(struct amdgpu_device *adev,
vm->mem_id, local_node, nid);
if (nid == local_node) {
uint64_t old_flags = *flags;
- unsigned int mtype_local = MTYPE_RW;
+ if ((*flags & AMDGPU_PTE_MTYPE_VG10_MASK) ==
+ AMDGPU_PTE_MTYPE_VG10(MTYPE_NC)) {
+ unsigned int mtype_local = MTYPE_RW;
- if (amdgpu_mtype_local == 1)
- mtype_local = MTYPE_NC;
- else if (amdgpu_mtype_local == 2)
- mtype_local = MTYPE_CC;
+ if (amdgpu_mtype_local == 1)
+ mtype_local = MTYPE_NC;
+ else if (amdgpu_mtype_local == 2)
+ mtype_local = MTYPE_CC;
+
+ *flags = (*flags & ~AMDGPU_PTE_MTYPE_VG10_MASK) |
+ AMDGPU_PTE_MTYPE_VG10(mtype_local);
+ } else {
+ /* MTYPE_UC case */
+ *flags = (*flags & ~AMDGPU_PTE_MTYPE_VG10_MASK) |
+ AMDGPU_PTE_MTYPE_VG10(MTYPE_CC);
+ }
- *flags = (*flags & ~AMDGPU_PTE_MTYPE_VG10_MASK) |
- AMDGPU_PTE_MTYPE_VG10(mtype_local);
dev_dbg_ratelimited(adev->dev, "flags updated from %llx to %llx\n",
old_flags, *flags);
}
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
index 7b81233bc9ae..bbaec633a806 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_svm.c
@@ -1321,7 +1321,7 @@ svm_range_unmap_from_gpu(struct amdgpu_device *adev, struct amdgpu_vm *vm,
pr_debug("[0x%llx 0x%llx]\n", start, last);
- return amdgpu_vm_update_range(adev, vm, false, true, true, NULL, start,
+ return amdgpu_vm_update_range(adev, vm, false, true, true, false, NULL, start,
last, init_pte_value, 0, 0, NULL, NULL,
fence);
}</pre>
</blockquote>
<p>This change from the last version is still needed, right?<br>
</p>
<p><span style="font-family: "Segoe UI", "Segoe UI
Web (West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">@@ -1286,7 +1286,7 @@
svm_range_get_pte_flags(struct kfd_node *node,</span><br style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; color: rgb(0, 0, 0);
background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0"> if
(num_possible_nodes() <= 1)</span><br style="font-family:
"Segoe UI", "Segoe UI Web (West European)",
"Segoe UI", -apple-system, BlinkMacSystemFont, Roboto,
"Helvetica Neue", sans-serif; font-size: 14.6667px;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">
mapping_flags |= mtype_local;</span><br style="font-family:
"Segoe UI", "Segoe UI Web (West European)",
"Segoe UI", -apple-system, BlinkMacSystemFont, Roboto,
"Helvetica Neue", sans-serif; font-size: 14.6667px;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0"> else</span><br style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; color: rgb(0, 0, 0);
background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">-
mapping_flags |= AMDGPU_VM_MTYPE_NC;</span><br style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; color: rgb(0, 0, 0);
background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">+
mapping_flags |= ext_coherent ? AMDGPU_VM_MTYPE_UC :
AMDGPU_VM_MTYPE_NC;</span><br style="font-family: "Segoe
UI", "Segoe UI Web (West European)", "Segoe
UI", -apple-system, BlinkMacSystemFont, Roboto,
"Helvetica Neue", sans-serif; font-size: 14.6667px;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0"> /* system memory
accessed by the dGPU */</span><br style="font-family:
"Segoe UI", "Segoe UI Web (West European)",
"Segoe UI", -apple-system, BlinkMacSystemFont, Roboto,
"Helvetica Neue", sans-serif; font-size: 14.6667px;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0"> } else {</span><br style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; color: rgb(0, 0, 0);
background-color: rgb(255, 255, 255);" class="ContentPasted0">
<span style="font-family: "Segoe UI", "Segoe UI Web
(West European)", "Segoe UI", -apple-system,
BlinkMacSystemFont, Roboto, "Helvetica Neue",
sans-serif; font-size: 14.6667px; display: inline !important;
color: rgb(0, 0, 0); background-color: rgb(255, 255, 255);" class="ContentPasted0"> mapping_flags |=
AMDGPU_VM_MTYPE_UC;</span></p>
<blockquote type="cite" cite="mid:20231012163941.553980-1-David.Francis@amd.com">
<pre class="moz-quote-pre" wrap="">
@@ -1428,8 +1428,8 @@ svm_range_map_to_gpu(struct kfd_process_device *pdd, struct svm_range *prange,
* different memory partition based on fpfn/lpfn, we should use
* same vm_manager.vram_base_offset regardless memory partition.
*/
- r = amdgpu_vm_update_range(adev, vm, false, false, flush_tlb, NULL,
- last_start, prange->start + i,
+ r = amdgpu_vm_update_range(adev, vm, false, false, flush_tlb, true,
+ NULL, last_start, prange->start + i,
pte_flags,
(last_start - prange->start) << PAGE_SHIFT,
bo_adev ? bo_adev->vm_manager.vram_base_offset : 0,
</pre>
</blockquote>
</body>
</html>