Re: Native Encryption for ZFS on FreeBSD CFT

From: Matthew Macy <mmacy_at_freebsd.org>
Date: Wed, 22 Aug 2018 00:20:59 -0700
Fixed. Pull.
bc2b257d1082112cc27e56db793f5c569f603bec

On Wed, Aug 22, 2018 at 12:10 AM Matthew Macy <mmacy_at_freebsd.org> wrote:

> Yes. I _just_ rebased and broke world in the process. Fix coming up
> momentarily.
> -M
>
> On Wed, Aug 22, 2018 at 12:06 AM Outback Dingo <outbackdingo_at_gmail.com>
> wrote:
>
>> of course interesting work, but unfortunately, and as you know me,
>> what would i say next
>>
>> cc -target x86_64-unknown-freebsd12.0
>> --sysroot=/usr/obj/usr/src/amd64.amd64/tmp
>> -B/usr/obj/usr/src/amd64.amd64/tmp/usr/bin  -O2 -pipe
>> -I/usr/src/sys/cddl/compat/opensolaris
>> -I/usr/src/cddl/compat/opensolaris/include
>> -I/usr/src/cddl/compat/opensolaris/lib/libumem
>> -I/usr/src/cddl/contrib/opensolaris/lib/libzpool/common
>> -I/usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs
>> -I/usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/lua
>> -I/usr/src/sys/cddl/contrib/opensolaris/common/zfs
>> -I/usr/src/sys/cddl/contrib/opensolaris/uts/common
>> -I/usr/src/cddl/contrib/opensolaris/head
>> -I/usr/src/cddl/contrib/opensolaris/lib/libnvpair
>> -I/usr/src/cddl/contrib/opensolaris/lib/libcmdutils
>> -DWANTS_MUTEX_OWNED -I/usr/src/lib/libpthread/thread
>> -I/usr/src/lib/libpthread/sys -I/usr/src/lib/libthr/arch/amd64/include
>> -g -DDEBUG=1 -DZFS_DEBUG=1   -DNEED_SOLARIS_BOOLEAN -g -MD
>> -MF.depend.freebsd_crypto.o -MTfreebsd_crypto.o -std=iso9899:1999
>> -fstack-protector-strong -Wno-pointer-sign -Wno-unknown-pragmas
>> -Wno-empty-body -Wno-string-plus-int -Wno-unused-const-variable
>> -Wno-tautological-compare -Wno-unused-value -Wno-parentheses-equality
>> -Wno-unused-function -Wno-enum-conversion -Wno-unused-local-typedef
>> -Wno-address-of-packed-member -Wno-switch -Wno-switch-enum
>> -Wno-knr-promoted-parameter -Wno-parentheses  -Qunused-arguments  -c
>> /usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/freebsd_crypto.c
>> -o freebsd_crypto.o
>>
>> /usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/freebsd_crypto.c:62:19:
>> warning: tentative definition of variable with internal linkage has
>> incomplete non-array type 'struct mtx'
>> [-Wtentative-definition-incomplete-type]
>> static struct mtx freebsd_crypto_mutex;
>>                   ^
>>
>> /usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/freebsd_crypto.c:62:15:
>> note: forward declaration of 'struct mtx'
>> static struct mtx freebsd_crypto_mutex;
>>               ^
>>
>> /usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/freebsd_crypto.c:63:35:
>> error: expected identifier
>> MTX_SYSINIT(freebsd_crypto_mutex, &freebsd_crypto_mutex, "FreeBSD ZFS
>> Crypto mutex", MTX_DEF);
>>                                   ^
>>
>> /usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/freebsd_crypto.c:63:1:
>> warning: type specifier missing, defaults to 'int' [-Wimplicit-int]
>> MTX_SYSINIT(freebsd_crypto_mutex, &freebsd_crypto_mutex, "FreeBSD ZFS
>> Crypto mutex", MTX_DEF);
>> ^
>>
>> /usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/freebsd_crypto.c:62:19:
>> error: tentative definition has type 'struct mtx' that is never
>> completed
>> static struct mtx freebsd_crypto_mutex;
>>                   ^
>>
>> /usr/src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/freebsd_crypto.c:62:15:
>> note: forward declaration of 'struct mtx'
>> static struct mtx freebsd_crypto_mutex;
>>               ^
>> 2 warnings and 2 errors generated.
>> *** Error code 1
>> On Wed, Aug 22, 2018 at 4:28 AM Matthew Macy <mmacy_at_freebsd.org> wrote:
>> >
>> > On Tue, Aug 21, 2018 at 6:55 PM Matthew Macy <mmacy_at_freebsd.org> wrote:
>> >
>> > > To anyone with an interest in native encryption in ZFS please test the
>> > > projects/zfs-crypto-merge-0820 branch in my freebsd repo:
>> > > https://github.com/mattmacy/networking.git
>> > >
>> > >
>> > Oh and I neglected to state that this work is being supported by iX
>> Systems
>> > and the tree is all built on work done by Sean Fagan at iX Systems.
>> Please
>> > keep him in the loop on any problems encountered.
>> > Thanks.
>> >
>> >
>> >
>> > > ( git clone  https://github.com/mattmacy/networking.git -b
>> > > projects/zfs-crypto-merge-0820 )
>> > >
>> > > The UI is quite close to the Oracle Solaris ZFS crypto with minor
>> > > differences for specifying key location.
>> > >
>> > > Please note that once a feature is enabled on a pool it can't be
>> > > disabled. This means that if you enable encryption support on a pool
>> > > you will never be able to import it in to a ZFS without encryption
>> > > support. For this reason I would strongly advise against using this on
>> > > any pool that can't be easily replaced until this change has made its
>> > > way in to HEAD after the freeze has been lifted.
>> > >
>> > >
>> > > By way of background the original ZoL commit can be found at:
>> > >
>> > >
>> https://github.com/zfsonlinux/zfs/pull/5769/commits/5aef9bedc801830264428c64cd2242d1b786fd49
>> > >
>> > > Thanks in advance.
>> > > -M
>> > >
>> > _______________________________________________
>> > freebsd-current_at_freebsd.org mailing list
>> > https://lists.freebsd.org/mailman/listinfo/freebsd-current
>> > To unsubscribe, send any mail to "
>> freebsd-current-unsubscribe_at_freebsd.org"
>>
>
Received on Wed Aug 22 2018 - 05:21:12 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:17 UTC