diff options
author | alc <alc@FreeBSD.org> | 2015-09-26 17:45:10 +0000 |
---|---|---|
committer | alc <alc@FreeBSD.org> | 2015-09-26 17:45:10 +0000 |
commit | 5b11ecb2fe08c58c8c245f3184b9e282d34d383f (patch) | |
tree | fd9f8e1db963ebf11fa162ad6e17280f6a8fc3b4 /sys | |
parent | fa0316d02ecef0c6a7fd8388860dc683514ae069 (diff) | |
download | FreeBSD-src-5b11ecb2fe08c58c8c245f3184b9e282d34d383f.zip FreeBSD-src-5b11ecb2fe08c58c8c245f3184b9e282d34d383f.tar.gz |
Exploit r288122 to address a cosmetic issue. Since the pages allocated
by noobj_alloc() don't belong to a vm object, they can't be paged out.
Since they can't be paged out, they are never enqueued in a paging queue.
Nonetheless, passing PQ_INACTIVE to vm_page_unwire() creates the appearance
that these pages are being enqueued in the inactive queue. As of r288122,
we can avoid giving this false impression by passing PQ_NONE.
Submitted by: kmacy
Differential Revision: https://reviews.freebsd.org/D1674
Diffstat (limited to 'sys')
-rw-r--r-- | sys/vm/uma_core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/vm/uma_core.c b/sys/vm/uma_core.c index 2a4a2cd..8169256 100644 --- a/sys/vm/uma_core.c +++ b/sys/vm/uma_core.c @@ -1153,7 +1153,7 @@ noobj_alloc(uma_zone_t zone, vm_size_t bytes, uint8_t *flags, int wait) * exit. */ TAILQ_FOREACH_SAFE(p, &alloctail, listq, p_next) { - vm_page_unwire(p, PQ_INACTIVE); + vm_page_unwire(p, PQ_NONE); vm_page_free(p); } return (NULL); |