]> git.proxmox.com Git - mirror_zfs.git/commit
zfs list: Allow more fields in ZFS_ITER_SIMPLE mode
authorAllan Jude <allan@klarasystems.com>
Wed, 14 Dec 2022 01:27:54 +0000 (20:27 -0500)
committerGitHub <noreply@github.com>
Wed, 14 Dec 2022 01:27:54 +0000 (17:27 -0800)
commitdc95911d21a19930848302aac9283fff68e4a41b
tree82f20fc9c52ce28b47987e8c7a089b37dbd100dd
parent70ac2654f579e138d04ed747967fb0d603b0cacb
zfs list: Allow more fields in ZFS_ITER_SIMPLE mode

If the fields to be listed and sorted by are constrained to those
populated by dsl_dataset_fast_stat(), then zfs list is much faster,
as it does not need to open each objset and reads its properties.

A previous optimization by Pawel Dawidek
(0cee24064a79f9c01fc4521543c37acea538405f) took advantage
of this to make listing snapshot names sorted only by name much faster.

However, it was limited to `-o name -s name`, this work extends this
optimization to work with:
  - name
  - guid
  - createtxg
  - numclones
  - inconsistent
  - redacted
  - origin
and could be further extended to any other properties supported by
dsl_dataset_fast_stat() or similar, that do not require extra locking
or reading from disk.

This was committed before (9a9e2e343dfa2af28bf7910de77ae73aa006de62),
but was reverted due to a regression when used with an older kernel.

If the kernel does not populate zc->zc_objset_stats, we now fallback
to getting the properties via the slower interface, to avoid problems
with newer userland and older kernels.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Allan Jude <allan@klarasystems.com>
Closes #14110
14 files changed:
cmd/zfs/zfs_iter.c
cmd/zfs/zfs_iter.h
cmd/zfs/zfs_main.c
cmd/zpool/zpool_main.c
contrib/pam_zfs_key/pam_zfs_key.c
include/libzfs.h
lib/libzfs/libzfs.abi
lib/libzfs/libzfs_changelist.c
lib/libzfs/libzfs_crypto.c
lib/libzfs/libzfs_dataset.c
lib/libzfs/libzfs_iter.c
lib/libzfs/libzfs_mount.c
lib/libzfs/libzfs_sendrecv.c
module/zfs/zfs_ioctl.c