c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2024-11-17 10:01:01 +03:00
Code Issues Packages Projects Releases Wiki Activity

Revert "dmu_objset_userquota_get_ids uses dn_bonus unsafely"

Browse Source 
This reverts commit 5f8e1e8505.  It
was determined that this patch introduced the quota regression
described in #3789.

Signed-off-by: Tim Chase <tim@chase2k.com>
Signed-off-by: Ned Bass <bass6@llnl.gov>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Issue #3443
Issue #3789
This commit is contained in:
Brian Behlendorf 2015-09-25 11:15:02 -07:00
parent 5592404784
commit a3000f9358
1 changed files with 3 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
module/zfs/dmu_objset.c
View File

@ -1342,7 +1342,6 @@ dmu_objset_userquota_get_ids(dnode_t *dn, boolean_t before, dmu_tx_t *tx)
int flags = dn->dn_id_flags;
int error;
boolean_t have_spill = B_FALSE;
boolean_t have_bonus = B_FALSE;
if (!dmu_objset_userused_enabled(dn->dn_objset))
return;
@ -1354,21 +1353,8 @@ dmu_objset_userquota_get_ids(dnode_t *dn, boolean_t before, dmu_tx_t *tx)
if (before && dn->dn_bonuslen != 0)
data = DN_BONUS(dn->dn_phys);
else if (!before && dn->dn_bonuslen != 0) {
db = dn->dn_bonus;
if (db != NULL) {
if (!RW_WRITE_HELD(&dn->dn_struct_rwlock)) {
have_bonus = dbuf_try_add_ref((dmu_buf_t *)db,
dn->dn_objset, dn->dn_object,
DMU_BONUS_BLKID, FTAG);
/*
* The hold will fail if the buffer is
* being evicted due to unlink, in which
* case nothing needs to be done.
*/
if (!have_bonus)
return;
}
if (dn->dn_bonus) {
db = dn->dn_bonus;
mutex_enter(&db->db_mtx);
data = dmu_objset_userquota_find_data(db, tx);
} else {
@ -1443,7 +1429,7 @@ dmu_objset_userquota_get_ids(dnode_t *dn, boolean_t before, dmu_tx_t *tx)
dn->dn_id_flags |= DN_ID_CHKED_BONUS;
}
mutex_exit(&dn->dn_mtx);
if (have_spill || have_bonus)
if (have_spill)
dmu_buf_rele((dmu_buf_t *)db, FTAG);
}