From nobody Mon Jul 4 10:42:19 2022 X-Original-To: dev-commits-src-main@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 B3BCF8B7167; Mon, 4 Jul 2022 10:42:28 +0000 (UTC) (envelope-from mjguzik@gmail.com) Received: from mail-lj1-x22a.google.com (mail-lj1-x22a.google.com [IPv6:2a00:1450:4864:20::22a]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1D4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Lc2SN0Tggz4r3F; Mon, 4 Jul 2022 10:42:28 +0000 (UTC) (envelope-from mjguzik@gmail.com) Received: by mail-lj1-x22a.google.com with SMTP id n15so10540554ljg.8; Mon, 04 Jul 2022 03:42:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=bpHnWl7o3wlDH1xgictYlHwR4RFct5XjnMXkg8unhyo=; b=oUWpqkICuxuIbK85WmVBvOhpKAgjzm8G7EIu6hih1e10TCjSD7Lt7BbeneYgyY5F7H ENBcz7Dj57yceYcVm2xvPDstxgN7gtB3RCKH0TnDukAuKbqJPn31bjCd13gSgXeQkrwR v0oVZLr66wmuI6llpjiDSFk2iv9vrS7gx9CVv7BIBTiyD1BgblPzz2LLH/ZJletHSL5F nSQfRJY3n519rHSk5/OJiRAT4rf9JnR5NH8BeOvl8tVHvpLOu0k+Uwaowh1fD1VZ+eLw MGEedlYZ1p9tZJk2IU3rtb6W5zIKVUmKVStkdtjoONnr2/FLN3INttqsiGHjXAImyLkU UX8A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=bpHnWl7o3wlDH1xgictYlHwR4RFct5XjnMXkg8unhyo=; b=heOTPIG1Zd07j2KI/bG2uCeJkeTMi87pY430TtnbZaywb9cpE/sPjmi5hazpN+gk7R rAvPZRxFyPIS0EKllhFPh6qw+QqoYe/8wScW7wcclQ7Of7ngtAukWzUguLE58VrKo9xI OsKqpGYAjo+XC//AWrfdDceQuUsz3lCyTjUmnfsGu2oMkWfBMaHChpdQD8b6HwHITcPk dCH/Qyk9iz7GGSGtGO4zR2Yr7LQSkKZx668S9/x4vLUTTeN2lqPwccazrhc52huyY7Y2 3f/0O/xeLtG1DHWqxj2X0gKWX3P5ed99G88cM/FEE3pE3OsgXehImeNHsAlMIj8v40h8 E3iA== X-Gm-Message-State: AJIora/XdUIHJzCcB/N1AGQiyHBhfl9qMmgg2VBToDxpH22b/OZxpLu5 hJEuPvkCbsOg8j+PHtSSYy6n2pkl0HWUhgj2spiN7xWt X-Google-Smtp-Source: AGRyM1tuzFz3jgPJwN2VXh2hoWUrDxLfOtY3pBUfmy+ratlo7bK1YzpfuS8JFc5Ti92DFKurpdteAvWx5L2AgbEusq0= X-Received: by 2002:a2e:a58e:0:b0:25b:f7f2:eb45 with SMTP id m14-20020a2ea58e000000b0025bf7f2eb45mr11974059ljp.507.1656931340732; Mon, 04 Jul 2022 03:42:20 -0700 (PDT) List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-main@freebsd.org X-BeenThere: dev-commits-src-main@freebsd.org MIME-Version: 1.0 Received: by 2002:a05:6520:3348:b0:1f7:d26b:9189 with HTTP; Mon, 4 Jul 2022 03:42:19 -0700 (PDT) In-Reply-To: References: <202206231550.25NFoBPQ012478@gitrepo.freebsd.org> <46C9AB26-0266-4E59-867F-3D57177E4AED@freebsd.org> From: Mateusz Guzik Date: Mon, 4 Jul 2022 12:42:19 +0200 Message-ID: Subject: Re: git: 1f1e2261e341 - main - zfs: merge openzfs/zfs@deb121309 To: John Baldwin Cc: Jessica Clarke , Martin Matuska , "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspamd-Queue-Id: 4Lc2SN0Tggz4r3F X-Spamd-Bar: --- Authentication-Results: mx1.freebsd.org; dkim=pass header.d=gmail.com header.s=20210112 header.b=oUWpqkIC; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (mx1.freebsd.org: domain of mjguzik@gmail.com designates 2a00:1450:4864:20::22a as permitted sender) smtp.mailfrom=mjguzik@gmail.com X-Spamd-Result: default: False [-4.00 / 15.00]; ARC_NA(0.00)[]; TO_DN_EQ_ADDR_SOME(0.00)[]; R_DKIM_ALLOW(-0.20)[gmail.com:s=20210112]; NEURAL_HAM_MEDIUM(-1.00)[-1.000]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; R_SPF_ALLOW(-0.20)[+ip6:2a00:1450:4000::/36:c]; FREEMAIL_FROM(0.00)[gmail.com]; MIME_GOOD(-0.10)[text/plain]; TO_MATCH_ENVRCPT_ALL(0.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; RCPT_COUNT_FIVE(0.00)[6]; MID_RHS_MATCH_FROMTLD(0.00)[]; DKIM_TRACE(0.00)[gmail.com:+]; DMARC_POLICY_ALLOW(-0.50)[gmail.com,none]; RCVD_IN_DNSWL_NONE(0.00)[2a00:1450:4864:20::22a:from]; NEURAL_HAM_SHORT(-1.00)[-1.000]; MLMMJ_DEST(0.00)[dev-commits-src-all,dev-commits-src-main]; FROM_EQ_ENVFROM(0.00)[]; MIME_TRACE(0.00)[0:+]; FREEMAIL_ENVFROM(0.00)[gmail.com]; ASN(0.00)[asn:15169, ipnet:2a00:1450::/32, country:US]; RCVD_COUNT_TWO(0.00)[2]; RCVD_TLS_ALL(0.00)[]; DWL_DNSWL_NONE(0.00)[gmail.com:dkim] X-ThisMailContainsUnwantedMimeParts: N [responding to a random e-mail] The following are still broken: powerpc LINT64LE kernel failed, check _.powerpc.LINT64LE for details powerpc GENERIC64LE-NODEBUG kernel failed, check _.powerpc.GENERIC64LE-NODEBUG for details powerpc GENERIC64LE kernel failed, check _.powerpc.GENERIC64LE for details with: /usr/src/sys/contrib/openzfs/module/icp/algs/blake3/blake3_x86-64.c:78:28: error: implicit declaration of function 'zfs_vsx_available' is invalid in C99 [-Werror,-Wimplicit-function-declaration] return (kfpu_allowed() && zfs_vsx_available()); ^ /usr/src/sys/contrib/openzfs/module/icp/algs/blake3/blake3_x86-64.c:144:28: error: implicit declaration of function 'zfs_vsx_available' is invalid in C99 [-Werror,-Wimplicit-function-declaration] return (kfpu_allowed() && zfs_vsx_available()); what's the status here On 7/1/22, John Baldwin wrote: > On 6/30/22 4:22 PM, Jessica Clarke wrote: >> On 23 Jun 2022, at 16:50, Martin Matuska wrote: >>> >>> The branch main has been updated by mm: >>> >>> URL: >>> https://cgit.FreeBSD.org/src/commit/?id=3D1f1e2261e341e6ca6862f82261066= ef1705f0a7a >>> >>> commit 1f1e2261e341e6ca6862f82261066ef1705f0a7a >>> Merge: a320e9dd51e6 deb1213098e2 >>> Author: Martin Matuska >>> AuthorDate: 2022-06-23 15:47:42 +0000 >>> Commit: Martin Matuska >>> CommitDate: 2022-06-23 15:49:33 +0000 >>> >>> zfs: merge openzfs/zfs@deb121309 >>> >>> Notable upstream pull request merges: >>> #12918 Introduce BLAKE3 checksums as an OpenZFS feature >>> #13553 Reduce ZIO io_lock contention on sorted scrub >>> #13537 Improve sorted scan memory accounting >>> #13540 AVL: Remove obsolete branching optimizations >>> #13563 FreeBSD: Improve crypto_dispatch() handling >>> >>> Obtained from: OpenZFS >>> OpenZFS commit: deb1213098e2dc10e6eee5e5c57bb40584e096a6 >>> >>> cddl/lib/libicp/Makefile | 29 +- >>> cddl/lib/libicp_rescue/Makefile | 29 +- >>> cddl/lib/libzpool/Makefile | 2 + >>> sys/contrib/openzfs/AUTHORS | 1 + >>> sys/contrib/openzfs/cmd/zfs/zfs_iter.c | 6 +- >>> sys/contrib/openzfs/cmd/zfs/zfs_main.c | 68 +- >>> sys/contrib/openzfs/cmd/zpool/zpool_main.c | 4 +- >>> sys/contrib/openzfs/cmd/ztest.c | 89 + >>> sys/contrib/openzfs/config/always-arch.m4 | 2 + >>> sys/contrib/openzfs/config/kernel-add-disk.m4 | 3 +- >>> sys/contrib/openzfs/config/kernel-blk-queue.m4 | 32 + >>> sys/contrib/openzfs/config/kernel-user-ns-inum.m4 | 23 + >>> sys/contrib/openzfs/config/kernel.m4 | 2 + >>> .../contrib/dracut/90zfs/module-setup.sh.in | 3 +- >>> .../contrib/pyzfs/libzfs_core/_constants.py | 1 + >>> sys/contrib/openzfs/etc/Makefile.am | 3 + >>> .../etc/systemd/system/zfs-trim-monthly@.timer.in | 12 + >>> .../etc/systemd/system/zfs-trim-weekly@.timer.in | 12 + >>> .../etc/systemd/system/zfs-trim@.service.in | 15 + >>> sys/contrib/openzfs/include/Makefile.am | 2 + >>> sys/contrib/openzfs/include/libzfs.h | 10 + >>> .../openzfs/include/os/freebsd/spl/sys/ccompile.h | 2 + >>> .../include/os/linux/kernel/linux/blkdev_compat.h | 111 + >>> .../include/os/linux/kernel/linux/simd_powerpc.h | 34 +- >>> .../include/os/linux/kernel/linux/vfs_compat.h | 16 - >>> sys/contrib/openzfs/include/os/linux/spl/sys/uio.h | 39 +- >>> .../openzfs/include/os/linux/spl/sys/zone.h | 31 +- >>> .../include/os/linux/zfs/sys/zfs_context_os.h | 5 + >>> sys/contrib/openzfs/include/sys/blake3.h | 125 + >>> sys/contrib/openzfs/include/sys/fs/zfs.h | 6 +- >>> sys/contrib/openzfs/include/sys/zfs_chksum.h | 48 + >>> sys/contrib/openzfs/include/sys/zfs_ioctl.h | 3 +- >>> sys/contrib/openzfs/include/sys/zio.h | 1 + >>> sys/contrib/openzfs/include/sys/zio_checksum.h | 12 +- >>> sys/contrib/openzfs/include/zfeature_common.h | 1 + >>> sys/contrib/openzfs/lib/libicp/Makefile.am | 25 +- >>> sys/contrib/openzfs/lib/libspl/include/sys/simd.h | 18 + >>> sys/contrib/openzfs/lib/libspl/include/sys/types.h | 2 +- >>> sys/contrib/openzfs/lib/libspl/include/zone.h | 12 +- >>> sys/contrib/openzfs/lib/libspl/os/linux/zone.c | 32 +- >>> sys/contrib/openzfs/lib/libuutil/libuutil.abi | 2 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs.abi | 18 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs_crypto.c | 10 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs_dataset.c | 64 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs_diff.c | 2 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs_impl.h | 4 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs_pool.c | 299 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs_sendrecv.c | 36 +- >>> sys/contrib/openzfs/lib/libzfs/libzfs_util.c | 12 +- >>> .../openzfs/lib/libzfs/os/freebsd/libzfs_compat.c | 2 - >>> .../openzfs/lib/libzfs/os/linux/libzfs_pool_os.c | 2 +- >>> .../openzfs/lib/libzfs/os/linux/libzfs_util_os.c | 71 + >>> .../openzfs/lib/libzfs_core/libzfs_core.abi | 2 +- >>> sys/contrib/openzfs/lib/libzpool/Makefile.am | 2 + >>> sys/contrib/openzfs/man/Makefile.am | 2 + >>> sys/contrib/openzfs/man/man4/zfs.4 | 69 +- >>> sys/contrib/openzfs/man/man7/zfsprops.7 | 10 +- >>> sys/contrib/openzfs/man/man7/zpool-features.7 | 8 + >>> sys/contrib/openzfs/man/man8/zfs-unzone.8 | 1 + >>> sys/contrib/openzfs/man/man8/zfs-zone.8 | 116 + >>> sys/contrib/openzfs/man/man8/zpool-trim.8 | 21 + >>> sys/contrib/openzfs/module/Kbuild.in | 38 +- >>> sys/contrib/openzfs/module/Makefile.bsd | 34 +- >>> sys/contrib/openzfs/module/avl/avl.c | 24 +- >>> .../openzfs/module/icp/algs/blake3/blake3.c | 732 +++++ >>> .../module/icp/algs/blake3/blake3_generic.c | 202 ++ >>> .../openzfs/module/icp/algs/blake3/blake3_impl.c | 284 ++ >>> .../openzfs/module/icp/algs/blake3/blake3_impl.h | 213 ++ >>> .../openzfs/module/icp/algs/blake3/blake3_x86-64.c | 248 ++ >>> .../icp/asm-aarch64/blake3/b3_aarch64_sse2.S | 2450 >>> ++++++++++++++++ >>> .../icp/asm-aarch64/blake3/b3_aarch64_sse41.S | 2463 >>> ++++++++++++++++ >>> .../module/icp/asm-ppc64/blake3/b3_ppc64le_sse2.S | 2823 >>> ++++++++++++++++++ >>> .../module/icp/asm-ppc64/blake3/b3_ppc64le_sse41.S | 3064 >>> ++++++++++++++++++++ >>> .../module/icp/asm-x86_64/blake3/blake3_avx2.S | 1845 ++++++++++++ >>> .../module/icp/asm-x86_64/blake3/blake3_avx512.S | 2618 >>> +++++++++++++++++ >>> .../module/icp/asm-x86_64/blake3/blake3_sse2.S | 2323 >>> +++++++++++++++ >>> .../module/icp/asm-x86_64/blake3/blake3_sse41.S | 2058 +++++++++++++ >>> .../openzfs/module/os/freebsd/zfs/crypto_os.c | 41 +- >>> .../openzfs/module/os/linux/spl/spl-generic.c | 6 + >>> sys/contrib/openzfs/module/os/linux/spl/spl-zone.c | 424 +++ >>> sys/contrib/openzfs/module/os/linux/zfs/policy.c | 2 +- >>> .../openzfs/module/os/linux/zfs/zfs_ioctl_os.c | 47 + >>> sys/contrib/openzfs/module/os/linux/zfs/zfs_uio.c | 154 +- >>> .../openzfs/module/os/linux/zfs/zfs_vfsops.c | 20 + >>> .../openzfs/module/os/linux/zfs/zpl_ctldir.c | 17 + >>> .../openzfs/module/os/linux/zfs/zpl_super.c | 1 + >>> .../openzfs/module/os/linux/zfs/zpl_xattr.c | 5 +- >>> sys/contrib/openzfs/module/os/linux/zfs/zvol_os.c | 632 +++- >>> .../openzfs/module/zcommon/zfeature_common.c | 31 +- >>> sys/contrib/openzfs/module/zcommon/zfs_prop.c | 8 +- >>> sys/contrib/openzfs/module/zfs/blake3_zfs.c | 117 + >>> sys/contrib/openzfs/module/zfs/dsl_prop.c | 10 +- >>> sys/contrib/openzfs/module/zfs/dsl_scan.c | 26 +- >>> sys/contrib/openzfs/module/zfs/spa_misc.c | 3 + >>> sys/contrib/openzfs/module/zfs/vdev.c | 6 +- >>> sys/contrib/openzfs/module/zfs/zcp_synctask.c | 2 +- >>> sys/contrib/openzfs/module/zfs/zfs_chksum.c | 323 +++ >>> sys/contrib/openzfs/module/zfs/zfs_ioctl.c | 14 +- >>> sys/contrib/openzfs/module/zfs/zio_checksum.c | 6 + >>> sys/contrib/openzfs/tests/runfiles/common.run | 10 +- >>> sys/contrib/openzfs/tests/runfiles/linux.run | 8 +- >>> sys/contrib/openzfs/tests/zfs-tests/cmd/.gitignore | 1 + >>> .../openzfs/tests/zfs-tests/cmd/Makefile.am | 6 +- >>> .../tests/zfs-tests/cmd/checksum/blake3_test.c | 575 ++++ >>> .../tests/zfs-tests/cmd/checksum/edonr_test.c | 3 - >>> .../tests/zfs-tests/cmd/checksum/sha2_test.c | 3 - >>> .../tests/zfs-tests/cmd/checksum/skein_test.c | 3 - >>> .../openzfs/tests/zfs-tests/include/commands.cfg | 5 + >>> .../openzfs/tests/zfs-tests/include/libtest.shlib | 45 +- >>> .../tests/zfs-tests/include/properties.shlib | 2 +- >>> .../openzfs/tests/zfs-tests/include/tunables.cfg | 1 + >>> .../openzfs/tests/zfs-tests/tests/Makefile.am | 9 + >>> .../tests/functional/checksum/default.cfg | 2 +- >>> .../tests/functional/checksum/run_blake3_test.ksh | 30 + >>> .../cli_root/zfs_set/checksum_001_pos.ksh | 2 +- >>> .../functional/cli_root/zpool_get/zpool_get.cfg | 1 + >>> .../user_namespace/user_namespace_001.ksh | 5 + >>> .../user_namespace/user_namespace_002.ksh | 115 + >>> .../user_namespace/user_namespace_003.ksh | 97 + >>> .../user_namespace/user_namespace_004.ksh | 67 + >>> .../tests/functional/zvol/zvol_common.shlib | 11 + >>> .../functional/zvol/zvol_misc/zvol_misc_fua.ksh | 96 + >>> .../functional/zvol/zvol_misc/zvol_misc_trim.ksh | 136 + >>> .../tests/functional/zvol/zvol_stress/cleanup.ksh | 36 + >>> .../tests/functional/zvol/zvol_stress/setup.ksh | 36 + >>> .../functional/zvol/zvol_stress/zvol_stress.ksh | 169 ++ >>> sys/modules/zfs/Makefile | 30 +- >>> sys/modules/zfs/zfs_config.h | 10 +- >>> sys/modules/zfs/zfs_gitrev.h | 2 +- >>> 129 files changed, 25938 insertions(+), 516 deletions(-) >>> >>> diff --cc cddl/lib/libicp/Makefile >>> index 6a8e979ea836,000000000000..5b316453af9a >>> mode 100644,000000..100644 >>> --- a/cddl/lib/libicp/Makefile >>> +++ b/cddl/lib/libicp/Makefile >>> @@@ -1,92 -1,0 +1,119 @@@ >>> +# $FreeBSD$ >>> + >>> +.PATH: ${SRCTOP}/sys/contrib/openzfs/module/icp >>> + >>> +PACKAGE=3D utilities >>> +LIB=3D icp >>> +LIBADD=3D >>> + >>> + >>> +.if ${MACHINE_ARCH} =3D=3D "amd64" >>> +ASM_SOURCES_C =3D asm-x86_64/aes/aeskey.c >>> +ASM_SOURCES_AS =3D \ >>> + asm-x86_64/aes/aes_amd64.S \ >>> + asm-x86_64/aes/aes_aesni.S \ >>> + asm-x86_64/modes/gcm_pclmulqdq.S \ >>> + asm-x86_64/modes/aesni-gcm-x86_64.S \ >>> + asm-x86_64/modes/ghash-x86_64.S \ >>> + asm-x86_64/sha2/sha256_impl.S \ >>> - asm-x86_64/sha2/sha512_impl.S >>> ++ asm-x86_64/sha2/sha512_impl.S \ >>> ++ asm-x86_64/blake3/blake3_avx2.S \ >>> ++ asm-x86_64/blake3/blake3_avx512.S \ >>> ++ asm-x86_64/blake3/blake3_sse2.S \ >>> ++ asm-x86_64/blake3/blake3_sse41.S >>> + >>> +CFLAGS+=3D -D__amd64 -D_SYS_STACK_H -UHAVE_AES >>> ++.elif ${MACHINE_ARCH} =3D=3D "aarch64" >>> ++ASM_SOURCES_C =3D >>> ++ASM_SOURCES_AS =3D \ >>> ++ asm-aarch64/blake3/b3_aarch64_sse2.S \ >>> ++ asm-aarch64/blake3/b3_aarch64_sse41.S >>> ++.elif ${MACHINE_ARCH} =3D=3D "powerpc64" >> >> This should be powerpc64le, as it=E2=80=99s for little-endian not big-en= dian. >> The powerpc64le build has been broken since this commit. > > Just commit it. It's a trivial fix and the build breakage spam for all t= he > commmits since this went in isn't worth waiting on further. > > -- > John Baldwin > --=20 Mateusz Guzik