vm_fault: expand KASSERT message in vm_fault_populate_cleanup

isoux · isoux · commit c78c1ea6e058 · 2026-02-20T22:10:17.000+02:00
Include index values and object pointer in the diagnostic
to improve debugging of pindex mismatches.

No functional change.

Signed-off-by: Isa Isoux &lt;isa@isoux.org&gt;
diff --git a/sys/vm/vm_fault.c b/sys/vm/vm_fault.c
@@ -509,7 +509,13 @@ vm_fault_populate_cleanup(vm_object_t object, vm_pindex_t first,
 		vm_page_deactivate(m);
 		vm_page_xunbusy(m);
 	}
-	KASSERT(pages.index == last, ("%s: pindex mismatch", __func__));
+	KASSERT(pages.index == last,
+	    ("%s: pindex mismatch index=%jx last=%jx first=%jx object=%p",
+	    __func__,
+	    (uintmax_t)pages.index,
+	    (uintmax_t)last,
+	    (uintmax_t)first,
+	    object));
 }
 
 static enum fault_status

Original file line number	Diff line number	Diff line change
`@@ -509,7 +509,13 @@ vm_fault_populate_cleanup(vm_object_t object, vm_pindex_t first,`
`509`	`509`	`vm_page_deactivate(m);`
`510`	`510`	`vm_page_xunbusy(m);`
`511`	`511`	`}`
`512`		`- KASSERT(pages.index == last, ("%s: pindex mismatch", __func__));`
	`512`	`+ KASSERT(pages.index == last,`
	`513`	`+ ("%s: pindex mismatch index=%jx last=%jx first=%jx object=%p",`
	`514`	`+ __func__,`
	`515`	`+ (uintmax_t)pages.index,`
	`516`	`+ (uintmax_t)last,`
	`517`	`+ (uintmax_t)first,`
	`518`	`+ object));`
`513`	`519`	`}`
`514`	`520`
`515`	`521`	`static enum fault_status`