author | Matthew Ahrens <mahrens@delphix.com> |
Tue, 20 Aug 2013 20:11:52 -0800 | |
changeset 14234 | 189ec27885ae |
parent 14204 | 6f5ac5d649af |
child 14247 | dceb17481b99 |
permissions | -rw-r--r-- |
789 | 1 |
/* |
2 |
* CDDL HEADER START |
|
3 |
* |
|
4 |
* The contents of this file are subject to the terms of the |
|
1491
bdcb30e07e7d
6389368 fat zap should use 16k blocks (with backwards compatability)
ahrens
parents:
873
diff
changeset
|
5 |
* Common Development and Distribution License (the "License"). |
bdcb30e07e7d
6389368 fat zap should use 16k blocks (with backwards compatability)
ahrens
parents:
873
diff
changeset
|
6 |
* You may not use this file except in compliance with the License. |
789 | 7 |
* |
8 |
* You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE |
|
9 |
* or http://www.opensolaris.org/os/licensing. |
|
10 |
* See the License for the specific language governing permissions |
|
11 |
* and limitations under the License. |
|
12 |
* |
|
13 |
* When distributing Covered Code, include this CDDL HEADER in each |
|
14 |
* file and include the License file at usr/src/OPENSOLARIS.LICENSE. |
|
15 |
* If applicable, add the following below this CDDL HEADER, with the |
|
16 |
* fields enclosed by brackets "[]" replaced with your own identifying |
|
17 |
* information: Portions Copyright [yyyy] [name of copyright owner] |
|
18 |
* |
|
19 |
* CDDL HEADER END |
|
20 |
*/ |
|
21 |
/* |
|
12411
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
22 |
* Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved. |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
23 |
* Copyright 2011 Nexenta Systems, Inc. All rights reserved. |
14005
9e23a7f7b812
3422 zpool create/syseventd race yield non-importable pool
George Wilson <george.wilson@delphix.com>
parents:
13882
diff
changeset
|
24 |
* Copyright (c) 2013 by Delphix. All rights reserved. |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
25 |
*/ |
789 | 26 |
|
27 |
#include <sys/dmu.h> |
|
28 |
#include <sys/dmu_impl.h> |
|
29 |
#include <sys/dbuf.h> |
|
30 |
#include <sys/dmu_tx.h> |
|
31 |
#include <sys/dmu_objset.h> |
|
32 |
#include <sys/dsl_dataset.h> /* for dsl_dataset_block_freeable() */ |
|
33 |
#include <sys/dsl_dir.h> /* for dsl_dir_tempreserve_*() */ |
|
34 |
#include <sys/dsl_pool.h> |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
35 |
#include <sys/zap_impl.h> /* for fzap_default_block_shift */ |
789 | 36 |
#include <sys/spa.h> |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
37 |
#include <sys/sa.h> |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
38 |
#include <sys/sa_impl.h> |
789 | 39 |
#include <sys/zfs_context.h> |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
40 |
#include <sys/varargs.h> |
789 | 41 |
|
1544 | 42 |
typedef void (*dmu_tx_hold_func_t)(dmu_tx_t *tx, struct dnode *dn, |
43 |
uint64_t arg1, uint64_t arg2); |
|
44 |
||
789 | 45 |
|
46 |
dmu_tx_t * |
|
2199 | 47 |
dmu_tx_create_dd(dsl_dir_t *dd) |
789 | 48 |
{ |
49 |
dmu_tx_t *tx = kmem_zalloc(sizeof (dmu_tx_t), KM_SLEEP); |
|
50 |
tx->tx_dir = dd; |
|
13797
95aba6e49b9f
2882 implement libzfs_core
Matthew Ahrens <mahrens@delphix.com>
parents:
13750
diff
changeset
|
51 |
if (dd != NULL) |
789 | 52 |
tx->tx_pool = dd->dd_pool; |
53 |
list_create(&tx->tx_holds, sizeof (dmu_tx_hold_t), |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
54 |
offsetof(dmu_tx_hold_t, txh_node)); |
10612
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
55 |
list_create(&tx->tx_callbacks, sizeof (dmu_tx_callback_t), |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
56 |
offsetof(dmu_tx_callback_t, dcb_node)); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
57 |
#ifdef ZFS_DEBUG |
789 | 58 |
refcount_create(&tx->tx_space_written); |
59 |
refcount_create(&tx->tx_space_freed); |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
60 |
#endif |
789 | 61 |
return (tx); |
62 |
} |
|
63 |
||
64 |
dmu_tx_t * |
|
65 |
dmu_tx_create(objset_t *os) |
|
66 |
{ |
|
10298
a0d52501437c
6860996 %temporary clones are not automatically destroyed on error
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
9873
diff
changeset
|
67 |
dmu_tx_t *tx = dmu_tx_create_dd(os->os_dsl_dataset->ds_dir); |
789 | 68 |
tx->tx_objset = os; |
10298
a0d52501437c
6860996 %temporary clones are not automatically destroyed on error
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
9873
diff
changeset
|
69 |
tx->tx_lastsnap_txg = dsl_dataset_prev_snap_txg(os->os_dsl_dataset); |
789 | 70 |
return (tx); |
71 |
} |
|
72 |
||
73 |
dmu_tx_t * |
|
74 |
dmu_tx_create_assigned(struct dsl_pool *dp, uint64_t txg) |
|
75 |
{ |
|
2199 | 76 |
dmu_tx_t *tx = dmu_tx_create_dd(NULL); |
789 | 77 |
|
78 |
ASSERT3U(txg, <=, dp->dp_tx.tx_open_txg); |
|
79 |
tx->tx_pool = dp; |
|
80 |
tx->tx_txg = txg; |
|
81 |
tx->tx_anyobj = TRUE; |
|
82 |
||
83 |
return (tx); |
|
84 |
} |
|
85 |
||
86 |
int |
|
87 |
dmu_tx_is_syncing(dmu_tx_t *tx) |
|
88 |
{ |
|
89 |
return (tx->tx_anyobj); |
|
90 |
} |
|
91 |
||
92 |
int |
|
93 |
dmu_tx_private_ok(dmu_tx_t *tx) |
|
94 |
{ |
|
1544 | 95 |
return (tx->tx_anyobj); |
789 | 96 |
} |
97 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
98 |
static dmu_tx_hold_t * |
789 | 99 |
dmu_tx_hold_object_impl(dmu_tx_t *tx, objset_t *os, uint64_t object, |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
100 |
enum dmu_tx_hold_type type, uint64_t arg1, uint64_t arg2) |
789 | 101 |
{ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
102 |
dmu_tx_hold_t *txh; |
789 | 103 |
dnode_t *dn = NULL; |
1544 | 104 |
int err; |
789 | 105 |
|
106 |
if (object != DMU_NEW_OBJECT) { |
|
10298
a0d52501437c
6860996 %temporary clones are not automatically destroyed on error
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
9873
diff
changeset
|
107 |
err = dnode_hold(os, object, tx, &dn); |
1544 | 108 |
if (err) { |
109 |
tx->tx_err = err; |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
110 |
return (NULL); |
1544 | 111 |
} |
789 | 112 |
|
1544 | 113 |
if (err == 0 && tx->tx_txg != 0) { |
789 | 114 |
mutex_enter(&dn->dn_mtx); |
115 |
/* |
|
116 |
* dn->dn_assigned_txg == tx->tx_txg doesn't pose a |
|
117 |
* problem, but there's no way for it to happen (for |
|
118 |
* now, at least). |
|
119 |
*/ |
|
120 |
ASSERT(dn->dn_assigned_txg == 0); |
|
121 |
dn->dn_assigned_txg = tx->tx_txg; |
|
122 |
(void) refcount_add(&dn->dn_tx_holds, tx); |
|
123 |
mutex_exit(&dn->dn_mtx); |
|
124 |
} |
|
125 |
} |
|
126 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
127 |
txh = kmem_zalloc(sizeof (dmu_tx_hold_t), KM_SLEEP); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
128 |
txh->txh_tx = tx; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
129 |
txh->txh_dnode = dn; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
130 |
#ifdef ZFS_DEBUG |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
131 |
txh->txh_type = type; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
132 |
txh->txh_arg1 = arg1; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
133 |
txh->txh_arg2 = arg2; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
134 |
#endif |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
135 |
list_insert_tail(&tx->tx_holds, txh); |
1544 | 136 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
137 |
return (txh); |
789 | 138 |
} |
139 |
||
140 |
void |
|
141 |
dmu_tx_add_new_object(dmu_tx_t *tx, objset_t *os, uint64_t object) |
|
142 |
{ |
|
143 |
/* |
|
144 |
* If we're syncing, they can manipulate any object anyhow, and |
|
145 |
* the hold on the dnode_t can cause problems. |
|
146 |
*/ |
|
147 |
if (!dmu_tx_is_syncing(tx)) { |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
148 |
(void) dmu_tx_hold_object_impl(tx, os, |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
149 |
object, THT_NEWOBJECT, 0, 0); |
789 | 150 |
} |
151 |
} |
|
152 |
||
1544 | 153 |
static int |
154 |
dmu_tx_check_ioerr(zio_t *zio, dnode_t *dn, int level, uint64_t blkid) |
|
155 |
{ |
|
156 |
int err; |
|
157 |
dmu_buf_impl_t *db; |
|
158 |
||
159 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
|
160 |
db = dbuf_hold_level(dn, level, blkid, FTAG); |
|
161 |
rw_exit(&dn->dn_struct_rwlock); |
|
162 |
if (db == NULL) |
|
14054
d7059eb1884c
3598 want to dtrace when errors are generated in zfs
Matthew Ahrens <mahrens@delphix.com>
parents:
14046
diff
changeset
|
163 |
return (SET_ERROR(EIO)); |
6245
1a2a7cfb9f26
6429205 each zpool needs to monitor its throughput and throttle heavy writers
maybee
parents:
5765
diff
changeset
|
164 |
err = dbuf_read(db, zio, DB_RF_CANFAIL | DB_RF_NOPREFETCH); |
1544 | 165 |
dbuf_rele(db, FTAG); |
166 |
return (err); |
|
167 |
} |
|
168 |
||
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
169 |
static void |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
170 |
dmu_tx_count_twig(dmu_tx_hold_t *txh, dnode_t *dn, dmu_buf_impl_t *db, |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
171 |
int level, uint64_t blkid, boolean_t freeable, uint64_t *history) |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
172 |
{ |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
173 |
objset_t *os = dn->dn_objset; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
174 |
dsl_dataset_t *ds = os->os_dsl_dataset; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
175 |
int epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
176 |
dmu_buf_impl_t *parent = NULL; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
177 |
blkptr_t *bp = NULL; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
178 |
uint64_t space; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
179 |
|
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
180 |
if (level >= dn->dn_nlevels || history[level] == blkid) |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
181 |
return; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
182 |
|
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
183 |
history[level] = blkid; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
184 |
|
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
185 |
space = (level == 0) ? dn->dn_datablksz : (1ULL << dn->dn_indblkshift); |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
186 |
|
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
187 |
if (db == NULL || db == dn->dn_dbuf) { |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
188 |
ASSERT(level != 0); |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
189 |
db = NULL; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
190 |
} else { |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
191 |
ASSERT(DB_DNODE(db) == dn); |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
192 |
ASSERT(db->db_level == level); |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
193 |
ASSERT(db->db.db_size == space); |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
194 |
ASSERT(db->db_blkid == blkid); |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
195 |
bp = db->db_blkptr; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
196 |
parent = db->db_parent; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
197 |
} |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
198 |
|
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
199 |
freeable = (bp && (freeable || |
12450
c77e20e4e046
6938089 dedup-induced latency causes FC initiator logouts/FC port resets
George Wilson <George.Wilson@Sun.COM>
parents:
12411
diff
changeset
|
200 |
dsl_dataset_block_freeable(ds, bp, bp->blk_birth))); |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
201 |
|
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
202 |
if (freeable) |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
203 |
txh->txh_space_tooverwrite += space; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
204 |
else |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
205 |
txh->txh_space_towrite += space; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
206 |
if (bp) |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
207 |
txh->txh_space_tounref += bp_get_dsize(os->os_spa, bp); |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
208 |
|
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
209 |
dmu_tx_count_twig(txh, dn, parent, level + 1, |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
210 |
blkid >> epbs, freeable, history); |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
211 |
} |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
212 |
|
789 | 213 |
/* ARGSUSED */ |
214 |
static void |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
215 |
dmu_tx_count_write(dmu_tx_hold_t *txh, uint64_t off, uint64_t len) |
789 | 216 |
{ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
217 |
dnode_t *dn = txh->txh_dnode; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
218 |
uint64_t start, end, i; |
789 | 219 |
int min_bs, max_bs, min_ibs, max_ibs, epbs, bits; |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
220 |
int err = 0; |
789 | 221 |
|
222 |
if (len == 0) |
|
223 |
return; |
|
224 |
||
225 |
min_bs = SPA_MINBLOCKSHIFT; |
|
226 |
max_bs = SPA_MAXBLOCKSHIFT; |
|
227 |
min_ibs = DN_MIN_INDBLKSHIFT; |
|
228 |
max_ibs = DN_MAX_INDBLKSHIFT; |
|
229 |
||
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
230 |
if (dn) { |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
231 |
uint64_t history[DN_MAX_LEVELS]; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
232 |
int nlvls = dn->dn_nlevels; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
233 |
int delta; |
1544 | 234 |
|
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
235 |
/* |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
236 |
* For i/o error checking, read the first and last level-0 |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
237 |
* blocks (if they are not aligned), and all the level-1 blocks. |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
238 |
*/ |
1544 | 239 |
if (dn->dn_maxblkid == 0) { |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
240 |
delta = dn->dn_datablksz; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
241 |
start = (off < dn->dn_datablksz) ? 0 : 1; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
242 |
end = (off+len <= dn->dn_datablksz) ? 0 : 1; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
243 |
if (start == 0 && (off > 0 || len < dn->dn_datablksz)) { |
7872
40a9434212f6
6646775 Speed up the dumpifying process for zvols
Tim Haley <Tim.Haley@Sun.COM>
parents:
7754
diff
changeset
|
244 |
err = dmu_tx_check_ioerr(NULL, dn, 0, 0); |
40a9434212f6
6646775 Speed up the dumpifying process for zvols
Tim Haley <Tim.Haley@Sun.COM>
parents:
7754
diff
changeset
|
245 |
if (err) |
40a9434212f6
6646775 Speed up the dumpifying process for zvols
Tim Haley <Tim.Haley@Sun.COM>
parents:
7754
diff
changeset
|
246 |
goto out; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
247 |
delta -= off; |
7872
40a9434212f6
6646775 Speed up the dumpifying process for zvols
Tim Haley <Tim.Haley@Sun.COM>
parents:
7754
diff
changeset
|
248 |
} |
1544 | 249 |
} else { |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
250 |
zio_t *zio = zio_root(dn->dn_objset->os_spa, |
1544 | 251 |
NULL, NULL, ZIO_FLAG_CANFAIL); |
252 |
||
253 |
/* first level-0 block */ |
|
2082 | 254 |
start = off >> dn->dn_datablkshift; |
255 |
if (P2PHASE(off, dn->dn_datablksz) || |
|
256 |
len < dn->dn_datablksz) { |
|
257 |
err = dmu_tx_check_ioerr(zio, dn, 0, start); |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
258 |
if (err) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
259 |
goto out; |
1544 | 260 |
} |
261 |
||
262 |
/* last level-0 block */ |
|
2082 | 263 |
end = (off+len-1) >> dn->dn_datablkshift; |
7872
40a9434212f6
6646775 Speed up the dumpifying process for zvols
Tim Haley <Tim.Haley@Sun.COM>
parents:
7754
diff
changeset
|
264 |
if (end != start && end <= dn->dn_maxblkid && |
2082 | 265 |
P2PHASE(off+len, dn->dn_datablksz)) { |
1544 | 266 |
err = dmu_tx_check_ioerr(zio, dn, 0, end); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
267 |
if (err) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
268 |
goto out; |
1544 | 269 |
} |
270 |
||
271 |
/* level-1 blocks */ |
|
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
272 |
if (nlvls > 1) { |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
273 |
int shft = dn->dn_indblkshift - SPA_BLKPTRSHIFT; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
274 |
for (i = (start>>shft)+1; i < end>>shft; i++) { |
1544 | 275 |
err = dmu_tx_check_ioerr(zio, dn, 1, i); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
276 |
if (err) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
277 |
goto out; |
1544 | 278 |
} |
279 |
} |
|
280 |
||
281 |
err = zio_wait(zio); |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
282 |
if (err) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
283 |
goto out; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
284 |
delta = P2NPHASE(off, dn->dn_datablksz); |
1544 | 285 |
} |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
286 |
|
14005
9e23a7f7b812
3422 zpool create/syseventd race yield non-importable pool
George Wilson <george.wilson@delphix.com>
parents:
13882
diff
changeset
|
287 |
min_ibs = max_ibs = dn->dn_indblkshift; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
288 |
if (dn->dn_maxblkid > 0) { |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
289 |
/* |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
290 |
* The blocksize can't change, |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
291 |
* so we can make a more precise estimate. |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
292 |
*/ |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
293 |
ASSERT(dn->dn_datablkshift != 0); |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
294 |
min_bs = max_bs = dn->dn_datablkshift; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
295 |
} |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
296 |
|
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
297 |
/* |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
298 |
* If this write is not off the end of the file |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
299 |
* we need to account for overwrites/unref. |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
300 |
*/ |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
301 |
if (start <= dn->dn_maxblkid) { |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
302 |
for (int l = 0; l < DN_MAX_LEVELS; l++) |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
303 |
history[l] = -1ULL; |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
304 |
} |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
305 |
while (start <= dn->dn_maxblkid) { |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
306 |
dmu_buf_impl_t *db; |
1544 | 307 |
|
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
308 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
12411
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
309 |
err = dbuf_hold_impl(dn, 0, start, FALSE, FTAG, &db); |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
310 |
rw_exit(&dn->dn_struct_rwlock); |
12411
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
311 |
|
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
312 |
if (err) { |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
313 |
txh->txh_tx->tx_err = err; |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
314 |
return; |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
315 |
} |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
316 |
|
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
317 |
dmu_tx_count_twig(txh, dn, db, 0, start, B_FALSE, |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
318 |
history); |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
319 |
dbuf_rele(db, FTAG); |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
320 |
if (++start > end) { |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
321 |
/* |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
322 |
* Account for new indirects appearing |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
323 |
* before this IO gets assigned into a txg. |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
324 |
*/ |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
325 |
bits = 64 - min_bs; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
326 |
epbs = min_ibs - SPA_BLKPTRSHIFT; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
327 |
for (bits -= epbs * (nlvls - 1); |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
328 |
bits >= 0; bits -= epbs) |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
329 |
txh->txh_fudge += 1ULL << max_ibs; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
330 |
goto out; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
331 |
} |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
332 |
off += delta; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
333 |
if (len >= delta) |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
334 |
len -= delta; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
335 |
delta = dn->dn_datablksz; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
336 |
} |
789 | 337 |
} |
338 |
||
339 |
/* |
|
340 |
* 'end' is the last thing we will access, not one past. |
|
341 |
* This way we won't overflow when accessing the last byte. |
|
342 |
*/ |
|
343 |
start = P2ALIGN(off, 1ULL << max_bs); |
|
344 |
end = P2ROUNDUP(off + len, 1ULL << max_bs) - 1; |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
345 |
txh->txh_space_towrite += end - start + 1; |
789 | 346 |
|
347 |
start >>= min_bs; |
|
348 |
end >>= min_bs; |
|
349 |
||
350 |
epbs = min_ibs - SPA_BLKPTRSHIFT; |
|
351 |
||
352 |
/* |
|
353 |
* The object contains at most 2^(64 - min_bs) blocks, |
|
354 |
* and each indirect level maps 2^epbs. |
|
355 |
*/ |
|
356 |
for (bits = 64 - min_bs; bits >= 0; bits -= epbs) { |
|
357 |
start >>= epbs; |
|
358 |
end >>= epbs; |
|
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
359 |
ASSERT3U(end, >=, start); |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
360 |
txh->txh_space_towrite += (end - start + 1) << max_ibs; |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
361 |
if (start != 0) { |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
362 |
/* |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
363 |
* We also need a new blkid=0 indirect block |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
364 |
* to reference any existing file data. |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
365 |
*/ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
366 |
txh->txh_space_towrite += 1ULL << max_ibs; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
367 |
} |
789 | 368 |
} |
369 |
||
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
370 |
out: |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
371 |
if (txh->txh_space_towrite + txh->txh_space_tooverwrite > |
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
372 |
2 * DMU_MAX_ACCESS) |
14054
d7059eb1884c
3598 want to dtrace when errors are generated in zfs
Matthew Ahrens <mahrens@delphix.com>
parents:
14046
diff
changeset
|
373 |
err = SET_ERROR(EFBIG); |
789 | 374 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
375 |
if (err) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
376 |
txh->txh_tx->tx_err = err; |
789 | 377 |
} |
378 |
||
379 |
static void |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
380 |
dmu_tx_count_dnode(dmu_tx_hold_t *txh) |
789 | 381 |
{ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
382 |
dnode_t *dn = txh->txh_dnode; |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
383 |
dnode_t *mdn = DMU_META_DNODE(txh->txh_tx->tx_objset); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
384 |
uint64_t space = mdn->dn_datablksz + |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
385 |
((mdn->dn_nlevels-1) << mdn->dn_indblkshift); |
789 | 386 |
|
387 |
if (dn && dn->dn_dbuf->db_blkptr && |
|
388 |
dsl_dataset_block_freeable(dn->dn_objset->os_dsl_dataset, |
|
12450
c77e20e4e046
6938089 dedup-induced latency causes FC initiator logouts/FC port resets
George Wilson <George.Wilson@Sun.COM>
parents:
12411
diff
changeset
|
389 |
dn->dn_dbuf->db_blkptr, dn->dn_dbuf->db_blkptr->blk_birth)) { |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
390 |
txh->txh_space_tooverwrite += space; |
8768
dfecfdbb27ed
6775697 oracle crashes when overwriting after hitting quota on zfs
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7872
diff
changeset
|
391 |
txh->txh_space_tounref += space; |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
392 |
} else { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
393 |
txh->txh_space_towrite += space; |
5378
111aa1baa84a
PSARC 2007/555 zfs fs-only quotas and reservations
ck153898
parents:
5329
diff
changeset
|
394 |
if (dn && dn->dn_dbuf->db_blkptr) |
111aa1baa84a
PSARC 2007/555 zfs fs-only quotas and reservations
ck153898
parents:
5329
diff
changeset
|
395 |
txh->txh_space_tounref += space; |
789 | 396 |
} |
397 |
} |
|
398 |
||
399 |
void |
|
400 |
dmu_tx_hold_write(dmu_tx_t *tx, uint64_t object, uint64_t off, int len) |
|
401 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
402 |
dmu_tx_hold_t *txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
403 |
|
789 | 404 |
ASSERT(tx->tx_txg == 0); |
1544 | 405 |
ASSERT(len < DMU_MAX_ACCESS); |
1819 | 406 |
ASSERT(len == 0 || UINT64_MAX - off >= len - 1); |
789 | 407 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
408 |
txh = dmu_tx_hold_object_impl(tx, tx->tx_objset, |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
409 |
object, THT_WRITE, off, len); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
410 |
if (txh == NULL) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
411 |
return; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
412 |
|
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
413 |
dmu_tx_count_write(txh, off, len); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
414 |
dmu_tx_count_dnode(txh); |
789 | 415 |
} |
416 |
||
417 |
static void |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
418 |
dmu_tx_count_free(dmu_tx_hold_t *txh, uint64_t off, uint64_t len) |
789 | 419 |
{ |
6992 | 420 |
uint64_t blkid, nblks, lastblk; |
421 |
uint64_t space = 0, unref = 0, skipped = 0; |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
422 |
dnode_t *dn = txh->txh_dnode; |
789 | 423 |
dsl_dataset_t *ds = dn->dn_objset->os_dsl_dataset; |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
424 |
spa_t *spa = txh->txh_tx->tx_pool->dp_spa; |
6992 | 425 |
int epbs; |
13847
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
426 |
uint64_t l0span = 0, nl1blks = 0; |
789 | 427 |
|
6992 | 428 |
if (dn->dn_nlevels == 0) |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
429 |
return; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
430 |
|
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
431 |
/* |
6992 | 432 |
* The struct_rwlock protects us against dn_nlevels |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
433 |
* changing, in case (against all odds) we manage to dirty & |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
434 |
* sync out the changes after we check for being dirty. |
12411
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
435 |
* Also, dbuf_hold_impl() wants us to have the struct_rwlock. |
789 | 436 |
*/ |
437 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
|
6992 | 438 |
epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT; |
439 |
if (dn->dn_maxblkid == 0) { |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
440 |
if (off == 0 && len >= dn->dn_datablksz) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
441 |
blkid = 0; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
442 |
nblks = 1; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
443 |
} else { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
444 |
rw_exit(&dn->dn_struct_rwlock); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
445 |
return; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
446 |
} |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
447 |
} else { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
448 |
blkid = off >> dn->dn_datablkshift; |
6992 | 449 |
nblks = (len + dn->dn_datablksz - 1) >> dn->dn_datablkshift; |
789 | 450 |
|
14204
6f5ac5d649af
3955 ztest failure: assertion refcount_count(&tx->tx_space_written) + delta <= tx->tx_space_towrite
Matthew Ahrens <mahrens@delphix.com>
parents:
14178
diff
changeset
|
451 |
if (blkid > dn->dn_maxblkid) { |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
452 |
rw_exit(&dn->dn_struct_rwlock); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
453 |
return; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
454 |
} |
6992 | 455 |
if (blkid + nblks > dn->dn_maxblkid) |
14204
6f5ac5d649af
3955 ztest failure: assertion refcount_count(&tx->tx_space_written) + delta <= tx->tx_space_towrite
Matthew Ahrens <mahrens@delphix.com>
parents:
14178
diff
changeset
|
456 |
nblks = dn->dn_maxblkid - blkid + 1; |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
457 |
|
1544 | 458 |
} |
13847
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
459 |
l0span = nblks; /* save for later use to calc level > 1 overhead */ |
6992 | 460 |
if (dn->dn_nlevels == 1) { |
789 | 461 |
int i; |
462 |
for (i = 0; i < nblks; i++) { |
|
463 |
blkptr_t *bp = dn->dn_phys->dn_blkptr; |
|
6992 | 464 |
ASSERT3U(blkid + i, <, dn->dn_nblkptr); |
789 | 465 |
bp += blkid + i; |
12450
c77e20e4e046
6938089 dedup-induced latency causes FC initiator logouts/FC port resets
George Wilson <George.Wilson@Sun.COM>
parents:
12411
diff
changeset
|
466 |
if (dsl_dataset_block_freeable(ds, bp, bp->blk_birth)) { |
789 | 467 |
dprintf_bp(bp, "can free old%s", ""); |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
468 |
space += bp_get_dsize(spa, bp); |
789 | 469 |
} |
5378
111aa1baa84a
PSARC 2007/555 zfs fs-only quotas and reservations
ck153898
parents:
5329
diff
changeset
|
470 |
unref += BP_GET_ASIZE(bp); |
789 | 471 |
} |
13847
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
472 |
nl1blks = 1; |
1544 | 473 |
nblks = 0; |
789 | 474 |
} |
475 |
||
6992 | 476 |
lastblk = blkid + nblks - 1; |
789 | 477 |
while (nblks) { |
478 |
dmu_buf_impl_t *dbuf; |
|
6992 | 479 |
uint64_t ibyte, new_blkid; |
480 |
int epb = 1 << epbs; |
|
481 |
int err, i, blkoff, tochk; |
|
482 |
blkptr_t *bp; |
|
789 | 483 |
|
6992 | 484 |
ibyte = blkid << dn->dn_datablkshift; |
485 |
err = dnode_next_offset(dn, |
|
486 |
DNODE_FIND_HAVELOCK, &ibyte, 2, 1, 0); |
|
487 |
new_blkid = ibyte >> dn->dn_datablkshift; |
|
7080
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
488 |
if (err == ESRCH) { |
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
489 |
skipped += (lastblk >> epbs) - (blkid >> epbs) + 1; |
6992 | 490 |
break; |
7080
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
491 |
} |
6992 | 492 |
if (err) { |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
493 |
txh->txh_tx->tx_err = err; |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
494 |
break; |
789 | 495 |
} |
7080
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
496 |
if (new_blkid > lastblk) { |
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
497 |
skipped += (lastblk >> epbs) - (blkid >> epbs) + 1; |
6992 | 498 |
break; |
7080
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
499 |
} |
6992 | 500 |
|
501 |
if (new_blkid > blkid) { |
|
7080
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
502 |
ASSERT((new_blkid >> epbs) > (blkid >> epbs)); |
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
503 |
skipped += (new_blkid >> epbs) - (blkid >> epbs) - 1; |
6992 | 504 |
nblks -= new_blkid - blkid; |
505 |
blkid = new_blkid; |
|
506 |
} |
|
507 |
blkoff = P2PHASE(blkid, epb); |
|
508 |
tochk = MIN(epb - blkoff, nblks); |
|
509 |
||
12411
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
510 |
err = dbuf_hold_impl(dn, 1, blkid >> epbs, FALSE, FTAG, &dbuf); |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
511 |
if (err) { |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
512 |
txh->txh_tx->tx_err = err; |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
513 |
break; |
3e9af067a727
6911391 panic in dmu_tx_count_write()
John Harres <John.Harres@Sun.COM>
parents:
11935
diff
changeset
|
514 |
} |
6992 | 515 |
|
516 |
txh->txh_memory_tohold += dbuf->db.db_size; |
|
11550
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
517 |
|
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
518 |
/* |
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
519 |
* We don't check memory_tohold against DMU_MAX_ACCESS because |
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
520 |
* memory_tohold is an over-estimation (especially the >L1 |
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
521 |
* indirect blocks), so it could fail. Callers should have |
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
522 |
* already verified that they will not be holding too much |
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
523 |
* memory. |
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
524 |
*/ |
e63888bb788a
6910767 deleting large holey objects hangs other I/Os
Lori Alt <Lori.Alt@Sun.COM>
parents:
10922
diff
changeset
|
525 |
|
6992 | 526 |
err = dbuf_read(dbuf, NULL, DB_RF_HAVESTRUCT | DB_RF_CANFAIL); |
527 |
if (err != 0) { |
|
528 |
txh->txh_tx->tx_err = err; |
|
529 |
dbuf_rele(dbuf, FTAG); |
|
530 |
break; |
|
531 |
} |
|
532 |
||
533 |
bp = dbuf->db.db_data; |
|
534 |
bp += blkoff; |
|
535 |
||
536 |
for (i = 0; i < tochk; i++) { |
|
12450
c77e20e4e046
6938089 dedup-induced latency causes FC initiator logouts/FC port resets
George Wilson <George.Wilson@Sun.COM>
parents:
12411
diff
changeset
|
537 |
if (dsl_dataset_block_freeable(ds, &bp[i], |
c77e20e4e046
6938089 dedup-induced latency causes FC initiator logouts/FC port resets
George Wilson <George.Wilson@Sun.COM>
parents:
12411
diff
changeset
|
538 |
bp[i].blk_birth)) { |
6992 | 539 |
dprintf_bp(&bp[i], "can free old%s", ""); |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
10612
diff
changeset
|
540 |
space += bp_get_dsize(spa, &bp[i]); |
6992 | 541 |
} |
542 |
unref += BP_GET_ASIZE(bp); |
|
543 |
} |
|
544 |
dbuf_rele(dbuf, FTAG); |
|
789 | 545 |
|
13847
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
546 |
++nl1blks; |
789 | 547 |
blkid += tochk; |
548 |
nblks -= tochk; |
|
549 |
} |
|
550 |
rw_exit(&dn->dn_struct_rwlock); |
|
551 |
||
13847
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
552 |
/* |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
553 |
* Add in memory requirements of higher-level indirects. |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
554 |
* This assumes a worst-possible scenario for dn_nlevels and a |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
555 |
* worst-possible distribution of l1-blocks over the region to free. |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
556 |
*/ |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
557 |
{ |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
558 |
uint64_t blkcnt = 1 + ((l0span >> epbs) >> epbs); |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
559 |
int level = 2; |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
560 |
/* |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
561 |
* Here we don't use DN_MAX_LEVEL, but calculate it with the |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
562 |
* given datablkshift and indblkshift. This makes the |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
563 |
* difference between 19 and 8 on large files. |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
564 |
*/ |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
565 |
int maxlevel = 2 + (DN_MAX_OFFSET_SHIFT - dn->dn_datablkshift) / |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
566 |
(dn->dn_indblkshift - SPA_BLKPTRSHIFT); |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
567 |
|
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
568 |
while (level++ < maxlevel) { |
13882
e9ad0a945d45
3189 kernel panic in ZFS test suite during hotspare_onoffline_004_neg
Christopher Siden <chris.siden@delphix.com>
parents:
13866
diff
changeset
|
569 |
txh->txh_memory_tohold += MAX(MIN(blkcnt, nl1blks), 1) |
13847
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
570 |
<< dn->dn_indblkshift; |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
571 |
blkcnt = 1 + (blkcnt >> epbs); |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
572 |
} |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
573 |
} |
f0c17d471b7a
1862 incremental zfs receive fails for sparse file > 8PB
Arne Jansen <sensille@gmx.net>
parents:
13819
diff
changeset
|
574 |
|
6992 | 575 |
/* account for new level 1 indirect blocks that might show up */ |
7080
13a30a37ddc0
6724533 assertion failed: file: ../../common/fs/zfs/dmu_tx.c, line: 983
maybee
parents:
7016
diff
changeset
|
576 |
if (skipped > 0) { |
7016
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
577 |
txh->txh_fudge += skipped << dn->dn_indblkshift; |
6992 | 578 |
skipped = MIN(skipped, DMU_MAX_DELETEBLKCNT >> epbs); |
579 |
txh->txh_memory_tohold += skipped << dn->dn_indblkshift; |
|
580 |
} |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
581 |
txh->txh_space_tofree += space; |
5378
111aa1baa84a
PSARC 2007/555 zfs fs-only quotas and reservations
ck153898
parents:
5329
diff
changeset
|
582 |
txh->txh_space_tounref += unref; |
789 | 583 |
} |
584 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
585 |
void |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
586 |
dmu_tx_hold_free(dmu_tx_t *tx, uint64_t object, uint64_t off, uint64_t len) |
789 | 587 |
{ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
588 |
dmu_tx_hold_t *txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
589 |
dnode_t *dn; |
14178
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
590 |
int err; |
1544 | 591 |
zio_t *zio; |
789 | 592 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
593 |
ASSERT(tx->tx_txg == 0); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
594 |
|
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
595 |
txh = dmu_tx_hold_object_impl(tx, tx->tx_objset, |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
596 |
object, THT_FREE, off, len); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
597 |
if (txh == NULL) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
598 |
return; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
599 |
dn = txh->txh_dnode; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
600 |
|
789 | 601 |
if (off >= (dn->dn_maxblkid+1) * dn->dn_datablksz) |
602 |
return; |
|
603 |
if (len == DMU_OBJECT_END) |
|
604 |
len = (dn->dn_maxblkid+1) * dn->dn_datablksz - off; |
|
605 |
||
14178
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
606 |
dmu_tx_count_dnode(txh); |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
607 |
|
1544 | 608 |
/* |
14178
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
609 |
* For i/o error checking, we read the first and last level-0 |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
610 |
* blocks if they are not aligned, and all the level-1 blocks. |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
611 |
* |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
612 |
* Note: dbuf_free_range() assumes that we have not instantiated |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
613 |
* any level-0 dbufs that will be completely freed. Therefore we must |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
614 |
* exercise care to not read or count the first and last blocks |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
615 |
* if they are blocksize-aligned. |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
616 |
*/ |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
617 |
if (dn->dn_datablkshift == 0) { |
14234
189ec27885ae
4047 panic from dbuf_free_range() from dmu_free_object() while doing zfs receive
Matthew Ahrens <mahrens@delphix.com>
parents:
14204
diff
changeset
|
618 |
if (off != 0 || len < dn->dn_datablksz) |
189ec27885ae
4047 panic from dbuf_free_range() from dmu_free_object() while doing zfs receive
Matthew Ahrens <mahrens@delphix.com>
parents:
14204
diff
changeset
|
619 |
dmu_tx_count_write(txh, off, len); |
14178
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
620 |
} else { |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
621 |
/* first block will be modified if it is not aligned */ |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
622 |
if (!IS_P2ALIGNED(off, 1 << dn->dn_datablkshift)) |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
623 |
dmu_tx_count_write(txh, off, 1); |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
624 |
/* last block will be modified if it is not aligned */ |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
625 |
if (!IS_P2ALIGNED(off + len, 1 << dn->dn_datablkshift)) |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
626 |
dmu_tx_count_write(txh, off+len, 1); |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
627 |
} |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
628 |
|
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
629 |
/* |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
630 |
* Check level-1 blocks. |
1544 | 631 |
*/ |
1793
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
632 |
if (dn->dn_nlevels > 1) { |
14178
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
633 |
int shift = dn->dn_datablkshift + dn->dn_indblkshift - |
1793
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
634 |
SPA_BLKPTRSHIFT; |
14178
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
635 |
uint64_t start = off >> shift; |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
636 |
uint64_t end = (off + len) >> shift; |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
637 |
|
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
638 |
ASSERT(dn->dn_datablkshift != 0); |
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
639 |
ASSERT(dn->dn_indblkshift != 0); |
1544 | 640 |
|
1793
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
641 |
zio = zio_root(tx->tx_pool->dp_spa, |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
642 |
NULL, NULL, ZIO_FLAG_CANFAIL); |
14178
277fa158020c
3834 incremental replication of 'holey' file systems is slow
Matthew Ahrens <mahrens@delphix.com>
parents:
14121
diff
changeset
|
643 |
for (uint64_t i = start; i <= end; i++) { |
1793
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
644 |
uint64_t ibyte = i << shift; |
6992 | 645 |
err = dnode_next_offset(dn, 0, &ibyte, 2, 1, 0); |
1793
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
646 |
i = ibyte >> shift; |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
647 |
if (err == ESRCH) |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
648 |
break; |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
649 |
if (err) { |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
650 |
tx->tx_err = err; |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
651 |
return; |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
652 |
} |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
653 |
|
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
654 |
err = dmu_tx_check_ioerr(zio, dn, 1, i); |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
655 |
if (err) { |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
656 |
tx->tx_err = err; |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
657 |
return; |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
658 |
} |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
659 |
} |
d371fba21a3e
6407444 unhandled i/o error from dnode_next_offset_level()
ahrens
parents:
1758
diff
changeset
|
660 |
err = zio_wait(zio); |
1544 | 661 |
if (err) { |
662 |
tx->tx_err = err; |
|
663 |
return; |
|
664 |
} |
|
665 |
} |
|
666 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
667 |
dmu_tx_count_free(txh, off, len); |
789 | 668 |
} |
669 |
||
670 |
void |
|
9396
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8890
diff
changeset
|
671 |
dmu_tx_hold_zap(dmu_tx_t *tx, uint64_t object, int add, const char *name) |
789 | 672 |
{ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
673 |
dmu_tx_hold_t *txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
674 |
dnode_t *dn; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
675 |
uint64_t nblocks; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
676 |
int epbs, err; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
677 |
|
789 | 678 |
ASSERT(tx->tx_txg == 0); |
679 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
680 |
txh = dmu_tx_hold_object_impl(tx, tx->tx_objset, |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
681 |
object, THT_ZAP, add, (uintptr_t)name); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
682 |
if (txh == NULL) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
683 |
return; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
684 |
dn = txh->txh_dnode; |
789 | 685 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
686 |
dmu_tx_count_dnode(txh); |
789 | 687 |
|
688 |
if (dn == NULL) { |
|
689 |
/* |
|
1544 | 690 |
* We will be able to fit a new object's entries into one leaf |
789 | 691 |
* block. So there will be at most 2 blocks total, |
692 |
* including the header block. |
|
693 |
*/ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
694 |
dmu_tx_count_write(txh, 0, 2 << fzap_default_block_shift); |
789 | 695 |
return; |
696 |
} |
|
697 |
||
13750
2889e2596bd6
2619 asynchronous destruction of ZFS file systems
Christopher Siden <chris.siden@delphix.com>
parents:
13497
diff
changeset
|
698 |
ASSERT3P(DMU_OT_BYTESWAP(dn->dn_type), ==, DMU_BSWAP_ZAP); |
789 | 699 |
|
1544 | 700 |
if (dn->dn_maxblkid == 0 && !add) { |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
701 |
blkptr_t *bp; |
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
702 |
|
789 | 703 |
/* |
704 |
* If there is only one block (i.e. this is a micro-zap) |
|
1544 | 705 |
* and we are not adding anything, the accounting is simple. |
789 | 706 |
*/ |
1544 | 707 |
err = dmu_tx_check_ioerr(NULL, dn, 0, 0); |
708 |
if (err) { |
|
709 |
tx->tx_err = err; |
|
710 |
return; |
|
711 |
} |
|
712 |
||
3245
0c86ad4b2673
6493634 zvol should return ENOTSUP on DKIOCGVTOC ioctl
maybee
parents:
3025
diff
changeset
|
713 |
/* |
0c86ad4b2673
6493634 zvol should return ENOTSUP on DKIOCGVTOC ioctl
maybee
parents:
3025
diff
changeset
|
714 |
* Use max block size here, since we don't know how much |
0c86ad4b2673
6493634 zvol should return ENOTSUP on DKIOCGVTOC ioctl
maybee
parents:
3025
diff
changeset
|
715 |
* the size will change between now and the dbuf dirty call. |
0c86ad4b2673
6493634 zvol should return ENOTSUP on DKIOCGVTOC ioctl
maybee
parents:
3025
diff
changeset
|
716 |
*/ |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
717 |
bp = &dn->dn_phys->dn_blkptr[0]; |
789 | 718 |
if (dsl_dataset_block_freeable(dn->dn_objset->os_dsl_dataset, |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
719 |
bp, bp->blk_birth)) |
3245
0c86ad4b2673
6493634 zvol should return ENOTSUP on DKIOCGVTOC ioctl
maybee
parents:
3025
diff
changeset
|
720 |
txh->txh_space_tooverwrite += SPA_MAXBLOCKSIZE; |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
721 |
else |
3245
0c86ad4b2673
6493634 zvol should return ENOTSUP on DKIOCGVTOC ioctl
maybee
parents:
3025
diff
changeset
|
722 |
txh->txh_space_towrite += SPA_MAXBLOCKSIZE; |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
723 |
if (!BP_IS_HOLE(bp)) |
8890
8c2bd5f17bf2
6798878 Unable to remove a file over NFS after hitting refquota limit
Chris Kirby <chris.kirby@sun.com>
parents:
8768
diff
changeset
|
724 |
txh->txh_space_tounref += SPA_MAXBLOCKSIZE; |
789 | 725 |
return; |
726 |
} |
|
727 |
||
1544 | 728 |
if (dn->dn_maxblkid > 0 && name) { |
729 |
/* |
|
730 |
* access the name in this fat-zap so that we'll check |
|
731 |
* for i/o errors to the leaf blocks, etc. |
|
732 |
*/ |
|
10298
a0d52501437c
6860996 %temporary clones are not automatically destroyed on error
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
9873
diff
changeset
|
733 |
err = zap_lookup(dn->dn_objset, dn->dn_object, name, |
1544 | 734 |
8, 0, NULL); |
735 |
if (err == EIO) { |
|
736 |
tx->tx_err = err; |
|
737 |
return; |
|
738 |
} |
|
739 |
} |
|
740 |
||
10298
a0d52501437c
6860996 %temporary clones are not automatically destroyed on error
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
9873
diff
changeset
|
741 |
err = zap_count_write(dn->dn_objset, dn->dn_object, name, add, |
9873
8ddc892eca6e
6847229 assertion failed: refcount_count(&tx->tx_space_written) + delta <= tx->tx_space_towrite in dmu_tx.c
Sanjeev Bagewadi <Sanjeev.Bagewadi@Sun.COM>
parents:
9653
diff
changeset
|
742 |
&txh->txh_space_towrite, &txh->txh_space_tooverwrite); |
789 | 743 |
|
744 |
/* |
|
745 |
* If the modified blocks are scattered to the four winds, |
|
746 |
* we'll have to modify an indirect twig for each. |
|
747 |
*/ |
|
748 |
epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT; |
|
749 |
for (nblocks = dn->dn_maxblkid >> epbs; nblocks != 0; nblocks >>= epbs) |
|
9653
a70048a304d1
6664765 Unable to remove files when using fat-zap and quota exceeded on ZFS filesystem
Sanjeev Bagewadi <Sanjeev.Bagewadi@Sun.COM>
parents:
9396
diff
changeset
|
750 |
if (dn->dn_objset->os_dsl_dataset->ds_phys->ds_prev_snap_obj) |
a70048a304d1
6664765 Unable to remove files when using fat-zap and quota exceeded on ZFS filesystem
Sanjeev Bagewadi <Sanjeev.Bagewadi@Sun.COM>
parents:
9396
diff
changeset
|
751 |
txh->txh_space_towrite += 3 << dn->dn_indblkshift; |
a70048a304d1
6664765 Unable to remove files when using fat-zap and quota exceeded on ZFS filesystem
Sanjeev Bagewadi <Sanjeev.Bagewadi@Sun.COM>
parents:
9396
diff
changeset
|
752 |
else |
a70048a304d1
6664765 Unable to remove files when using fat-zap and quota exceeded on ZFS filesystem
Sanjeev Bagewadi <Sanjeev.Bagewadi@Sun.COM>
parents:
9396
diff
changeset
|
753 |
txh->txh_space_tooverwrite += 3 << dn->dn_indblkshift; |
789 | 754 |
} |
755 |
||
756 |
void |
|
757 |
dmu_tx_hold_bonus(dmu_tx_t *tx, uint64_t object) |
|
758 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
759 |
dmu_tx_hold_t *txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
760 |
|
789 | 761 |
ASSERT(tx->tx_txg == 0); |
762 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
763 |
txh = dmu_tx_hold_object_impl(tx, tx->tx_objset, |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
764 |
object, THT_BONUS, 0, 0); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
765 |
if (txh) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
766 |
dmu_tx_count_dnode(txh); |
789 | 767 |
} |
768 |
||
769 |
void |
|
770 |
dmu_tx_hold_space(dmu_tx_t *tx, uint64_t space) |
|
771 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
772 |
dmu_tx_hold_t *txh; |
789 | 773 |
ASSERT(tx->tx_txg == 0); |
774 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
775 |
txh = dmu_tx_hold_object_impl(tx, tx->tx_objset, |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
776 |
DMU_NEW_OBJECT, THT_SPACE, space, 0); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
777 |
|
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
778 |
txh->txh_space_towrite += space; |
789 | 779 |
} |
780 |
||
781 |
int |
|
782 |
dmu_tx_holds(dmu_tx_t *tx, uint64_t object) |
|
783 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
784 |
dmu_tx_hold_t *txh; |
789 | 785 |
int holds = 0; |
786 |
||
787 |
/* |
|
788 |
* By asserting that the tx is assigned, we're counting the |
|
789 |
* number of dn_tx_holds, which is the same as the number of |
|
790 |
* dn_holds. Otherwise, we'd be counting dn_holds, but |
|
791 |
* dn_tx_holds could be 0. |
|
792 |
*/ |
|
793 |
ASSERT(tx->tx_txg != 0); |
|
794 |
||
795 |
/* if (tx->tx_anyobj == TRUE) */ |
|
796 |
/* return (0); */ |
|
797 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
798 |
for (txh = list_head(&tx->tx_holds); txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
799 |
txh = list_next(&tx->tx_holds, txh)) { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
800 |
if (txh->txh_dnode && txh->txh_dnode->dn_object == object) |
789 | 801 |
holds++; |
802 |
} |
|
803 |
||
804 |
return (holds); |
|
805 |
} |
|
806 |
||
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
807 |
#ifdef ZFS_DEBUG |
789 | 808 |
void |
809 |
dmu_tx_dirty_buf(dmu_tx_t *tx, dmu_buf_impl_t *db) |
|
810 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
811 |
dmu_tx_hold_t *txh; |
789 | 812 |
int match_object = FALSE, match_offset = FALSE; |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
813 |
dnode_t *dn; |
789 | 814 |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
815 |
DB_DNODE_ENTER(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
816 |
dn = DB_DNODE(db); |
789 | 817 |
ASSERT(tx->tx_txg != 0); |
10298
a0d52501437c
6860996 %temporary clones are not automatically destroyed on error
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
9873
diff
changeset
|
818 |
ASSERT(tx->tx_objset == NULL || dn->dn_objset == tx->tx_objset); |
789 | 819 |
ASSERT3U(dn->dn_object, ==, db->db.db_object); |
820 |
||
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
821 |
if (tx->tx_anyobj) { |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
822 |
DB_DNODE_EXIT(db); |
789 | 823 |
return; |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
824 |
} |
789 | 825 |
|
826 |
/* XXX No checking on the meta dnode for now */ |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
827 |
if (db->db.db_object == DMU_META_DNODE_OBJECT) { |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
828 |
DB_DNODE_EXIT(db); |
789 | 829 |
return; |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
830 |
} |
789 | 831 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
832 |
for (txh = list_head(&tx->tx_holds); txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
833 |
txh = list_next(&tx->tx_holds, txh)) { |
789 | 834 |
ASSERT(dn == NULL || dn->dn_assigned_txg == tx->tx_txg); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
835 |
if (txh->txh_dnode == dn && txh->txh_type != THT_NEWOBJECT) |
789 | 836 |
match_object = TRUE; |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
837 |
if (txh->txh_dnode == NULL || txh->txh_dnode == dn) { |
789 | 838 |
int datablkshift = dn->dn_datablkshift ? |
839 |
dn->dn_datablkshift : SPA_MAXBLOCKSHIFT; |
|
840 |
int epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT; |
|
841 |
int shift = datablkshift + epbs * db->db_level; |
|
842 |
uint64_t beginblk = shift >= 64 ? 0 : |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
843 |
(txh->txh_arg1 >> shift); |
789 | 844 |
uint64_t endblk = shift >= 64 ? 0 : |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
845 |
((txh->txh_arg1 + txh->txh_arg2 - 1) >> shift); |
789 | 846 |
uint64_t blkid = db->db_blkid; |
847 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
848 |
/* XXX txh_arg2 better not be zero... */ |
789 | 849 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
850 |
dprintf("found txh type %x beginblk=%llx endblk=%llx\n", |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
851 |
txh->txh_type, beginblk, endblk); |
789 | 852 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
853 |
switch (txh->txh_type) { |
789 | 854 |
case THT_WRITE: |
855 |
if (blkid >= beginblk && blkid <= endblk) |
|
856 |
match_offset = TRUE; |
|
857 |
/* |
|
858 |
* We will let this hold work for the bonus |
|
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
859 |
* or spill buffer so that we don't need to |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
860 |
* hold it when creating a new object. |
789 | 861 |
*/ |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
862 |
if (blkid == DMU_BONUS_BLKID || |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
863 |
blkid == DMU_SPILL_BLKID) |
789 | 864 |
match_offset = TRUE; |
865 |
/* |
|
866 |
* They might have to increase nlevels, |
|
867 |
* thus dirtying the new TLIBs. Or the |
|
868 |
* might have to change the block size, |
|
869 |
* thus dirying the new lvl=0 blk=0. |
|
870 |
*/ |
|
871 |
if (blkid == 0) |
|
872 |
match_offset = TRUE; |
|
873 |
break; |
|
874 |
case THT_FREE: |
|
6992 | 875 |
/* |
876 |
* We will dirty all the level 1 blocks in |
|
877 |
* the free range and perhaps the first and |
|
878 |
* last level 0 block. |
|
879 |
*/ |
|
880 |
if (blkid >= beginblk && (blkid <= endblk || |
|
881 |
txh->txh_arg2 == DMU_OBJECT_END)) |
|
789 | 882 |
match_offset = TRUE; |
883 |
break; |
|
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
884 |
case THT_SPILL: |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
885 |
if (blkid == DMU_SPILL_BLKID) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
886 |
match_offset = TRUE; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
887 |
break; |
789 | 888 |
case THT_BONUS: |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
889 |
if (blkid == DMU_BONUS_BLKID) |
789 | 890 |
match_offset = TRUE; |
891 |
break; |
|
892 |
case THT_ZAP: |
|
893 |
match_offset = TRUE; |
|
894 |
break; |
|
895 |
case THT_NEWOBJECT: |
|
896 |
match_object = TRUE; |
|
897 |
break; |
|
898 |
default: |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
899 |
ASSERT(!"bad txh_type"); |
789 | 900 |
} |
901 |
} |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
902 |
if (match_object && match_offset) { |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
903 |
DB_DNODE_EXIT(db); |
789 | 904 |
return; |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
905 |
} |
789 | 906 |
} |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
907 |
DB_DNODE_EXIT(db); |
789 | 908 |
panic("dirtying dbuf obj=%llx lvl=%u blkid=%llx but not tx_held\n", |
909 |
(u_longlong_t)db->db.db_object, db->db_level, |
|
910 |
(u_longlong_t)db->db_blkid); |
|
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
911 |
} |
789 | 912 |
#endif |
913 |
||
914 |
static int |
|
14046
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
915 |
dmu_tx_try_assign(dmu_tx_t *tx, txg_how_t txg_how) |
789 | 916 |
{ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
917 |
dmu_tx_hold_t *txh; |
5329 | 918 |
spa_t *spa = tx->tx_pool->dp_spa; |
6992 | 919 |
uint64_t memory, asize, fsize, usize; |
7016
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
920 |
uint64_t towrite, tofree, tooverwrite, tounref, tohold, fudge; |
789 | 921 |
|
13866
e3a9ae14a119
3006 VERIFY[S,U,P] and ASSERT[S,U,P] frequently check if first argument is zero
Madhav Suresh <madhav.suresh@delphix.com>
parents:
13847
diff
changeset
|
922 |
ASSERT0(tx->tx_txg); |
5329 | 923 |
|
1544 | 924 |
if (tx->tx_err) |
925 |
return (tx->tx_err); |
|
789 | 926 |
|
7754
b80e4842ad54
6754011 SPA 3.0: lock breakup, i/o pipeline refactoring, device failure handling
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
7161
diff
changeset
|
927 |
if (spa_suspended(spa)) { |
5329 | 928 |
/* |
929 |
* If the user has indicated a blocking failure mode |
|
930 |
* then return ERESTART which will block in dmu_tx_wait(). |
|
931 |
* Otherwise, return EIO so that an error can get |
|
932 |
* propagated back to the VOP calls. |
|
933 |
* |
|
934 |
* Note that we always honor the txg_how flag regardless |
|
935 |
* of the failuremode setting. |
|
936 |
*/ |
|
937 |
if (spa_get_failmode(spa) == ZIO_FAILURE_MODE_CONTINUE && |
|
938 |
txg_how != TXG_WAIT) |
|
14054
d7059eb1884c
3598 want to dtrace when errors are generated in zfs
Matthew Ahrens <mahrens@delphix.com>
parents:
14046
diff
changeset
|
939 |
return (SET_ERROR(EIO)); |
5329 | 940 |
|
14054
d7059eb1884c
3598 want to dtrace when errors are generated in zfs
Matthew Ahrens <mahrens@delphix.com>
parents:
14046
diff
changeset
|
941 |
return (SET_ERROR(ERESTART)); |
5329 | 942 |
} |
943 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
944 |
tx->tx_txg = txg_hold_open(tx->tx_pool, &tx->tx_txgh); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
945 |
tx->tx_needassign_txh = NULL; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
946 |
|
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
947 |
/* |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
948 |
* NB: No error returns are allowed after txg_hold_open, but |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
949 |
* before processing the dnode holds, due to the |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
950 |
* dmu_tx_unassign() logic. |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
951 |
*/ |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
952 |
|
7016
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
953 |
towrite = tofree = tooverwrite = tounref = tohold = fudge = 0; |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
954 |
for (txh = list_head(&tx->tx_holds); txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
955 |
txh = list_next(&tx->tx_holds, txh)) { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
956 |
dnode_t *dn = txh->txh_dnode; |
789 | 957 |
if (dn != NULL) { |
958 |
mutex_enter(&dn->dn_mtx); |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
959 |
if (dn->dn_assigned_txg == tx->tx_txg - 1) { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
960 |
mutex_exit(&dn->dn_mtx); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
961 |
tx->tx_needassign_txh = txh; |
14054
d7059eb1884c
3598 want to dtrace when errors are generated in zfs
Matthew Ahrens <mahrens@delphix.com>
parents:
14046
diff
changeset
|
962 |
return (SET_ERROR(ERESTART)); |
789 | 963 |
} |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
964 |
if (dn->dn_assigned_txg == 0) |
789 | 965 |
dn->dn_assigned_txg = tx->tx_txg; |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
966 |
ASSERT3U(dn->dn_assigned_txg, ==, tx->tx_txg); |
789 | 967 |
(void) refcount_add(&dn->dn_tx_holds, tx); |
968 |
mutex_exit(&dn->dn_mtx); |
|
969 |
} |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
970 |
towrite += txh->txh_space_towrite; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
971 |
tofree += txh->txh_space_tofree; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
972 |
tooverwrite += txh->txh_space_tooverwrite; |
5378
111aa1baa84a
PSARC 2007/555 zfs fs-only quotas and reservations
ck153898
parents:
5329
diff
changeset
|
973 |
tounref += txh->txh_space_tounref; |
6992 | 974 |
tohold += txh->txh_memory_tohold; |
7016
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
975 |
fudge += txh->txh_fudge; |
1544 | 976 |
} |
977 |
||
978 |
/* |
|
979 |
* If a snapshot has been taken since we made our estimates, |
|
980 |
* assume that we won't be able to free or overwrite anything. |
|
981 |
*/ |
|
982 |
if (tx->tx_objset && |
|
10298
a0d52501437c
6860996 %temporary clones are not automatically destroyed on error
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
9873
diff
changeset
|
983 |
dsl_dataset_prev_snap_txg(tx->tx_objset->os_dsl_dataset) > |
1544 | 984 |
tx->tx_lastsnap_txg) { |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
985 |
towrite += tooverwrite; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
986 |
tooverwrite = tofree = 0; |
789 | 987 |
} |
988 |
||
6992 | 989 |
/* needed allocation: worst-case estimate of write space */ |
990 |
asize = spa_get_asize(tx->tx_pool->dp_spa, towrite + tooverwrite); |
|
991 |
/* freed space estimate: worst-case overwrite + free estimate */ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
992 |
fsize = spa_get_asize(tx->tx_pool->dp_spa, tooverwrite) + tofree; |
6992 | 993 |
/* convert unrefd space to worst-case estimate */ |
5378
111aa1baa84a
PSARC 2007/555 zfs fs-only quotas and reservations
ck153898
parents:
5329
diff
changeset
|
994 |
usize = spa_get_asize(tx->tx_pool->dp_spa, tounref); |
6992 | 995 |
/* calculate memory footprint estimate */ |
996 |
memory = towrite + tooverwrite + tohold; |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
997 |
|
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
998 |
#ifdef ZFS_DEBUG |
7016
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
999 |
/* |
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
1000 |
* Add in 'tohold' to account for our dirty holds on this memory |
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
1001 |
* XXX - the "fudge" factor is to account for skipped blocks that |
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
1002 |
* we missed because dnode_next_offset() misses in-core-only blocks. |
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
1003 |
*/ |
6992 | 1004 |
tx->tx_space_towrite = asize + |
7016
8b85f8b0129c
6721941 dmu_tx_count_free() does not account for nlevels increasing
maybee
parents:
6992
diff
changeset
|
1005 |
spa_get_asize(tx->tx_pool->dp_spa, tohold + fudge); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1006 |
tx->tx_space_tofree = tofree; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1007 |
tx->tx_space_tooverwrite = tooverwrite; |
5378
111aa1baa84a
PSARC 2007/555 zfs fs-only quotas and reservations
ck153898
parents:
5329
diff
changeset
|
1008 |
tx->tx_space_tounref = tounref; |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1009 |
#endif |
789 | 1010 |
|
1011 |
if (tx->tx_dir && asize != 0) { |
|
6992 | 1012 |
int err = dsl_dir_tempreserve_space(tx->tx_dir, memory, |
1013 |
asize, fsize, usize, &tx->tx_tempreserve_cookie, tx); |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1014 |
if (err) |
789 | 1015 |
return (err); |
1016 |
} |
|
1017 |
||
1018 |
return (0); |
|
1019 |
} |
|
1020 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1021 |
static void |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1022 |
dmu_tx_unassign(dmu_tx_t *tx) |
789 | 1023 |
{ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1024 |
dmu_tx_hold_t *txh; |
789 | 1025 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1026 |
if (tx->tx_txg == 0) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1027 |
return; |
789 | 1028 |
|
1029 |
txg_rele_to_quiesce(&tx->tx_txgh); |
|
1030 |
||
14121
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1031 |
/* |
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1032 |
* Walk the transaction's hold list, removing the hold on the |
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1033 |
* associated dnode, and notifying waiters if the refcount drops to 0. |
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1034 |
*/ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1035 |
for (txh = list_head(&tx->tx_holds); txh != tx->tx_needassign_txh; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1036 |
txh = list_next(&tx->tx_holds, txh)) { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1037 |
dnode_t *dn = txh->txh_dnode; |
789 | 1038 |
|
1039 |
if (dn == NULL) |
|
1040 |
continue; |
|
1041 |
mutex_enter(&dn->dn_mtx); |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1042 |
ASSERT3U(dn->dn_assigned_txg, ==, tx->tx_txg); |
789 | 1043 |
|
1044 |
if (refcount_remove(&dn->dn_tx_holds, tx) == 0) { |
|
1045 |
dn->dn_assigned_txg = 0; |
|
1046 |
cv_broadcast(&dn->dn_notxholds); |
|
1047 |
} |
|
1048 |
mutex_exit(&dn->dn_mtx); |
|
1049 |
} |
|
1050 |
||
1051 |
txg_rele_to_sync(&tx->tx_txgh); |
|
1052 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1053 |
tx->tx_lasttried_txg = tx->tx_txg; |
789 | 1054 |
tx->tx_txg = 0; |
1055 |
} |
|
1056 |
||
1057 |
/* |
|
1058 |
* Assign tx to a transaction group. txg_how can be one of: |
|
1059 |
* |
|
1060 |
* (1) TXG_WAIT. If the current open txg is full, waits until there's |
|
1061 |
* a new one. This should be used when you're not holding locks. |
|
14046
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1062 |
* It will only fail if we're truly out of space (or over quota). |
789 | 1063 |
* |
1064 |
* (2) TXG_NOWAIT. If we can't assign into the current open txg without |
|
1065 |
* blocking, returns immediately with ERESTART. This should be used |
|
1066 |
* whenever you're holding locks. On an ERESTART error, the caller |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1067 |
* should drop locks, do a dmu_tx_wait(tx), and try again. |
789 | 1068 |
*/ |
1069 |
int |
|
14046
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1070 |
dmu_tx_assign(dmu_tx_t *tx, txg_how_t txg_how) |
789 | 1071 |
{ |
1072 |
int err; |
|
1073 |
||
1074 |
ASSERT(tx->tx_txg == 0); |
|
14046
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1075 |
ASSERT(txg_how == TXG_WAIT || txg_how == TXG_NOWAIT); |
789 | 1076 |
ASSERT(!dsl_pool_sync_context(tx->tx_pool)); |
1077 |
||
14046
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1078 |
/* If we might wait, we must not hold the config lock. */ |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1079 |
ASSERT(txg_how != TXG_WAIT || !dsl_pool_config_held(tx->tx_pool)); |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1080 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1081 |
while ((err = dmu_tx_try_assign(tx, txg_how)) != 0) { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1082 |
dmu_tx_unassign(tx); |
789 | 1083 |
|
1084 |
if (err != ERESTART || txg_how != TXG_WAIT) |
|
1085 |
return (err); |
|
1086 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1087 |
dmu_tx_wait(tx); |
789 | 1088 |
} |
1089 |
||
1090 |
txg_rele_to_quiesce(&tx->tx_txgh); |
|
1091 |
||
1092 |
return (0); |
|
1093 |
} |
|
1094 |
||
1095 |
void |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1096 |
dmu_tx_wait(dmu_tx_t *tx) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1097 |
{ |
5329 | 1098 |
spa_t *spa = tx->tx_pool->dp_spa; |
1099 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1100 |
ASSERT(tx->tx_txg == 0); |
14046
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1101 |
ASSERT(!dsl_pool_config_held(tx->tx_pool)); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1102 |
|
5329 | 1103 |
/* |
1104 |
* It's possible that the pool has become active after this thread |
|
1105 |
* has tried to obtain a tx. If that's the case then his |
|
1106 |
* tx_lasttried_txg would not have been assigned. |
|
1107 |
*/ |
|
7754
b80e4842ad54
6754011 SPA 3.0: lock breakup, i/o pipeline refactoring, device failure handling
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
7161
diff
changeset
|
1108 |
if (spa_suspended(spa) || tx->tx_lasttried_txg == 0) { |
5329 | 1109 |
txg_wait_synced(tx->tx_pool, spa_last_synced_txg(spa) + 1); |
1110 |
} else if (tx->tx_needassign_txh) { |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1111 |
dnode_t *dn = tx->tx_needassign_txh->txh_dnode; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1112 |
|
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1113 |
mutex_enter(&dn->dn_mtx); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1114 |
while (dn->dn_assigned_txg == tx->tx_lasttried_txg - 1) |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1115 |
cv_wait(&dn->dn_notxholds, &dn->dn_mtx); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1116 |
mutex_exit(&dn->dn_mtx); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1117 |
tx->tx_needassign_txh = NULL; |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1118 |
} else { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1119 |
txg_wait_open(tx->tx_pool, tx->tx_lasttried_txg + 1); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1120 |
} |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1121 |
} |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1122 |
|
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1123 |
void |
789 | 1124 |
dmu_tx_willuse_space(dmu_tx_t *tx, int64_t delta) |
1125 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1126 |
#ifdef ZFS_DEBUG |
789 | 1127 |
if (tx->tx_dir == NULL || delta == 0) |
1128 |
return; |
|
1129 |
||
1130 |
if (delta > 0) { |
|
1131 |
ASSERT3U(refcount_count(&tx->tx_space_written) + delta, <=, |
|
1132 |
tx->tx_space_towrite); |
|
1133 |
(void) refcount_add_many(&tx->tx_space_written, delta, NULL); |
|
1134 |
} else { |
|
1135 |
(void) refcount_add_many(&tx->tx_space_freed, -delta, NULL); |
|
1136 |
} |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1137 |
#endif |
789 | 1138 |
} |
1139 |
||
1140 |
void |
|
1141 |
dmu_tx_commit(dmu_tx_t *tx) |
|
1142 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1143 |
dmu_tx_hold_t *txh; |
789 | 1144 |
|
1145 |
ASSERT(tx->tx_txg != 0); |
|
1146 |
||
14121
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1147 |
/* |
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1148 |
* Go through the transaction's hold list and remove holds on |
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1149 |
* associated dnodes, notifying waiters if no holds remain. |
9475b3fef59d
3741 zfs needs better comments
Will Andrews <will@firepipe.net>
parents:
14054
diff
changeset
|
1150 |
*/ |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1151 |
while (txh = list_head(&tx->tx_holds)) { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1152 |
dnode_t *dn = txh->txh_dnode; |
789 | 1153 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1154 |
list_remove(&tx->tx_holds, txh); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1155 |
kmem_free(txh, sizeof (dmu_tx_hold_t)); |
789 | 1156 |
if (dn == NULL) |
1157 |
continue; |
|
1158 |
mutex_enter(&dn->dn_mtx); |
|
1159 |
ASSERT3U(dn->dn_assigned_txg, ==, tx->tx_txg); |
|
1160 |
||
1161 |
if (refcount_remove(&dn->dn_tx_holds, tx) == 0) { |
|
1162 |
dn->dn_assigned_txg = 0; |
|
1163 |
cv_broadcast(&dn->dn_notxholds); |
|
1164 |
} |
|
1165 |
mutex_exit(&dn->dn_mtx); |
|
1166 |
dnode_rele(dn, tx); |
|
1167 |
} |
|
1168 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1169 |
if (tx->tx_tempreserve_cookie) |
789 | 1170 |
dsl_dir_tempreserve_clear(tx->tx_tempreserve_cookie, tx); |
1171 |
||
10612
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1172 |
if (!list_is_empty(&tx->tx_callbacks)) |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1173 |
txg_register_callbacks(&tx->tx_txgh, &tx->tx_callbacks); |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1174 |
|
789 | 1175 |
if (tx->tx_anyobj == FALSE) |
1176 |
txg_rele_to_sync(&tx->tx_txgh); |
|
10612
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1177 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1178 |
list_destroy(&tx->tx_callbacks); |
5765
86c482ef4a34
6646545 Condvars in tx_cpu aren't being initialized/destroyed
ek110237
parents:
5378
diff
changeset
|
1179 |
list_destroy(&tx->tx_holds); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1180 |
#ifdef ZFS_DEBUG |
789 | 1181 |
dprintf("towrite=%llu written=%llu tofree=%llu freed=%llu\n", |
1182 |
tx->tx_space_towrite, refcount_count(&tx->tx_space_written), |
|
1183 |
tx->tx_space_tofree, refcount_count(&tx->tx_space_freed)); |
|
1184 |
refcount_destroy_many(&tx->tx_space_written, |
|
1185 |
refcount_count(&tx->tx_space_written)); |
|
1186 |
refcount_destroy_many(&tx->tx_space_freed, |
|
1187 |
refcount_count(&tx->tx_space_freed)); |
|
1188 |
#endif |
|
1189 |
kmem_free(tx, sizeof (dmu_tx_t)); |
|
1190 |
} |
|
1191 |
||
1192 |
void |
|
1193 |
dmu_tx_abort(dmu_tx_t *tx) |
|
1194 |
{ |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1195 |
dmu_tx_hold_t *txh; |
789 | 1196 |
|
1197 |
ASSERT(tx->tx_txg == 0); |
|
1198 |
||
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1199 |
while (txh = list_head(&tx->tx_holds)) { |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1200 |
dnode_t *dn = txh->txh_dnode; |
789 | 1201 |
|
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1202 |
list_remove(&tx->tx_holds, txh); |
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1203 |
kmem_free(txh, sizeof (dmu_tx_hold_t)); |
789 | 1204 |
if (dn != NULL) |
1205 |
dnode_rele(dn, tx); |
|
1206 |
} |
|
10612
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1207 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1208 |
/* |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1209 |
* Call any registered callbacks with an error code. |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1210 |
*/ |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1211 |
if (!list_is_empty(&tx->tx_callbacks)) |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1212 |
dmu_tx_do_callbacks(&tx->tx_callbacks, ECANCELED); |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1213 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1214 |
list_destroy(&tx->tx_callbacks); |
5765
86c482ef4a34
6646545 Condvars in tx_cpu aren't being initialized/destroyed
ek110237
parents:
5378
diff
changeset
|
1215 |
list_destroy(&tx->tx_holds); |
2113
0510bb40c993
6430121 3-way deadlock involving tc_lock within zfs
ahrens
parents:
2082
diff
changeset
|
1216 |
#ifdef ZFS_DEBUG |
789 | 1217 |
refcount_destroy_many(&tx->tx_space_written, |
1218 |
refcount_count(&tx->tx_space_written)); |
|
1219 |
refcount_destroy_many(&tx->tx_space_freed, |
|
1220 |
refcount_count(&tx->tx_space_freed)); |
|
1221 |
#endif |
|
1222 |
kmem_free(tx, sizeof (dmu_tx_t)); |
|
1223 |
} |
|
1224 |
||
1225 |
uint64_t |
|
1226 |
dmu_tx_get_txg(dmu_tx_t *tx) |
|
1227 |
{ |
|
1228 |
ASSERT(tx->tx_txg != 0); |
|
1229 |
return (tx->tx_txg); |
|
1230 |
} |
|
10612
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1231 |
|
14046
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1232 |
dsl_pool_t * |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1233 |
dmu_tx_pool(dmu_tx_t *tx) |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1234 |
{ |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1235 |
ASSERT(tx->tx_pool != NULL); |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1236 |
return (tx->tx_pool); |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1237 |
} |
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1238 |
|
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
14005
diff
changeset
|
1239 |
|
10612
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1240 |
void |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1241 |
dmu_tx_callback_register(dmu_tx_t *tx, dmu_tx_callback_func_t *func, void *data) |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1242 |
{ |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1243 |
dmu_tx_callback_t *dcb; |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1244 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1245 |
dcb = kmem_alloc(sizeof (dmu_tx_callback_t), KM_SLEEP); |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1246 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1247 |
dcb->dcb_func = func; |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1248 |
dcb->dcb_data = data; |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1249 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1250 |
list_insert_tail(&tx->tx_callbacks, dcb); |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1251 |
} |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1252 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1253 |
/* |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1254 |
* Call all the commit callbacks on a list, with a given error code. |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1255 |
*/ |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1256 |
void |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1257 |
dmu_tx_do_callbacks(list_t *cb_list, int error) |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1258 |
{ |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1259 |
dmu_tx_callback_t *dcb; |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1260 |
|
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1261 |
while (dcb = list_head(cb_list)) { |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1262 |
list_remove(cb_list, dcb); |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1263 |
dcb->dcb_func(dcb->dcb_data, error); |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1264 |
kmem_free(dcb, sizeof (dmu_tx_callback_t)); |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1265 |
} |
89423355fa6f
6650218 Commit callbacks API for the DMU
Ricardo M. Correia <Ricardo.M.Correia@Sun.COM>
parents:
10298
diff
changeset
|
1266 |
} |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1267 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1268 |
/* |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1269 |
* Interface to hold a bunch of attributes. |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1270 |
* used for creating new files. |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1271 |
* attrsize is the total size of all attributes |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1272 |
* to be added during object creation |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1273 |
* |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1274 |
* For updating/adding a single attribute dmu_tx_hold_sa() should be used. |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1275 |
*/ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1276 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1277 |
/* |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1278 |
* hold necessary attribute name for attribute registration. |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1279 |
* should be a very rare case where this is needed. If it does |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1280 |
* happen it would only happen on the first write to the file system. |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1281 |
*/ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1282 |
static void |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1283 |
dmu_tx_sa_registration_hold(sa_os_t *sa, dmu_tx_t *tx) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1284 |
{ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1285 |
int i; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1286 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1287 |
if (!sa->sa_need_attr_registration) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1288 |
return; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1289 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1290 |
for (i = 0; i != sa->sa_num_attrs; i++) { |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1291 |
if (!sa->sa_attr_table[i].sa_registered) { |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1292 |
if (sa->sa_reg_attr_obj) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1293 |
dmu_tx_hold_zap(tx, sa->sa_reg_attr_obj, |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1294 |
B_TRUE, sa->sa_attr_table[i].sa_name); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1295 |
else |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1296 |
dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1297 |
B_TRUE, sa->sa_attr_table[i].sa_name); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1298 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1299 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1300 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1301 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1302 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1303 |
void |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1304 |
dmu_tx_hold_spill(dmu_tx_t *tx, uint64_t object) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1305 |
{ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1306 |
dnode_t *dn; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1307 |
dmu_tx_hold_t *txh; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1308 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1309 |
txh = dmu_tx_hold_object_impl(tx, tx->tx_objset, object, |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1310 |
THT_SPILL, 0, 0); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1311 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1312 |
dn = txh->txh_dnode; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1313 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1314 |
if (dn == NULL) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1315 |
return; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1316 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1317 |
/* If blkptr doesn't exist then add space to towrite */ |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
1318 |
if (!(dn->dn_phys->dn_flags & DNODE_FLAG_SPILL_BLKPTR)) { |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1319 |
txh->txh_space_towrite += SPA_MAXBLOCKSIZE; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1320 |
} else { |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
1321 |
blkptr_t *bp; |
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
1322 |
|
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
1323 |
bp = &dn->dn_phys->dn_spill; |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1324 |
if (dsl_dataset_block_freeable(dn->dn_objset->os_dsl_dataset, |
12450
c77e20e4e046
6938089 dedup-induced latency causes FC initiator logouts/FC port resets
George Wilson <George.Wilson@Sun.COM>
parents:
12411
diff
changeset
|
1325 |
bp, bp->blk_birth)) |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1326 |
txh->txh_space_tooverwrite += SPA_MAXBLOCKSIZE; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1327 |
else |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1328 |
txh->txh_space_towrite += SPA_MAXBLOCKSIZE; |
13497
b8e89e5c4167
1475 zfs spill block hold can access invalid spill blkptr
Albert Lee <trisk@nexenta.com>
parents:
12684
diff
changeset
|
1329 |
if (!BP_IS_HOLE(bp)) |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1330 |
txh->txh_space_tounref += SPA_MAXBLOCKSIZE; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1331 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1332 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1333 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1334 |
void |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1335 |
dmu_tx_hold_sa_create(dmu_tx_t *tx, int attrsize) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1336 |
{ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1337 |
sa_os_t *sa = tx->tx_objset->os_sa; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1338 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1339 |
dmu_tx_hold_bonus(tx, DMU_NEW_OBJECT); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1340 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1341 |
if (tx->tx_objset->os_sa->sa_master_obj == 0) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1342 |
return; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1343 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1344 |
if (tx->tx_objset->os_sa->sa_layout_attr_obj) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1345 |
dmu_tx_hold_zap(tx, sa->sa_layout_attr_obj, B_TRUE, NULL); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1346 |
else { |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1347 |
dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_LAYOUTS); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1348 |
dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_REGISTRY); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1349 |
dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1350 |
dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1351 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1352 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1353 |
dmu_tx_sa_registration_hold(sa, tx); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1354 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1355 |
if (attrsize <= DN_MAX_BONUSLEN && !sa->sa_force_spill) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1356 |
return; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1357 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1358 |
(void) dmu_tx_hold_object_impl(tx, tx->tx_objset, DMU_NEW_OBJECT, |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1359 |
THT_SPILL, 0, 0); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1360 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1361 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1362 |
/* |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1363 |
* Hold SA attribute |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1364 |
* |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1365 |
* dmu_tx_hold_sa(dmu_tx_t *tx, sa_handle_t *, attribute, add, size) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1366 |
* |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1367 |
* variable_size is the total size of all variable sized attributes |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1368 |
* passed to this function. It is not the total size of all |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1369 |
* variable size attributes that *may* exist on this object. |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1370 |
*/ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1371 |
void |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1372 |
dmu_tx_hold_sa(dmu_tx_t *tx, sa_handle_t *hdl, boolean_t may_grow) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1373 |
{ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1374 |
uint64_t object; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1375 |
sa_os_t *sa = tx->tx_objset->os_sa; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1376 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1377 |
ASSERT(hdl != NULL); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1378 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1379 |
object = sa_handle_object(hdl); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1380 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1381 |
dmu_tx_hold_bonus(tx, object); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1382 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1383 |
if (tx->tx_objset->os_sa->sa_master_obj == 0) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1384 |
return; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1385 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1386 |
if (tx->tx_objset->os_sa->sa_reg_attr_obj == 0 || |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1387 |
tx->tx_objset->os_sa->sa_layout_attr_obj == 0) { |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1388 |
dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_LAYOUTS); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1389 |
dmu_tx_hold_zap(tx, sa->sa_master_obj, B_TRUE, SA_REGISTRY); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1390 |
dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1391 |
dmu_tx_hold_zap(tx, DMU_NEW_OBJECT, B_TRUE, NULL); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1392 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1393 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1394 |
dmu_tx_sa_registration_hold(sa, tx); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1395 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1396 |
if (may_grow && tx->tx_objset->os_sa->sa_layout_attr_obj) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1397 |
dmu_tx_hold_zap(tx, sa->sa_layout_attr_obj, B_TRUE, NULL); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1398 |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1399 |
if (sa->sa_force_spill || may_grow || hdl->sa_spill) { |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1400 |
ASSERT(tx->tx_txg == 0); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1401 |
dmu_tx_hold_spill(tx, object); |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1402 |
} else { |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1403 |
dmu_buf_impl_t *db = (dmu_buf_impl_t *)hdl->sa_bonus; |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1404 |
dnode_t *dn; |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1405 |
|
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1406 |
DB_DNODE_ENTER(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1407 |
dn = DB_DNODE(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1408 |
if (dn->dn_have_spill) { |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1409 |
ASSERT(tx->tx_txg == 0); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1410 |
dmu_tx_hold_spill(tx, object); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1411 |
} |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12450
diff
changeset
|
1412 |
DB_DNODE_EXIT(db); |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1413 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11550
diff
changeset
|
1414 |
} |