Skip to content

Commit

Permalink
OS-6387 1K zfs_abd_chunk_size is inducing excessive fragmentation
Browse files Browse the repository at this point in the history
Reviewed by: Joshua M. Clulow <jmc@joyent.com>
Approved by: Joshua M. Clulow <jmc@joyent.com>
  • Loading branch information
bcantrill authored and jclulow committed Oct 8, 2017
1 parent a5857e8 commit 2bd6ca8
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions usr/src/uts/common/fs/zfs/abd.c
Expand Up @@ -146,9 +146,12 @@ boolean_t zfs_abd_scatter_enabled = B_TRUE;
* it at runtime would cause ABD iteration to work incorrectly for ABDs which
* were allocated with the old size, so a safeguard has been put in place which
* will cause the machine to panic if you change it and try to access the data
* within a scattered ABD.
* within a scattered ABD. Note that tuning this value to be smaller than the
* page size can induce heavy fragmentation in the slab layer, which may itself
* result in more memory waste than is saved by the smaller chunk size -- and
* will induces more computational work in the slab layer. Tune with caution!
*/
size_t zfs_abd_chunk_size = 1024;
size_t zfs_abd_chunk_size = 4096;

#ifdef _KERNEL
extern vmem_t *zio_alloc_arena;
Expand Down

0 comments on commit 2bd6ca8

Please sign in to comment.