From nobody Wed Jan 25 19:17:31 2023 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4P2DB90HK9z3bCl7; Wed, 25 Jan 2023 19:17:37 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4P2DB9014fz3l4H; Wed, 25 Jan 2023 19:17:37 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1674674257; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=0B3XlE6rJwOMvSXoL/WajEnQoePqpa6cs/u5Pqe2O2c=; b=WKUonYR3Uu5aPXS9SCIBDwBgh3Fvclmg9F1wG76ja4FccfjRvgIf4MoQqx6MraQE2Na5mx PDeqU8zzy5kVzy47e6ZSJs+uIno0dZ1ZungWMDTS/t+FFY+BIWXmvCmM01bMkW1+6pRjbf RzFkcIbdB24x9yd1RUpCqk8y4glUtqvVkKO/VmADDsFFOhUuV9Zj6o9eMdoUVuCWEZMREd 2Yx0fxPCfBKSVaYuobc2cMLEHi4yd2AwnD/IqPHV4Zrq+jAPB2UVKFXR5r3/Nht9Fxdh7B ZjZtznTHTdjnBg0jXlDB5J7qp1kb8jz80m0wT1ucRbbXYha5qycDrXBN+qqKog== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1674674257; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=0B3XlE6rJwOMvSXoL/WajEnQoePqpa6cs/u5Pqe2O2c=; b=xjPHxZOd+yqAN/zMmjTZxzgQRkBe0ai/yQpFDDBuRuetpe8QR7RlxHGjoT/FmrzdV7GO3f 2QOGEgr98yGehKSquEgvMg+TUwUL+WaS7ZXYE1DYx/u2YRCrvq21BhuhdhJYTjszvtSDyf M3GTQl8P02qjE2JzRRmOTuofv5Uur2qpsGA3JxTE52gONAjo8vvtCmuBRT1JmcMJXSK6Yp h1T1RAmmSifnRdg/gy6brJFvetW6UrloBVtFN4UhNsOyiffiKyt6/PBRRVGaMw/1zQK2YC nV3j1kq/Qc/tfnN2Gr8Y6fnNyREuZe+NPRbUXy5u048hvh4XPeUe7PAHNZWJpA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1674674257; a=rsa-sha256; cv=none; b=eu0Ez43CHqNEAVXANojePDs2E50ChThhHTEVTBlqGQFz1uDLZEgXHw9kXdr6+t286M05F1 fi1pIpfwYpbWFjnJ4y2zdJhbXFRHTEBsBMsa64ty6cAH6xeyOK7jSYSlCfZq78j89KWvRF GamsJXip1rEkXRwceVhTfOw8krjb5xDCA4JArgx+KK4p5rrxo+IuVhWY+DesXiZxSvTbri 0Lc/28GBjrpNCwNytY5qhr4CcT2YsI1UhdGoAa4Sv1xycEZbV4HflqtDVht20XzdnTO1Pd 31GtvophhlbFcMzYsnaFdsPTCsg3TXaOq7HVrbwiEWR0+vgUmFT1Hs2cGaEywA== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4P2DB86B1Xz10bb; Wed, 25 Jan 2023 19:17:36 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 30PJHa2S005946; Wed, 25 Jan 2023 19:17:36 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 30PJHVP4005933; Wed, 25 Jan 2023 19:17:31 GMT (envelope-from git) Date: Wed, 25 Jan 2023 19:17:31 GMT Message-Id: <202301251917.30PJHVP4005933@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Martin Matuska Subject: git: 15f0b8c309de - main - zfs: merge openzfs/zfs@9cd71c860 (master) List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: mm X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 15f0b8c309dea1dcb14d3e374686576ff68ac43f Auto-Submitted: auto-generated X-ThisMailContainsUnwantedMimeParts: N The branch main has been updated by mm: URL: https://cgit.FreeBSD.org/src/commit/?id=15f0b8c309dea1dcb14d3e374686576ff68ac43f commit 15f0b8c309dea1dcb14d3e374686576ff68ac43f Merge: da52fc464a4c 9cd71c8604d5 Author: Martin Matuska AuthorDate: 2023-01-25 18:50:29 +0000 Commit: Martin Matuska CommitDate: 2023-01-25 18:50:29 +0000 zfs: merge openzfs/zfs@9cd71c860 (master) Notable upstream pull request merges: #13805 Configure zed's diagnosis engine with vdev properties #14110 zfs list: Allow more fields in ZFS_ITER_SIMPLE mode #14121 Batch enqueue/dequeue for bqueue #14123 arc_read()/arc_access() refactoring and cleanup #14159 Bypass metaslab throttle for removal allocations #14243 Implement uncached prefetch #14251 Cache dbuf_hash() calculation #14253 Allow reciever to override encryption property in case of replication #14254 Restrict visibility of per-dataset kstats inside FreeBSD jails #14255 Zero end of embedded block buffer in dump_write_embedded() #14263 Cleanups identified by CodeQL and Coverity #14264 Miscellaneous fixes #14272 Change ZEVENT_POOL_GUID to ZEVENT_POOL to display pool names #14287 FreeBSD: Remove stray debug printf #14288 Colorize zfs diff output #14289 deadlock between spa_errlog_lock and dp_config_rwlock #14291 FreeBSD: Fix potential boot panic with bad label #14292 Add tunable to allow changing micro ZAP's max size #14293 Turn default_bs and default_ibs into ZFS_MODULE_PARAMs #14295 zed: add hotplug support for spare vdevs #14304 Activate filesystem features only in syncing context #14311 zpool: do guid-based comparison in is_vdev_cb() #14317 Pack zrlock_t by 8 bytes #14320 Update arc_summary and arcstat outputs #14328 FreeBSD: catch up to 1400077 #14376 Use setproctitle to report progress of zfs send #14340 Remove some dead ARC code #14358 Wait for txg sync if the last DRR_FREEOBJECTS might result in a hole #14360 libzpool: fix ddi_strtoull to update nptr #14364 Fix unprotected zfs_znode_dmu_fini #14379 zfs_receive_one: Check for the more likely error first #14380 Cleanup of dead code suggested by Clang Static Analyzer #14397 Avoid passing an uninitialized index to dsl_prop_known_index #14404 Fix reading uninitialized variable in receive_read #14407 free_blocks(): Fix reports from 2016 PVS Studio FreeBSD report #14418 Introduce minimal ZIL block commit delay #14422 x86 assembly: fix .size placement and replace .align with .balign Obtained from: OpenZFS OpenZFS commit: 9cd71c8604d52def22ffaddc35755712f0fb9349 lib/libbe/be.c | 8 +- lib/libbe/be_access.c | 6 +- lib/libbe/be_info.c | 4 +- .../.github/workflows/build-dependencies.txt | 6 + .../.github/workflows/zfs-tests-functional.yml | 17 +- .../openzfs/.github/workflows/zfs-tests-sanity.yml | 17 +- sys/contrib/openzfs/CODE_OF_CONDUCT.md | 2 +- sys/contrib/openzfs/META | 2 +- sys/contrib/openzfs/cmd/arc_summary | 173 +++-- sys/contrib/openzfs/cmd/arcstat.in | 119 ++- sys/contrib/openzfs/cmd/zdb/zdb.c | 44 +- sys/contrib/openzfs/cmd/zed/agents/zfs_agents.c | 2 +- sys/contrib/openzfs/cmd/zed/agents/zfs_diagnosis.c | 40 +- sys/contrib/openzfs/cmd/zed/agents/zfs_mod.c | 21 +- sys/contrib/openzfs/cmd/zed/agents/zfs_retire.c | 89 ++- .../openzfs/cmd/zed/zed.d/statechange-notify.sh | 4 +- sys/contrib/openzfs/cmd/zfs/zfs_iter.c | 65 +- sys/contrib/openzfs/cmd/zfs/zfs_iter.h | 12 +- sys/contrib/openzfs/cmd/zfs/zfs_main.c | 65 +- sys/contrib/openzfs/cmd/zhack.c | 3 +- sys/contrib/openzfs/cmd/zpool/zpool_main.c | 96 +-- .../openzfs/cmd/zstream/zstream_decompress.c | 22 +- .../openzfs/cmd/zstream/zstream_recompress.c | 22 +- sys/contrib/openzfs/cmd/zstream/zstream_redup.c | 22 +- sys/contrib/openzfs/cmd/ztest.c | 48 +- sys/contrib/openzfs/config/ax_python_devel.m4 | 71 +- sys/contrib/openzfs/config/deb.am | 28 +- sys/contrib/openzfs/config/kernel-acl.m4 | 50 +- sys/contrib/openzfs/config/kernel-tmpfile.m4 | 32 +- sys/contrib/openzfs/config/kernel.m4 | 32 + sys/contrib/openzfs/config/zfs-build.m4 | 1 + sys/contrib/openzfs/configure.ac | 2 + sys/contrib/openzfs/contrib/debian/.gitignore | 1 + sys/contrib/openzfs/contrib/debian/control | 3 +- .../contrib/debian/openzfs-libpam-zfs.postinst | 4 +- .../contrib/debian/openzfs-zfs-zed.postinst | 7 - .../openzfs/contrib/debian/openzfs-zfs-zed.prerm | 16 - .../contrib/debian/openzfs-zfsutils.install | 1 - .../contrib/debian/openzfs-zfsutils.postinst | 15 - .../openzfs/contrib/debian/{rules => rules.in} | 4 +- .../contrib/dracut/90zfs/zfs-load-key.sh.in | 2 +- sys/contrib/openzfs/contrib/initramfs/scripts/zfs | 22 +- .../openzfs/contrib/pam_zfs_key/pam_zfs_key.c | 2 +- .../system-generators/zfs-mount-generator.c | 9 +- .../openzfs/etc/systemd/system/zfs-zed.service.in | 2 +- sys/contrib/openzfs/include/Makefile.am | 1 + sys/contrib/openzfs/include/libzfs.h | 25 +- sys/contrib/openzfs/include/libzutil.h | 9 + sys/contrib/openzfs/include/os/freebsd/Makefile.am | 3 + .../include/os/freebsd/spl/sys/ia32/asm_linkage.h | 178 +++++ sys/contrib/openzfs/include/os/linux/Makefile.am | 4 + .../include/os/linux/kernel/linux/blkdev_compat.h | 4 + .../include/os/linux/kernel/linux/dcache_compat.h | 8 +- .../include/os/linux/kernel/linux/simd_powerpc.h | 15 + .../include/os/linux/spl/sys/ia32/asm_linkage.h | 212 ++++++ .../openzfs/include/os/linux/spl/sys/kmem.h | 3 + .../openzfs/include/os/linux/spl/sys/kmem_cache.h | 3 - .../openzfs/include/os/linux/zfs/sys/trace_arc.h | 6 +- sys/contrib/openzfs/include/os/linux/zfs/sys/zpl.h | 5 +- sys/contrib/openzfs/include/sys/arc.h | 8 +- sys/contrib/openzfs/include/sys/arc_impl.h | 63 +- sys/contrib/openzfs/include/sys/asm_linkage.h | 48 ++ sys/contrib/openzfs/include/sys/bqueue.h | 7 +- sys/contrib/openzfs/include/sys/dbuf.h | 9 +- sys/contrib/openzfs/include/sys/dnode.h | 8 +- sys/contrib/openzfs/include/sys/dsl_dir.h | 2 +- sys/contrib/openzfs/include/sys/fm/fs/zfs.h | 4 + sys/contrib/openzfs/include/sys/fs/zfs.h | 5 + sys/contrib/openzfs/include/sys/spa.h | 2 +- sys/contrib/openzfs/include/sys/vdev_impl.h | 8 + sys/contrib/openzfs/include/sys/zfs_znode.h | 2 + sys/contrib/openzfs/include/sys/zrlock.h | 3 +- sys/contrib/openzfs/lib/libicp/Makefile.am | 1 + sys/contrib/openzfs/lib/libspl/include/Makefile.am | 10 + .../include/os/freebsd/sys/ia32/asm_linkage.h | 184 +++++ .../include/os/linux}/sys/ia32/asm_linkage.h | 31 +- .../icp => lib/libspl}/include/sys/asm_linkage.h | 0 sys/contrib/openzfs/lib/libzfs/libzfs.abi | 27 +- sys/contrib/openzfs/lib/libzfs/libzfs_changelist.c | 6 +- sys/contrib/openzfs/lib/libzfs/libzfs_crypto.c | 2 +- sys/contrib/openzfs/lib/libzfs/libzfs_dataset.c | 72 +- sys/contrib/openzfs/lib/libzfs/libzfs_diff.c | 40 + sys/contrib/openzfs/lib/libzfs/libzfs_iter.c | 53 +- sys/contrib/openzfs/lib/libzfs/libzfs_mount.c | 4 +- sys/contrib/openzfs/lib/libzfs/libzfs_pool.c | 62 +- sys/contrib/openzfs/lib/libzfs/libzfs_sendrecv.c | 114 ++- sys/contrib/openzfs/lib/libzfs/libzfs_status.c | 2 +- sys/contrib/openzfs/lib/libzfs/libzfs_util.c | 21 +- sys/contrib/openzfs/lib/libzpool/kernel.c | 6 +- sys/contrib/openzfs/lib/libzutil/Makefile.am | 1 + .../lib/libzutil/os/linux/zutil_setproctitle.c | 299 ++++++++ sys/contrib/openzfs/man/man1/arcstat.1 | 82 +- sys/contrib/openzfs/man/man4/zfs.4 | 26 +- sys/contrib/openzfs/man/man7/vdevprops.7 | 17 +- sys/contrib/openzfs/man/man8/zfs-send.8 | 24 +- sys/contrib/openzfs/man/man8/zfs.8 | 4 + sys/contrib/openzfs/module/Kbuild.in | 4 +- sys/contrib/openzfs/module/icp/algs/aes/aes_impl.c | 2 +- .../openzfs/module/icp/algs/aes/aes_impl_aesni.c | 9 +- .../openzfs/module/icp/algs/blake3/blake3_impl.h | 1 + .../openzfs/module/icp/algs/blake3/blake3_x86-64.c | 20 +- sys/contrib/openzfs/module/icp/algs/modes/ccm.c | 2 +- sys/contrib/openzfs/module/icp/algs/modes/gcm.c | 20 +- .../openzfs/module/icp/algs/modes/gcm_pclmulqdq.c | 3 +- sys/contrib/openzfs/module/icp/algs/sha2/sha2.c | 5 +- sys/contrib/openzfs/module/icp/api/kcf_ctxops.c | 2 +- .../openzfs/module/icp/asm-x86_64/aes/aes_aesni.S | 16 +- .../openzfs/module/icp/asm-x86_64/aes/aes_amd64.S | 60 +- .../module/icp/asm-x86_64/blake3/blake3_avx2.S | 12 +- .../module/icp/asm-x86_64/blake3/blake3_avx512.S | 30 +- .../module/icp/asm-x86_64/blake3/blake3_sse2.S | 32 +- .../module/icp/asm-x86_64/blake3/blake3_sse41.S | 31 +- .../module/icp/asm-x86_64/modes/aesni-gcm-x86_64.S | 86 +-- .../module/icp/asm-x86_64/modes/gcm_pclmulqdq.S | 2 +- .../module/icp/asm-x86_64/modes/ghash-x86_64.S | 58 +- .../module/icp/asm-x86_64/sha2/sha256_impl.S | 8 +- .../module/icp/asm-x86_64/sha2/sha512_impl.S | 11 +- sys/contrib/openzfs/module/icp/core/kcf_callprov.c | 2 +- .../openzfs/module/icp/include/aes/aes_impl.h | 9 +- .../openzfs/module/icp/include/sys/ia32/stack.h | 160 ---- .../openzfs/module/icp/include/sys/ia32/trap.h | 107 --- sys/contrib/openzfs/module/lua/ldo.c | 6 +- sys/contrib/openzfs/module/lua/lfunc.h | 4 +- sys/contrib/openzfs/module/lua/lgc.c | 2 +- sys/contrib/openzfs/module/lua/lgc.h | 2 +- sys/contrib/openzfs/module/lua/lobject.h | 4 +- sys/contrib/openzfs/module/lua/lvm.c | 2 +- .../openzfs/module/lua/setjmp/setjmp_aarch64.S | 2 +- sys/contrib/openzfs/module/lua/setjmp/setjmp_arm.S | 2 +- .../openzfs/module/lua/setjmp/setjmp_i386.S | 2 +- sys/contrib/openzfs/module/lua/setjmp/setjmp_ppc.S | 4 +- .../openzfs/module/lua/setjmp/setjmp_sparc64.S | 2 +- .../openzfs/module/lua/setjmp/setjmp_x86_64.S | 22 +- sys/contrib/openzfs/module/os/freebsd/spl/callb.c | 4 +- .../openzfs/module/os/freebsd/spl/spl_kstat.c | 89 ++- .../openzfs/module/os/freebsd/zfs/kmod_core.c | 7 +- sys/contrib/openzfs/module/os/freebsd/zfs/spa_os.c | 2 + .../openzfs/module/os/freebsd/zfs/sysctl_os.c | 31 +- .../openzfs/module/os/freebsd/zfs/zfs_acl.c | 4 +- .../openzfs/module/os/freebsd/zfs/zfs_debug.c | 4 +- .../openzfs/module/os/freebsd/zfs/zfs_dir.c | 7 + .../openzfs/module/os/freebsd/zfs/zfs_znode.c | 4 +- .../openzfs/module/os/freebsd/zfs/zio_crypt.c | 3 +- .../openzfs/module/os/linux/spl/spl-kmem-cache.c | 10 +- .../openzfs/module/os/linux/spl/spl-thread.c | 5 +- sys/contrib/openzfs/module/os/linux/spl/spl-zone.c | 2 +- .../openzfs/module/os/linux/zfs/vdev_disk.c | 13 +- sys/contrib/openzfs/module/os/linux/zfs/zfs_acl.c | 6 +- .../openzfs/module/os/linux/zfs/zfs_debug.c | 6 +- sys/contrib/openzfs/module/os/linux/zfs/zfs_dir.c | 13 +- .../openzfs/module/os/linux/zfs/zfs_vnops_os.c | 6 +- .../openzfs/module/os/linux/zfs/zfs_znode.c | 9 +- .../openzfs/module/os/linux/zfs/zpl_inode.c | 27 + .../openzfs/module/os/linux/zfs/zpl_xattr.c | 11 +- sys/contrib/openzfs/module/unicode/u8_textprep.c | 21 +- sys/contrib/openzfs/module/zcommon/zfs_fletcher.c | 4 +- sys/contrib/openzfs/module/zcommon/zpool_prop.c | 12 + sys/contrib/openzfs/module/zfs/arc.c | 848 +++++++++++---------- sys/contrib/openzfs/module/zfs/blake3_zfs.c | 4 +- sys/contrib/openzfs/module/zfs/bqueue.c | 117 +-- sys/contrib/openzfs/module/zfs/dbuf.c | 136 ++-- sys/contrib/openzfs/module/zfs/dmu.c | 18 +- sys/contrib/openzfs/module/zfs/dmu_objset.c | 9 +- sys/contrib/openzfs/module/zfs/dmu_recv.c | 47 +- sys/contrib/openzfs/module/zfs/dmu_send.c | 15 +- sys/contrib/openzfs/module/zfs/dmu_traverse.c | 3 +- sys/contrib/openzfs/module/zfs/dmu_tx.c | 10 +- sys/contrib/openzfs/module/zfs/dmu_zfetch.c | 6 +- sys/contrib/openzfs/module/zfs/dnode.c | 5 + sys/contrib/openzfs/module/zfs/dnode_sync.c | 31 +- sys/contrib/openzfs/module/zfs/dsl_dataset.c | 23 +- sys/contrib/openzfs/module/zfs/dsl_dir.c | 5 +- sys/contrib/openzfs/module/zfs/dsl_prop.c | 2 +- sys/contrib/openzfs/module/zfs/dsl_scan.c | 10 +- sys/contrib/openzfs/module/zfs/fm.c | 3 +- sys/contrib/openzfs/module/zfs/metaslab.c | 13 +- sys/contrib/openzfs/module/zfs/spa.c | 2 +- sys/contrib/openzfs/module/zfs/spa_config.c | 2 + sys/contrib/openzfs/module/zfs/spa_errlog.c | 267 +++---- sys/contrib/openzfs/module/zfs/vdev.c | 137 +++- sys/contrib/openzfs/module/zfs/vdev_indirect.c | 2 +- .../zfs/vdev_raidz_math_powerpc_altivec_common.h | 44 +- sys/contrib/openzfs/module/zfs/vdev_removal.c | 6 +- sys/contrib/openzfs/module/zfs/zap.c | 4 +- sys/contrib/openzfs/module/zfs/zap_micro.c | 8 +- sys/contrib/openzfs/module/zfs/zcp.c | 10 +- sys/contrib/openzfs/module/zfs/zfs_chksum.c | 2 +- sys/contrib/openzfs/module/zfs/zfs_fm.c | 79 ++ sys/contrib/openzfs/module/zfs/zfs_fuid.c | 2 +- sys/contrib/openzfs/module/zfs/zfs_ioctl.c | 24 +- sys/contrib/openzfs/module/zfs/zfs_vnops.c | 2 +- sys/contrib/openzfs/module/zfs/zil.c | 36 +- sys/contrib/openzfs/module/zfs/zio.c | 2 +- sys/contrib/openzfs/module/zfs/zvol.c | 4 +- .../module/zstd/include/zstd_compat_wrapper.h | 2 - .../module/zstd/lib/common/fse_decompress.c | 11 - sys/contrib/openzfs/module/zstd/zfs_zstd.c | 4 +- sys/contrib/openzfs/scripts/debian-packaging.sh | 36 - sys/contrib/openzfs/tests/runfiles/common.run | 1 + sys/contrib/openzfs/tests/runfiles/linux.run | 2 +- sys/contrib/openzfs/tests/runfiles/sanity.run | 3 + .../openzfs/tests/test-runner/bin/zts-report.py.in | 10 +- .../openzfs/tests/zfs-tests/cmd/mmapwrite.c | 12 +- .../openzfs/tests/zfs-tests/tests/Makefile.am | 12 +- .../tests/functional/acl/posix/posix_004_pos.ksh | 3 + .../zfs_property/zfs_written_property_001_pos.ksh | 1 + .../zfs_receive/zfs_receive_to_encrypted.ksh | 14 + .../cli_root/zfs_send/zfs_send_006_pos.ksh | 32 +- .../tests/functional/events/zed_cksum_config.ksh | 158 ++++ .../tests/functional/events/zed_io_config.ksh | 150 ++++ .../tests/functional/l2arc/l2arc_mfuonly_pos.ksh | 2 +- .../tests/functional/renameat2/Makefile.am | 7 - .../rsend/send_encrypted_freeobjects.ksh | 87 +++ .../rsend/send_encrypted_incremental.ksh | 93 +++ .../functional/rsend/send_raw_large_blocks.ksh | 2 +- .../tests/functional/trim/autotrim_config.ksh | 1 + .../tests/functional/trim/trim_config.ksh | 1 + .../functional/zvol/zvol_misc/zvol_misc_trim.ksh | 1 + sys/modules/zfs/zfs_config.h | 17 +- sys/modules/zfs/zfs_gitrev.h | 2 +- 220 files changed, 4467 insertions(+), 2127 deletions(-) diff --cc lib/libbe/be.c index 13f7a59d5215,000000000000..cc6b8533a8f9 mode 100644,000000..100644 --- a/lib/libbe/be.c +++ b/lib/libbe/be.c @@@ -1,1309 -1,0 +1,1309 @@@ +/*- + * SPDX-License-Identifier: BSD-2-Clause-FreeBSD + * + * Copyright (c) 2017 Kyle J. Kneitinger + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * 1. Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * 2. Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in the + * documentation and/or other materials provided with the distribution. + * + * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND + * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE + * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE + * ARE DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE + * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL + * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS + * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY + * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF + * SUCH DAMAGE. + */ + +#include +__FBSDID("$FreeBSD$"); + +#include +#include +#include +#include +#include +#include +#include +#include + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include "be.h" +#include "be_impl.h" + +struct promote_entry { + char name[BE_MAXPATHLEN]; + SLIST_ENTRY(promote_entry) link; +}; + +struct be_destroy_data { + libbe_handle_t *lbh; + char target_name[BE_MAXPATHLEN]; + char *snapname; + SLIST_HEAD(, promote_entry) promotelist; +}; + +#if SOON +static int be_create_child_noent(libbe_handle_t *lbh, const char *active, + const char *child_path); +static int be_create_child_cloned(libbe_handle_t *lbh, const char *active); +#endif + +/* Arbitrary... should tune */ +#define BE_SNAP_SERIAL_MAX 1024 + +/* + * Iterator function for locating the rootfs amongst the children of the + * zfs_be_root set by loader(8). data is expected to be a libbe_handle_t *. + */ +static int +be_locate_rootfs(libbe_handle_t *lbh) +{ + struct statfs sfs; + struct mnttab entry; + zfs_handle_t *zfs; + + /* + * Check first if root is ZFS; if not, we'll bail on rootfs capture. + * Unfortunately needed because zfs_path_to_zhandle will emit to + * stderr if / isn't actually a ZFS filesystem, which we'd like + * to avoid. + */ + if (statfs("/", &sfs) == 0) { + statfs2mnttab(&sfs, &entry); + if (strcmp(entry.mnt_fstype, MNTTYPE_ZFS) != 0) + return (1); + } else + return (1); + zfs = zfs_path_to_zhandle(lbh->lzh, "/", ZFS_TYPE_FILESYSTEM); + if (zfs == NULL) + return (1); + + strlcpy(lbh->rootfs, zfs_get_name(zfs), sizeof(lbh->rootfs)); + zfs_close(zfs); + return (0); +} + +/* + * Initializes the libbe context to operate in the root boot environment + * dataset, for example, zroot/ROOT. + */ +libbe_handle_t * +libbe_init(const char *root) +{ + char altroot[MAXPATHLEN]; + libbe_handle_t *lbh; + char *poolname, *pos; + int pnamelen; + + lbh = NULL; + poolname = pos = NULL; + + if ((lbh = calloc(1, sizeof(libbe_handle_t))) == NULL) + goto err; + + if ((lbh->lzh = libzfs_init()) == NULL) + goto err; + + /* + * Grab rootfs, we'll work backwards from there if an optional BE root + * has not been passed in. + */ + if (be_locate_rootfs(lbh) != 0) { + if (root == NULL) + goto err; + *lbh->rootfs = '\0'; + } + if (root == NULL) { + /* Strip off the final slash from rootfs to get the be root */ + strlcpy(lbh->root, lbh->rootfs, sizeof(lbh->root)); + pos = strrchr(lbh->root, '/'); + if (pos == NULL) + goto err; + *pos = '\0'; + } else + strlcpy(lbh->root, root, sizeof(lbh->root)); + + if ((pos = strchr(lbh->root, '/')) == NULL) + goto err; + + pnamelen = pos - lbh->root; + poolname = malloc(pnamelen + 1); + if (poolname == NULL) + goto err; + + strlcpy(poolname, lbh->root, pnamelen + 1); + if ((lbh->active_phandle = zpool_open(lbh->lzh, poolname)) == NULL) + goto err; + free(poolname); + poolname = NULL; + + if (zpool_get_prop(lbh->active_phandle, ZPOOL_PROP_BOOTFS, lbh->bootfs, + sizeof(lbh->bootfs), NULL, true) != 0) + goto err; + + if (zpool_get_prop(lbh->active_phandle, ZPOOL_PROP_ALTROOT, + altroot, sizeof(altroot), NULL, true) == 0 && + strcmp(altroot, "-") != 0) + lbh->altroot_len = strlen(altroot); + + return (lbh); +err: + if (lbh != NULL) { + if (lbh->active_phandle != NULL) + zpool_close(lbh->active_phandle); + if (lbh->lzh != NULL) + libzfs_fini(lbh->lzh); + free(lbh); + } + free(poolname); + return (NULL); +} + + +/* + * Free memory allocated by libbe_init() + */ +void +libbe_close(libbe_handle_t *lbh) +{ + + if (lbh->active_phandle != NULL) + zpool_close(lbh->active_phandle); + libzfs_fini(lbh->lzh); + free(lbh); +} + +/* + * Proxy through to libzfs for the moment. + */ +void +be_nicenum(uint64_t num, char *buf, size_t buflen) +{ + + zfs_nicenum(num, buf, buflen); +} + +static bool +be_should_promote_clones(zfs_handle_t *zfs_hdl, struct be_destroy_data *bdd) +{ + char *atpos; + + if (zfs_get_type(zfs_hdl) != ZFS_TYPE_SNAPSHOT) + return (false); + + /* + * If we're deleting a snapshot, we need to make sure we only promote + * clones that are derived from one of the snapshots we're deleting, + * rather than that of a snapshot we're not touching. This keeps stuff + * in a consistent state, making sure that we don't error out unless + * we really need to. + */ + if (bdd->snapname == NULL) + return (true); + + atpos = strchr(zfs_get_name(zfs_hdl), '@'); + return (strcmp(atpos + 1, bdd->snapname) == 0); +} + +/* + * This is executed from be_promote_dependent_clones via zfs_iter_dependents, + * It checks if the dependent type is a snapshot then attempts to find any + * clones associated with it. Any clones not related to the destroy target are + * added to the promote list. + */ +static int +be_dependent_clone_cb(zfs_handle_t *zfs_hdl, void *data) +{ + int err; + bool found; + char *name; + struct nvlist *nvl; + struct nvpair *nvp; + struct be_destroy_data *bdd; + struct promote_entry *entry, *newentry; + + nvp = NULL; + err = 0; + bdd = (struct be_destroy_data *)data; + + if (be_should_promote_clones(zfs_hdl, bdd) && + (nvl = zfs_get_clones_nvl(zfs_hdl)) != NULL) { + while ((nvp = nvlist_next_nvpair(nvl, nvp)) != NULL) { + name = nvpair_name(nvp); + + /* + * Skip if the clone is equal to, or a child of, the + * destroy target. + */ + if (strncmp(name, bdd->target_name, + strlen(bdd->target_name)) == 0 || + strstr(name, bdd->target_name) == name) { + continue; + } + + found = false; + SLIST_FOREACH(entry, &bdd->promotelist, link) { + if (strcmp(entry->name, name) == 0) { + found = true; + break; + } + } + + if (found) + continue; + + newentry = malloc(sizeof(struct promote_entry)); + if (newentry == NULL) { + err = ENOMEM; + break; + } + +#define BE_COPY_NAME(entry, src) \ + strlcpy((entry)->name, (src), sizeof((entry)->name)) + if (BE_COPY_NAME(newentry, name) >= + sizeof(newentry->name)) { + /* Shouldn't happen. */ + free(newentry); + err = ENAMETOOLONG; + break; + } +#undef BE_COPY_NAME + + /* + * We're building up a SLIST here to make sure both that + * we get the order right and so that we don't + * inadvertently observe the wrong state by promoting + * datasets while we're still walking the tree. The + * latter can lead to situations where we promote a BE + * then effectively demote it again. + */ + SLIST_INSERT_HEAD(&bdd->promotelist, newentry, link); + } + nvlist_free(nvl); + } + zfs_close(zfs_hdl); + return (err); +} + +/* + * This is called before a destroy, so that any datasets(environments) that are + * dependent on this one get promoted before destroying the target. + */ +static int +be_promote_dependent_clones(zfs_handle_t *zfs_hdl, struct be_destroy_data *bdd) +{ + int err; + zfs_handle_t *clone; + struct promote_entry *entry; + + snprintf(bdd->target_name, BE_MAXPATHLEN, "%s/", zfs_get_name(zfs_hdl)); - err = zfs_iter_dependents(zfs_hdl, true, be_dependent_clone_cb, bdd); ++ err = zfs_iter_dependents(zfs_hdl, 0, true, be_dependent_clone_cb, bdd); + + /* + * Drain the list and walk away from it if we're only deleting a + * snapshot. + */ + if (bdd->snapname != NULL && !SLIST_EMPTY(&bdd->promotelist)) + err = BE_ERR_HASCLONES; + while (!SLIST_EMPTY(&bdd->promotelist)) { + entry = SLIST_FIRST(&bdd->promotelist); + SLIST_REMOVE_HEAD(&bdd->promotelist, link); + +#define ZFS_GRAB_CLONE() \ + zfs_open(bdd->lbh->lzh, entry->name, ZFS_TYPE_FILESYSTEM) + /* + * Just skip this part on error, we still want to clean up the + * promotion list after the first error. We'll then preserve it + * all the way back. + */ + if (err == 0 && (clone = ZFS_GRAB_CLONE()) != NULL) { + err = zfs_promote(clone); + if (err != 0) + err = BE_ERR_DESTROYMNT; + zfs_close(clone); + } +#undef ZFS_GRAB_CLONE + free(entry); + } + + return (err); +} + +static int +be_destroy_cb(zfs_handle_t *zfs_hdl, void *data) +{ + char path[BE_MAXPATHLEN]; + struct be_destroy_data *bdd; + zfs_handle_t *snap; + int err; + + bdd = (struct be_destroy_data *)data; + if (bdd->snapname == NULL) { - err = zfs_iter_children(zfs_hdl, be_destroy_cb, data); ++ err = zfs_iter_children(zfs_hdl, 0, be_destroy_cb, data); + if (err != 0) + return (err); + return (zfs_destroy(zfs_hdl, false)); + } + /* If we're dealing with snapshots instead, delete that one alone */ - err = zfs_iter_filesystems(zfs_hdl, be_destroy_cb, data); ++ err = zfs_iter_filesystems(zfs_hdl, 0, be_destroy_cb, data); + if (err != 0) + return (err); + /* + * This part is intentionally glossing over any potential errors, + * because there's a lot less potential for errors when we're cleaning + * up snapshots rather than a full deep BE. The primary error case + * here being if the snapshot doesn't exist in the first place, which + * the caller will likely deem insignificant as long as it doesn't + * exist after the call. Thus, such a missing snapshot shouldn't jam + * up the destruction. + */ + snprintf(path, sizeof(path), "%s@%s", zfs_get_name(zfs_hdl), + bdd->snapname); + if (!zfs_dataset_exists(bdd->lbh->lzh, path, ZFS_TYPE_SNAPSHOT)) + return (0); + snap = zfs_open(bdd->lbh->lzh, path, ZFS_TYPE_SNAPSHOT); + if (snap != NULL) + zfs_destroy(snap, false); + return (0); +} + +#define BE_DESTROY_WANTORIGIN (BE_DESTROY_ORIGIN | BE_DESTROY_AUTOORIGIN) +/* + * Destroy the boot environment or snapshot specified by the name + * parameter. Options are or'd together with the possible values: + * BE_DESTROY_FORCE : forces operation on mounted datasets + * BE_DESTROY_ORIGIN: destroy the origin snapshot as well + */ +static int +be_destroy_internal(libbe_handle_t *lbh, const char *name, int options, + bool odestroyer) +{ + struct be_destroy_data bdd; + char origin[BE_MAXPATHLEN], path[BE_MAXPATHLEN]; + zfs_handle_t *fs; + char *snapdelim; + int err, force, mounted; + size_t rootlen; + + bdd.lbh = lbh; + bdd.snapname = NULL; + SLIST_INIT(&bdd.promotelist); + force = options & BE_DESTROY_FORCE; + *origin = '\0'; + + be_root_concat(lbh, name, path); + + if ((snapdelim = strchr(path, '@')) == NULL) { + if (!zfs_dataset_exists(lbh->lzh, path, ZFS_TYPE_FILESYSTEM)) + return (set_error(lbh, BE_ERR_NOENT)); + + if (strcmp(path, lbh->rootfs) == 0 || + strcmp(path, lbh->bootfs) == 0) + return (set_error(lbh, BE_ERR_DESTROYACT)); + + fs = zfs_open(lbh->lzh, path, ZFS_TYPE_FILESYSTEM); + if (fs == NULL) + return (set_error(lbh, BE_ERR_ZFSOPEN)); + + /* Don't destroy a mounted dataset unless force is specified */ + if ((mounted = zfs_is_mounted(fs, NULL)) != 0) { + if (force) { + zfs_unmount(fs, NULL, 0); + } else { + free(bdd.snapname); + return (set_error(lbh, BE_ERR_DESTROYMNT)); + } + } + } else { + /* + * If we're initially destroying a snapshot, origin options do + * not make sense. If we're destroying the origin snapshot of + * a BE, we want to maintain the options in case we need to + * fake success after failing to promote. + */ + if (!odestroyer) + options &= ~BE_DESTROY_WANTORIGIN; + if (!zfs_dataset_exists(lbh->lzh, path, ZFS_TYPE_SNAPSHOT)) + return (set_error(lbh, BE_ERR_NOENT)); + + bdd.snapname = strdup(snapdelim + 1); + if (bdd.snapname == NULL) + return (set_error(lbh, BE_ERR_NOMEM)); + *snapdelim = '\0'; + fs = zfs_open(lbh->lzh, path, ZFS_TYPE_DATASET); + if (fs == NULL) { + free(bdd.snapname); + return (set_error(lbh, BE_ERR_ZFSOPEN)); + } + } + + /* + * Whether we're destroying a BE or a single snapshot, we need to walk + * the tree of what we're going to destroy and promote everything in our + * path so that we can make it happen. + */ + if ((err = be_promote_dependent_clones(fs, &bdd)) != 0) { + free(bdd.snapname); + + /* + * If we're just destroying the origin of some other dataset + * we were invoked to destroy, then we just ignore + * BE_ERR_HASCLONES and return success unless the caller wanted + * to force the issue. + */ + if (odestroyer && err == BE_ERR_HASCLONES && + (options & BE_DESTROY_AUTOORIGIN) != 0) + return (0); + return (set_error(lbh, err)); + } + + /* + * This was deferred until after we promote all of the derivatives so + * that we grab the new origin after everything's settled down. + */ + if ((options & BE_DESTROY_WANTORIGIN) != 0 && + zfs_prop_get(fs, ZFS_PROP_ORIGIN, origin, sizeof(origin), + NULL, NULL, 0, 1) != 0 && + (options & BE_DESTROY_ORIGIN) != 0) + return (set_error(lbh, BE_ERR_NOORIGIN)); + + /* + * If the caller wants auto-origin destruction and the origin + * name matches one of our automatically created snapshot names + * (i.e. strftime("%F-%T") with a serial at the end), then + * we'll set the DESTROY_ORIGIN flag and nuke it + * be_is_auto_snapshot_name is exported from libbe(3) so that + * the caller can determine if it needs to warn about the origin + * not being destroyed or not. + */ + if ((options & BE_DESTROY_AUTOORIGIN) != 0 && *origin != '\0' && + be_is_auto_snapshot_name(lbh, origin)) + options |= BE_DESTROY_ORIGIN; + + err = be_destroy_cb(fs, &bdd); + zfs_close(fs); + free(bdd.snapname); + if (err != 0) { + /* Children are still present or the mount is referenced */ + if (err == EBUSY) + return (set_error(lbh, BE_ERR_DESTROYMNT)); + return (set_error(lbh, BE_ERR_UNKNOWN)); + } + + if ((options & BE_DESTROY_ORIGIN) == 0) + return (0); + + /* The origin can't possibly be shorter than the BE root */ + rootlen = strlen(lbh->root); + if (*origin == '\0' || strlen(origin) <= rootlen + 1) + return (set_error(lbh, BE_ERR_INVORIGIN)); + + /* + * We'll be chopping off the BE root and running this back through + * be_destroy, so that we properly handle the origin snapshot whether + * it be that of a deep BE or not. + */ + if (strncmp(origin, lbh->root, rootlen) != 0 || origin[rootlen] != '/') + return (0); + + return (be_destroy_internal(lbh, origin + rootlen + 1, + options & ~BE_DESTROY_ORIGIN, true)); +} + +int +be_destroy(libbe_handle_t *lbh, const char *name, int options) +{ + + /* + * The consumer must not set both BE_DESTROY_AUTOORIGIN and + * BE_DESTROY_ORIGIN. Internally, we'll set the latter from the former. + * The latter should imply that we must succeed at destroying the + * origin, or complain otherwise. + */ + if ((options & BE_DESTROY_WANTORIGIN) == BE_DESTROY_WANTORIGIN) + return (set_error(lbh, BE_ERR_UNKNOWN)); + return (be_destroy_internal(lbh, name, options, false)); +} + +static void +be_setup_snapshot_name(libbe_handle_t *lbh, char *buf, size_t buflen) +{ + time_t rawtime; + int len, serial; + + time(&rawtime); + len = strlen(buf); + len += strftime(buf + len, buflen - len, "@%F-%T", localtime(&rawtime)); + /* No room for serial... caller will do its best */ + if (buflen - len < 2) + return; + + for (serial = 0; serial < BE_SNAP_SERIAL_MAX; ++serial) { + snprintf(buf + len, buflen - len, "-%d", serial); + if (!zfs_dataset_exists(lbh->lzh, buf, ZFS_TYPE_SNAPSHOT)) + return; + } +} + +bool +be_is_auto_snapshot_name(libbe_handle_t *lbh __unused, const char *name) +{ + const char *snap; + int day, hour, minute, month, second, serial, year; + + if ((snap = strchr(name, '@')) == NULL) + return (false); + ++snap; + /* We'll grab the individual components and do some light validation. */ + if (sscanf(snap, "%d-%d-%d-%d:%d:%d-%d", &year, &month, &day, &hour, + &minute, &second, &serial) != 7) + return (false); + return (year >= 1970) && (month >= 1 && month <= 12) && + (day >= 1 && day <= 31) && (hour >= 0 && hour <= 23) && + (minute >= 0 && minute <= 59) && (second >= 0 && second <= 60) && + serial >= 0; +} + +int +be_snapshot(libbe_handle_t *lbh, const char *source, const char *snap_name, + bool recursive, char *result) +{ + char buf[BE_MAXPATHLEN]; + int err; + + be_root_concat(lbh, source, buf); + + if ((err = be_exists(lbh, buf)) != 0) + return (set_error(lbh, err)); + + if (snap_name != NULL) { + if (strlcat(buf, "@", sizeof(buf)) >= sizeof(buf)) + return (set_error(lbh, BE_ERR_INVALIDNAME)); + + if (strlcat(buf, snap_name, sizeof(buf)) >= sizeof(buf)) + return (set_error(lbh, BE_ERR_INVALIDNAME)); + + if (result != NULL) + snprintf(result, BE_MAXPATHLEN, "%s@%s", source, + snap_name); + } else { + be_setup_snapshot_name(lbh, buf, sizeof(buf)); + + if (result != NULL && strlcpy(result, strrchr(buf, '/') + 1, + sizeof(buf)) >= sizeof(buf)) + return (set_error(lbh, BE_ERR_INVALIDNAME)); + } + if ((err = zfs_snapshot(lbh->lzh, buf, recursive, NULL)) != 0) { + switch (err) { + case EZFS_INVALIDNAME: + return (set_error(lbh, BE_ERR_INVALIDNAME)); + + default: + /* + * The other errors that zfs_ioc_snapshot might return + * shouldn't happen if we've set things up properly, so + * we'll gloss over them and call it UNKNOWN as it will + * require further triage. + */ + if (errno == ENOTSUP) + return (set_error(lbh, BE_ERR_NOPOOL)); + return (set_error(lbh, BE_ERR_UNKNOWN)); + } + } + + return (BE_ERR_SUCCESS); +} + + +/* + * Create the boot environment specified by the name parameter + */ +int +be_create(libbe_handle_t *lbh, const char *name) +{ + int err; + + err = be_create_from_existing(lbh, name, be_active_path(lbh)); + + return (set_error(lbh, err)); +} + +static int +be_deep_clone_prop(int prop, void *cb) +{ + int err; + struct libbe_dccb *dccb; + zprop_source_t src; + char pval[BE_MAXPATHLEN]; + char source[BE_MAXPATHLEN]; + char *val; + + dccb = cb; + /* Skip some properties we don't want to touch */ + if (prop == ZFS_PROP_CANMOUNT) + return (ZPROP_CONT); + + /* Don't copy readonly properties */ + if (zfs_prop_readonly(prop)) + return (ZPROP_CONT); + + if ((err = zfs_prop_get(dccb->zhp, prop, (char *)&pval, + sizeof(pval), &src, (char *)&source, sizeof(source), false))) + /* Just continue if we fail to read a property */ + return (ZPROP_CONT); + + /* + * Only copy locally defined or received properties. This continues + * to avoid temporary/default/local properties intentionally without + * breaking received datasets. + */ + if (src != ZPROP_SRC_LOCAL && src != ZPROP_SRC_RECEIVED) + return (ZPROP_CONT); + + /* Augment mountpoint with altroot, if needed */ + val = pval; + if (prop == ZFS_PROP_MOUNTPOINT) + val = be_mountpoint_augmented(dccb->lbh, val); + + nvlist_add_string(dccb->props, zfs_prop_to_name(prop), val); + + return (ZPROP_CONT); +} + +/* + * Return the corresponding boot environment path for a given + * dataset path, the constructed path is placed in 'result'. + * + * example: say our new boot environment name is 'bootenv' and + * the dataset path is 'zroot/ROOT/default/data/set'. + * + * result should produce: 'zroot/ROOT/bootenv/data/set' *** 2396 LINES SKIPPED ***