Author: tsoome
Date: Wed Jan  2 07:30:27 2019
New Revision: 342683
URL: https://svnweb.freebsd.org/changeset/base/342683

Log:
  MFC r341438:
  zfs: we can boot from dataset with large_dnode enabled
  
  loader has been supporting large_dnode for some time, no need to block the
  feature for boot dataset.
  
  Relnotes:     yes

Modified:
  stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c
  stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_ioctl.c
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c
==============================================================================
--- stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c      Wed Jan 
 2 06:48:53 2019        (r342682)
+++ stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/spa.c      Wed Jan 
 2 07:30:27 2019        (r342683)
@@ -615,9 +615,7 @@ spa_prop_validate(spa_t *spa, nvlist_t *props)
 
                                /*
                                 * Must be ZPL, and its property settings
-                                * must be supported by GRUB (compression
-                                * is not gzip, and large blocks or large
-                                * dnodes are not used).
+                                * must be supported.
                                 */
 
                                if (dmu_objset_type(os) != DMU_OST_ZFS) {
@@ -627,12 +625,6 @@ spa_prop_validate(spa_t *spa, nvlist_t *props)
                                    zfs_prop_to_name(ZFS_PROP_COMPRESSION),
                                    &propval)) == 0 &&
                                    !BOOTFS_COMPRESS_VALID(propval)) {
-                                       error = SET_ERROR(ENOTSUP);
-                               } else if ((error =
-                                   dsl_prop_get_int_ds(dmu_objset_ds(os),
-                                   zfs_prop_to_name(ZFS_PROP_DNODESIZE),
-                                   &propval)) == 0 &&
-                                   propval != ZFS_DNSIZE_LEGACY) {
                                        error = SET_ERROR(ENOTSUP);
                                } else {
                                        objnum = dmu_objset_id(os);

Modified: stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_ioctl.c
==============================================================================
--- stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_ioctl.c        
Wed Jan  2 06:48:53 2019        (r342682)
+++ stable/12/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_ioctl.c        
Wed Jan  2 07:30:27 2019        (r342683)
@@ -4208,16 +4208,6 @@ zfs_check_settable(const char *dsname, nvpair_t *pair,
                    intval != ZFS_DNSIZE_LEGACY) {
                        spa_t *spa;
 
-                       /*
-                        * If this is a bootable dataset then
-                        * we don't allow large (>512B) dnodes,
-                        * because GRUB doesn't support them.
-                        */
-                       if (zfs_is_bootfs(dsname) &&
-                               intval != ZFS_DNSIZE_LEGACY) {
-                               return (SET_ERROR(EDOM));
-                       }
-
                        if ((err = spa_open(dsname, &spa, FTAG)) != 0)
                                return (err);
 
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to