diff options
| author | Qiang Liu <liuqiang@kylinos.cn> | 2026-08-12 17:28:56 +0800 |
|---|---|---|
| committer | Andrew Morton <akpm@linux-foundation.org> | 2026-08-24 18:43:24 -0700 |
| commit | 0d0878fd7c9b49b8eff95c4426a2121c5f8f31cc (patch) | |
| tree | 5e8011bdcb5953786c67b159b54a756b28b5c1c1 /lib | |
| parent | f525001b3309a0decdcdcdaeceafd7ab9dc927fe (diff) | |
| download | linux-stable-0d0878fd7c9b49b8eff95c4426a2121c5f8f31cc.tar.gz linux-stable-0d0878fd7c9b49b8eff95c4426a2121c5f8f31cc.zip | |
lib/test_hmm: fix garbage pfn and wrong direction in devmem fault debug
Move pr_debug() inside the `if (dpage)` block to avoid printing garbage
pfn for NULL dpage, and correct the direction label from "sys to dev" to
"dev to sys".
Link: https://lore.kernel.org/20260812092856.55296-1-liuqiangneo@163.com
Signed-off-by: Qiang Liu <liuqiang@kylinos.cn>
Assisted-by: Qoder:Qwen-3.8-MAX-Preview
Cc: Jason Gunthorpe <jgg@ziepe.ca>
Cc: Leon Romanovsky <leon@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/test_hmm.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/test_hmm.c b/lib/test_hmm.c index 6205fb313bd0..7c4d10eae6fe 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)); } |