aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMark Johnston <markj@FreeBSD.org>2021-02-22 20:50:09 +0000
committerMark Johnston <markj@FreeBSD.org>2021-02-22 20:50:09 +0000
commit23e875fd97fb9f17b3f5dc2b26082f25e1a86b6f (patch)
treedcaf37efd3586a08372e02e59d51da3bcea151a4
parent5ce2d4a1c2922821b9940473213c0d75469755fc (diff)
downloadsrc-23e875fd97fb9f17b3f5dc2b26082f25e1a86b6f.tar.gz
src-23e875fd97fb9f17b3f5dc2b26082f25e1a86b6f.zip
vm_kern: Avoid sign extension in the KVA_QUANTUM definition
Otherwise, on a powerpc64 NUMA system with hashed page tables, the first-level superpage reservation size is large enough that the value of the kernel KVA arena import quantum, KVA_NUMA_IMPORT_QUANTUM, is negative and gets sign-extended when passed to vmem_set_import(). This results in a boot-time hang on such platforms. Reported by: bdragon MFC after: 3 days
-rw-r--r--sys/vm/vm_kern.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/vm/vm_kern.c b/sys/vm/vm_kern.c
index e8e24c3ca8a3..637298eb2caa 100644
--- a/sys/vm/vm_kern.c
+++ b/sys/vm/vm_kern.c
@@ -127,7 +127,7 @@ SYSCTL_ULONG(_vm, OID_AUTO, max_kernel_address, CTLFLAG_RD,
/* On non-superpage architectures we want large import sizes. */
#define KVA_QUANTUM_SHIFT (8 + PAGE_SHIFT)
#endif
-#define KVA_QUANTUM (1 << KVA_QUANTUM_SHIFT)
+#define KVA_QUANTUM (1ul << KVA_QUANTUM_SHIFT)
#define KVA_NUMA_IMPORT_QUANTUM (KVA_QUANTUM * 128)
extern void uma_startup2(void);