Am 18.07.19 um 18:53 schrieb StDenis, Tom:
The specification says to treat a PTE with the F bit set "like a PDE"
which means that all but the lower 6 bits are part of the page base
address. Indeed, in the wild a comment came back indicating that
we were stripping off bits needed to properly fetch the next
PTE.
(v2): Only capture excess bits if it is a PTE-FURTHER entry
Signed-off-by: Tom St Denis <tom.stdenis@xxxxxxx>
Acked-by: Christian König <christian.koenig@xxxxxxx>
---
src/lib/read_vram.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/lib/read_vram.c b/src/lib/read_vram.c
index cba8f6b..e83a85c 100644
--- a/src/lib/read_vram.c
+++ b/src/lib/read_vram.c
@@ -676,7 +676,7 @@ pde_is_pte:
pte_idx = (address >> 12) & ((1ULL << pde_fields.frag_size) - 1);
// grab PTE base address from the PTE that has the F bit set.
- pde_fields.pte_base_addr = pte_fields.page_base_addr;
+ pde_fields.pte_base_addr = pte_entry & 0xFFFFFFFFFFC0ULL;
goto pte_further;
}
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
