author | maybee |
Sun, 26 Aug 2007 11:19:04 -0700 | |
changeset 4944 | 96d96f8de974 |
parent 4831 | 41ec732c6d9f |
child 5370 | eb153afce98e |
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:
1199
diff
changeset
|
5 |
* Common Development and Distribution License (the "License"). |
bdcb30e07e7d
6389368 fat zap should use 16k blocks (with backwards compatability)
ahrens
parents:
1199
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 |
/* |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
22 |
* Copyright 2007 Sun Microsystems, Inc. All rights reserved. |
789 | 23 |
* Use is subject to license terms. |
24 |
*/ |
|
25 |
||
26 |
#pragma ident "%Z%%M% %I% %E% SMI" |
|
27 |
||
28 |
#include <sys/zfs_context.h> |
|
29 |
#include <sys/dmu.h> |
|
30 |
#include <sys/dmu_impl.h> |
|
31 |
#include <sys/dbuf.h> |
|
32 |
#include <sys/dmu_objset.h> |
|
33 |
#include <sys/dsl_dataset.h> |
|
34 |
#include <sys/dsl_dir.h> |
|
35 |
#include <sys/dmu_tx.h> |
|
36 |
#include <sys/spa.h> |
|
37 |
#include <sys/zio.h> |
|
38 |
#include <sys/dmu_zfetch.h> |
|
39 |
||
40 |
static void dbuf_destroy(dmu_buf_impl_t *db); |
|
41 |
static int dbuf_undirty(dmu_buf_impl_t *db, dmu_tx_t *tx); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
42 |
static void dbuf_write(dbuf_dirty_record_t *dr, arc_buf_t *data, int checksum, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
43 |
int compress, dmu_tx_t *tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
44 |
static arc_done_func_t dbuf_write_ready; |
789 | 45 |
static arc_done_func_t dbuf_write_done; |
46 |
||
2986 | 47 |
int zfs_mdcomp_disable = 0; |
48 |
||
789 | 49 |
/* |
50 |
* Global data structures and functions for the dbuf cache. |
|
51 |
*/ |
|
52 |
static kmem_cache_t *dbuf_cache; |
|
53 |
||
54 |
/* ARGSUSED */ |
|
55 |
static int |
|
56 |
dbuf_cons(void *vdb, void *unused, int kmflag) |
|
57 |
{ |
|
58 |
dmu_buf_impl_t *db = vdb; |
|
59 |
bzero(db, sizeof (dmu_buf_impl_t)); |
|
60 |
||
61 |
mutex_init(&db->db_mtx, NULL, MUTEX_DEFAULT, NULL); |
|
62 |
cv_init(&db->db_changed, NULL, CV_DEFAULT, NULL); |
|
63 |
refcount_create(&db->db_holds); |
|
64 |
return (0); |
|
65 |
} |
|
66 |
||
67 |
/* ARGSUSED */ |
|
68 |
static void |
|
69 |
dbuf_dest(void *vdb, void *unused) |
|
70 |
{ |
|
71 |
dmu_buf_impl_t *db = vdb; |
|
72 |
mutex_destroy(&db->db_mtx); |
|
73 |
cv_destroy(&db->db_changed); |
|
74 |
refcount_destroy(&db->db_holds); |
|
75 |
} |
|
76 |
||
77 |
/* |
|
78 |
* dbuf hash table routines |
|
79 |
*/ |
|
80 |
static dbuf_hash_table_t dbuf_hash_table; |
|
81 |
||
82 |
static uint64_t dbuf_hash_count; |
|
83 |
||
84 |
static uint64_t |
|
85 |
dbuf_hash(void *os, uint64_t obj, uint8_t lvl, uint64_t blkid) |
|
86 |
{ |
|
87 |
uintptr_t osv = (uintptr_t)os; |
|
88 |
uint64_t crc = -1ULL; |
|
89 |
||
90 |
ASSERT(zfs_crc64_table[128] == ZFS_CRC64_POLY); |
|
91 |
crc = (crc >> 8) ^ zfs_crc64_table[(crc ^ (lvl)) & 0xFF]; |
|
92 |
crc = (crc >> 8) ^ zfs_crc64_table[(crc ^ (osv >> 6)) & 0xFF]; |
|
93 |
crc = (crc >> 8) ^ zfs_crc64_table[(crc ^ (obj >> 0)) & 0xFF]; |
|
94 |
crc = (crc >> 8) ^ zfs_crc64_table[(crc ^ (obj >> 8)) & 0xFF]; |
|
95 |
crc = (crc >> 8) ^ zfs_crc64_table[(crc ^ (blkid >> 0)) & 0xFF]; |
|
96 |
crc = (crc >> 8) ^ zfs_crc64_table[(crc ^ (blkid >> 8)) & 0xFF]; |
|
97 |
||
98 |
crc ^= (osv>>14) ^ (obj>>16) ^ (blkid>>16); |
|
99 |
||
100 |
return (crc); |
|
101 |
} |
|
102 |
||
103 |
#define DBUF_HASH(os, obj, level, blkid) dbuf_hash(os, obj, level, blkid); |
|
104 |
||
105 |
#define DBUF_EQUAL(dbuf, os, obj, level, blkid) \ |
|
106 |
((dbuf)->db.db_object == (obj) && \ |
|
107 |
(dbuf)->db_objset == (os) && \ |
|
108 |
(dbuf)->db_level == (level) && \ |
|
109 |
(dbuf)->db_blkid == (blkid)) |
|
110 |
||
111 |
dmu_buf_impl_t * |
|
112 |
dbuf_find(dnode_t *dn, uint8_t level, uint64_t blkid) |
|
113 |
{ |
|
114 |
dbuf_hash_table_t *h = &dbuf_hash_table; |
|
115 |
objset_impl_t *os = dn->dn_objset; |
|
116 |
uint64_t obj = dn->dn_object; |
|
117 |
uint64_t hv = DBUF_HASH(os, obj, level, blkid); |
|
118 |
uint64_t idx = hv & h->hash_table_mask; |
|
119 |
dmu_buf_impl_t *db; |
|
120 |
||
121 |
mutex_enter(DBUF_HASH_MUTEX(h, idx)); |
|
122 |
for (db = h->hash_table[idx]; db != NULL; db = db->db_hash_next) { |
|
123 |
if (DBUF_EQUAL(db, os, obj, level, blkid)) { |
|
124 |
mutex_enter(&db->db_mtx); |
|
1544 | 125 |
if (db->db_state != DB_EVICTING) { |
789 | 126 |
mutex_exit(DBUF_HASH_MUTEX(h, idx)); |
127 |
return (db); |
|
128 |
} |
|
129 |
mutex_exit(&db->db_mtx); |
|
130 |
} |
|
131 |
} |
|
132 |
mutex_exit(DBUF_HASH_MUTEX(h, idx)); |
|
133 |
return (NULL); |
|
134 |
} |
|
135 |
||
136 |
/* |
|
137 |
* Insert an entry into the hash table. If there is already an element |
|
138 |
* equal to elem in the hash table, then the already existing element |
|
139 |
* will be returned and the new element will not be inserted. |
|
140 |
* Otherwise returns NULL. |
|
141 |
*/ |
|
142 |
static dmu_buf_impl_t * |
|
143 |
dbuf_hash_insert(dmu_buf_impl_t *db) |
|
144 |
{ |
|
145 |
dbuf_hash_table_t *h = &dbuf_hash_table; |
|
146 |
objset_impl_t *os = db->db_objset; |
|
147 |
uint64_t obj = db->db.db_object; |
|
148 |
int level = db->db_level; |
|
149 |
uint64_t blkid = db->db_blkid; |
|
150 |
uint64_t hv = DBUF_HASH(os, obj, level, blkid); |
|
151 |
uint64_t idx = hv & h->hash_table_mask; |
|
152 |
dmu_buf_impl_t *dbf; |
|
153 |
||
154 |
mutex_enter(DBUF_HASH_MUTEX(h, idx)); |
|
155 |
for (dbf = h->hash_table[idx]; dbf != NULL; dbf = dbf->db_hash_next) { |
|
156 |
if (DBUF_EQUAL(dbf, os, obj, level, blkid)) { |
|
157 |
mutex_enter(&dbf->db_mtx); |
|
1544 | 158 |
if (dbf->db_state != DB_EVICTING) { |
789 | 159 |
mutex_exit(DBUF_HASH_MUTEX(h, idx)); |
160 |
return (dbf); |
|
161 |
} |
|
162 |
mutex_exit(&dbf->db_mtx); |
|
163 |
} |
|
164 |
} |
|
165 |
||
166 |
mutex_enter(&db->db_mtx); |
|
167 |
db->db_hash_next = h->hash_table[idx]; |
|
168 |
h->hash_table[idx] = db; |
|
169 |
mutex_exit(DBUF_HASH_MUTEX(h, idx)); |
|
170 |
atomic_add_64(&dbuf_hash_count, 1); |
|
171 |
||
172 |
return (NULL); |
|
173 |
} |
|
174 |
||
175 |
/* |
|
176 |
* Remove an entry from the hash table. This operation will |
|
177 |
* fail if there are any existing holds on the db. |
|
178 |
*/ |
|
179 |
static void |
|
180 |
dbuf_hash_remove(dmu_buf_impl_t *db) |
|
181 |
{ |
|
182 |
dbuf_hash_table_t *h = &dbuf_hash_table; |
|
183 |
uint64_t hv = DBUF_HASH(db->db_objset, db->db.db_object, |
|
184 |
db->db_level, db->db_blkid); |
|
185 |
uint64_t idx = hv & h->hash_table_mask; |
|
186 |
dmu_buf_impl_t *dbf, **dbp; |
|
187 |
||
188 |
/* |
|
189 |
* We musn't hold db_mtx to maintin lock ordering: |
|
190 |
* DBUF_HASH_MUTEX > db_mtx. |
|
191 |
*/ |
|
192 |
ASSERT(refcount_is_zero(&db->db_holds)); |
|
1544 | 193 |
ASSERT(db->db_state == DB_EVICTING); |
789 | 194 |
ASSERT(!MUTEX_HELD(&db->db_mtx)); |
195 |
||
196 |
mutex_enter(DBUF_HASH_MUTEX(h, idx)); |
|
197 |
dbp = &h->hash_table[idx]; |
|
198 |
while ((dbf = *dbp) != db) { |
|
199 |
dbp = &dbf->db_hash_next; |
|
200 |
ASSERT(dbf != NULL); |
|
201 |
} |
|
202 |
*dbp = db->db_hash_next; |
|
203 |
db->db_hash_next = NULL; |
|
204 |
mutex_exit(DBUF_HASH_MUTEX(h, idx)); |
|
205 |
atomic_add_64(&dbuf_hash_count, -1); |
|
206 |
} |
|
207 |
||
1544 | 208 |
static arc_evict_func_t dbuf_do_evict; |
789 | 209 |
|
210 |
static void |
|
211 |
dbuf_evict_user(dmu_buf_impl_t *db) |
|
212 |
{ |
|
213 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
|
214 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
215 |
if (db->db_level != 0 || db->db_evict_func == NULL) |
789 | 216 |
return; |
217 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
218 |
if (db->db_user_data_ptr_ptr) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
219 |
*db->db_user_data_ptr_ptr = db->db.db_data; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
220 |
db->db_evict_func(&db->db, db->db_user_ptr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
221 |
db->db_user_ptr = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
222 |
db->db_user_data_ptr_ptr = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
223 |
db->db_evict_func = NULL; |
789 | 224 |
} |
225 |
||
226 |
void |
|
1544 | 227 |
dbuf_evict(dmu_buf_impl_t *db) |
228 |
{ |
|
229 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
|
230 |
ASSERT(db->db_buf == NULL); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
231 |
ASSERT(db->db_data_pending == NULL); |
1544 | 232 |
|
233 |
dbuf_clear(db); |
|
234 |
dbuf_destroy(db); |
|
235 |
} |
|
236 |
||
237 |
void |
|
789 | 238 |
dbuf_init(void) |
239 |
{ |
|
1544 | 240 |
uint64_t hsize = 1ULL << 16; |
789 | 241 |
dbuf_hash_table_t *h = &dbuf_hash_table; |
242 |
int i; |
|
243 |
||
244 |
/* |
|
245 |
* The hash table is big enough to fill all of physical memory |
|
1544 | 246 |
* with an average 4K block size. The table will take up |
247 |
* totalmem*sizeof(void*)/4K (i.e. 2MB/GB with 8-byte pointers). |
|
789 | 248 |
*/ |
1544 | 249 |
while (hsize * 4096 < physmem * PAGESIZE) |
789 | 250 |
hsize <<= 1; |
251 |
||
1544 | 252 |
retry: |
789 | 253 |
h->hash_table_mask = hsize - 1; |
1544 | 254 |
h->hash_table = kmem_zalloc(hsize * sizeof (void *), KM_NOSLEEP); |
255 |
if (h->hash_table == NULL) { |
|
256 |
/* XXX - we should really return an error instead of assert */ |
|
257 |
ASSERT(hsize > (1ULL << 10)); |
|
258 |
hsize >>= 1; |
|
259 |
goto retry; |
|
260 |
} |
|
789 | 261 |
|
262 |
dbuf_cache = kmem_cache_create("dmu_buf_impl_t", |
|
263 |
sizeof (dmu_buf_impl_t), |
|
264 |
0, dbuf_cons, dbuf_dest, NULL, NULL, NULL, 0); |
|
265 |
||
266 |
for (i = 0; i < DBUF_MUTEXES; i++) |
|
267 |
mutex_init(&h->hash_mutexes[i], NULL, MUTEX_DEFAULT, NULL); |
|
268 |
} |
|
269 |
||
270 |
void |
|
271 |
dbuf_fini(void) |
|
272 |
{ |
|
273 |
dbuf_hash_table_t *h = &dbuf_hash_table; |
|
274 |
int i; |
|
275 |
||
276 |
for (i = 0; i < DBUF_MUTEXES; i++) |
|
277 |
mutex_destroy(&h->hash_mutexes[i]); |
|
278 |
kmem_free(h->hash_table, (h->hash_table_mask + 1) * sizeof (void *)); |
|
279 |
kmem_cache_destroy(dbuf_cache); |
|
280 |
} |
|
281 |
||
282 |
/* |
|
283 |
* Other stuff. |
|
284 |
*/ |
|
285 |
||
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
286 |
#ifdef ZFS_DEBUG |
789 | 287 |
static void |
288 |
dbuf_verify(dmu_buf_impl_t *db) |
|
289 |
{ |
|
290 |
dnode_t *dn = db->db_dnode; |
|
291 |
||
292 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
|
293 |
||
294 |
if (!(zfs_flags & ZFS_DEBUG_DBUF_VERIFY)) |
|
295 |
return; |
|
296 |
||
297 |
ASSERT(db->db_objset != NULL); |
|
298 |
if (dn == NULL) { |
|
299 |
ASSERT(db->db_parent == NULL); |
|
300 |
ASSERT(db->db_blkptr == NULL); |
|
301 |
} else { |
|
302 |
ASSERT3U(db->db.db_object, ==, dn->dn_object); |
|
303 |
ASSERT3P(db->db_objset, ==, dn->dn_objset); |
|
304 |
ASSERT3U(db->db_level, <, dn->dn_nlevels); |
|
1544 | 305 |
ASSERT(db->db_blkid == DB_BONUS_BLKID || |
306 |
list_head(&dn->dn_dbufs)); |
|
789 | 307 |
} |
308 |
if (db->db_blkid == DB_BONUS_BLKID) { |
|
309 |
ASSERT(dn != NULL); |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
310 |
ASSERT3U(db->db.db_size, >=, dn->dn_bonuslen); |
789 | 311 |
ASSERT3U(db->db.db_offset, ==, DB_BONUS_BLKID); |
312 |
} else { |
|
313 |
ASSERT3U(db->db.db_offset, ==, db->db_blkid * db->db.db_size); |
|
314 |
} |
|
315 |
||
316 |
if (db->db_level == 0) { |
|
317 |
/* we can be momentarily larger in dnode_set_blksz() */ |
|
318 |
if (db->db_blkid != DB_BONUS_BLKID && dn) { |
|
319 |
ASSERT3U(db->db.db_size, >=, dn->dn_datablksz); |
|
320 |
} |
|
1544 | 321 |
if (db->db.db_object == DMU_META_DNODE_OBJECT) { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
322 |
dbuf_dirty_record_t *dr = db->db_data_pending; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
323 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
324 |
* it should only be modified in syncing |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
325 |
* context, so make sure we only have |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
326 |
* one copy of the data. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
327 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
328 |
ASSERT(dr == NULL || dr->dt.dl.dr_data == db->db_buf); |
789 | 329 |
} |
330 |
} |
|
331 |
||
332 |
/* verify db->db_blkptr */ |
|
333 |
if (db->db_blkptr) { |
|
334 |
if (db->db_parent == dn->dn_dbuf) { |
|
335 |
/* db is pointed to by the dnode */ |
|
336 |
/* ASSERT3U(db->db_blkid, <, dn->dn_nblkptr); */ |
|
1544 | 337 |
if (db->db.db_object == DMU_META_DNODE_OBJECT) |
789 | 338 |
ASSERT(db->db_parent == NULL); |
339 |
else |
|
340 |
ASSERT(db->db_parent != NULL); |
|
341 |
ASSERT3P(db->db_blkptr, ==, |
|
342 |
&dn->dn_phys->dn_blkptr[db->db_blkid]); |
|
343 |
} else { |
|
344 |
/* db is pointed to by an indirect block */ |
|
345 |
int epb = db->db_parent->db.db_size >> SPA_BLKPTRSHIFT; |
|
346 |
ASSERT3U(db->db_parent->db_level, ==, db->db_level+1); |
|
347 |
ASSERT3U(db->db_parent->db.db_object, ==, |
|
348 |
db->db.db_object); |
|
349 |
/* |
|
350 |
* dnode_grow_indblksz() can make this fail if we don't |
|
351 |
* have the struct_rwlock. XXX indblksz no longer |
|
352 |
* grows. safe to do this now? |
|
353 |
*/ |
|
354 |
if (RW_WRITE_HELD(&db->db_dnode->dn_struct_rwlock)) { |
|
355 |
ASSERT3P(db->db_blkptr, ==, |
|
356 |
((blkptr_t *)db->db_parent->db.db_data + |
|
357 |
db->db_blkid % epb)); |
|
358 |
} |
|
359 |
} |
|
360 |
} |
|
361 |
if ((db->db_blkptr == NULL || BP_IS_HOLE(db->db_blkptr)) && |
|
362 |
db->db.db_data && db->db_blkid != DB_BONUS_BLKID && |
|
363 |
db->db_state != DB_FILL && !dn->dn_free_txg) { |
|
364 |
/* |
|
365 |
* If the blkptr isn't set but they have nonzero data, |
|
366 |
* it had better be dirty, otherwise we'll lose that |
|
367 |
* data when we evict this buffer. |
|
368 |
*/ |
|
369 |
if (db->db_dirtycnt == 0) { |
|
370 |
uint64_t *buf = db->db.db_data; |
|
371 |
int i; |
|
372 |
||
373 |
for (i = 0; i < db->db.db_size >> 3; i++) { |
|
374 |
ASSERT(buf[i] == 0); |
|
375 |
} |
|
376 |
} |
|
377 |
} |
|
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
378 |
} |
789 | 379 |
#endif |
380 |
||
381 |
static void |
|
382 |
dbuf_update_data(dmu_buf_impl_t *db) |
|
383 |
{ |
|
384 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
385 |
if (db->db_level == 0 && db->db_user_data_ptr_ptr) { |
789 | 386 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
387 |
*db->db_user_data_ptr_ptr = db->db.db_data; |
789 | 388 |
} |
389 |
} |
|
390 |
||
391 |
static void |
|
392 |
dbuf_set_data(dmu_buf_impl_t *db, arc_buf_t *buf) |
|
393 |
{ |
|
394 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
|
1544 | 395 |
ASSERT(db->db_buf == NULL || !arc_has_callback(db->db_buf)); |
789 | 396 |
db->db_buf = buf; |
1544 | 397 |
if (buf != NULL) { |
398 |
ASSERT(buf->b_data != NULL); |
|
399 |
db->db.db_data = buf->b_data; |
|
400 |
if (!arc_released(buf)) |
|
401 |
arc_set_callback(buf, dbuf_do_evict, db); |
|
402 |
dbuf_update_data(db); |
|
403 |
} else { |
|
404 |
dbuf_evict_user(db); |
|
405 |
db->db.db_data = NULL; |
|
406 |
db->db_state = DB_UNCACHED; |
|
407 |
} |
|
789 | 408 |
} |
409 |
||
410 |
uint64_t |
|
411 |
dbuf_whichblock(dnode_t *dn, uint64_t offset) |
|
412 |
{ |
|
413 |
if (dn->dn_datablkshift) { |
|
414 |
return (offset >> dn->dn_datablkshift); |
|
415 |
} else { |
|
416 |
ASSERT3U(offset, <, dn->dn_datablksz); |
|
417 |
return (0); |
|
418 |
} |
|
419 |
} |
|
420 |
||
421 |
static void |
|
422 |
dbuf_read_done(zio_t *zio, arc_buf_t *buf, void *vdb) |
|
423 |
{ |
|
424 |
dmu_buf_impl_t *db = vdb; |
|
425 |
||
426 |
mutex_enter(&db->db_mtx); |
|
427 |
ASSERT3U(db->db_state, ==, DB_READ); |
|
428 |
/* |
|
429 |
* All reads are synchronous, so we must have a hold on the dbuf |
|
430 |
*/ |
|
431 |
ASSERT(refcount_count(&db->db_holds) > 0); |
|
1544 | 432 |
ASSERT(db->db_buf == NULL); |
789 | 433 |
ASSERT(db->db.db_data == NULL); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
434 |
if (db->db_level == 0 && db->db_freed_in_flight) { |
789 | 435 |
/* we were freed in flight; disregard any error */ |
436 |
arc_release(buf, db); |
|
437 |
bzero(buf->b_data, db->db.db_size); |
|
3093
71525e4187d5
6490569 verify arc bufs are not modified when they shouldn't be
ahrens
parents:
2986
diff
changeset
|
438 |
arc_buf_freeze(buf); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
439 |
db->db_freed_in_flight = FALSE; |
789 | 440 |
dbuf_set_data(db, buf); |
441 |
db->db_state = DB_CACHED; |
|
442 |
} else if (zio == NULL || zio->io_error == 0) { |
|
443 |
dbuf_set_data(db, buf); |
|
444 |
db->db_state = DB_CACHED; |
|
445 |
} else { |
|
446 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
|
1544 | 447 |
ASSERT3P(db->db_buf, ==, NULL); |
448 |
VERIFY(arc_buf_remove_ref(buf, db) == 1); |
|
789 | 449 |
db->db_state = DB_UNCACHED; |
450 |
} |
|
451 |
cv_broadcast(&db->db_changed); |
|
452 |
mutex_exit(&db->db_mtx); |
|
1544 | 453 |
dbuf_rele(db, NULL); |
789 | 454 |
} |
455 |
||
1544 | 456 |
static void |
2391 | 457 |
dbuf_read_impl(dmu_buf_impl_t *db, zio_t *zio, uint32_t *flags) |
789 | 458 |
{ |
459 |
blkptr_t *bp; |
|
1544 | 460 |
zbookmark_t zb; |
2391 | 461 |
uint32_t aflags = ARC_NOWAIT; |
789 | 462 |
|
463 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
|
464 |
/* We need the struct_rwlock to prevent db_blkptr from changing. */ |
|
465 |
ASSERT(RW_LOCK_HELD(&db->db_dnode->dn_struct_rwlock)); |
|
1544 | 466 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
467 |
ASSERT(db->db_state == DB_UNCACHED); |
|
468 |
ASSERT(db->db_buf == NULL); |
|
789 | 469 |
|
470 |
if (db->db_blkid == DB_BONUS_BLKID) { |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
471 |
int bonuslen = db->db_dnode->dn_bonuslen; |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
472 |
|
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
473 |
ASSERT3U(bonuslen, <=, db->db.db_size); |
1544 | 474 |
db->db.db_data = zio_buf_alloc(DN_MAX_BONUSLEN); |
4506 | 475 |
arc_space_consume(DN_MAX_BONUSLEN); |
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
476 |
if (bonuslen < DN_MAX_BONUSLEN) |
1544 | 477 |
bzero(db->db.db_data, DN_MAX_BONUSLEN); |
478 |
bcopy(DN_BONUS(db->db_dnode->dn_phys), db->db.db_data, |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
479 |
bonuslen); |
1544 | 480 |
dbuf_update_data(db); |
789 | 481 |
db->db_state = DB_CACHED; |
482 |
mutex_exit(&db->db_mtx); |
|
483 |
return; |
|
484 |
} |
|
485 |
||
486 |
if (db->db_level == 0 && dnode_block_freed(db->db_dnode, db->db_blkid)) |
|
487 |
bp = NULL; |
|
488 |
else |
|
489 |
bp = db->db_blkptr; |
|
490 |
||
491 |
if (bp == NULL) |
|
492 |
dprintf_dbuf(db, "blkptr: %s\n", "NULL"); |
|
493 |
else |
|
494 |
dprintf_dbuf_bp(db, bp, "%s", "blkptr:"); |
|
495 |
||
496 |
if (bp == NULL || BP_IS_HOLE(bp)) { |
|
3290 | 497 |
arc_buf_contents_t type = DBUF_GET_BUFC_TYPE(db); |
498 |
||
789 | 499 |
ASSERT(bp == NULL || BP_IS_HOLE(bp)); |
500 |
dbuf_set_data(db, arc_buf_alloc(db->db_dnode->dn_objset->os_spa, |
|
3290 | 501 |
db->db.db_size, db, type)); |
789 | 502 |
bzero(db->db.db_data, db->db.db_size); |
503 |
db->db_state = DB_CACHED; |
|
2391 | 504 |
*flags |= DB_RF_CACHED; |
789 | 505 |
mutex_exit(&db->db_mtx); |
506 |
return; |
|
507 |
} |
|
508 |
||
509 |
db->db_state = DB_READ; |
|
510 |
mutex_exit(&db->db_mtx); |
|
511 |
||
1544 | 512 |
zb.zb_objset = db->db_objset->os_dsl_dataset ? |
513 |
db->db_objset->os_dsl_dataset->ds_object : 0; |
|
514 |
zb.zb_object = db->db.db_object; |
|
515 |
zb.zb_level = db->db_level; |
|
516 |
zb.zb_blkid = db->db_blkid; |
|
517 |
||
518 |
dbuf_add_ref(db, NULL); |
|
789 | 519 |
/* ZIO_FLAG_CANFAIL callers have to check the parent zio's error */ |
3882 | 520 |
ASSERT3U(db->db_dnode->dn_type, <, DMU_OT_NUMTYPES); |
789 | 521 |
(void) arc_read(zio, db->db_dnode->dn_objset->os_spa, bp, |
522 |
db->db_level > 0 ? byteswap_uint64_array : |
|
523 |
dmu_ot[db->db_dnode->dn_type].ot_byteswap, |
|
524 |
dbuf_read_done, db, ZIO_PRIORITY_SYNC_READ, |
|
2391 | 525 |
(*flags & DB_RF_CANFAIL) ? ZIO_FLAG_CANFAIL : ZIO_FLAG_MUSTSUCCEED, |
526 |
&aflags, &zb); |
|
527 |
if (aflags & ARC_CACHED) |
|
528 |
*flags |= DB_RF_CACHED; |
|
789 | 529 |
} |
530 |
||
1544 | 531 |
int |
532 |
dbuf_read(dmu_buf_impl_t *db, zio_t *zio, uint32_t flags) |
|
789 | 533 |
{ |
1544 | 534 |
int err = 0; |
535 |
int havepzio = (zio != NULL); |
|
2391 | 536 |
int prefetch; |
789 | 537 |
|
538 |
/* |
|
539 |
* We don't have to hold the mutex to check db_state because it |
|
540 |
* can't be freed while we have a hold on the buffer. |
|
541 |
*/ |
|
542 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
|
543 |
||
1544 | 544 |
if ((flags & DB_RF_HAVESTRUCT) == 0) |
545 |
rw_enter(&db->db_dnode->dn_struct_rwlock, RW_READER); |
|
789 | 546 |
|
2391 | 547 |
prefetch = db->db_level == 0 && db->db_blkid != DB_BONUS_BLKID && |
548 |
(flags & DB_RF_NOPREFETCH) == 0 && db->db_dnode != NULL; |
|
549 |
||
789 | 550 |
mutex_enter(&db->db_mtx); |
1544 | 551 |
if (db->db_state == DB_CACHED) { |
552 |
mutex_exit(&db->db_mtx); |
|
2391 | 553 |
if (prefetch) |
554 |
dmu_zfetch(&db->db_dnode->dn_zfetch, db->db.db_offset, |
|
555 |
db->db.db_size, TRUE); |
|
1544 | 556 |
if ((flags & DB_RF_HAVESTRUCT) == 0) |
557 |
rw_exit(&db->db_dnode->dn_struct_rwlock); |
|
558 |
} else if (db->db_state == DB_UNCACHED) { |
|
559 |
if (zio == NULL) { |
|
560 |
zio = zio_root(db->db_dnode->dn_objset->os_spa, |
|
561 |
NULL, NULL, ZIO_FLAG_CANFAIL); |
|
562 |
} |
|
2391 | 563 |
dbuf_read_impl(db, zio, &flags); |
564 |
||
1544 | 565 |
/* dbuf_read_impl has dropped db_mtx for us */ |
789 | 566 |
|
2391 | 567 |
if (prefetch) |
1544 | 568 |
dmu_zfetch(&db->db_dnode->dn_zfetch, db->db.db_offset, |
2391 | 569 |
db->db.db_size, flags & DB_RF_CACHED); |
789 | 570 |
|
1544 | 571 |
if ((flags & DB_RF_HAVESTRUCT) == 0) |
572 |
rw_exit(&db->db_dnode->dn_struct_rwlock); |
|
789 | 573 |
|
1544 | 574 |
if (!havepzio) |
575 |
err = zio_wait(zio); |
|
576 |
} else { |
|
2391 | 577 |
mutex_exit(&db->db_mtx); |
578 |
if (prefetch) |
|
579 |
dmu_zfetch(&db->db_dnode->dn_zfetch, db->db.db_offset, |
|
580 |
db->db.db_size, TRUE); |
|
1544 | 581 |
if ((flags & DB_RF_HAVESTRUCT) == 0) |
582 |
rw_exit(&db->db_dnode->dn_struct_rwlock); |
|
2391 | 583 |
|
584 |
mutex_enter(&db->db_mtx); |
|
1544 | 585 |
if ((flags & DB_RF_NEVERWAIT) == 0) { |
586 |
while (db->db_state == DB_READ || |
|
587 |
db->db_state == DB_FILL) { |
|
588 |
ASSERT(db->db_state == DB_READ || |
|
589 |
(flags & DB_RF_HAVESTRUCT) == 0); |
|
590 |
cv_wait(&db->db_changed, &db->db_mtx); |
|
591 |
} |
|
592 |
if (db->db_state == DB_UNCACHED) |
|
593 |
err = EIO; |
|
594 |
} |
|
595 |
mutex_exit(&db->db_mtx); |
|
596 |
} |
|
789 | 597 |
|
1544 | 598 |
ASSERT(err || havepzio || db->db_state == DB_CACHED); |
599 |
return (err); |
|
789 | 600 |
} |
601 |
||
602 |
static void |
|
603 |
dbuf_noread(dmu_buf_impl_t *db) |
|
604 |
{ |
|
605 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
|
1544 | 606 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
789 | 607 |
mutex_enter(&db->db_mtx); |
608 |
while (db->db_state == DB_READ || db->db_state == DB_FILL) |
|
609 |
cv_wait(&db->db_changed, &db->db_mtx); |
|
610 |
if (db->db_state == DB_UNCACHED) { |
|
3290 | 611 |
arc_buf_contents_t type = DBUF_GET_BUFC_TYPE(db); |
612 |
||
1544 | 613 |
ASSERT(db->db_buf == NULL); |
789 | 614 |
ASSERT(db->db.db_data == NULL); |
615 |
dbuf_set_data(db, arc_buf_alloc(db->db_dnode->dn_objset->os_spa, |
|
3290 | 616 |
db->db.db_size, db, type)); |
789 | 617 |
db->db_state = DB_FILL; |
618 |
} else { |
|
619 |
ASSERT3U(db->db_state, ==, DB_CACHED); |
|
620 |
} |
|
621 |
mutex_exit(&db->db_mtx); |
|
622 |
} |
|
623 |
||
624 |
/* |
|
625 |
* This is our just-in-time copy function. It makes a copy of |
|
626 |
* buffers, that have been modified in a previous transaction |
|
627 |
* group, before we modify them in the current active group. |
|
628 |
* |
|
629 |
* This function is used in two places: when we are dirtying a |
|
630 |
* buffer for the first time in a txg, and when we are freeing |
|
631 |
* a range in a dnode that includes this buffer. |
|
632 |
* |
|
633 |
* Note that when we are called from dbuf_free_range() we do |
|
634 |
* not put a hold on the buffer, we just traverse the active |
|
635 |
* dbuf list for the dnode. |
|
636 |
*/ |
|
637 |
static void |
|
638 |
dbuf_fix_old_data(dmu_buf_impl_t *db, uint64_t txg) |
|
639 |
{ |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
640 |
dbuf_dirty_record_t *dr = db->db_last_dirty; |
789 | 641 |
|
642 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
|
643 |
ASSERT(db->db.db_data != NULL); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
644 |
ASSERT(db->db_level == 0); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
645 |
ASSERT(db->db.db_object != DMU_META_DNODE_OBJECT); |
789 | 646 |
|
3711
2226ffbe7873
6526196 ZFS: assertion failed: zp->z_phys->zp_links > zp_is_dir
maybee
parents:
3689
diff
changeset
|
647 |
if (dr == NULL || |
2226ffbe7873
6526196 ZFS: assertion failed: zp->z_phys->zp_links > zp_is_dir
maybee
parents:
3689
diff
changeset
|
648 |
(dr->dt.dl.dr_data != |
2226ffbe7873
6526196 ZFS: assertion failed: zp->z_phys->zp_links > zp_is_dir
maybee
parents:
3689
diff
changeset
|
649 |
((db->db_blkid == DB_BONUS_BLKID) ? db->db.db_data : db->db_buf))) |
789 | 650 |
return; |
651 |
||
652 |
/* |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
653 |
* If the last dirty record for this dbuf has not yet synced |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
654 |
* and its referencing the dbuf data, either: |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
655 |
* reset the reference to point to a new copy, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
656 |
* or (if there a no active holders) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
657 |
* just null out the current db_data pointer. |
789 | 658 |
*/ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
659 |
ASSERT(dr->dr_txg >= txg - 2); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
660 |
if (db->db_blkid == DB_BONUS_BLKID) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
661 |
/* Note that the data bufs here are zio_bufs */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
662 |
dr->dt.dl.dr_data = zio_buf_alloc(DN_MAX_BONUSLEN); |
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
663 |
arc_space_consume(DN_MAX_BONUSLEN); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
664 |
bcopy(db->db.db_data, dr->dt.dl.dr_data, DN_MAX_BONUSLEN); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
665 |
} else if (refcount_count(&db->db_holds) > db->db_dirtycnt) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
666 |
int size = db->db.db_size; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
667 |
arc_buf_contents_t type = DBUF_GET_BUFC_TYPE(db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
668 |
dr->dt.dl.dr_data = arc_buf_alloc( |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
669 |
db->db_dnode->dn_objset->os_spa, size, db, type); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
670 |
bcopy(db->db.db_data, dr->dt.dl.dr_data->b_data, size); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
671 |
} else { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
672 |
dbuf_set_data(db, NULL); |
1544 | 673 |
} |
674 |
} |
|
675 |
||
789 | 676 |
void |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
677 |
dbuf_unoverride(dbuf_dirty_record_t *dr) |
789 | 678 |
{ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
679 |
dmu_buf_impl_t *db = dr->dr_dbuf; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
680 |
uint64_t txg = dr->dr_txg; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
681 |
|
789 | 682 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
683 |
ASSERT(dr->dt.dl.dr_override_state != DR_IN_DMU_SYNC); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
684 |
ASSERT(db->db_level == 0); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
685 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
686 |
if (db->db_blkid == DB_BONUS_BLKID || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
687 |
dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
688 |
return; |
2237 | 689 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
690 |
/* free this block */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
691 |
if (!BP_IS_HOLE(&dr->dt.dl.dr_overridden_by)) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
692 |
/* XXX can get silent EIO here */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
693 |
(void) arc_free(NULL, db->db_dnode->dn_objset->os_spa, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
694 |
txg, &dr->dt.dl.dr_overridden_by, NULL, NULL, ARC_WAIT); |
789 | 695 |
} |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
696 |
dr->dt.dl.dr_override_state = DR_NOT_OVERRIDDEN; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
697 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
698 |
* Release the already-written buffer, so we leave it in |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
699 |
* a consistent dirty state. Note that all callers are |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
700 |
* modifying the buffer, so they will immediately do |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
701 |
* another (redundant) arc_release(). Therefore, leave |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
702 |
* the buf thawed to save the effort of freezing & |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
703 |
* immediately re-thawing it. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
704 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
705 |
arc_release(dr->dt.dl.dr_data, db); |
789 | 706 |
} |
707 |
||
708 |
void |
|
709 |
dbuf_free_range(dnode_t *dn, uint64_t blkid, uint64_t nblks, dmu_tx_t *tx) |
|
710 |
{ |
|
711 |
dmu_buf_impl_t *db, *db_next; |
|
712 |
uint64_t txg = tx->tx_txg; |
|
713 |
||
714 |
dprintf_dnode(dn, "blkid=%llu nblks=%llu\n", blkid, nblks); |
|
715 |
mutex_enter(&dn->dn_dbufs_mtx); |
|
716 |
for (db = list_head(&dn->dn_dbufs); db; db = db_next) { |
|
717 |
db_next = list_next(&dn->dn_dbufs, db); |
|
1544 | 718 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
719 |
if (db->db_level != 0) |
|
789 | 720 |
continue; |
721 |
dprintf_dbuf(db, "found buf %s\n", ""); |
|
722 |
if (db->db_blkid < blkid || |
|
723 |
db->db_blkid >= blkid+nblks) |
|
724 |
continue; |
|
725 |
||
726 |
/* found a level 0 buffer in the range */ |
|
727 |
if (dbuf_undirty(db, tx)) |
|
728 |
continue; |
|
729 |
||
730 |
mutex_enter(&db->db_mtx); |
|
1544 | 731 |
if (db->db_state == DB_UNCACHED || |
732 |
db->db_state == DB_EVICTING) { |
|
789 | 733 |
ASSERT(db->db.db_data == NULL); |
734 |
mutex_exit(&db->db_mtx); |
|
735 |
continue; |
|
736 |
} |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
737 |
if (db->db_state == DB_READ || db->db_state == DB_FILL) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
738 |
/* will be handled in dbuf_read_done or dbuf_rele */ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
739 |
db->db_freed_in_flight = TRUE; |
789 | 740 |
mutex_exit(&db->db_mtx); |
741 |
continue; |
|
742 |
} |
|
1544 | 743 |
if (refcount_count(&db->db_holds) == 0) { |
744 |
ASSERT(db->db_buf); |
|
745 |
dbuf_clear(db); |
|
746 |
continue; |
|
747 |
} |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
748 |
/* The dbuf is referenced */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
749 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
750 |
if (db->db_last_dirty != NULL) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
751 |
dbuf_dirty_record_t *dr = db->db_last_dirty; |
789 | 752 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
753 |
if (dr->dr_txg == txg) { |
2688
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
754 |
/* |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
755 |
* This buffer is "in-use", re-adjust the file |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
756 |
* size to reflect that this buffer may |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
757 |
* contain new data when we sync. |
2688
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
758 |
*/ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
759 |
if (db->db_blkid > dn->dn_maxblkid) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
760 |
dn->dn_maxblkid = db->db_blkid; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
761 |
dbuf_unoverride(dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
762 |
} else { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
763 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
764 |
* This dbuf is not dirty in the open context. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
765 |
* Either uncache it (if its not referenced in |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
766 |
* the open context) or reset its contents to |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
767 |
* empty. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
768 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
769 |
dbuf_fix_old_data(db, txg); |
2688
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
770 |
} |
1544 | 771 |
} |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
772 |
/* clear the contents if its cached */ |
1544 | 773 |
if (db->db_state == DB_CACHED) { |
774 |
ASSERT(db->db.db_data != NULL); |
|
789 | 775 |
arc_release(db->db_buf, db); |
776 |
bzero(db->db.db_data, db->db.db_size); |
|
3093
71525e4187d5
6490569 verify arc bufs are not modified when they shouldn't be
ahrens
parents:
2986
diff
changeset
|
777 |
arc_buf_freeze(db->db_buf); |
789 | 778 |
} |
1544 | 779 |
|
789 | 780 |
mutex_exit(&db->db_mtx); |
781 |
} |
|
782 |
mutex_exit(&dn->dn_dbufs_mtx); |
|
783 |
} |
|
784 |
||
785 |
static int |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
786 |
dbuf_block_freeable(dmu_buf_impl_t *db) |
789 | 787 |
{ |
788 |
dsl_dataset_t *ds = db->db_objset->os_dsl_dataset; |
|
789 |
uint64_t birth_txg = 0; |
|
790 |
||
791 |
/* |
|
792 |
* We don't need any locking to protect db_blkptr: |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
793 |
* If it's syncing, then db_last_dirty will be set |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
794 |
* so we'll ignore db_blkptr. |
789 | 795 |
*/ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
796 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
797 |
if (db->db_last_dirty) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
798 |
birth_txg = db->db_last_dirty->dr_txg; |
789 | 799 |
else if (db->db_blkptr) |
800 |
birth_txg = db->db_blkptr->blk_birth; |
|
801 |
||
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
802 |
/* If we don't exist or are in a snapshot, we can't be freed */ |
789 | 803 |
if (birth_txg) |
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
804 |
return (ds == NULL || |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
805 |
dsl_dataset_block_freeable(ds, birth_txg)); |
789 | 806 |
else |
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
807 |
return (FALSE); |
789 | 808 |
} |
809 |
||
810 |
void |
|
811 |
dbuf_new_size(dmu_buf_impl_t *db, int size, dmu_tx_t *tx) |
|
812 |
{ |
|
813 |
arc_buf_t *buf, *obuf; |
|
814 |
int osize = db->db.db_size; |
|
3290 | 815 |
arc_buf_contents_t type = DBUF_GET_BUFC_TYPE(db); |
789 | 816 |
|
1544 | 817 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
818 |
||
789 | 819 |
/* XXX does *this* func really need the lock? */ |
820 |
ASSERT(RW_WRITE_HELD(&db->db_dnode->dn_struct_rwlock)); |
|
821 |
||
822 |
/* |
|
823 |
* This call to dbuf_will_dirty() with the dn_struct_rwlock held |
|
824 |
* is OK, because there can be no other references to the db |
|
825 |
* when we are changing its size, so no concurrent DB_FILL can |
|
826 |
* be happening. |
|
827 |
*/ |
|
1544 | 828 |
/* |
829 |
* XXX we should be doing a dbuf_read, checking the return |
|
830 |
* value and returning that up to our callers |
|
831 |
*/ |
|
789 | 832 |
dbuf_will_dirty(db, tx); |
833 |
||
834 |
/* create the data buffer for the new block */ |
|
3290 | 835 |
buf = arc_buf_alloc(db->db_dnode->dn_objset->os_spa, size, db, type); |
789 | 836 |
|
837 |
/* copy old block data to the new block */ |
|
838 |
obuf = db->db_buf; |
|
1491
bdcb30e07e7d
6389368 fat zap should use 16k blocks (with backwards compatability)
ahrens
parents:
1199
diff
changeset
|
839 |
bcopy(obuf->b_data, buf->b_data, MIN(osize, size)); |
789 | 840 |
/* zero the remainder */ |
1491
bdcb30e07e7d
6389368 fat zap should use 16k blocks (with backwards compatability)
ahrens
parents:
1199
diff
changeset
|
841 |
if (size > osize) |
bdcb30e07e7d
6389368 fat zap should use 16k blocks (with backwards compatability)
ahrens
parents:
1199
diff
changeset
|
842 |
bzero((uint8_t *)buf->b_data + osize, size - osize); |
789 | 843 |
|
844 |
mutex_enter(&db->db_mtx); |
|
845 |
dbuf_set_data(db, buf); |
|
1544 | 846 |
VERIFY(arc_buf_remove_ref(obuf, db) == 1); |
789 | 847 |
db->db.db_size = size; |
848 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
849 |
if (db->db_level == 0) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
850 |
ASSERT3U(db->db_last_dirty->dr_txg, ==, tx->tx_txg); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
851 |
db->db_last_dirty->dt.dl.dr_data = buf; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
852 |
} |
789 | 853 |
mutex_exit(&db->db_mtx); |
854 |
||
855 |
dnode_willuse_space(db->db_dnode, size-osize, tx); |
|
856 |
} |
|
857 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
858 |
dbuf_dirty_record_t * |
789 | 859 |
dbuf_dirty(dmu_buf_impl_t *db, dmu_tx_t *tx) |
860 |
{ |
|
861 |
dnode_t *dn = db->db_dnode; |
|
862 |
objset_impl_t *os = dn->dn_objset; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
863 |
dbuf_dirty_record_t **drp, *dr; |
789 | 864 |
int drop_struct_lock = FALSE; |
865 |
int txgoff = tx->tx_txg & TXG_MASK; |
|
866 |
||
867 |
ASSERT(tx->tx_txg != 0); |
|
868 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
|
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
869 |
DMU_TX_DIRTY_BUF(tx, db); |
789 | 870 |
|
871 |
/* |
|
872 |
* Shouldn't dirty a regular buffer in syncing context. Private |
|
873 |
* objects may be dirtied in syncing context, but only if they |
|
874 |
* were already pre-dirtied in open context. |
|
875 |
* XXX We may want to prohibit dirtying in syncing context even |
|
876 |
* if they did pre-dirty. |
|
877 |
*/ |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
878 |
ASSERT(!dmu_tx_is_syncing(tx) || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
879 |
BP_IS_HOLE(dn->dn_objset->os_rootbp) || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
880 |
dn->dn_object == DMU_META_DNODE_OBJECT || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
881 |
dn->dn_objset->os_dsl_dataset == NULL || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
882 |
dsl_dir_is_private(dn->dn_objset->os_dsl_dataset->ds_dir)); |
789 | 883 |
|
884 |
/* |
|
885 |
* We make this assert for private objects as well, but after we |
|
886 |
* check if we're already dirty. They are allowed to re-dirty |
|
887 |
* in syncing context. |
|
888 |
*/ |
|
1544 | 889 |
ASSERT(dn->dn_object == DMU_META_DNODE_OBJECT || |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
890 |
dn->dn_dirtyctx == DN_UNDIRTIED || dn->dn_dirtyctx == |
789 | 891 |
(dmu_tx_is_syncing(tx) ? DN_DIRTY_SYNC : DN_DIRTY_OPEN)); |
892 |
||
893 |
mutex_enter(&db->db_mtx); |
|
894 |
/* |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
895 |
* XXX make this true for indirects too? The problem is that |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
896 |
* transactions created with dmu_tx_create_assigned() from |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
897 |
* syncing context don't bother holding ahead. |
789 | 898 |
*/ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
899 |
ASSERT(db->db_level != 0 || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
900 |
db->db_state == DB_CACHED || db->db_state == DB_FILL); |
789 | 901 |
|
902 |
mutex_enter(&dn->dn_mtx); |
|
903 |
/* |
|
904 |
* Don't set dirtyctx to SYNC if we're just modifying this as we |
|
905 |
* initialize the objset. |
|
906 |
*/ |
|
907 |
if (dn->dn_dirtyctx == DN_UNDIRTIED && |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
908 |
!BP_IS_HOLE(dn->dn_objset->os_rootbp)) { |
789 | 909 |
dn->dn_dirtyctx = |
910 |
(dmu_tx_is_syncing(tx) ? DN_DIRTY_SYNC : DN_DIRTY_OPEN); |
|
911 |
ASSERT(dn->dn_dirtyctx_firstset == NULL); |
|
912 |
dn->dn_dirtyctx_firstset = kmem_alloc(1, KM_SLEEP); |
|
913 |
} |
|
914 |
mutex_exit(&dn->dn_mtx); |
|
915 |
||
916 |
/* |
|
917 |
* If this buffer is already dirty, we're done. |
|
918 |
*/ |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
919 |
drp = &db->db_last_dirty; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
920 |
ASSERT(*drp == NULL || (*drp)->dr_txg <= tx->tx_txg || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
921 |
db->db.db_object == DMU_META_DNODE_OBJECT); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
922 |
while (*drp && (*drp)->dr_txg > tx->tx_txg) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
923 |
drp = &(*drp)->dr_next; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
924 |
if (*drp && (*drp)->dr_txg == tx->tx_txg) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
925 |
if (db->db_level == 0 && db->db_blkid != DB_BONUS_BLKID) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
926 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
927 |
* If this buffer has already been written out, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
928 |
* we now need to reset its state. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
929 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
930 |
dbuf_unoverride(*drp); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
931 |
if (db->db.db_object != DMU_META_DNODE_OBJECT) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
932 |
arc_buf_thaw(db->db_buf); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
933 |
} |
789 | 934 |
mutex_exit(&db->db_mtx); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
935 |
return (*drp); |
789 | 936 |
} |
937 |
||
938 |
/* |
|
939 |
* Only valid if not already dirty. |
|
940 |
*/ |
|
941 |
ASSERT(dn->dn_dirtyctx == DN_UNDIRTIED || dn->dn_dirtyctx == |
|
942 |
(dmu_tx_is_syncing(tx) ? DN_DIRTY_SYNC : DN_DIRTY_OPEN)); |
|
943 |
||
944 |
ASSERT3U(dn->dn_nlevels, >, db->db_level); |
|
945 |
ASSERT((dn->dn_phys->dn_nlevels == 0 && db->db_level == 0) || |
|
946 |
dn->dn_phys->dn_nlevels > db->db_level || |
|
947 |
dn->dn_next_nlevels[txgoff] > db->db_level || |
|
948 |
dn->dn_next_nlevels[(tx->tx_txg-1) & TXG_MASK] > db->db_level || |
|
949 |
dn->dn_next_nlevels[(tx->tx_txg-2) & TXG_MASK] > db->db_level); |
|
950 |
||
951 |
/* |
|
952 |
* We should only be dirtying in syncing context if it's the |
|
953 |
* mos, a spa os, or we're initializing the os. However, we are |
|
954 |
* allowed to dirty in syncing context provided we already |
|
955 |
* dirtied it in open context. Hence we must make this |
|
956 |
* assertion only if we're not already dirty. |
|
957 |
*/ |
|
958 |
ASSERT(!dmu_tx_is_syncing(tx) || |
|
959 |
os->os_dsl_dataset == NULL || |
|
960 |
!dsl_dir_is_private(os->os_dsl_dataset->ds_dir) || |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
961 |
!BP_IS_HOLE(os->os_rootbp)); |
789 | 962 |
ASSERT(db->db.db_size != 0); |
963 |
||
964 |
dprintf_dbuf(db, "size=%llx\n", (u_longlong_t)db->db.db_size); |
|
965 |
||
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
966 |
if (db->db_blkid != DB_BONUS_BLKID) { |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
967 |
/* |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
968 |
* Update the accounting. |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
969 |
*/ |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
970 |
if (dbuf_block_freeable(db)) { |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
971 |
blkptr_t *bp = db->db_blkptr; |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
972 |
int64_t willfree = (bp && !BP_IS_HOLE(bp)) ? |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
973 |
bp_get_dasize(os->os_spa, bp) : db->db.db_size; |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
974 |
/* |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
975 |
* This is only a guess -- if the dbuf is dirty |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
976 |
* in a previous txg, we don't know how much |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
977 |
* space it will use on disk yet. We should |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
978 |
* really have the struct_rwlock to access |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
979 |
* db_blkptr, but since this is just a guess, |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
980 |
* it's OK if we get an odd answer. |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
981 |
*/ |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
982 |
dnode_willuse_space(dn, -willfree, tx); |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
983 |
} |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
984 |
dnode_willuse_space(dn, db->db.db_size, tx); |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
985 |
} |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
986 |
|
1544 | 987 |
/* |
988 |
* If this buffer is dirty in an old transaction group we need |
|
989 |
* to make a copy of it so that the changes we make in this |
|
990 |
* transaction group won't leak out when we sync the older txg. |
|
991 |
*/ |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
992 |
dr = kmem_zalloc(sizeof (dbuf_dirty_record_t), KM_SLEEP); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
993 |
if (db->db_level == 0) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
994 |
void *data_old = db->db_buf; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
995 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
996 |
if (db->db_blkid == DB_BONUS_BLKID) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
997 |
dbuf_fix_old_data(db, tx->tx_txg); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
998 |
data_old = db->db.db_data; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
999 |
} else if (db->db.db_object != DMU_META_DNODE_OBJECT) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1000 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1001 |
* Release the data buffer from the cache so that we |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1002 |
* can modify it without impacting possible other users |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1003 |
* of this cached data block. Note that indirect |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1004 |
* blocks and private objects are not released until the |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1005 |
* syncing state (since they are only modified then). |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1006 |
*/ |
789 | 1007 |
arc_release(db->db_buf, db); |
1008 |
dbuf_fix_old_data(db, tx->tx_txg); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1009 |
data_old = db->db_buf; |
789 | 1010 |
} |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1011 |
ASSERT(data_old != NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1012 |
dr->dt.dl.dr_data = data_old; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1013 |
} else { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1014 |
mutex_init(&dr->dt.di.dr_mtx, NULL, MUTEX_DEFAULT, NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1015 |
list_create(&dr->dt.di.dr_children, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1016 |
sizeof (dbuf_dirty_record_t), |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1017 |
offsetof(dbuf_dirty_record_t, dr_dirty_node)); |
789 | 1018 |
} |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1019 |
dr->dr_dbuf = db; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1020 |
dr->dr_txg = tx->tx_txg; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1021 |
dr->dr_next = *drp; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1022 |
*drp = dr; |
789 | 1023 |
|
1024 |
/* |
|
1025 |
* We could have been freed_in_flight between the dbuf_noread |
|
1026 |
* and dbuf_dirty. We win, as though the dbuf_noread() had |
|
1027 |
* happened after the free. |
|
1028 |
*/ |
|
1029 |
if (db->db_level == 0 && db->db_blkid != DB_BONUS_BLKID) { |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1030 |
mutex_enter(&dn->dn_mtx); |
789 | 1031 |
dnode_clear_range(dn, db->db_blkid, 1, tx); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1032 |
mutex_exit(&dn->dn_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1033 |
db->db_freed_in_flight = FALSE; |
789 | 1034 |
} |
1035 |
||
1036 |
/* |
|
1037 |
* This buffer is now part of this txg |
|
1038 |
*/ |
|
1039 |
dbuf_add_ref(db, (void *)(uintptr_t)tx->tx_txg); |
|
1040 |
db->db_dirtycnt += 1; |
|
1041 |
ASSERT3U(db->db_dirtycnt, <=, 3); |
|
1042 |
||
1043 |
mutex_exit(&db->db_mtx); |
|
1044 |
||
1045 |
if (db->db_blkid == DB_BONUS_BLKID) { |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1046 |
mutex_enter(&dn->dn_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1047 |
ASSERT(!list_link_active(&dr->dr_dirty_node)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1048 |
list_insert_tail(&dn->dn_dirty_records[txgoff], dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1049 |
mutex_exit(&dn->dn_mtx); |
789 | 1050 |
dnode_setdirty(dn, tx); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1051 |
return (dr); |
789 | 1052 |
} |
1053 |
||
2688
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
1054 |
if (db->db_level == 0) { |
789 | 1055 |
dnode_new_blkid(dn, db->db_blkid, tx); |
2688
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
1056 |
ASSERT(dn->dn_maxblkid >= db->db_blkid); |
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
1057 |
} |
789 | 1058 |
|
1059 |
if (!RW_WRITE_HELD(&dn->dn_struct_rwlock)) { |
|
1060 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
|
1061 |
drop_struct_lock = TRUE; |
|
1062 |
} |
|
1063 |
||
2688
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
1064 |
if (db->db_level+1 < dn->dn_nlevels) { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1065 |
dmu_buf_impl_t *parent = db->db_parent; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1066 |
dbuf_dirty_record_t *di; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1067 |
int parent_held = FALSE; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1068 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1069 |
if (db->db_parent == NULL || db->db_parent == dn->dn_dbuf) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1070 |
int epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1071 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1072 |
parent = dbuf_hold_level(dn, db->db_level+1, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1073 |
db->db_blkid >> epbs, FTAG); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1074 |
parent_held = TRUE; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1075 |
} |
789 | 1076 |
if (drop_struct_lock) |
1077 |
rw_exit(&dn->dn_struct_rwlock); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1078 |
ASSERT3U(db->db_level+1, ==, parent->db_level); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1079 |
di = dbuf_dirty(parent, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1080 |
if (parent_held) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1081 |
dbuf_rele(parent, FTAG); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1082 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1083 |
mutex_enter(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1084 |
/* possible race with dbuf_undirty() */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1085 |
if (db->db_last_dirty == dr || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1086 |
dn->dn_object == DMU_META_DNODE_OBJECT) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1087 |
mutex_enter(&di->dt.di.dr_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1088 |
ASSERT3U(di->dr_txg, ==, tx->tx_txg); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1089 |
ASSERT(!list_link_active(&dr->dr_dirty_node)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1090 |
list_insert_tail(&di->dt.di.dr_children, dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1091 |
mutex_exit(&di->dt.di.dr_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1092 |
dr->dr_parent = di; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1093 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1094 |
mutex_exit(&db->db_mtx); |
789 | 1095 |
} else { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1096 |
ASSERT(db->db_level+1 == dn->dn_nlevels); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1097 |
ASSERT(db->db_blkid < dn->dn_nblkptr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1098 |
ASSERT(db->db_parent == NULL || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1099 |
db->db_parent == db->db_dnode->dn_dbuf); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1100 |
mutex_enter(&dn->dn_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1101 |
ASSERT(!list_link_active(&dr->dr_dirty_node)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1102 |
list_insert_tail(&dn->dn_dirty_records[txgoff], dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1103 |
mutex_exit(&dn->dn_mtx); |
789 | 1104 |
if (drop_struct_lock) |
1105 |
rw_exit(&dn->dn_struct_rwlock); |
|
1106 |
} |
|
1107 |
||
1108 |
dnode_setdirty(dn, tx); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1109 |
return (dr); |
789 | 1110 |
} |
1111 |
||
1112 |
static int |
|
1113 |
dbuf_undirty(dmu_buf_impl_t *db, dmu_tx_t *tx) |
|
1114 |
{ |
|
1115 |
dnode_t *dn = db->db_dnode; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1116 |
uint64_t txg = tx->tx_txg; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1117 |
dbuf_dirty_record_t *dr; |
789 | 1118 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1119 |
ASSERT(txg != 0); |
1544 | 1120 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
789 | 1121 |
|
1122 |
mutex_enter(&db->db_mtx); |
|
1123 |
||
1124 |
/* |
|
1125 |
* If this buffer is not dirty, we're done. |
|
1126 |
*/ |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1127 |
for (dr = db->db_last_dirty; dr; dr = dr->dr_next) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1128 |
if (dr->dr_txg <= txg) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1129 |
break; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1130 |
if (dr == NULL || dr->dr_txg < txg) { |
789 | 1131 |
mutex_exit(&db->db_mtx); |
1132 |
return (0); |
|
1133 |
} |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1134 |
ASSERT(dr->dr_txg == txg); |
789 | 1135 |
|
1136 |
/* |
|
1137 |
* If this buffer is currently held, we cannot undirty |
|
1138 |
* it, since one of the current holders may be in the |
|
1139 |
* middle of an update. Note that users of dbuf_undirty() |
|
1140 |
* should not place a hold on the dbuf before the call. |
|
1141 |
*/ |
|
1142 |
if (refcount_count(&db->db_holds) > db->db_dirtycnt) { |
|
1143 |
mutex_exit(&db->db_mtx); |
|
2688
12e2ad22b970
6447701 ZFS hangs when iSCSI Target attempts to initialize its backing store
maybee
parents:
2417
diff
changeset
|
1144 |
/* Make sure we don't toss this buffer at sync phase */ |
789 | 1145 |
mutex_enter(&dn->dn_mtx); |
1146 |
dnode_clear_range(dn, db->db_blkid, 1, tx); |
|
1147 |
mutex_exit(&dn->dn_mtx); |
|
1148 |
return (0); |
|
1149 |
} |
|
1150 |
||
1151 |
dprintf_dbuf(db, "size=%llx\n", (u_longlong_t)db->db.db_size); |
|
1152 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1153 |
ASSERT(db->db.db_size != 0); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1154 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1155 |
/* XXX would be nice to fix up dn_towrite_space[] */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1156 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1157 |
db->db_last_dirty = dr->dr_next; |
789 | 1158 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1159 |
if (dr->dr_parent) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1160 |
mutex_enter(&dr->dr_parent->dt.di.dr_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1161 |
list_remove(&dr->dr_parent->dt.di.dr_children, dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1162 |
mutex_exit(&dr->dr_parent->dt.di.dr_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1163 |
} else if (db->db_level+1 == dn->dn_nlevels) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1164 |
ASSERT3P(db->db_parent, ==, dn->dn_dbuf); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1165 |
mutex_enter(&dn->dn_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1166 |
list_remove(&dn->dn_dirty_records[txg & TXG_MASK], dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1167 |
mutex_exit(&dn->dn_mtx); |
789 | 1168 |
} |
1169 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1170 |
if (db->db_level == 0) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1171 |
dbuf_unoverride(dr); |
789 | 1172 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1173 |
ASSERT(db->db_buf != NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1174 |
ASSERT(dr->dt.dl.dr_data != NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1175 |
if (dr->dt.dl.dr_data != db->db_buf) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1176 |
VERIFY(arc_buf_remove_ref(dr->dt.dl.dr_data, db) == 1); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1177 |
} else { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1178 |
ASSERT(db->db_buf != NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1179 |
ASSERT(list_head(&dr->dt.di.dr_children) == NULL); |
4831
41ec732c6d9f
6584470 zdb needs to initialize the bpl_lock mutex
gw25295
parents:
4506
diff
changeset
|
1180 |
mutex_destroy(&dr->dt.di.dr_mtx); |
41ec732c6d9f
6584470 zdb needs to initialize the bpl_lock mutex
gw25295
parents:
4506
diff
changeset
|
1181 |
list_destroy(&dr->dt.di.dr_children); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1182 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1183 |
kmem_free(dr, sizeof (dbuf_dirty_record_t)); |
789 | 1184 |
|
1185 |
ASSERT(db->db_dirtycnt > 0); |
|
1186 |
db->db_dirtycnt -= 1; |
|
1187 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1188 |
if (refcount_remove(&db->db_holds, (void *)(uintptr_t)txg) == 0) { |
1544 | 1189 |
arc_buf_t *buf = db->db_buf; |
789 | 1190 |
|
1544 | 1191 |
ASSERT(arc_released(buf)); |
1192 |
dbuf_set_data(db, NULL); |
|
1193 |
VERIFY(arc_buf_remove_ref(buf, db) == 1); |
|
789 | 1194 |
dbuf_evict(db); |
1195 |
return (1); |
|
1196 |
} |
|
1197 |
||
1198 |
mutex_exit(&db->db_mtx); |
|
1199 |
return (0); |
|
1200 |
} |
|
1201 |
||
1202 |
#pragma weak dmu_buf_will_dirty = dbuf_will_dirty |
|
1203 |
void |
|
1204 |
dbuf_will_dirty(dmu_buf_impl_t *db, dmu_tx_t *tx) |
|
1205 |
{ |
|
1206 |
int rf = DB_RF_MUST_SUCCEED; |
|
1207 |
||
1208 |
ASSERT(tx->tx_txg != 0); |
|
1209 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
|
1210 |
||
1211 |
if (RW_WRITE_HELD(&db->db_dnode->dn_struct_rwlock)) |
|
1212 |
rf |= DB_RF_HAVESTRUCT; |
|
1544 | 1213 |
(void) dbuf_read(db, NULL, rf); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1214 |
(void) dbuf_dirty(db, tx); |
789 | 1215 |
} |
1216 |
||
1217 |
void |
|
1544 | 1218 |
dmu_buf_will_fill(dmu_buf_t *db_fake, dmu_tx_t *tx) |
789 | 1219 |
{ |
1544 | 1220 |
dmu_buf_impl_t *db = (dmu_buf_impl_t *)db_fake; |
1221 |
||
1222 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
|
789 | 1223 |
ASSERT(tx->tx_txg != 0); |
1224 |
ASSERT(db->db_level == 0); |
|
1225 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
|
1226 |
||
1544 | 1227 |
ASSERT(db->db.db_object != DMU_META_DNODE_OBJECT || |
789 | 1228 |
dmu_tx_private_ok(tx)); |
1229 |
||
1230 |
dbuf_noread(db); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1231 |
(void) dbuf_dirty(db, tx); |
789 | 1232 |
} |
1233 |
||
1234 |
#pragma weak dmu_buf_fill_done = dbuf_fill_done |
|
1235 |
/* ARGSUSED */ |
|
1236 |
void |
|
1237 |
dbuf_fill_done(dmu_buf_impl_t *db, dmu_tx_t *tx) |
|
1238 |
{ |
|
1239 |
mutex_enter(&db->db_mtx); |
|
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
1240 |
DBUF_VERIFY(db); |
789 | 1241 |
|
1242 |
if (db->db_state == DB_FILL) { |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1243 |
if (db->db_level == 0 && db->db_freed_in_flight) { |
1544 | 1244 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
789 | 1245 |
/* we were freed while filling */ |
1246 |
/* XXX dbuf_undirty? */ |
|
1247 |
bzero(db->db.db_data, db->db.db_size); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1248 |
db->db_freed_in_flight = FALSE; |
789 | 1249 |
} |
1250 |
db->db_state = DB_CACHED; |
|
1251 |
cv_broadcast(&db->db_changed); |
|
1252 |
} |
|
1253 |
mutex_exit(&db->db_mtx); |
|
1254 |
} |
|
1255 |
||
1544 | 1256 |
/* |
1257 |
* "Clear" the contents of this dbuf. This will mark the dbuf |
|
1258 |
* EVICTING and clear *most* of its references. Unfortunetely, |
|
1259 |
* when we are not holding the dn_dbufs_mtx, we can't clear the |
|
1260 |
* entry in the dn_dbufs list. We have to wait until dbuf_destroy() |
|
1261 |
* in this case. For callers from the DMU we will usually see: |
|
1262 |
* dbuf_clear()->arc_buf_evict()->dbuf_do_evict()->dbuf_destroy() |
|
1263 |
* For the arc callback, we will usually see: |
|
1264 |
* dbuf_do_evict()->dbuf_clear();dbuf_destroy() |
|
1265 |
* Sometimes, though, we will get a mix of these two: |
|
1266 |
* DMU: dbuf_clear()->arc_buf_evict() |
|
1267 |
* ARC: dbuf_do_evict()->dbuf_destroy() |
|
1268 |
*/ |
|
1269 |
void |
|
789 | 1270 |
dbuf_clear(dmu_buf_impl_t *db) |
1271 |
{ |
|
1272 |
dnode_t *dn = db->db_dnode; |
|
1544 | 1273 |
dmu_buf_impl_t *parent = db->db_parent; |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1274 |
dmu_buf_impl_t *dndb = dn->dn_dbuf; |
1544 | 1275 |
int dbuf_gone = FALSE; |
789 | 1276 |
|
1277 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
|
1278 |
ASSERT(refcount_is_zero(&db->db_holds)); |
|
1279 |
||
1544 | 1280 |
dbuf_evict_user(db); |
1281 |
||
789 | 1282 |
if (db->db_state == DB_CACHED) { |
1544 | 1283 |
ASSERT(db->db.db_data != NULL); |
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1284 |
if (db->db_blkid == DB_BONUS_BLKID) { |
1544 | 1285 |
zio_buf_free(db->db.db_data, DN_MAX_BONUSLEN); |
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1286 |
arc_space_return(DN_MAX_BONUSLEN); |
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1287 |
} |
789 | 1288 |
db->db.db_data = NULL; |
1289 |
db->db_state = DB_UNCACHED; |
|
1290 |
} |
|
1291 |
||
1292 |
ASSERT3U(db->db_state, ==, DB_UNCACHED); |
|
1293 |
ASSERT(db->db_data_pending == NULL); |
|
1294 |
||
1544 | 1295 |
db->db_state = DB_EVICTING; |
1296 |
db->db_blkptr = NULL; |
|
1297 |
||
1298 |
if (db->db_blkid != DB_BONUS_BLKID && MUTEX_HELD(&dn->dn_dbufs_mtx)) { |
|
1299 |
list_remove(&dn->dn_dbufs, db); |
|
1300 |
dnode_rele(dn, db); |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1301 |
db->db_dnode = NULL; |
1544 | 1302 |
} |
1303 |
||
1304 |
if (db->db_buf) |
|
1305 |
dbuf_gone = arc_buf_evict(db->db_buf); |
|
1306 |
||
1307 |
if (!dbuf_gone) |
|
1308 |
mutex_exit(&db->db_mtx); |
|
789 | 1309 |
|
1310 |
/* |
|
1311 |
* If this dbuf is referened from an indirect dbuf, |
|
1312 |
* decrement the ref count on the indirect dbuf. |
|
1313 |
*/ |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1314 |
if (parent && parent != dndb) |
1544 | 1315 |
dbuf_rele(parent, db); |
789 | 1316 |
} |
1317 |
||
1318 |
static int |
|
1319 |
dbuf_findbp(dnode_t *dn, int level, uint64_t blkid, int fail_sparse, |
|
1320 |
dmu_buf_impl_t **parentp, blkptr_t **bpp) |
|
1321 |
{ |
|
1322 |
int nlevels, epbs; |
|
1323 |
||
2417 | 1324 |
*parentp = NULL; |
1325 |
*bpp = NULL; |
|
1326 |
||
1544 | 1327 |
ASSERT(blkid != DB_BONUS_BLKID); |
1328 |
||
789 | 1329 |
if (dn->dn_phys->dn_nlevels == 0) |
1330 |
nlevels = 1; |
|
1331 |
else |
|
1332 |
nlevels = dn->dn_phys->dn_nlevels; |
|
1333 |
||
1334 |
epbs = dn->dn_indblkshift - SPA_BLKPTRSHIFT; |
|
1335 |
||
1336 |
ASSERT3U(level * epbs, <, 64); |
|
1337 |
ASSERT(RW_LOCK_HELD(&dn->dn_struct_rwlock)); |
|
1544 | 1338 |
if (level >= nlevels || |
789 | 1339 |
(blkid > (dn->dn_phys->dn_maxblkid >> (level * epbs)))) { |
1340 |
/* the buffer has no parent yet */ |
|
1341 |
return (ENOENT); |
|
1342 |
} else if (level < nlevels-1) { |
|
1343 |
/* this block is referenced from an indirect block */ |
|
1344 |
int err = dbuf_hold_impl(dn, level+1, |
|
1345 |
blkid >> epbs, fail_sparse, NULL, parentp); |
|
1346 |
if (err) |
|
1347 |
return (err); |
|
1544 | 1348 |
err = dbuf_read(*parentp, NULL, |
1349 |
(DB_RF_HAVESTRUCT | DB_RF_NOPREFETCH | DB_RF_CANFAIL)); |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1350 |
if (err) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1351 |
dbuf_rele(*parentp, NULL); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1352 |
*parentp = NULL; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1353 |
return (err); |
1544 | 1354 |
} |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1355 |
*bpp = ((blkptr_t *)(*parentp)->db.db_data) + |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1356 |
(blkid & ((1ULL << epbs) - 1)); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1357 |
return (0); |
789 | 1358 |
} else { |
1359 |
/* the block is referenced from the dnode */ |
|
1360 |
ASSERT3U(level, ==, nlevels-1); |
|
1361 |
ASSERT(dn->dn_phys->dn_nblkptr == 0 || |
|
1362 |
blkid < dn->dn_phys->dn_nblkptr); |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1363 |
if (dn->dn_dbuf) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1364 |
dbuf_add_ref(dn->dn_dbuf, NULL); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1365 |
*parentp = dn->dn_dbuf; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1366 |
} |
789 | 1367 |
*bpp = &dn->dn_phys->dn_blkptr[blkid]; |
1368 |
return (0); |
|
1369 |
} |
|
1370 |
} |
|
1371 |
||
1372 |
static dmu_buf_impl_t * |
|
1373 |
dbuf_create(dnode_t *dn, uint8_t level, uint64_t blkid, |
|
1374 |
dmu_buf_impl_t *parent, blkptr_t *blkptr) |
|
1375 |
{ |
|
1376 |
objset_impl_t *os = dn->dn_objset; |
|
1377 |
dmu_buf_impl_t *db, *odb; |
|
1378 |
||
1379 |
ASSERT(RW_LOCK_HELD(&dn->dn_struct_rwlock)); |
|
1380 |
ASSERT(dn->dn_type != DMU_OT_NONE); |
|
1381 |
||
1382 |
db = kmem_cache_alloc(dbuf_cache, KM_SLEEP); |
|
1383 |
||
1384 |
db->db_objset = os; |
|
1385 |
db->db.db_object = dn->dn_object; |
|
1386 |
db->db_level = level; |
|
1387 |
db->db_blkid = blkid; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1388 |
db->db_last_dirty = NULL; |
1544 | 1389 |
db->db_dirtycnt = 0; |
1390 |
db->db_dnode = dn; |
|
1391 |
db->db_parent = parent; |
|
1392 |
db->db_blkptr = blkptr; |
|
789 | 1393 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1394 |
db->db_user_ptr = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1395 |
db->db_user_data_ptr_ptr = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1396 |
db->db_evict_func = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1397 |
db->db_immediate_evict = 0; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1398 |
db->db_freed_in_flight = 0; |
1544 | 1399 |
|
1400 |
if (blkid == DB_BONUS_BLKID) { |
|
1401 |
ASSERT3P(parent, ==, dn->dn_dbuf); |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1402 |
db->db.db_size = DN_MAX_BONUSLEN - |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1403 |
(dn->dn_nblkptr-1) * sizeof (blkptr_t); |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1404 |
ASSERT3U(db->db.db_size, >=, dn->dn_bonuslen); |
789 | 1405 |
db->db.db_offset = DB_BONUS_BLKID; |
1544 | 1406 |
db->db_state = DB_UNCACHED; |
1407 |
/* the bonus dbuf is not placed in the hash table */ |
|
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1408 |
arc_space_consume(sizeof (dmu_buf_impl_t)); |
1544 | 1409 |
return (db); |
789 | 1410 |
} else { |
1411 |
int blocksize = |
|
1412 |
db->db_level ? 1<<dn->dn_indblkshift : dn->dn_datablksz; |
|
1413 |
db->db.db_size = blocksize; |
|
1414 |
db->db.db_offset = db->db_blkid * blocksize; |
|
1415 |
} |
|
1416 |
||
1417 |
/* |
|
1418 |
* Hold the dn_dbufs_mtx while we get the new dbuf |
|
1419 |
* in the hash table *and* added to the dbufs list. |
|
1420 |
* This prevents a possible deadlock with someone |
|
1421 |
* trying to look up this dbuf before its added to the |
|
1422 |
* dn_dbufs list. |
|
1423 |
*/ |
|
1424 |
mutex_enter(&dn->dn_dbufs_mtx); |
|
1544 | 1425 |
db->db_state = DB_EVICTING; |
789 | 1426 |
if ((odb = dbuf_hash_insert(db)) != NULL) { |
1427 |
/* someone else inserted it first */ |
|
1428 |
kmem_cache_free(dbuf_cache, db); |
|
1429 |
mutex_exit(&dn->dn_dbufs_mtx); |
|
1430 |
return (odb); |
|
1431 |
} |
|
1432 |
list_insert_head(&dn->dn_dbufs, db); |
|
1544 | 1433 |
db->db_state = DB_UNCACHED; |
789 | 1434 |
mutex_exit(&dn->dn_dbufs_mtx); |
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1435 |
arc_space_consume(sizeof (dmu_buf_impl_t)); |
789 | 1436 |
|
1437 |
if (parent && parent != dn->dn_dbuf) |
|
1438 |
dbuf_add_ref(parent, db); |
|
1439 |
||
1544 | 1440 |
ASSERT(dn->dn_object == DMU_META_DNODE_OBJECT || |
1441 |
refcount_count(&dn->dn_holds) > 0); |
|
789 | 1442 |
(void) refcount_add(&dn->dn_holds, db); |
1443 |
||
1444 |
dprintf_dbuf(db, "db=%p\n", db); |
|
1445 |
||
1446 |
return (db); |
|
1447 |
} |
|
1448 |
||
1449 |
static int |
|
1544 | 1450 |
dbuf_do_evict(void *private) |
789 | 1451 |
{ |
1544 | 1452 |
arc_buf_t *buf = private; |
1453 |
dmu_buf_impl_t *db = buf->b_private; |
|
789 | 1454 |
|
1544 | 1455 |
if (!MUTEX_HELD(&db->db_mtx)) |
1456 |
mutex_enter(&db->db_mtx); |
|
789 | 1457 |
|
1544 | 1458 |
ASSERT(refcount_is_zero(&db->db_holds)); |
789 | 1459 |
|
1544 | 1460 |
if (db->db_state != DB_EVICTING) { |
1461 |
ASSERT(db->db_state == DB_CACHED); |
|
1462 |
DBUF_VERIFY(db); |
|
1463 |
db->db_buf = NULL; |
|
1464 |
dbuf_evict(db); |
|
1465 |
} else { |
|
1466 |
mutex_exit(&db->db_mtx); |
|
1467 |
dbuf_destroy(db); |
|
789 | 1468 |
} |
1544 | 1469 |
return (0); |
789 | 1470 |
} |
1471 |
||
1472 |
static void |
|
1473 |
dbuf_destroy(dmu_buf_impl_t *db) |
|
1474 |
{ |
|
1475 |
ASSERT(refcount_is_zero(&db->db_holds)); |
|
1476 |
||
1544 | 1477 |
if (db->db_blkid != DB_BONUS_BLKID) { |
1478 |
/* |
|
1479 |
* If this dbuf is still on the dn_dbufs list, |
|
1480 |
* remove it from that list. |
|
1481 |
*/ |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1482 |
if (db->db_dnode) { |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1483 |
dnode_t *dn = db->db_dnode; |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1484 |
|
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1485 |
mutex_enter(&dn->dn_dbufs_mtx); |
1544 | 1486 |
list_remove(&dn->dn_dbufs, db); |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1487 |
mutex_exit(&dn->dn_dbufs_mtx); |
1544 | 1488 |
|
1489 |
dnode_rele(dn, db); |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1490 |
db->db_dnode = NULL; |
1544 | 1491 |
} |
1492 |
dbuf_hash_remove(db); |
|
1493 |
} |
|
1494 |
db->db_parent = NULL; |
|
1495 |
db->db_buf = NULL; |
|
1496 |
||
4312
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1497 |
ASSERT(!list_link_active(&db->db_link)); |
789 | 1498 |
ASSERT(db->db.db_data == NULL); |
1499 |
ASSERT(db->db_hash_next == NULL); |
|
1500 |
ASSERT(db->db_blkptr == NULL); |
|
1501 |
ASSERT(db->db_data_pending == NULL); |
|
1502 |
||
1503 |
kmem_cache_free(dbuf_cache, db); |
|
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1504 |
arc_space_return(sizeof (dmu_buf_impl_t)); |
789 | 1505 |
} |
1506 |
||
1507 |
void |
|
1508 |
dbuf_prefetch(dnode_t *dn, uint64_t blkid) |
|
1509 |
{ |
|
2391 | 1510 |
dmu_buf_impl_t *db = NULL; |
789 | 1511 |
blkptr_t *bp = NULL; |
1512 |
||
1513 |
ASSERT(blkid != DB_BONUS_BLKID); |
|
1514 |
ASSERT(RW_LOCK_HELD(&dn->dn_struct_rwlock)); |
|
1515 |
||
1516 |
if (dnode_block_freed(dn, blkid)) |
|
1517 |
return; |
|
1518 |
||
1519 |
/* dbuf_find() returns with db_mtx held */ |
|
1520 |
if (db = dbuf_find(dn, 0, blkid)) { |
|
2391 | 1521 |
if (refcount_count(&db->db_holds) > 0) { |
1522 |
/* |
|
1523 |
* This dbuf is active. We assume that it is |
|
1524 |
* already CACHED, or else about to be either |
|
1525 |
* read or filled. |
|
1526 |
*/ |
|
1527 |
mutex_exit(&db->db_mtx); |
|
1528 |
return; |
|
1529 |
} |
|
789 | 1530 |
mutex_exit(&db->db_mtx); |
2417 | 1531 |
db = NULL; |
789 | 1532 |
} |
1533 |
||
2391 | 1534 |
if (dbuf_findbp(dn, 0, blkid, TRUE, &db, &bp) == 0) { |
789 | 1535 |
if (bp && !BP_IS_HOLE(bp)) { |
2391 | 1536 |
uint32_t aflags = ARC_NOWAIT | ARC_PREFETCH; |
1544 | 1537 |
zbookmark_t zb; |
1538 |
zb.zb_objset = dn->dn_objset->os_dsl_dataset ? |
|
1539 |
dn->dn_objset->os_dsl_dataset->ds_object : 0; |
|
1540 |
zb.zb_object = dn->dn_object; |
|
1541 |
zb.zb_level = 0; |
|
1542 |
zb.zb_blkid = blkid; |
|
1543 |
||
789 | 1544 |
(void) arc_read(NULL, dn->dn_objset->os_spa, bp, |
1545 |
dmu_ot[dn->dn_type].ot_byteswap, |
|
1546 |
NULL, NULL, ZIO_PRIORITY_ASYNC_READ, |
|
1547 |
ZIO_FLAG_CANFAIL | ZIO_FLAG_SPECULATIVE, |
|
2391 | 1548 |
&aflags, &zb); |
789 | 1549 |
} |
2391 | 1550 |
if (db) |
1551 |
dbuf_rele(db, NULL); |
|
789 | 1552 |
} |
1553 |
} |
|
1554 |
||
1555 |
/* |
|
1556 |
* Returns with db_holds incremented, and db_mtx not held. |
|
1557 |
* Note: dn_struct_rwlock must be held. |
|
1558 |
*/ |
|
1559 |
int |
|
1560 |
dbuf_hold_impl(dnode_t *dn, uint8_t level, uint64_t blkid, int fail_sparse, |
|
1561 |
void *tag, dmu_buf_impl_t **dbp) |
|
1562 |
{ |
|
1563 |
dmu_buf_impl_t *db, *parent = NULL; |
|
1564 |
||
1544 | 1565 |
ASSERT(blkid != DB_BONUS_BLKID); |
789 | 1566 |
ASSERT(RW_LOCK_HELD(&dn->dn_struct_rwlock)); |
1567 |
ASSERT3U(dn->dn_nlevels, >, level); |
|
1568 |
||
1569 |
*dbp = NULL; |
|
1544 | 1570 |
top: |
789 | 1571 |
/* dbuf_find() returns with db_mtx held */ |
1572 |
db = dbuf_find(dn, level, blkid); |
|
1573 |
||
1574 |
if (db == NULL) { |
|
1575 |
blkptr_t *bp = NULL; |
|
1576 |
int err; |
|
1577 |
||
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1578 |
ASSERT3P(parent, ==, NULL); |
789 | 1579 |
err = dbuf_findbp(dn, level, blkid, fail_sparse, &parent, &bp); |
1580 |
if (fail_sparse) { |
|
1581 |
if (err == 0 && bp && BP_IS_HOLE(bp)) |
|
1582 |
err = ENOENT; |
|
1583 |
if (err) { |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1584 |
if (parent) |
1544 | 1585 |
dbuf_rele(parent, NULL); |
789 | 1586 |
return (err); |
1587 |
} |
|
1588 |
} |
|
1544 | 1589 |
if (err && err != ENOENT) |
1590 |
return (err); |
|
789 | 1591 |
db = dbuf_create(dn, level, blkid, parent, bp); |
1592 |
} |
|
1593 |
||
1544 | 1594 |
if (db->db_buf && refcount_is_zero(&db->db_holds)) { |
1595 |
arc_buf_add_ref(db->db_buf, db); |
|
1596 |
if (db->db_buf->b_data == NULL) { |
|
1597 |
dbuf_clear(db); |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1598 |
if (parent) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1599 |
dbuf_rele(parent, NULL); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1600 |
parent = NULL; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1601 |
} |
1544 | 1602 |
goto top; |
1603 |
} |
|
1604 |
ASSERT3P(db->db.db_data, ==, db->db_buf->b_data); |
|
1605 |
} |
|
1606 |
||
1607 |
ASSERT(db->db_buf == NULL || arc_referenced(db->db_buf)); |
|
1608 |
||
789 | 1609 |
/* |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1610 |
* If this buffer is currently syncing out, and we are are |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1611 |
* still referencing it from db_data, we need to make a copy |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1612 |
* of it in case we decide we want to dirty it again in this txg. |
789 | 1613 |
*/ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1614 |
if (db->db_level == 0 && db->db_blkid != DB_BONUS_BLKID && |
1544 | 1615 |
dn->dn_object != DMU_META_DNODE_OBJECT && |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1616 |
db->db_state == DB_CACHED && db->db_data_pending) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1617 |
dbuf_dirty_record_t *dr = db->db_data_pending; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1618 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1619 |
if (dr->dt.dl.dr_data == db->db_buf) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1620 |
arc_buf_contents_t type = DBUF_GET_BUFC_TYPE(db); |
789 | 1621 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1622 |
dbuf_set_data(db, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1623 |
arc_buf_alloc(db->db_dnode->dn_objset->os_spa, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1624 |
db->db.db_size, db, type)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1625 |
bcopy(dr->dt.dl.dr_data->b_data, db->db.db_data, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1626 |
db->db.db_size); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1627 |
} |
789 | 1628 |
} |
1629 |
||
1544 | 1630 |
(void) refcount_add(&db->db_holds, tag); |
789 | 1631 |
dbuf_update_data(db); |
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
1632 |
DBUF_VERIFY(db); |
789 | 1633 |
mutex_exit(&db->db_mtx); |
1634 |
||
1635 |
/* NOTE: we can't rele the parent until after we drop the db_mtx */ |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
1636 |
if (parent) |
1544 | 1637 |
dbuf_rele(parent, NULL); |
789 | 1638 |
|
1639 |
ASSERT3P(db->db_dnode, ==, dn); |
|
1640 |
ASSERT3U(db->db_blkid, ==, blkid); |
|
1641 |
ASSERT3U(db->db_level, ==, level); |
|
1642 |
*dbp = db; |
|
1643 |
||
1644 |
return (0); |
|
1645 |
} |
|
1646 |
||
1647 |
dmu_buf_impl_t * |
|
1544 | 1648 |
dbuf_hold(dnode_t *dn, uint64_t blkid, void *tag) |
789 | 1649 |
{ |
1650 |
dmu_buf_impl_t *db; |
|
1544 | 1651 |
int err = dbuf_hold_impl(dn, 0, blkid, FALSE, tag, &db); |
1652 |
return (err ? NULL : db); |
|
789 | 1653 |
} |
1654 |
||
1655 |
dmu_buf_impl_t * |
|
1656 |
dbuf_hold_level(dnode_t *dn, int level, uint64_t blkid, void *tag) |
|
1657 |
{ |
|
1658 |
dmu_buf_impl_t *db; |
|
1544 | 1659 |
int err = dbuf_hold_impl(dn, level, blkid, FALSE, tag, &db); |
1660 |
return (err ? NULL : db); |
|
789 | 1661 |
} |
1662 |
||
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1663 |
void |
1544 | 1664 |
dbuf_create_bonus(dnode_t *dn) |
789 | 1665 |
{ |
1544 | 1666 |
ASSERT(RW_WRITE_HELD(&dn->dn_struct_rwlock)); |
1667 |
||
1668 |
ASSERT(dn->dn_bonus == NULL); |
|
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1669 |
dn->dn_bonus = dbuf_create(dn, 0, DB_BONUS_BLKID, dn->dn_dbuf, NULL); |
789 | 1670 |
} |
1671 |
||
1544 | 1672 |
#pragma weak dmu_buf_add_ref = dbuf_add_ref |
789 | 1673 |
void |
1674 |
dbuf_add_ref(dmu_buf_impl_t *db, void *tag) |
|
1675 |
{ |
|
1544 | 1676 |
int64_t holds = refcount_add(&db->db_holds, tag); |
1677 |
ASSERT(holds > 1); |
|
789 | 1678 |
} |
1679 |
||
1544 | 1680 |
#pragma weak dmu_buf_rele = dbuf_rele |
789 | 1681 |
void |
1544 | 1682 |
dbuf_rele(dmu_buf_impl_t *db, void *tag) |
789 | 1683 |
{ |
1684 |
int64_t holds; |
|
1685 |
||
1686 |
mutex_enter(&db->db_mtx); |
|
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
1687 |
DBUF_VERIFY(db); |
789 | 1688 |
|
1689 |
holds = refcount_remove(&db->db_holds, tag); |
|
1544 | 1690 |
ASSERT(holds >= 0); |
1691 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1692 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1693 |
* We can't freeze indirects if there is a possibility that they |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1694 |
* may be modified in the current syncing context. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1695 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1696 |
if (db->db_buf && holds == (db->db_level == 0 ? db->db_dirtycnt : 0)) |
3093
71525e4187d5
6490569 verify arc bufs are not modified when they shouldn't be
ahrens
parents:
2986
diff
changeset
|
1697 |
arc_buf_freeze(db->db_buf); |
71525e4187d5
6490569 verify arc bufs are not modified when they shouldn't be
ahrens
parents:
2986
diff
changeset
|
1698 |
|
1544 | 1699 |
if (holds == db->db_dirtycnt && |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1700 |
db->db_level == 0 && db->db_immediate_evict) |
1544 | 1701 |
dbuf_evict_user(db); |
789 | 1702 |
|
1703 |
if (holds == 0) { |
|
1544 | 1704 |
if (db->db_blkid == DB_BONUS_BLKID) { |
1705 |
mutex_exit(&db->db_mtx); |
|
1706 |
dnode_rele(db->db_dnode, db); |
|
1707 |
} else if (db->db_buf == NULL) { |
|
1708 |
/* |
|
1709 |
* This is a special case: we never associated this |
|
1710 |
* dbuf with any data allocated from the ARC. |
|
1711 |
*/ |
|
1712 |
ASSERT3U(db->db_state, ==, DB_UNCACHED); |
|
1713 |
dbuf_evict(db); |
|
3093
71525e4187d5
6490569 verify arc bufs are not modified when they shouldn't be
ahrens
parents:
2986
diff
changeset
|
1714 |
} else if (arc_released(db->db_buf)) { |
1544 | 1715 |
arc_buf_t *buf = db->db_buf; |
1716 |
/* |
|
1717 |
* This dbuf has anonymous data associated with it. |
|
1718 |
*/ |
|
1719 |
dbuf_set_data(db, NULL); |
|
1720 |
VERIFY(arc_buf_remove_ref(buf, db) == 1); |
|
1721 |
dbuf_evict(db); |
|
1722 |
} else { |
|
1723 |
VERIFY(arc_buf_remove_ref(db->db_buf, db) == 0); |
|
1724 |
mutex_exit(&db->db_mtx); |
|
1725 |
} |
|
789 | 1726 |
} else { |
1727 |
mutex_exit(&db->db_mtx); |
|
1728 |
} |
|
1729 |
} |
|
1730 |
||
1731 |
#pragma weak dmu_buf_refcount = dbuf_refcount |
|
1732 |
uint64_t |
|
1733 |
dbuf_refcount(dmu_buf_impl_t *db) |
|
1734 |
{ |
|
1735 |
return (refcount_count(&db->db_holds)); |
|
1736 |
} |
|
1737 |
||
1738 |
void * |
|
1739 |
dmu_buf_set_user(dmu_buf_t *db_fake, void *user_ptr, void *user_data_ptr_ptr, |
|
1740 |
dmu_buf_evict_func_t *evict_func) |
|
1741 |
{ |
|
1742 |
return (dmu_buf_update_user(db_fake, NULL, user_ptr, |
|
1743 |
user_data_ptr_ptr, evict_func)); |
|
1744 |
} |
|
1745 |
||
1746 |
void * |
|
1747 |
dmu_buf_set_user_ie(dmu_buf_t *db_fake, void *user_ptr, void *user_data_ptr_ptr, |
|
1748 |
dmu_buf_evict_func_t *evict_func) |
|
1749 |
{ |
|
1750 |
dmu_buf_impl_t *db = (dmu_buf_impl_t *)db_fake; |
|
1751 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1752 |
db->db_immediate_evict = TRUE; |
789 | 1753 |
return (dmu_buf_update_user(db_fake, NULL, user_ptr, |
1754 |
user_data_ptr_ptr, evict_func)); |
|
1755 |
} |
|
1756 |
||
1757 |
void * |
|
1758 |
dmu_buf_update_user(dmu_buf_t *db_fake, void *old_user_ptr, void *user_ptr, |
|
1759 |
void *user_data_ptr_ptr, dmu_buf_evict_func_t *evict_func) |
|
1760 |
{ |
|
1761 |
dmu_buf_impl_t *db = (dmu_buf_impl_t *)db_fake; |
|
1762 |
ASSERT(db->db_level == 0); |
|
1763 |
||
1764 |
ASSERT((user_ptr == NULL) == (evict_func == NULL)); |
|
1765 |
||
1766 |
mutex_enter(&db->db_mtx); |
|
1767 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1768 |
if (db->db_user_ptr == old_user_ptr) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1769 |
db->db_user_ptr = user_ptr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1770 |
db->db_user_data_ptr_ptr = user_data_ptr_ptr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1771 |
db->db_evict_func = evict_func; |
789 | 1772 |
|
1773 |
dbuf_update_data(db); |
|
1774 |
} else { |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1775 |
old_user_ptr = db->db_user_ptr; |
789 | 1776 |
} |
1777 |
||
1778 |
mutex_exit(&db->db_mtx); |
|
1779 |
return (old_user_ptr); |
|
1780 |
} |
|
1781 |
||
1782 |
void * |
|
1783 |
dmu_buf_get_user(dmu_buf_t *db_fake) |
|
1784 |
{ |
|
1785 |
dmu_buf_impl_t *db = (dmu_buf_impl_t *)db_fake; |
|
1786 |
ASSERT(!refcount_is_zero(&db->db_holds)); |
|
1787 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1788 |
return (db->db_user_ptr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1789 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1790 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1791 |
static void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1792 |
dbuf_check_blkptr(dnode_t *dn, dmu_buf_impl_t *db) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1793 |
{ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1794 |
/* ASSERT(dmu_tx_is_syncing(tx) */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1795 |
ASSERT(MUTEX_HELD(&db->db_mtx)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1796 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1797 |
if (db->db_blkptr != NULL) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1798 |
return; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1799 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1800 |
if (db->db_level == dn->dn_phys->dn_nlevels-1) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1801 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1802 |
* This buffer was allocated at a time when there was |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1803 |
* no available blkptrs from the dnode, or it was |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1804 |
* inappropriate to hook it in (i.e., nlevels mis-match). |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1805 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1806 |
ASSERT(db->db_blkid < dn->dn_phys->dn_nblkptr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1807 |
ASSERT(db->db_parent == NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1808 |
db->db_parent = dn->dn_dbuf; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1809 |
db->db_blkptr = &dn->dn_phys->dn_blkptr[db->db_blkid]; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1810 |
DBUF_VERIFY(db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1811 |
} else { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1812 |
dmu_buf_impl_t *parent = db->db_parent; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1813 |
int epbs = dn->dn_phys->dn_indblkshift - SPA_BLKPTRSHIFT; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1814 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1815 |
ASSERT(dn->dn_phys->dn_nlevels > 1); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1816 |
if (parent == NULL) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1817 |
mutex_exit(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1818 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1819 |
(void) dbuf_hold_impl(dn, db->db_level+1, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1820 |
db->db_blkid >> epbs, FALSE, db, &parent); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1821 |
rw_exit(&dn->dn_struct_rwlock); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1822 |
mutex_enter(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1823 |
db->db_parent = parent; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1824 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1825 |
db->db_blkptr = (blkptr_t *)parent->db.db_data + |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1826 |
(db->db_blkid & ((1ULL << epbs) - 1)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1827 |
DBUF_VERIFY(db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1828 |
} |
789 | 1829 |
} |
1830 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1831 |
static void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1832 |
dbuf_sync_indirect(dbuf_dirty_record_t *dr, dmu_tx_t *tx) |
789 | 1833 |
{ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1834 |
dmu_buf_impl_t *db = dr->dr_dbuf; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1835 |
dnode_t *dn = db->db_dnode; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1836 |
zio_t *zio; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1837 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1838 |
ASSERT(dmu_tx_is_syncing(tx)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1839 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1840 |
dprintf_dbuf_bp(db, db->db_blkptr, "blkptr=%p", db->db_blkptr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1841 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1842 |
mutex_enter(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1843 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1844 |
ASSERT(db->db_level > 0); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1845 |
DBUF_VERIFY(db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1846 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1847 |
if (db->db_buf == NULL) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1848 |
mutex_exit(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1849 |
(void) dbuf_read(db, NULL, DB_RF_MUST_SUCCEED); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1850 |
mutex_enter(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1851 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1852 |
ASSERT3U(db->db_state, ==, DB_CACHED); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1853 |
ASSERT3U(db->db.db_size, ==, 1<<dn->dn_phys->dn_indblkshift); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1854 |
ASSERT(db->db_buf != NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1855 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1856 |
dbuf_check_blkptr(dn, db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1857 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1858 |
db->db_data_pending = dr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1859 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1860 |
arc_release(db->db_buf, db); |
3897
278bade789ba
6437750 panic: db->db_buf==0||arc_referenced(db->db_buf), file: dbuf.c,line:1539
maybee
parents:
3882
diff
changeset
|
1861 |
mutex_exit(&db->db_mtx); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1862 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1863 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1864 |
* XXX -- we should design a compression algorithm |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1865 |
* that specializes in arrays of bps. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1866 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1867 |
dbuf_write(dr, db->db_buf, ZIO_CHECKSUM_FLETCHER_4, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1868 |
zfs_mdcomp_disable ? ZIO_COMPRESS_EMPTY : ZIO_COMPRESS_LZJB, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1869 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1870 |
zio = dr->dr_zio; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1871 |
mutex_enter(&dr->dt.di.dr_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1872 |
dbuf_sync_list(&dr->dt.di.dr_children, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1873 |
ASSERT(list_head(&dr->dt.di.dr_children) == NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1874 |
mutex_exit(&dr->dt.di.dr_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1875 |
zio_nowait(zio); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1876 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1877 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1878 |
static void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1879 |
dbuf_sync_leaf(dbuf_dirty_record_t *dr, dmu_tx_t *tx) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1880 |
{ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1881 |
arc_buf_t **datap = &dr->dt.dl.dr_data; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1882 |
dmu_buf_impl_t *db = dr->dr_dbuf; |
789 | 1883 |
dnode_t *dn = db->db_dnode; |
1884 |
objset_impl_t *os = dn->dn_objset; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1885 |
uint64_t txg = tx->tx_txg; |
1544 | 1886 |
int checksum, compress; |
789 | 1887 |
int blksz; |
1888 |
||
1889 |
ASSERT(dmu_tx_is_syncing(tx)); |
|
1890 |
||
1891 |
dprintf_dbuf_bp(db, db->db_blkptr, "blkptr=%p", db->db_blkptr); |
|
1892 |
||
1893 |
mutex_enter(&db->db_mtx); |
|
1894 |
/* |
|
1895 |
* To be synced, we must be dirtied. But we |
|
1896 |
* might have been freed after the dirty. |
|
1897 |
*/ |
|
1898 |
if (db->db_state == DB_UNCACHED) { |
|
1899 |
/* This buffer has been freed since it was dirtied */ |
|
1900 |
ASSERT(db->db.db_data == NULL); |
|
1901 |
} else if (db->db_state == DB_FILL) { |
|
1902 |
/* This buffer was freed and is now being re-filled */ |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1903 |
ASSERT(db->db.db_data != dr->dt.dl.dr_data); |
789 | 1904 |
} else { |
1905 |
ASSERT3U(db->db_state, ==, DB_CACHED); |
|
1906 |
} |
|
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
1907 |
DBUF_VERIFY(db); |
789 | 1908 |
|
1909 |
/* |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1910 |
* If this is a bonus buffer, simply copy the bonus data into the |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1911 |
* dnode. It will be written out when the dnode is synced (and it |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1912 |
* will be synced, since it must have been dirty for dbuf_sync to |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1913 |
* be called). |
789 | 1914 |
*/ |
1544 | 1915 |
if (db->db_blkid == DB_BONUS_BLKID) { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1916 |
dbuf_dirty_record_t **drp; |
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4831
diff
changeset
|
1917 |
|
1544 | 1918 |
ASSERT(*datap != NULL); |
1919 |
ASSERT3U(db->db_level, ==, 0); |
|
1920 |
ASSERT3U(dn->dn_phys->dn_bonuslen, <=, DN_MAX_BONUSLEN); |
|
1921 |
bcopy(*datap, DN_BONUS(dn->dn_phys), dn->dn_phys->dn_bonuslen); |
|
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1922 |
if (*datap != db->db.db_data) { |
1544 | 1923 |
zio_buf_free(*datap, DN_MAX_BONUSLEN); |
4309
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1924 |
arc_space_return(DN_MAX_BONUSLEN); |
3dfde0f4662d
6542676 ARC needs to track meta-data memory overhead
maybee
parents:
3897
diff
changeset
|
1925 |
} |
1544 | 1926 |
db->db_data_pending = NULL; |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1927 |
drp = &db->db_last_dirty; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1928 |
while (*drp != dr) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1929 |
drp = &(*drp)->dr_next; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1930 |
ASSERT((*drp)->dr_next == NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1931 |
*drp = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1932 |
kmem_free(dr, sizeof (dbuf_dirty_record_t)); |
1544 | 1933 |
ASSERT(db->db_dirtycnt > 0); |
1934 |
db->db_dirtycnt -= 1; |
|
1935 |
mutex_exit(&db->db_mtx); |
|
1936 |
dbuf_rele(db, (void *)(uintptr_t)txg); |
|
1937 |
return; |
|
1938 |
} |
|
1939 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1940 |
/* |
4312
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1941 |
* This function may have dropped the db_mtx lock allowing a dmu_sync |
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1942 |
* operation to sneak in. As a result, we need to ensure that we |
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1943 |
* don't check the dr_override_state until we have returned from |
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1944 |
* dbuf_check_blkptr. |
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1945 |
*/ |
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1946 |
dbuf_check_blkptr(dn, db); |
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1947 |
|
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
4309
diff
changeset
|
1948 |
/* |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1949 |
* If this buffer is in the middle of an immdiate write, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1950 |
* wait for the synchronous IO to complete. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1951 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1952 |
while (dr->dt.dl.dr_override_state == DR_IN_DMU_SYNC) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1953 |
ASSERT(dn->dn_object != DMU_META_DNODE_OBJECT); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1954 |
cv_wait(&db->db_changed, &db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1955 |
ASSERT(dr->dt.dl.dr_override_state != DR_NOT_OVERRIDDEN); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1956 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1957 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1958 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1959 |
* If this dbuf has already been written out via an immediate write, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1960 |
* just complete the write by copying over the new block pointer and |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1961 |
* updating the accounting via the write-completion functions. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1962 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1963 |
if (dr->dt.dl.dr_override_state == DR_OVERRIDDEN) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1964 |
zio_t zio_fake; |
2237 | 1965 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1966 |
zio_fake.io_private = &db; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1967 |
zio_fake.io_error = 0; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1968 |
zio_fake.io_bp = db->db_blkptr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1969 |
zio_fake.io_bp_orig = *db->db_blkptr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1970 |
zio_fake.io_txg = txg; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1971 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1972 |
*db->db_blkptr = dr->dt.dl.dr_overridden_by; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1973 |
dr->dt.dl.dr_override_state = DR_NOT_OVERRIDDEN; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1974 |
db->db_data_pending = dr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1975 |
dr->dr_zio = &zio_fake; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1976 |
mutex_exit(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1977 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1978 |
if (BP_IS_OLDER(&zio_fake.io_bp_orig, txg)) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1979 |
dsl_dataset_block_kill(os->os_dsl_dataset, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1980 |
&zio_fake.io_bp_orig, dn->dn_zio, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1981 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1982 |
dbuf_write_ready(&zio_fake, db->db_buf, db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1983 |
dbuf_write_done(&zio_fake, db->db_buf, db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1984 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1985 |
return; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1986 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1987 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1988 |
blksz = arc_buf_size(*datap); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1989 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1990 |
if (dn->dn_object != DMU_META_DNODE_OBJECT) { |
789 | 1991 |
/* |
1992 |
* If this buffer is currently "in use" (i.e., there are |
|
1993 |
* active holds and db_data still references it), then make |
|
1994 |
* a copy before we start the write so that any modifications |
|
1995 |
* from the open txg will not leak into this write. |
|
1996 |
* |
|
1997 |
* NOTE: this copy does not need to be made for objects only |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
1998 |
* modified in the syncing context (e.g. DNONE_DNODE blocks). |
789 | 1999 |
*/ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2000 |
if (refcount_count(&db->db_holds) > 1 && *datap == db->db_buf) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2001 |
arc_buf_contents_t type = DBUF_GET_BUFC_TYPE(db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2002 |
*datap = arc_buf_alloc(os->os_spa, blksz, db, type); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2003 |
bcopy(db->db.db_data, (*datap)->b_data, blksz); |
789 | 2004 |
} |
2005 |
} else { |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2006 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2007 |
* Private object buffers are released here rather |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2008 |
* than in dbuf_dirty() since they are only modified |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2009 |
* in the syncing context and we don't want the |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2010 |
* overhead of making multiple copies of the data. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2011 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2012 |
arc_release(db->db_buf, db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2013 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2014 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2015 |
ASSERT(*datap != NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2016 |
db->db_data_pending = dr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2017 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2018 |
mutex_exit(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2019 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2020 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2021 |
* Allow dnode settings to override objset settings, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2022 |
* except for metadata checksums. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2023 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2024 |
if (dmu_ot[dn->dn_type].ot_metadata) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2025 |
checksum = os->os_md_checksum; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2026 |
compress = zio_compress_select(dn->dn_compress, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2027 |
os->os_md_compress); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2028 |
} else { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2029 |
checksum = zio_checksum_select(dn->dn_checksum, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2030 |
os->os_checksum); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2031 |
compress = zio_compress_select(dn->dn_compress, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2032 |
os->os_compress); |
789 | 2033 |
} |
2034 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2035 |
dbuf_write(dr, *datap, checksum, compress, tx); |
789 | 2036 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2037 |
ASSERT(!list_link_active(&dr->dr_dirty_node)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2038 |
if (dn->dn_object == DMU_META_DNODE_OBJECT) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2039 |
list_insert_tail(&dn->dn_dirty_records[txg&TXG_MASK], dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2040 |
else |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2041 |
zio_nowait(dr->dr_zio); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2042 |
} |
789 | 2043 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2044 |
void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2045 |
dbuf_sync_list(list_t *list, dmu_tx_t *tx) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2046 |
{ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2047 |
dbuf_dirty_record_t *dr; |
789 | 2048 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2049 |
while (dr = list_head(list)) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2050 |
if (dr->dr_zio != NULL) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2051 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2052 |
* If we find an already initialized zio then we |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2053 |
* are processing the meta-dnode, and we have finished. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2054 |
* The dbufs for all dnodes are put back on the list |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2055 |
* during processing, so that we can zio_wait() |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2056 |
* these IOs after initiating all child IOs. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2057 |
*/ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2058 |
ASSERT3U(dr->dr_dbuf->db.db_object, ==, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2059 |
DMU_META_DNODE_OBJECT); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2060 |
break; |
1163
4ba797920cc2
6343073 buf[i] == 0 assertion failure when running zvol_pstress
maybee
parents:
982
diff
changeset
|
2061 |
} |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2062 |
list_remove(list, dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2063 |
if (dr->dr_dbuf->db_level > 0) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2064 |
dbuf_sync_indirect(dr, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2065 |
else |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2066 |
dbuf_sync_leaf(dr, tx); |
1163
4ba797920cc2
6343073 buf[i] == 0 assertion failure when running zvol_pstress
maybee
parents:
982
diff
changeset
|
2067 |
} |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2068 |
} |
1163
4ba797920cc2
6343073 buf[i] == 0 assertion failure when running zvol_pstress
maybee
parents:
982
diff
changeset
|
2069 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2070 |
static void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2071 |
dbuf_write(dbuf_dirty_record_t *dr, arc_buf_t *data, int checksum, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2072 |
int compress, dmu_tx_t *tx) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2073 |
{ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2074 |
dmu_buf_impl_t *db = dr->dr_dbuf; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2075 |
dnode_t *dn = db->db_dnode; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2076 |
objset_impl_t *os = dn->dn_objset; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2077 |
dmu_buf_impl_t *parent = db->db_parent; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2078 |
uint64_t txg = tx->tx_txg; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2079 |
zbookmark_t zb; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2080 |
zio_t *zio; |
3689 | 2081 |
int zio_flags; |
789 | 2082 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2083 |
if (parent != dn->dn_dbuf) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2084 |
ASSERT(parent && parent->db_data_pending); |
789 | 2085 |
ASSERT(db->db_level == parent->db_level-1); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2086 |
ASSERT(arc_released(parent->db_buf)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2087 |
zio = parent->db_data_pending->dr_zio; |
1544 | 2088 |
} else { |
789 | 2089 |
ASSERT(db->db_level == dn->dn_phys->dn_nlevels-1); |
2090 |
ASSERT3P(db->db_blkptr, ==, |
|
2091 |
&dn->dn_phys->dn_blkptr[db->db_blkid]); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2092 |
zio = dn->dn_zio; |
1544 | 2093 |
} |
789 | 2094 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2095 |
ASSERT(db->db_level == 0 || data == db->db_buf); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2096 |
ASSERT3U(db->db_blkptr->blk_birth, <=, txg); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2097 |
ASSERT(zio); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2098 |
|
1544 | 2099 |
zb.zb_objset = os->os_dsl_dataset ? os->os_dsl_dataset->ds_object : 0; |
2100 |
zb.zb_object = db->db.db_object; |
|
2101 |
zb.zb_level = db->db_level; |
|
2102 |
zb.zb_blkid = db->db_blkid; |
|
1775
e51e26b432c0
6410698 ZFS metadata needs to be more highly replicated (ditto blocks)
billm
parents:
1596
diff
changeset
|
2103 |
|
3689 | 2104 |
zio_flags = ZIO_FLAG_MUSTSUCCEED; |
2105 |
if (dmu_ot[dn->dn_type].ot_metadata || zb.zb_level != 0) |
|
2106 |
zio_flags |= ZIO_FLAG_METADATA; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2107 |
if (BP_IS_OLDER(db->db_blkptr, txg)) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2108 |
dsl_dataset_block_kill( |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2109 |
os->os_dsl_dataset, db->db_blkptr, zio, tx); |
789 | 2110 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2111 |
dr->dr_zio = arc_write(zio, os->os_spa, checksum, compress, |
3835 | 2112 |
dmu_get_replication_level(os, &zb, dn->dn_type), txg, |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2113 |
db->db_blkptr, data, dbuf_write_ready, dbuf_write_done, db, |
3689 | 2114 |
ZIO_PRIORITY_ASYNC_WRITE, zio_flags, &zb); |
789 | 2115 |
} |
2116 |
||
2117 |
/* ARGSUSED */ |
|
2118 |
static void |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2119 |
dbuf_write_ready(zio_t *zio, arc_buf_t *buf, void *vdb) |
789 | 2120 |
{ |
2121 |
dmu_buf_impl_t *db = vdb; |
|
2122 |
dnode_t *dn = db->db_dnode; |
|
2123 |
objset_impl_t *os = dn->dn_objset; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2124 |
blkptr_t *bp_orig = &zio->io_bp_orig; |
789 | 2125 |
uint64_t fill = 0; |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2126 |
int old_size, new_size, i; |
789 | 2127 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2128 |
dprintf_dbuf_bp(db, bp_orig, "bp_orig: %s", ""); |
789 | 2129 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2130 |
old_size = bp_get_dasize(os->os_spa, bp_orig); |
2082 | 2131 |
new_size = bp_get_dasize(os->os_spa, zio->io_bp); |
789 | 2132 |
|
2133 |
dnode_diduse_space(dn, new_size-old_size); |
|
2134 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2135 |
if (BP_IS_HOLE(zio->io_bp)) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2136 |
dsl_dataset_t *ds = os->os_dsl_dataset; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2137 |
dmu_tx_t *tx = os->os_synctx; |
789 | 2138 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2139 |
if (bp_orig->blk_birth == tx->tx_txg) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2140 |
dsl_dataset_block_kill(ds, bp_orig, NULL, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2141 |
ASSERT3U(db->db_blkptr->blk_fill, ==, 0); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2142 |
return; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2143 |
} |
2237 | 2144 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2145 |
mutex_enter(&db->db_mtx); |
789 | 2146 |
|
2147 |
if (db->db_level == 0) { |
|
2148 |
mutex_enter(&dn->dn_mtx); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2149 |
if (db->db_blkid > dn->dn_phys->dn_maxblkid) |
789 | 2150 |
dn->dn_phys->dn_maxblkid = db->db_blkid; |
2151 |
mutex_exit(&dn->dn_mtx); |
|
2152 |
||
2153 |
if (dn->dn_type == DMU_OT_DNODE) { |
|
2154 |
dnode_phys_t *dnp = db->db.db_data; |
|
2155 |
for (i = db->db.db_size >> DNODE_SHIFT; i > 0; |
|
2156 |
i--, dnp++) { |
|
2157 |
if (dnp->dn_type != DMU_OT_NONE) |
|
2158 |
fill++; |
|
2159 |
} |
|
2160 |
} else { |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2161 |
fill = 1; |
789 | 2162 |
} |
2163 |
} else { |
|
2164 |
blkptr_t *bp = db->db.db_data; |
|
2165 |
ASSERT3U(db->db.db_size, ==, 1<<dn->dn_phys->dn_indblkshift); |
|
2166 |
for (i = db->db.db_size >> SPA_BLKPTRSHIFT; i > 0; i--, bp++) { |
|
2167 |
if (BP_IS_HOLE(bp)) |
|
2168 |
continue; |
|
2169 |
ASSERT3U(BP_GET_LSIZE(bp), ==, |
|
2170 |
db->db_level == 1 ? dn->dn_datablksz : |
|
2171 |
(1<<dn->dn_phys->dn_indblkshift)); |
|
2172 |
fill += bp->blk_fill; |
|
2173 |
} |
|
2174 |
} |
|
2175 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2176 |
db->db_blkptr->blk_fill = fill; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2177 |
BP_SET_TYPE(db->db_blkptr, dn->dn_type); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2178 |
BP_SET_LEVEL(db->db_blkptr, db->db_level); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2179 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2180 |
mutex_exit(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2181 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2182 |
/* We must do this after we've set the bp's type and level */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2183 |
if (!DVA_EQUAL(BP_IDENTITY(zio->io_bp), BP_IDENTITY(bp_orig))) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2184 |
dsl_dataset_t *ds = os->os_dsl_dataset; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2185 |
dmu_tx_t *tx = os->os_synctx; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2186 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2187 |
if (bp_orig->blk_birth == tx->tx_txg) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2188 |
dsl_dataset_block_kill(ds, bp_orig, NULL, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2189 |
dsl_dataset_block_born(ds, zio->io_bp, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2190 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2191 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2192 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2193 |
/* ARGSUSED */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2194 |
static void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2195 |
dbuf_write_done(zio_t *zio, arc_buf_t *buf, void *vdb) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2196 |
{ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2197 |
dmu_buf_impl_t *db = vdb; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2198 |
uint64_t txg = zio->io_txg; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2199 |
dbuf_dirty_record_t **drp, *dr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2200 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2201 |
ASSERT3U(zio->io_error, ==, 0); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2202 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2203 |
mutex_enter(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2204 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2205 |
drp = &db->db_last_dirty; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2206 |
while (*drp != db->db_data_pending) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2207 |
drp = &(*drp)->dr_next; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2208 |
ASSERT(!list_link_active(&(*drp)->dr_dirty_node)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2209 |
ASSERT((*drp)->dr_txg == txg); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2210 |
ASSERT((*drp)->dr_next == NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2211 |
dr = *drp; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2212 |
*drp = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2213 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2214 |
if (db->db_level == 0) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2215 |
ASSERT(db->db_blkid != DB_BONUS_BLKID); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2216 |
ASSERT(dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2217 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2218 |
if (dr->dt.dl.dr_data != db->db_buf) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2219 |
VERIFY(arc_buf_remove_ref(dr->dt.dl.dr_data, db) == 1); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2220 |
else if (!BP_IS_HOLE(db->db_blkptr)) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2221 |
arc_set_callback(db->db_buf, dbuf_do_evict, db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2222 |
else |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2223 |
ASSERT(arc_released(db->db_buf)); |
789 | 2224 |
} else { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2225 |
dnode_t *dn = db->db_dnode; |
789 | 2226 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2227 |
ASSERT(list_head(&dr->dt.di.dr_children) == NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2228 |
ASSERT3U(db->db.db_size, ==, 1<<dn->dn_phys->dn_indblkshift); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2229 |
if (!BP_IS_HOLE(db->db_blkptr)) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2230 |
int epbs = |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2231 |
dn->dn_phys->dn_indblkshift - SPA_BLKPTRSHIFT; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2232 |
ASSERT3U(BP_GET_LSIZE(db->db_blkptr), ==, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2233 |
db->db.db_size); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2234 |
ASSERT3U(dn->dn_phys->dn_maxblkid |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2235 |
>> (db->db_level * epbs), >=, db->db_blkid); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2236 |
arc_set_callback(db->db_buf, dbuf_do_evict, db); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2237 |
} |
4831
41ec732c6d9f
6584470 zdb needs to initialize the bpl_lock mutex
gw25295
parents:
4506
diff
changeset
|
2238 |
mutex_destroy(&dr->dt.di.dr_mtx); |
41ec732c6d9f
6584470 zdb needs to initialize the bpl_lock mutex
gw25295
parents:
4506
diff
changeset
|
2239 |
list_destroy(&dr->dt.di.dr_children); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2240 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2241 |
kmem_free(dr, sizeof (dbuf_dirty_record_t)); |
789 | 2242 |
|
2243 |
cv_broadcast(&db->db_changed); |
|
2244 |
ASSERT(db->db_dirtycnt > 0); |
|
2245 |
db->db_dirtycnt -= 1; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2246 |
db->db_data_pending = NULL; |
789 | 2247 |
mutex_exit(&db->db_mtx); |
2248 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3290
diff
changeset
|
2249 |
dprintf_dbuf_bp(db, zio->io_bp, "bp: %s", ""); |
789 | 2250 |
|
1544 | 2251 |
dbuf_rele(db, (void *)(uintptr_t)txg); |
789 | 2252 |
} |