From 7a364d458ac518befc5ee5aa794195668b86f535 Mon Sep 17 00:00:00 2001 From: Mark Johnston Date: Mon, 10 Sep 2018 18:59:23 +0000 Subject: [PATCH] Split some checks in vm_page_activate() to make it easier to read. No functional change intended. Reviewed by: alc, kib Approved by: re (gjb) Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D17028 --- sys/vm/vm_page.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c index 23b2cb4873ad..d022435704f1 100644 --- a/sys/vm/vm_page.c +++ b/sys/vm/vm_page.c @@ -3406,13 +3406,13 @@ vm_page_requeue(vm_page_t m) void vm_page_activate(vm_page_t m) { - int queue; vm_page_assert_locked(m); - if ((queue = vm_page_queue(m)) == PQ_ACTIVE || m->wire_count > 0 || - (m->oflags & VPO_UNMANAGED) != 0) { - if (queue == PQ_ACTIVE && m->act_count < ACT_INIT) + if (m->wire_count > 0 || (m->oflags & VPO_UNMANAGED) != 0) + return; + if (vm_page_queue(m) == PQ_ACTIVE) { + if (m->act_count < ACT_INIT) m->act_count = ACT_INIT; return; }