Add default user/group/project quota properties

This adds default userquota, groupquota, and projectquota properties to
MASTER_NODE_OBJ to make them accessible during zfsvfs_init() (regular
DSL properties require dsl_config_lock, which cannot be safely acquired
in this context). The zfs_fill_zplprops_impl() logic is updated to read
these default properties directly from MASTER_NODE_OBJ.

Signed-off-by: Ameer Hamza <ahamza@ixsystems.com>

Reviewed-by: Alexander Motin <mav@FreeBSD.org>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
This commit is contained in:
Ameer Hamza
2025-03-09 21:37:26 +05:00
committed by Tony Hutter
parent 7be9fa259e
commit 2a8d9d9607
10 changed files with 333 additions and 2 deletions
+86
View File
@@ -861,6 +861,36 @@ zfsvfs_init(zfsvfs_t *zfsvfs, objset_t *os)
zfsvfs->z_xattr_sa = B_TRUE;
}
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTUSERQUOTA,
&zfsvfs->z_defaultuserquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTGROUPQUOTA,
&zfsvfs->z_defaultgroupquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTPROJECTQUOTA,
&zfsvfs->z_defaultprojectquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTUSEROBJQUOTA,
&zfsvfs->z_defaultuserobjquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTGROUPOBJQUOTA,
&zfsvfs->z_defaultgroupobjquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTPROJECTOBJQUOTA,
&zfsvfs->z_defaultprojectobjquota);
if (error != 0)
return (error);
error = sa_setup(os, sa_obj, zfs_attr_table, ZPL_END,
&zfsvfs->z_attr_table);
if (error != 0)
@@ -2241,6 +2271,62 @@ zfs_set_version(zfsvfs_t *zfsvfs, uint64_t newvers)
return (0);
}
int
zfs_set_default_quota(zfsvfs_t *zfsvfs, zfs_prop_t prop, uint64_t quota)
{
int error;
objset_t *os = zfsvfs->z_os;
const char *propstr = zfs_prop_to_name(prop);
dmu_tx_t *tx;
tx = dmu_tx_create(os);
dmu_tx_hold_zap(tx, MASTER_NODE_OBJ, B_FALSE, propstr);
error = dmu_tx_assign(tx, DMU_TX_WAIT);
if (error) {
dmu_tx_abort(tx);
return (error);
}
if (quota == 0) {
error = zap_remove(os, MASTER_NODE_OBJ, propstr, tx);
if (error == ENOENT)
error = 0;
} else {
error = zap_update(os, MASTER_NODE_OBJ, propstr, 8, 1,
&quota, tx);
}
if (error)
goto out;
switch (prop) {
case ZFS_PROP_DEFAULTUSERQUOTA:
zfsvfs->z_defaultuserquota = quota;
break;
case ZFS_PROP_DEFAULTGROUPQUOTA:
zfsvfs->z_defaultgroupquota = quota;
break;
case ZFS_PROP_DEFAULTPROJECTQUOTA:
zfsvfs->z_defaultprojectquota = quota;
break;
case ZFS_PROP_DEFAULTUSEROBJQUOTA:
zfsvfs->z_defaultuserobjquota = quota;
break;
case ZFS_PROP_DEFAULTGROUPOBJQUOTA:
zfsvfs->z_defaultgroupobjquota = quota;
break;
case ZFS_PROP_DEFAULTPROJECTOBJQUOTA:
zfsvfs->z_defaultprojectobjquota = quota;
break;
default:
break;
}
out:
dmu_tx_commit(tx);
return (error);
}
/*
* Return true if the corresponding vfs's unmounted flag is set.
* Otherwise return false.
+87
View File
@@ -697,6 +697,36 @@ zfsvfs_init(zfsvfs_t *zfsvfs, objset_t *os)
zfsvfs->z_xattr_sa = B_TRUE;
}
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTUSERQUOTA,
&zfsvfs->z_defaultuserquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTGROUPQUOTA,
&zfsvfs->z_defaultgroupquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTPROJECTQUOTA,
&zfsvfs->z_defaultprojectquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTUSEROBJQUOTA,
&zfsvfs->z_defaultuserobjquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTGROUPOBJQUOTA,
&zfsvfs->z_defaultgroupobjquota);
if (error != 0)
return (error);
error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTPROJECTOBJQUOTA,
&zfsvfs->z_defaultprojectobjquota);
if (error != 0)
return (error);
error = zap_lookup(os, MASTER_NODE_OBJ, ZFS_ROOT_OBJ, 8, 1,
&zfsvfs->z_root);
if (error != 0)
@@ -2005,6 +2035,62 @@ zfs_set_version(zfsvfs_t *zfsvfs, uint64_t newvers)
return (0);
}
int
zfs_set_default_quota(zfsvfs_t *zfsvfs, zfs_prop_t prop, uint64_t quota)
{
int error;
objset_t *os = zfsvfs->z_os;
const char *propstr = zfs_prop_to_name(prop);
dmu_tx_t *tx;
tx = dmu_tx_create(os);
dmu_tx_hold_zap(tx, MASTER_NODE_OBJ, B_FALSE, propstr);
error = dmu_tx_assign(tx, DMU_TX_WAIT);
if (error) {
dmu_tx_abort(tx);
return (error);
}
if (quota == 0) {
error = zap_remove(os, MASTER_NODE_OBJ, propstr, tx);
if (error == ENOENT)
error = 0;
} else {
error = zap_update(os, MASTER_NODE_OBJ, propstr, 8, 1,
&quota, tx);
}
if (error)
goto out;
switch (prop) {
case ZFS_PROP_DEFAULTUSERQUOTA:
zfsvfs->z_defaultuserquota = quota;
break;
case ZFS_PROP_DEFAULTGROUPQUOTA:
zfsvfs->z_defaultgroupquota = quota;
break;
case ZFS_PROP_DEFAULTPROJECTQUOTA:
zfsvfs->z_defaultprojectquota = quota;
break;
case ZFS_PROP_DEFAULTUSEROBJQUOTA:
zfsvfs->z_defaultuserobjquota = quota;
break;
case ZFS_PROP_DEFAULTGROUPOBJQUOTA:
zfsvfs->z_defaultgroupobjquota = quota;
break;
case ZFS_PROP_DEFAULTPROJECTOBJQUOTA:
zfsvfs->z_defaultprojectobjquota = quota;
break;
default:
break;
}
out:
dmu_tx_commit(tx);
return (error);
}
/*
* Return true if the corresponding vfs's unmounted flag is set.
* Otherwise return false.
@@ -2073,4 +2159,5 @@ EXPORT_SYMBOL(zfs_remount);
EXPORT_SYMBOL(zfs_statvfs);
EXPORT_SYMBOL(zfs_vget);
EXPORT_SYMBOL(zfs_prune);
EXPORT_SYMBOL(zfs_set_default_quota);
#endif
+22
View File
@@ -708,6 +708,28 @@ zfs_prop_init(void)
zprop_register_number(ZFS_PROP_SNAPSHOT_LIMIT, "snapshot_limit",
UINT64_MAX, PROP_DEFAULT, ZFS_TYPE_FILESYSTEM | ZFS_TYPE_VOLUME,
"<count> | none", "SSLIMIT", B_FALSE, sfeatures);
zprop_register_number(ZFS_PROP_DEFAULTUSERQUOTA, "defaultuserquota", 0,
PROP_DEFAULT, ZFS_TYPE_FILESYSTEM | ZFS_TYPE_SNAPSHOT,
"<size> | none", "DEFAULTUSERQUOTA", B_FALSE, sfeatures);
zprop_register_number(ZFS_PROP_DEFAULTGROUPQUOTA, "defaultgroupquota",
0, PROP_DEFAULT, ZFS_TYPE_FILESYSTEM | ZFS_TYPE_SNAPSHOT,
"<size> | none", "DEFAULTGROUPQUOTA", B_FALSE, sfeatures);
zprop_register_number(ZFS_PROP_DEFAULTPROJECTQUOTA,
"defaultprojectquota", 0, PROP_DEFAULT,
ZFS_TYPE_FILESYSTEM | ZFS_TYPE_SNAPSHOT, "<size> | none",
"DEFAULTPROJECTQUOTA", B_FALSE, sfeatures);
zprop_register_number(ZFS_PROP_DEFAULTUSEROBJQUOTA,
"defaultuserobjquota", 0, PROP_DEFAULT,
ZFS_TYPE_FILESYSTEM | ZFS_TYPE_SNAPSHOT, "<size> | none",
"DEFAULTUSEROBJQUOTA", B_FALSE, sfeatures);
zprop_register_number(ZFS_PROP_DEFAULTGROUPOBJQUOTA,
"defaultgroupobjquota", 0, PROP_DEFAULT,
ZFS_TYPE_FILESYSTEM | ZFS_TYPE_SNAPSHOT, "<size> | none",
"DEFAULTGROUPOBJQUOTA", B_FALSE, sfeatures);
zprop_register_number(ZFS_PROP_DEFAULTPROJECTOBJQUOTA,
"defaultprojectobjquota", 0, PROP_DEFAULT,
ZFS_TYPE_FILESYSTEM | ZFS_TYPE_SNAPSHOT, "<size> | none",
"DEFAULTPROJECTOBJQUOTA", B_FALSE, sfeatures);
/* inherit number properties */
zprop_register_number(ZFS_PROP_RECORDSIZE, "recordsize",
+95 -1
View File
@@ -2259,7 +2259,19 @@ zfs_ioc_objset_zplprops(zfs_cmd_t *zc)
if ((err = nvl_add_zplprop(os, nv, ZFS_PROP_VERSION)) == 0 &&
(err = nvl_add_zplprop(os, nv, ZFS_PROP_NORMALIZE)) == 0 &&
(err = nvl_add_zplprop(os, nv, ZFS_PROP_UTF8ONLY)) == 0 &&
(err = nvl_add_zplprop(os, nv, ZFS_PROP_CASE)) == 0)
(err = nvl_add_zplprop(os, nv, ZFS_PROP_CASE)) == 0 &&
(err = nvl_add_zplprop(os, nv,
ZFS_PROP_DEFAULTUSERQUOTA)) == 0 &&
(err = nvl_add_zplprop(os, nv,
ZFS_PROP_DEFAULTGROUPQUOTA)) == 0 &&
(err = nvl_add_zplprop(os, nv,
ZFS_PROP_DEFAULTPROJECTQUOTA)) == 0 &&
(err = nvl_add_zplprop(os, nv,
ZFS_PROP_DEFAULTUSEROBJQUOTA)) == 0 &&
(err = nvl_add_zplprop(os, nv,
ZFS_PROP_DEFAULTGROUPOBJQUOTA)) == 0 &&
(err = nvl_add_zplprop(os, nv,
ZFS_PROP_DEFAULTPROJECTOBJQUOTA)) == 0)
err = put_nvlist(zc, nv);
nvlist_free(nv);
} else {
@@ -2642,6 +2654,20 @@ zfs_prop_set_special(const char *dsname, zprop_source_t source,
zfsvfs_rele(zfsvfs, FTAG);
break;
}
case ZFS_PROP_DEFAULTUSERQUOTA:
case ZFS_PROP_DEFAULTGROUPQUOTA:
case ZFS_PROP_DEFAULTPROJECTQUOTA:
case ZFS_PROP_DEFAULTUSEROBJQUOTA:
case ZFS_PROP_DEFAULTGROUPOBJQUOTA:
case ZFS_PROP_DEFAULTPROJECTOBJQUOTA:
{
zfsvfs_t *zfsvfs;
if ((err = zfsvfs_hold(dsname, FTAG, &zfsvfs, B_TRUE)) != 0)
break;
err = zfs_set_default_quota(zfsvfs, prop, intval);
zfsvfs_rele(zfsvfs, FTAG);
break;
}
default:
err = -1;
}
@@ -3331,6 +3357,9 @@ zfs_fill_zplprops_impl(objset_t *os, uint64_t zplver,
uint64_t sense = ZFS_PROP_UNDEFINED;
uint64_t norm = ZFS_PROP_UNDEFINED;
uint64_t u8 = ZFS_PROP_UNDEFINED;
uint64_t duq = ZFS_PROP_UNDEFINED, duoq = ZFS_PROP_UNDEFINED;
uint64_t dgq = ZFS_PROP_UNDEFINED, dgoq = ZFS_PROP_UNDEFINED;
uint64_t dpq = ZFS_PROP_UNDEFINED, dpoq = ZFS_PROP_UNDEFINED;
int error;
ASSERT(zplprops != NULL);
@@ -3357,6 +3386,30 @@ zfs_fill_zplprops_impl(objset_t *os, uint64_t zplver,
zfs_prop_to_name(ZFS_PROP_CASE), &sense);
(void) nvlist_remove_all(createprops,
zfs_prop_to_name(ZFS_PROP_CASE));
(void) nvlist_lookup_uint64(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTUSERQUOTA), &duq);
(void) nvlist_remove_all(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTUSERQUOTA));
(void) nvlist_lookup_uint64(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTGROUPQUOTA), &dgq);
(void) nvlist_remove_all(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTGROUPQUOTA));
(void) nvlist_lookup_uint64(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTPROJECTQUOTA), &dpq);
(void) nvlist_remove_all(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTPROJECTQUOTA));
(void) nvlist_lookup_uint64(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTUSEROBJQUOTA), &duoq);
(void) nvlist_remove_all(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTUSEROBJQUOTA));
(void) nvlist_lookup_uint64(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTGROUPOBJQUOTA), &dgoq);
(void) nvlist_remove_all(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTGROUPOBJQUOTA));
(void) nvlist_lookup_uint64(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTPROJECTOBJQUOTA), &dpoq);
(void) nvlist_remove_all(createprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTPROJECTOBJQUOTA));
}
/*
@@ -3402,6 +3455,47 @@ zfs_fill_zplprops_impl(objset_t *os, uint64_t zplver,
VERIFY(nvlist_add_uint64(zplprops,
zfs_prop_to_name(ZFS_PROP_CASE), sense) == 0);
if (duq == ZFS_PROP_UNDEFINED &&
(error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTUSERQUOTA, &duq)) != 0)
return (error);
VERIFY(nvlist_add_uint64(zplprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTUSERQUOTA), duq) == 0);
if (dgq == ZFS_PROP_UNDEFINED &&
(error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTGROUPQUOTA,
&dgq)) != 0)
return (error);
VERIFY(nvlist_add_uint64(zplprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTGROUPQUOTA), dgq) == 0);
if (dpq == ZFS_PROP_UNDEFINED &&
(error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTPROJECTQUOTA,
&dpq)) != 0)
return (error);
VERIFY(nvlist_add_uint64(zplprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTPROJECTQUOTA), dpq) == 0);
if (duoq == ZFS_PROP_UNDEFINED &&
(error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTUSEROBJQUOTA,
&duoq)) != 0)
return (error);
VERIFY(nvlist_add_uint64(zplprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTUSEROBJQUOTA), duoq) == 0);
if (dgoq == ZFS_PROP_UNDEFINED &&
(error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTGROUPOBJQUOTA,
&dgoq)) != 0)
return (error);
VERIFY(nvlist_add_uint64(zplprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTGROUPOBJQUOTA), dgoq) == 0);
if (dpoq == ZFS_PROP_UNDEFINED &&
(error = zfs_get_zplprop(os, ZFS_PROP_DEFAULTPROJECTOBJQUOTA,
&dpoq)) != 0)
return (error);
VERIFY(nvlist_add_uint64(zplprops,
zfs_prop_to_name(ZFS_PROP_DEFAULTPROJECTOBJQUOTA), dpoq) == 0);
if (is_ci)
*is_ci = (sense == ZFS_CASE_INSENSITIVE);
+8
View File
@@ -384,6 +384,14 @@ zfs_get_zplprop(objset_t *os, zfs_prop_t prop, uint64_t *value)
*value = ZFS_ACLTYPE_OFF;
#endif
break;
case ZFS_PROP_DEFAULTUSERQUOTA:
case ZFS_PROP_DEFAULTGROUPQUOTA:
case ZFS_PROP_DEFAULTPROJECTQUOTA:
case ZFS_PROP_DEFAULTUSEROBJQUOTA:
case ZFS_PROP_DEFAULTGROUPOBJQUOTA:
case ZFS_PROP_DEFAULTPROJECTOBJQUOTA:
*value = 0;
return (0);
default:
return (error);
}