aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKonstantin Belousov <kib@FreeBSD.org>2021-09-14 12:25:37 +0000
committerKonstantin Belousov <kib@FreeBSD.org>2021-09-21 13:31:27 +0000
commit62faf669ac4b779d7444e72ad97ebfefbfaaaf63 (patch)
tree71b8b080ab750d042fe27da128a04c7b0221bc07
parent3917b3c7e5e18cccd3216bc3925efb0bc1e8affe (diff)
downloadsrc-62faf669ac4b779d7444e72ad97ebfefbfaaaf63.tar.gz
src-62faf669ac4b779d7444e72ad97ebfefbfaaaf63.zip
vm_phys: do not ignore phys_avail[] segments that do not fit completely into vm_phys segments
(cherry picked from commit 181bfb42fd01bfa9f4636e803ccb3eeed8ac8ba4)
-rw-r--r--sys/vm/vm_page.c25
1 files changed, 12 insertions, 13 deletions
diff --git a/sys/vm/vm_page.c b/sys/vm/vm_page.c
index aae45513a44c..dc56ca9bae05 100644
--- a/sys/vm/vm_page.c
+++ b/sys/vm/vm_page.c
@@ -552,11 +552,12 @@ vm_offset_t
vm_page_startup(vm_offset_t vaddr)
{
struct vm_phys_seg *seg;
+ struct vm_domain *vmd;
vm_page_t m;
char *list, *listend;
vm_paddr_t end, high_avail, low_avail, new_end, size;
vm_paddr_t page_range __unused;
- vm_paddr_t last_pa, pa;
+ vm_paddr_t last_pa, pa, startp, endp;
u_long pagecount;
#if MINIDUMP_PAGE_TRACKING
u_long vm_page_dump_size;
@@ -770,21 +771,20 @@ vm_page_startup(vm_offset_t vaddr)
vm_page_init_page(m, pa, segind);
/*
- * Add the segment to the free lists only if it is covered by
- * one of the ranges in phys_avail. Because we've added the
- * ranges to the vm_phys_segs array, we can assume that each
- * segment is either entirely contained in one of the ranges,
- * or doesn't overlap any of them.
+ * Add the segment's pages that are covered by one of
+ * phys_avail's ranges to the free lists.
*/
for (i = 0; phys_avail[i + 1] != 0; i += 2) {
- struct vm_domain *vmd;
-
- if (seg->start < phys_avail[i] ||
- seg->end > phys_avail[i + 1])
+ if (seg->end < phys_avail[i] ||
+ seg->start >= phys_avail[i + 1])
continue;
- m = seg->first_page;
- pagecount = (u_long)atop(seg->end - seg->start);
+ startp = MAX(seg->start, phys_avail[i]);
+ m = seg->first_page + atop(seg->start - startp);
+ endp = MIN(seg->end, phys_avail[i + 1]);
+ pagecount = (u_long)atop(endp - startp);
+ if (pagecount == 0)
+ continue;
vmd = VM_DOMAIN(seg->domain);
vm_domain_free_lock(vmd);
@@ -796,7 +796,6 @@ vm_page_startup(vm_offset_t vaddr)
vmd = VM_DOMAIN(seg->domain);
vmd->vmd_page_count += (u_int)pagecount;
vmd->vmd_segs |= 1UL << m->segind;
- break;
}
}