diff options
| author | Bjoern A. Zeeb <bz@FreeBSD.org> | 2021-08-22 17:21:01 +0000 |
|---|---|---|
| committer | Bjoern A. Zeeb <bz@FreeBSD.org> | 2021-08-22 17:43:12 +0000 |
| commit | eccb516db8b7fec957b139c3a9a2e16175551067 (patch) | |
| tree | 61995f62db08bb81b712a018af4495941acd86b2 /sys/vm | |
| parent | 614faa32691bf9ccd2690624ad37470e08fd14d3 (diff) | |
| download | src-eccb516db8b7fec957b139c3a9a2e16175551067.tar.gz src-eccb516db8b7fec957b139c3a9a2e16175551067.zip | |
vm: use __func__ for the correct function name
In fee2a2fa39834d8d5eaa981298fce9d2ed31546d the KASSERTs in
vm_page_unwire_noq() changed from "vm_page_unwire" to "vm_page_unref".
While the former no longer was part of that function the latter does
not exist as a function and is highly confusing when hit when using
tools to lookup the functions and not doing a full-text search.
Use %s __func__ for printing the function name, as that will do the
right thing as code moves around and functions get renamed.
Hit: while debugging a wired page leak with linuxkpi/iwlwifi
Sponsored by: The FreeBSD Foundation
Reviewed by: markj
Differential Revision: https://reviews.freebsd.org/D31635
Diffstat (limited to 'sys/vm')
| -rw-r--r-- | sys/vm/vm_page.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c index d435b871b7cb..4ecea30e9219 100644 --- a/sys/vm/vm_page.c +++ b/sys/vm/vm_page.c @@ -4050,9 +4050,9 @@ vm_page_unwire_noq(vm_page_t m) old = vm_page_drop(m, 1); KASSERT(VPRC_WIRE_COUNT(old) != 0, - ("vm_page_unref: counter underflow for page %p", m)); + ("%s: counter underflow for page %p", __func__, m)); KASSERT((m->flags & PG_FICTITIOUS) == 0 || VPRC_WIRE_COUNT(old) > 1, - ("vm_page_unref: missing ref on fictitious page %p", m)); + ("%s: missing ref on fictitious page %p", __func__, m)); if (VPRC_WIRE_COUNT(old) > 1) return (false); |