diff options
| author | alc <alc@FreeBSD.org> | 2008-04-04 18:41:12 +0000 |
|---|---|---|
| committer | alc <alc@FreeBSD.org> | 2008-04-04 18:41:12 +0000 |
| commit | 067dba5f97f5aec73b22ae390047bd858ca388e1 (patch) | |
| tree | da7f6a5b63eaf179d69327307f8b69f29e18aaf3 /sys/vm | |
| parent | 8353654e9666e7668d486e7e2a72051f06fcc9d3 (diff) | |
| download | FreeBSD-src-067dba5f97f5aec73b22ae390047bd858ca388e1.zip FreeBSD-src-067dba5f97f5aec73b22ae390047bd858ca388e1.tar.gz | |
Reintroduce UMA_SLAB_KMAP; however, change its spelling to
UMA_SLAB_KERNEL for consistency with its sibling UMA_SLAB_KMEM.
(UMA_SLAB_KMAP met its original demise in revision 1.30 of
vm/uma_core.c.) UMA_SLAB_KERNEL is now required by the jumbo frame
allocators. Without it, UMA cannot correctly return pages from the
jumbo frame zones to the VM system because it resets the pages' object
field to NULL instead of the kernel object. In more detail, the jumbo
frame zones are created with the option UMA_ZONE_REFCNT. This causes
UMA to overwrite the pages' object field with the address of the slab.
However, when UMA wants to release these pages, it doesn't know how to
restore the object field, so it sets it to NULL. This change teaches
UMA how to reset the object field to the kernel object.
Crashes reported by: kris
Fix tested by: kris
Fix discussed with: jeff
MFC after: 6 weeks
Diffstat (limited to 'sys/vm')
| -rw-r--r-- | sys/vm/uma.h | 1 | ||||
| -rw-r--r-- | sys/vm/uma_core.c | 4 |
2 files changed, 5 insertions, 0 deletions
diff --git a/sys/vm/uma.h b/sys/vm/uma.h index c082e96..ffa8d62 100644 --- a/sys/vm/uma.h +++ b/sys/vm/uma.h @@ -488,6 +488,7 @@ void uma_zone_set_freef(uma_zone_t zone, uma_free freef); */ #define UMA_SLAB_BOOT 0x01 /* Slab alloced from boot pages */ #define UMA_SLAB_KMEM 0x02 /* Slab alloced from kmem_map */ +#define UMA_SLAB_KERNEL 0x04 /* Slab alloced from kernel_map */ #define UMA_SLAB_PRIV 0x08 /* Slab alloced from priv allocator */ #define UMA_SLAB_OFFP 0x10 /* Slab is managed separately */ #define UMA_SLAB_MALLOC 0x20 /* Slab is a large malloc slab */ diff --git a/sys/vm/uma_core.c b/sys/vm/uma_core.c index c32f4d7..a74ca32 100644 --- a/sys/vm/uma_core.c +++ b/sys/vm/uma_core.c @@ -751,6 +751,8 @@ finished: if (flags & UMA_SLAB_KMEM) obj = kmem_object; + else if (flags & UMA_SLAB_KERNEL) + obj = kernel_object; else obj = NULL; for (i = 0; i < keg->uk_ppera; i++) @@ -871,6 +873,8 @@ slab_zalloc(uma_zone_t zone, int wait) if (flags & UMA_SLAB_KMEM) obj = kmem_object; + else if (flags & UMA_SLAB_KERNEL) + obj = kernel_object; else obj = NULL; for (i = 0; i < keg->uk_ppera; i++) |
