Re: ZFS patches.

On Sun, Jul 27, 2008 at 8:54 AM, Pawel Jakub Dawidek <pjd_at_freebsd.org>wrote:

> Hi.
>
>        http://people.freebsd.org/~pjd/patches/zfs_20080727.patch.bz2<http://people.freebsd.org/%7Epjd/patches/zfs_20080727.patch.bz2>



> Please test, test, test. If I get enough positive feedback, I may be
> able to squeeze it into 7.1-RELEASE, but this might be hard.


If the near term goal is to have this in 7.1, it may help to post a patch
that works with 7-STABLE to test.  I've tried several bits of advice from
this thread to compile it.  I even nuked my src tree and pulled a virgin one
from cvsup.

Right now, on 7-STABLE, it stops at:

  cc -O2 -fno-strict-aliasing -pipe  -DZFS_NO_ACL
-I/usr/src/cddl/lib/libzfs/../../../sbin/mount
-I/usr/src/cddl/lib/libzfs/../../../cddl/lib/libumem
-I/usr/src/cddl/lib/libzfs/../../../sys/cddl/compat/opensolaris
-I/usr/src/cddl/lib/libzfs/../../../cddl/compat/opensolaris/include
-I/usr/src/cddl/lib/libzfs/../../../cddl/compat/opensolaris/lib/libumem
-I/usr/src/cddl/lib/libzfs/../../../cddl/contrib/opensolaris/lib/libzpool/common
-I/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/common/zfs
-I/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/fs/zfs
-I/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/sys
-I/usr/src/cddl/lib/libzfs/../../../cddl/contrib/opensolaris/head
-I/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common
-I/usr/src/cddl/lib/libzfs/../../../cddl/contrib/opensolaris/lib/libnvpair
-I/usr/src/cddl/lib/libzfs/../../../cddl/contrib/opensolaris/lib/libuutil/common
-I/usr/src/cddl/lib/libzfs/../../../cddl/contrib/opensolaris/lib/libzfs/common
-D_SOLARIS_C_SOURCE  -c
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/common/zfs/zfs_prop.c
In file included from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/sys/u8_textprep.h:31,
                 from
/usr/src/cddl/lib/libzfs/../../../cddl/contrib/opensolaris/lib/libzpool/common/sys/zfs_context.h:83,
                 from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zio.h:32,
                 from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/common/zfs/zfs_prop.c:28:
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/sys/isa_defs.h:232:1:
warning: "_LITTLE_ENDIAN" redefined
In file included from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/compat/opensolaris/machine/endian.h:32,
                 from /usr/obj/usr/src/tmp/usr/include/sys/types.h:44,
                 from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/compat/opensolaris/sys/types.h:37,
                 from /usr/obj/usr/src/tmp/usr/include/unistd.h:41,
                 from
/usr/src/cddl/lib/libzfs/../../../cddl/contrib/opensolaris/lib/libzpool/common/sys/zfs_context.h:53,
                 from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zio.h:32,
                 from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/common/zfs/zfs_prop.c:28:
/usr/obj/usr/src/tmp/usr/include/machine/endian.h:53:1: warning: this is the
location of the previous definition
In file included from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/zfs_acl.h:34,
                 from
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/common/zfs/zfs_prop.c:31:
/usr/src/cddl/lib/libzfs/../../../sys/cddl/contrib/opensolaris/uts/common/sys/acl.h:42:
error: expected specifier-qualifier-list before 'o_mode_t'
*** Error code 1

Stop in /usr/src/cddl/lib/libzfs.
*** Error code 1