diff options
| author | Austin Shafer <ashafer@FreeBSD.org> | 2025-12-09 15:33:17 +0000 |
|---|---|---|
| committer | Austin Shafer <ashafer@FreeBSD.org> | 2025-12-10 16:00:08 +0000 |
| commit | 4ae02d9792272fa8c43f4982cf338123d9bcaed2 (patch) | |
| tree | 55bebb90b7c0938123e13016bcaf99f14ac36441 | |
| parent | 255af72c8059b0117db646f82efa2e4848fa7570 (diff) | |
linuxkpi: clean up stray pctrie_iter_reset
This removes an extraneous pctrie_iter_reset before returning.
This is not needed as it simply clears a local variable that
will get cleaned up anyway as we immediately return from the
function.
MFC after: 1 week
Sponsored by: NVIDIA
Reviewed by: alc
Differential Revision: https://reviews.freebsd.org/D54153
| -rw-r--r-- | sys/compat/linuxkpi/common/src/linux_page.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/sys/compat/linuxkpi/common/src/linux_page.c b/sys/compat/linuxkpi/common/src/linux_page.c index 2b2827579cba..c0d9d71ba1ce 100644 --- a/sys/compat/linuxkpi/common/src/linux_page.c +++ b/sys/compat/linuxkpi/common/src/linux_page.c @@ -345,10 +345,8 @@ retry: page = vm_page_grab_iter(vm_obj, pindex, VM_ALLOC_NOCREAT, &pages); if (page == NULL) { page = PHYS_TO_VM_PAGE(IDX_TO_OFF(pfn)); - if (page == NULL) { - pctrie_iter_reset(&pages); + if (page == NULL) return (VM_FAULT_SIGBUS); - } if (!vm_page_busy_acquire(page, VM_ALLOC_WAITFAIL)) { pctrie_iter_reset(&pages); goto retry; |