From: Qiang Liu Move pr_debug() inside the `if (dpage)` block to avoid a NULL deref, and fix the direction label from "sys to dev" to "dev to sys" to match the device-to-system copy. Assisted-by: Qoder:Qwen-3.8-MAX-Preview Signed-off-by: Qiang Liu --- lib/test_hmm.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/test_hmm.c b/lib/test_hmm.c index 6e72a0d8a53d..1dae93ab1e93 100644 --- a/lib/test_hmm.c +++ b/lib/test_hmm.c @@ -1151,10 +1151,9 @@ static vm_fault_t dmirror_devmem_fault_alloc_and_copy(struct migrate_vma *args, if (!dpage && !order) return VM_FAULT_OOM; - pr_debug("migrating from sys to dev pfn src: 0x%lx pfn dst: 0x%lx\n", - page_to_pfn(spage), page_to_pfn(dpage)); - if (dpage) { + pr_debug("migrating from dev to sys pfn src: 0x%lx pfn dst: 0x%lx\n", + page_to_pfn(spage), page_to_pfn(dpage)); lock_page(dpage); *dst |= migrate_pfn(page_to_pfn(dpage)); } -- 2.43.0 No virus found Checked by Hillstone Network AntiVirus