diff options
| author | Brian Behlendorf <[email protected]> | 2020-12-21 20:17:13 -0800 |
|---|---|---|
| committer | GitHub <[email protected]> | 2020-12-21 20:17:13 -0800 |
| commit | 0c763f76b1bc6d5e38a638493d13c9b54222743d (patch) | |
| tree | 626db2112d518bc6d8083ebd6c64c47ddcc65823 | |
| parent | af9593903ef287bc1c0cb1829f0863773ce93c78 (diff) | |
Remove unused check from dmu_tx_count_write()
Individual transactions may not be larger than DMU_MAX_ACCESS.
This is enforced by the assertions in dmu_tx_hold_write() and
dmu_tx_hold_write_by_dnode(). There's an additional check in
dmu_tx_count_write() however it has no effect and only sets a
local err variable. We could enable this check, however since
it's already enforced by ASSERTs elsewhere I opted to remove it
instead.
Reviewed-by: Matthew Ahrens <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #3731
Closes #11384
| -rw-r--r-- | module/zfs/dmu_tx.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/module/zfs/dmu_tx.c b/module/zfs/dmu_tx.c index 09ef2be94..0ebed4e6f 100644 --- a/module/zfs/dmu_tx.c +++ b/module/zfs/dmu_tx.c @@ -230,9 +230,6 @@ dmu_tx_count_write(dmu_tx_hold_t *txh, uint64_t off, uint64_t len) (void) zfs_refcount_add_many(&txh->txh_space_towrite, len, FTAG); - if (zfs_refcount_count(&txh->txh_space_towrite) > 2 * DMU_MAX_ACCESS) - err = SET_ERROR(EFBIG); - if (dn == NULL) return; |