From ee06cffcd2d7940ae1ffd8e2d50704203fbfc439 Mon Sep 17 00:00:00 2001 From: Konstantin Belousov Date: Sat, 27 Jun 2020 02:31:39 +0000 Subject: [PATCH] vm_page_free_prep(): correct description of the required page and object state. Reviewed by: markj Sponsored by: The FreeBSD Foundation Differential revision: https://reviews.freebsd.org/D25482 --- sys/vm/vm_page.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c index 22eacf423b95..08420d5da19e 100644 --- a/sys/vm/vm_page.c +++ b/sys/vm/vm_page.c @@ -3666,8 +3666,9 @@ vm_page_enqueue(vm_page_t m, uint8_t queue) * disassociating it from any VM object. The caller may return * the page to the free list only if this function returns true. * - * The object must be locked. The page must be locked if it is - * managed. + * The object, if it exists, must be locked, and then the page must + * be xbusy. Otherwise the page must be not busied. A managed + * page must be unmapped. */ static bool vm_page_free_prep(vm_page_t m)