diff options
author | Matthew Ahrens <[email protected]> | 2019-08-20 11:34:52 -0700 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2019-08-20 11:34:52 -0700 |
commit | 325d288c5d536227010ff4dfcf66df89f123d166 (patch) | |
tree | 86807360e97638a06b6cce82a168b75a36a3caaf /include/sys/policy.h | |
parent | 3beb0a7694df5d1d4314179147aaa1d40b63fe76 (diff) |
Add fast path for zfs_ioc_space_snaps() handling of empty_bpobj
When there are many snapshots, calls to zfs_ioc_space_snaps() (e.g. from
`zfs destroy -nv pool/fs@snap1%snap10000`) can be very slow, resulting
in poor performance because we are holding the dp_config_rwlock the
entire time, blocking spa_sync() from continuing. With around ten
thousand snapshots, we've seen up to 500 seconds in this ioctl,
iterating over up to 50,000,000 bpobjs, ~99% of which are the empty
bpobj.
By creating a fast path for zfs_ioc_space_snaps() handling of the
empty_bpobj, we can achieve a ~5x performance improvement of this ioctl
(when there are many snapshots, and the deadlist is mostly
empty_bpobj's).
Reviewed-by: Pavel Zakharov <[email protected]>
Reviewed-by: Brian Behlendorf <[email protected]>
Reviewed-by: Paul Dagnelie <[email protected]>
Signed-off-by: Matthew Ahrens <[email protected]>
External-issue: DLPX-58348
Closes #8744
Diffstat (limited to 'include/sys/policy.h')
0 files changed, 0 insertions, 0 deletions