Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/sys/uvm make ubc_winshift / ubc_winsize constant, and based ...
details: https://anonhg.NetBSD.org/src/rev/29c8de970c25
branches: trunk
changeset: 973260:29c8de970c25
user: jdolecek <jdolecek%NetBSD.org@localhost>
date: Thu Jun 25 14:04:30 2020 +0000
description:
make ubc_winshift / ubc_winsize constant, and based on whatever is bigger
of (1 << UBC_WINSHIFT, MAX_PAGE_SIZE)
given that default UBC_WINSHIFT is 13, this changes behaviour only
for mips and powerpc (BookE/OEA), which will now have twice as much
memory used for UBC windows; if this ever becomes a problem, it's
possible to reduce ubc_nwins in MD code similar to what is done on sparc
this eliminates variable-length arrays in ubc_fault(),
ubc_uiomove(), and ubc_zerorange() so that the stack usage can be
determined and checked in compile time
diffstat:
sys/uvm/uvm_bio.c | 27 +++++++++++++++------------
1 files changed, 15 insertions(+), 12 deletions(-)
diffs (98 lines):
diff -r e225a784c6c8 -r 29c8de970c25 sys/uvm/uvm_bio.c
--- a/sys/uvm/uvm_bio.c Thu Jun 25 11:48:39 2020 +0000
+++ b/sys/uvm/uvm_bio.c Thu Jun 25 14:04:30 2020 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: uvm_bio.c,v 1.117 2020/05/25 19:29:08 ad Exp $ */
+/* $NetBSD: uvm_bio.c,v 1.118 2020/06/25 14:04:30 jdolecek Exp $ */
/*
* Copyright (c) 1998 Chuck Silvers.
@@ -34,7 +34,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: uvm_bio.c,v 1.117 2020/05/25 19:29:08 ad Exp $");
+__KERNEL_RCSID(0, "$NetBSD: uvm_bio.c,v 1.118 2020/06/25 14:04:30 jdolecek Exp $");
#include "opt_uvmhist.h"
#include "opt_ubc.h"
@@ -45,6 +45,7 @@
#include <sys/kernel.h>
#include <sys/proc.h>
#include <sys/vnode.h>
+#include <sys/bitops.h> /* for ilog2() */
#include <uvm/uvm.h>
#include <uvm/uvm_pdpolicy.h>
@@ -120,9 +121,13 @@
/* ... rest are NULL */
};
+/* Use value at least as big as maximum page size supported by architecture */
+#define UBC_MAX_WINSHIFT \
+ ((1 << UBC_WINSHIFT) > MAX_PAGE_SIZE ? UBC_WINSHIFT : ilog2(MAX_PAGE_SIZE))
+
int ubc_nwins = UBC_NWINS;
-int ubc_winshift __read_mostly = UBC_WINSHIFT;
-int ubc_winsize __read_mostly;
+const int ubc_winshift = UBC_MAX_WINSHIFT;
+const int ubc_winsize = 1 << UBC_MAX_WINSHIFT;
#if defined(PMAP_PREFER)
int ubc_nqueues;
#define UBC_NQUEUES ubc_nqueues
@@ -161,9 +166,7 @@
/*
* Make sure ubc_winshift is sane.
*/
- if (ubc_winshift < PAGE_SHIFT)
- ubc_winshift = PAGE_SHIFT;
- ubc_winsize = 1 << ubc_winshift;
+ KASSERT(ubc_winshift >= PAGE_SHIFT);
/*
* init ubc_object.
@@ -304,7 +307,7 @@
struct uvm_object *uobj;
struct ubc_map *umap;
vaddr_t va, eva, ubc_offset, slot_offset;
- struct vm_page *pgs[ubc_winsize >> PAGE_SHIFT];
+ struct vm_page *pgs[howmany(ubc_winsize, MIN_PAGE_SIZE)];
int i, error, npages;
vm_prot_t prot;
@@ -732,7 +735,7 @@
int flags)
{
const bool overwrite = (flags & UBC_FAULTBUSY) != 0;
- struct vm_page *pgs[ubc_winsize >> PAGE_SHIFT];
+ struct vm_page *pgs[howmany(ubc_winsize, MIN_PAGE_SIZE)];
voff_t off;
int error, npages;
@@ -798,7 +801,7 @@
void
ubc_zerorange(struct uvm_object *uobj, off_t off, size_t len, int flags)
{
- struct vm_page *pgs[ubc_winsize >> PAGE_SHIFT];
+ struct vm_page *pgs[howmany(ubc_winsize, MIN_PAGE_SIZE)];
int npages;
#ifdef UBC_USE_PMAP_DIRECT
@@ -975,7 +978,7 @@
const bool overwrite = (flags & UBC_FAULTBUSY) != 0;
voff_t off;
int error, npages;
- struct vm_page *pgs[ubc_winsize >> PAGE_SHIFT];
+ struct vm_page *pgs[howmany(ubc_winsize, MIN_PAGE_SIZE)];
KASSERT(todo <= uio->uio_resid);
KASSERT(((flags & UBC_WRITE) != 0 && uio->uio_rw == UIO_WRITE) ||
@@ -1043,7 +1046,7 @@
ubc_zerorange_direct(struct uvm_object *uobj, off_t off, size_t todo, int flags)
{
int error, npages;
- struct vm_page *pgs[ubc_winsize >> PAGE_SHIFT];
+ struct vm_page *pgs[howmany(ubc_winsize, MIN_PAGE_SIZE)];
flags |= UBC_WRITE;
Home |
Main Index |
Thread Index |
Old Index