author | Joshua M. Clulow <jmc@joyent.com> |
Mon, 04 Mar 2013 23:52:56 +0000 | |
changeset 14188 | afe390b9f1e0 |
parent 14046 | 6a5a2f4a47db |
permissions | -rw-r--r-- |
789 | 1 |
/* |
2 |
* CDDL HEADER START |
|
3 |
* |
|
4 |
* The contents of this file are subject to the terms of the |
|
1544 | 5 |
* Common Development and Distribution License (the "License"). |
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 |
*/ |
|
13700
2889e2596bd6
2619 asynchronous destruction of ZFS file systems
Christopher Siden <chris.siden@delphix.com>
parents:
12684
diff
changeset
|
21 |
|
789 | 22 |
/* |
12178
93ec1371f750
6939983 assertion failed: zap_count(os, DMU_USERUSED_OBJECT, &count) != 0||count == 0, in dsl_dataset.c
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11935
diff
changeset
|
23 |
* Copyright (c) 2005, 2010, Oracle and/or its affiliates. All rights reserved. |
13700
2889e2596bd6
2619 asynchronous destruction of ZFS file systems
Christopher Siden <chris.siden@delphix.com>
parents:
12684
diff
changeset
|
24 |
* Copyright (c) 2012 by Delphix. All rights reserved. |
789 | 25 |
*/ |
26 |
||
27 |
#include <sys/zfs_context.h> |
|
28 |
#include <sys/dbuf.h> |
|
29 |
#include <sys/dnode.h> |
|
30 |
#include <sys/dmu.h> |
|
31 |
#include <sys/dmu_tx.h> |
|
32 |
#include <sys/dmu_objset.h> |
|
33 |
#include <sys/dsl_dataset.h> |
|
34 |
#include <sys/spa.h> |
|
35 |
||
36 |
static void |
|
37 |
dnode_increase_indirection(dnode_t *dn, dmu_tx_t *tx) |
|
38 |
{ |
|
39 |
dmu_buf_impl_t *db; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
40 |
int txgoff = tx->tx_txg & TXG_MASK; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
41 |
int nblkptr = dn->dn_phys->dn_nblkptr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
42 |
int old_toplvl = dn->dn_phys->dn_nlevels - 1; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
43 |
int new_level = dn->dn_next_nlevels[txgoff]; |
789 | 44 |
int i; |
45 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
46 |
rw_enter(&dn->dn_struct_rwlock, RW_WRITER); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
47 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
48 |
/* this dnode can't be paged out because it's dirty */ |
789 | 49 |
ASSERT(dn->dn_phys->dn_type != DMU_OT_NONE); |
50 |
ASSERT(RW_WRITE_HELD(&dn->dn_struct_rwlock)); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
51 |
ASSERT(new_level > 1 && dn->dn_phys->dn_nlevels > 0); |
789 | 52 |
|
53 |
db = dbuf_hold_level(dn, dn->dn_phys->dn_nlevels, 0, FTAG); |
|
1544 | 54 |
ASSERT(db != NULL); |
789 | 55 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
56 |
dn->dn_phys->dn_nlevels = new_level; |
4312
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
3547
diff
changeset
|
57 |
dprintf("os=%p obj=%llu, increase to %d\n", dn->dn_objset, |
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
3547
diff
changeset
|
58 |
dn->dn_object, dn->dn_phys->dn_nlevels); |
789 | 59 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
60 |
/* check for existing blkptrs in the dnode */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
61 |
for (i = 0; i < nblkptr; i++) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
62 |
if (!BP_IS_HOLE(&dn->dn_phys->dn_blkptr[i])) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
63 |
break; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
64 |
if (i != nblkptr) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
65 |
/* transfer dnode's block pointers to new indirect block */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
66 |
(void) dbuf_read(db, NULL, DB_RF_MUST_SUCCEED|DB_RF_HAVESTRUCT); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
67 |
ASSERT(db->db.db_data); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
68 |
ASSERT(arc_released(db->db_buf)); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
69 |
ASSERT3U(sizeof (blkptr_t) * nblkptr, <=, db->db.db_size); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
70 |
bcopy(dn->dn_phys->dn_blkptr, db->db.db_data, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
71 |
sizeof (blkptr_t) * nblkptr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
72 |
arc_buf_freeze(db->db_buf); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
73 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
74 |
|
789 | 75 |
/* set dbuf's parent pointers to new indirect buf */ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
76 |
for (i = 0; i < nblkptr; i++) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
77 |
dmu_buf_impl_t *child = dbuf_find(dn, old_toplvl, i); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
78 |
|
789 | 79 |
if (child == NULL) |
80 |
continue; |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
81 |
#ifdef DEBUG |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
82 |
DB_DNODE_ENTER(child); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
83 |
ASSERT3P(DB_DNODE(child), ==, dn); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
84 |
DB_DNODE_EXIT(child); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
85 |
#endif /* DEBUG */ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
86 |
if (child->db_parent && child->db_parent != dn->dn_dbuf) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
87 |
ASSERT(child->db_parent->db_level == db->db_level); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
88 |
ASSERT(child->db_blkptr != |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
89 |
&dn->dn_phys->dn_blkptr[child->db_blkid]); |
789 | 90 |
mutex_exit(&child->db_mtx); |
91 |
continue; |
|
92 |
} |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
93 |
ASSERT(child->db_parent == NULL || |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
94 |
child->db_parent == dn->dn_dbuf); |
789 | 95 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
96 |
child->db_parent = db; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
97 |
dbuf_add_ref(db, child); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
98 |
if (db->db.db_data) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
99 |
child->db_blkptr = (blkptr_t *)db->db.db_data + i; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
100 |
else |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
101 |
child->db_blkptr = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
102 |
dprintf_dbuf_bp(child, child->db_blkptr, |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
103 |
"changed db_blkptr to new indirect %s", ""); |
789 | 104 |
|
105 |
mutex_exit(&child->db_mtx); |
|
106 |
} |
|
107 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
108 |
bzero(dn->dn_phys->dn_blkptr, sizeof (blkptr_t) * nblkptr); |
789 | 109 |
|
1544 | 110 |
dbuf_rele(db, FTAG); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
111 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
112 |
rw_exit(&dn->dn_struct_rwlock); |
789 | 113 |
} |
114 |
||
6992 | 115 |
static int |
789 | 116 |
free_blocks(dnode_t *dn, blkptr_t *bp, int num, dmu_tx_t *tx) |
117 |
{ |
|
6992 | 118 |
dsl_dataset_t *ds = dn->dn_objset->os_dsl_dataset; |
789 | 119 |
uint64_t bytesfreed = 0; |
6992 | 120 |
int i, blocks_freed = 0; |
789 | 121 |
|
6992 | 122 |
dprintf("ds=%p obj=%llx num=%d\n", ds, dn->dn_object, num); |
789 | 123 |
|
124 |
for (i = 0; i < num; i++, bp++) { |
|
125 |
if (BP_IS_HOLE(bp)) |
|
126 |
continue; |
|
127 |
||
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
9396
diff
changeset
|
128 |
bytesfreed += dsl_dataset_block_kill(ds, bp, tx, B_FALSE); |
2082 | 129 |
ASSERT3U(bytesfreed, <=, DN_USED_BYTES(dn->dn_phys)); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
130 |
bzero(bp, sizeof (blkptr_t)); |
6992 | 131 |
blocks_freed += 1; |
789 | 132 |
} |
133 |
dnode_diduse_space(dn, -bytesfreed); |
|
6992 | 134 |
return (blocks_freed); |
789 | 135 |
} |
136 |
||
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
137 |
#ifdef ZFS_DEBUG |
789 | 138 |
static void |
139 |
free_verify(dmu_buf_impl_t *db, uint64_t start, uint64_t end, dmu_tx_t *tx) |
|
140 |
{ |
|
141 |
int off, num; |
|
142 |
int i, err, epbs; |
|
143 |
uint64_t txg = tx->tx_txg; |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
144 |
dnode_t *dn; |
789 | 145 |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
146 |
DB_DNODE_ENTER(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
147 |
dn = DB_DNODE(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
148 |
epbs = dn->dn_phys->dn_indblkshift - SPA_BLKPTRSHIFT; |
789 | 149 |
off = start - (db->db_blkid * 1<<epbs); |
150 |
num = end - start + 1; |
|
151 |
||
152 |
ASSERT3U(off, >=, 0); |
|
153 |
ASSERT3U(num, >=, 0); |
|
154 |
ASSERT3U(db->db_level, >, 0); |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
155 |
ASSERT3U(db->db.db_size, ==, 1 << dn->dn_phys->dn_indblkshift); |
789 | 156 |
ASSERT3U(off+num, <=, db->db.db_size >> SPA_BLKPTRSHIFT); |
157 |
ASSERT(db->db_blkptr != NULL); |
|
158 |
||
159 |
for (i = off; i < off+num; i++) { |
|
160 |
uint64_t *buf; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
161 |
dmu_buf_impl_t *child; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
162 |
dbuf_dirty_record_t *dr; |
789 | 163 |
int j; |
164 |
||
165 |
ASSERT(db->db_level == 1); |
|
166 |
||
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
167 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
168 |
err = dbuf_hold_impl(dn, db->db_level-1, |
4312
4fef416ca8cb
6523336 panic dr->dt.dl.dr_override_state == DR_NOT_OVERRIDDEN, file: ../../ common/fs/zfs/dbuf.c line: 2195
gw25295
parents:
3547
diff
changeset
|
169 |
(db->db_blkid << epbs) + i, TRUE, FTAG, &child); |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
170 |
rw_exit(&dn->dn_struct_rwlock); |
789 | 171 |
if (err == ENOENT) |
172 |
continue; |
|
173 |
ASSERT(err == 0); |
|
174 |
ASSERT(child->db_level == 0); |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
175 |
dr = child->db_last_dirty; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
176 |
while (dr && dr->dr_txg > txg) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
177 |
dr = dr->dr_next; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
178 |
ASSERT(dr == NULL || dr->dr_txg == txg); |
789 | 179 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
180 |
/* data_old better be zeroed */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
181 |
if (dr) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
182 |
buf = dr->dt.dl.dr_data->b_data; |
789 | 183 |
for (j = 0; j < child->db.db_size >> 3; j++) { |
184 |
if (buf[j] != 0) { |
|
185 |
panic("freed data not zero: " |
|
186 |
"child=%p i=%d off=%d num=%d\n", |
|
7240
c4957ab6a78e
6698108 Sun Studio 12 finds many lint errors in ON source.
rh87107
parents:
7046
diff
changeset
|
187 |
(void *)child, i, off, num); |
789 | 188 |
} |
189 |
} |
|
190 |
} |
|
191 |
||
192 |
/* |
|
193 |
* db_data better be zeroed unless it's dirty in a |
|
194 |
* future txg. |
|
195 |
*/ |
|
196 |
mutex_enter(&child->db_mtx); |
|
197 |
buf = child->db.db_data; |
|
198 |
if (buf != NULL && child->db_state != DB_FILL && |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
199 |
child->db_last_dirty == NULL) { |
789 | 200 |
for (j = 0; j < child->db.db_size >> 3; j++) { |
201 |
if (buf[j] != 0) { |
|
202 |
panic("freed data not zero: " |
|
203 |
"child=%p i=%d off=%d num=%d\n", |
|
7240
c4957ab6a78e
6698108 Sun Studio 12 finds many lint errors in ON source.
rh87107
parents:
7046
diff
changeset
|
204 |
(void *)child, i, off, num); |
789 | 205 |
} |
206 |
} |
|
207 |
} |
|
208 |
mutex_exit(&child->db_mtx); |
|
209 |
||
1544 | 210 |
dbuf_rele(child, FTAG); |
789 | 211 |
} |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
212 |
DB_DNODE_EXIT(db); |
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
213 |
} |
789 | 214 |
#endif |
215 |
||
6992 | 216 |
#define ALL -1 |
217 |
||
789 | 218 |
static int |
219 |
free_children(dmu_buf_impl_t *db, uint64_t blkid, uint64_t nblks, int trunc, |
|
220 |
dmu_tx_t *tx) |
|
221 |
{ |
|
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
222 |
dnode_t *dn; |
789 | 223 |
blkptr_t *bp; |
224 |
dmu_buf_impl_t *subdb; |
|
225 |
uint64_t start, end, dbstart, dbend, i; |
|
226 |
int epbs, shift, err; |
|
227 |
int all = TRUE; |
|
6992 | 228 |
int blocks_freed = 0; |
789 | 229 |
|
6992 | 230 |
/* |
231 |
* There is a small possibility that this block will not be cached: |
|
232 |
* 1 - if level > 1 and there are no children with level <= 1 |
|
233 |
* 2 - if we didn't get a dirty hold (because this block had just |
|
234 |
* finished being written -- and so had no holds), and then this |
|
235 |
* block got evicted before we got here. |
|
236 |
*/ |
|
237 |
if (db->db_state != DB_CACHED) |
|
238 |
(void) dbuf_read(db, NULL, DB_RF_MUST_SUCCEED); |
|
239 |
||
12296
7cf402a7f374
6675946 'zpool status' should show the progress of resilvering for individual disk.
Lin Ling <Lin.Ling@Sun.COM>
parents:
12178
diff
changeset
|
240 |
dbuf_release_bp(db); |
789 | 241 |
bp = (blkptr_t *)db->db.db_data; |
242 |
||
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
243 |
DB_DNODE_ENTER(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
244 |
dn = DB_DNODE(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
245 |
epbs = dn->dn_phys->dn_indblkshift - SPA_BLKPTRSHIFT; |
789 | 246 |
shift = (db->db_level - 1) * epbs; |
247 |
dbstart = db->db_blkid << epbs; |
|
248 |
start = blkid >> shift; |
|
249 |
if (dbstart < start) { |
|
250 |
bp += start - dbstart; |
|
251 |
all = FALSE; |
|
252 |
} else { |
|
253 |
start = dbstart; |
|
254 |
} |
|
255 |
dbend = ((db->db_blkid + 1) << epbs) - 1; |
|
256 |
end = (blkid + nblks - 1) >> shift; |
|
257 |
if (dbend <= end) |
|
258 |
end = dbend; |
|
259 |
else if (all) |
|
260 |
all = trunc; |
|
261 |
ASSERT3U(start, <=, end); |
|
262 |
||
263 |
if (db->db_level == 1) { |
|
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
264 |
FREE_VERIFY(db, start, end, tx); |
6992 | 265 |
blocks_freed = free_blocks(dn, bp, end-start+1, tx); |
3093
71525e4187d5
6490569 verify arc bufs are not modified when they shouldn't be
ahrens
parents:
3025
diff
changeset
|
266 |
arc_buf_freeze(db->db_buf); |
6992 | 267 |
ASSERT(all || blocks_freed == 0 || db->db_last_dirty); |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
268 |
DB_DNODE_EXIT(db); |
6992 | 269 |
return (all ? ALL : blocks_freed); |
789 | 270 |
} |
271 |
||
272 |
for (i = start; i <= end; i++, bp++) { |
|
273 |
if (BP_IS_HOLE(bp)) |
|
274 |
continue; |
|
275 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
|
276 |
err = dbuf_hold_impl(dn, db->db_level-1, i, TRUE, FTAG, &subdb); |
|
13805
e3a9ae14a119
3006 VERIFY[S,U,P] and ASSERT[S,U,P] frequently check if first argument is zero
Madhav Suresh <madhav.suresh@delphix.com>
parents:
13765
diff
changeset
|
277 |
ASSERT0(err); |
789 | 278 |
rw_exit(&dn->dn_struct_rwlock); |
279 |
||
6992 | 280 |
if (free_children(subdb, blkid, nblks, trunc, tx) == ALL) { |
789 | 281 |
ASSERT3P(subdb->db_blkptr, ==, bp); |
6992 | 282 |
blocks_freed += free_blocks(dn, bp, 1, tx); |
1163
4ba797920cc2
6343073 buf[i] == 0 assertion failure when running zvol_pstress
maybee
parents:
873
diff
changeset
|
283 |
} else { |
4ba797920cc2
6343073 buf[i] == 0 assertion failure when running zvol_pstress
maybee
parents:
873
diff
changeset
|
284 |
all = FALSE; |
789 | 285 |
} |
1544 | 286 |
dbuf_rele(subdb, FTAG); |
789 | 287 |
} |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
288 |
DB_DNODE_EXIT(db); |
3093
71525e4187d5
6490569 verify arc bufs are not modified when they shouldn't be
ahrens
parents:
3025
diff
changeset
|
289 |
arc_buf_freeze(db->db_buf); |
789 | 290 |
#ifdef ZFS_DEBUG |
291 |
bp -= (end-start)+1; |
|
292 |
for (i = start; i <= end; i++, bp++) { |
|
293 |
if (i == start && blkid != 0) |
|
294 |
continue; |
|
295 |
else if (i == end && !trunc) |
|
296 |
continue; |
|
13805
e3a9ae14a119
3006 VERIFY[S,U,P] and ASSERT[S,U,P] frequently check if first argument is zero
Madhav Suresh <madhav.suresh@delphix.com>
parents:
13765
diff
changeset
|
297 |
ASSERT0(bp->blk_birth); |
789 | 298 |
} |
299 |
#endif |
|
6992 | 300 |
ASSERT(all || blocks_freed == 0 || db->db_last_dirty); |
301 |
return (all ? ALL : blocks_freed); |
|
789 | 302 |
} |
303 |
||
304 |
/* |
|
14046
6a5a2f4a47db
3742 zfs comments need cleaner, more consistent style
Will Andrews <will@firepipe.net>
parents:
13973
diff
changeset
|
305 |
* Traverse the indicated range of the provided file |
789 | 306 |
* and "free" all the blocks contained there. |
307 |
*/ |
|
308 |
static void |
|
309 |
dnode_sync_free_range(dnode_t *dn, uint64_t blkid, uint64_t nblks, dmu_tx_t *tx) |
|
310 |
{ |
|
311 |
blkptr_t *bp = dn->dn_phys->dn_blkptr; |
|
312 |
dmu_buf_impl_t *db; |
|
313 |
int trunc, start, end, shift, i, err; |
|
314 |
int dnlevel = dn->dn_phys->dn_nlevels; |
|
315 |
||
316 |
if (blkid > dn->dn_phys->dn_maxblkid) |
|
317 |
return; |
|
318 |
||
319 |
ASSERT(dn->dn_phys->dn_maxblkid < UINT64_MAX); |
|
320 |
trunc = blkid + nblks > dn->dn_phys->dn_maxblkid; |
|
321 |
if (trunc) |
|
322 |
nblks = dn->dn_phys->dn_maxblkid - blkid + 1; |
|
323 |
||
324 |
/* There are no indirect blocks in the object */ |
|
325 |
if (dnlevel == 1) { |
|
326 |
if (blkid >= dn->dn_phys->dn_nblkptr) { |
|
327 |
/* this range was never made persistent */ |
|
328 |
return; |
|
329 |
} |
|
330 |
ASSERT3U(blkid + nblks, <=, dn->dn_phys->dn_nblkptr); |
|
6992 | 331 |
(void) free_blocks(dn, bp + blkid, nblks, tx); |
789 | 332 |
if (trunc) { |
333 |
uint64_t off = (dn->dn_phys->dn_maxblkid + 1) * |
|
334 |
(dn->dn_phys->dn_datablkszsec << SPA_MINBLOCKSHIFT); |
|
335 |
dn->dn_phys->dn_maxblkid = (blkid ? blkid - 1 : 0); |
|
336 |
ASSERT(off < dn->dn_phys->dn_maxblkid || |
|
337 |
dn->dn_phys->dn_maxblkid == 0 || |
|
6992 | 338 |
dnode_next_offset(dn, 0, &off, 1, 1, 0) != 0); |
789 | 339 |
} |
340 |
return; |
|
341 |
} |
|
342 |
||
343 |
shift = (dnlevel - 1) * (dn->dn_phys->dn_indblkshift - SPA_BLKPTRSHIFT); |
|
344 |
start = blkid >> shift; |
|
345 |
ASSERT(start < dn->dn_phys->dn_nblkptr); |
|
346 |
end = (blkid + nblks - 1) >> shift; |
|
347 |
bp += start; |
|
348 |
for (i = start; i <= end; i++, bp++) { |
|
349 |
if (BP_IS_HOLE(bp)) |
|
350 |
continue; |
|
351 |
rw_enter(&dn->dn_struct_rwlock, RW_READER); |
|
352 |
err = dbuf_hold_impl(dn, dnlevel-1, i, TRUE, FTAG, &db); |
|
13805
e3a9ae14a119
3006 VERIFY[S,U,P] and ASSERT[S,U,P] frequently check if first argument is zero
Madhav Suresh <madhav.suresh@delphix.com>
parents:
13765
diff
changeset
|
353 |
ASSERT0(err); |
789 | 354 |
rw_exit(&dn->dn_struct_rwlock); |
355 |
||
6992 | 356 |
if (free_children(db, blkid, nblks, trunc, tx) == ALL) { |
789 | 357 |
ASSERT3P(db->db_blkptr, ==, bp); |
6992 | 358 |
(void) free_blocks(dn, bp, 1, tx); |
789 | 359 |
} |
1544 | 360 |
dbuf_rele(db, FTAG); |
789 | 361 |
} |
362 |
if (trunc) { |
|
363 |
uint64_t off = (dn->dn_phys->dn_maxblkid + 1) * |
|
364 |
(dn->dn_phys->dn_datablkszsec << SPA_MINBLOCKSHIFT); |
|
365 |
dn->dn_phys->dn_maxblkid = (blkid ? blkid - 1 : 0); |
|
366 |
ASSERT(off < dn->dn_phys->dn_maxblkid || |
|
367 |
dn->dn_phys->dn_maxblkid == 0 || |
|
6992 | 368 |
dnode_next_offset(dn, 0, &off, 1, 1, 0) != 0); |
789 | 369 |
} |
370 |
} |
|
371 |
||
1544 | 372 |
/* |
14046
6a5a2f4a47db
3742 zfs comments need cleaner, more consistent style
Will Andrews <will@firepipe.net>
parents:
13973
diff
changeset
|
373 |
* Try to kick all the dnode's dbufs out of the cache... |
1544 | 374 |
*/ |
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
375 |
void |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
376 |
dnode_evict_dbufs(dnode_t *dn) |
1544 | 377 |
{ |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
378 |
int progress; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
379 |
int pass = 0; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
380 |
|
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
381 |
do { |
1602
a0fdd5599c90
6397271 panic: mutex_enter: bad mutex, lp=f074ea3c owner=deadbee8 thread=d794fde0
maybee
parents:
1600
diff
changeset
|
382 |
dmu_buf_impl_t *db, marker; |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
383 |
int evicting = FALSE; |
1544 | 384 |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
385 |
progress = FALSE; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
386 |
mutex_enter(&dn->dn_dbufs_mtx); |
1602
a0fdd5599c90
6397271 panic: mutex_enter: bad mutex, lp=f074ea3c owner=deadbee8 thread=d794fde0
maybee
parents:
1600
diff
changeset
|
387 |
list_insert_tail(&dn->dn_dbufs, &marker); |
a0fdd5599c90
6397271 panic: mutex_enter: bad mutex, lp=f074ea3c owner=deadbee8 thread=d794fde0
maybee
parents:
1600
diff
changeset
|
388 |
db = list_head(&dn->dn_dbufs); |
a0fdd5599c90
6397271 panic: mutex_enter: bad mutex, lp=f074ea3c owner=deadbee8 thread=d794fde0
maybee
parents:
1600
diff
changeset
|
389 |
for (; db != ▮ db = list_head(&dn->dn_dbufs)) { |
a0fdd5599c90
6397271 panic: mutex_enter: bad mutex, lp=f074ea3c owner=deadbee8 thread=d794fde0
maybee
parents:
1600
diff
changeset
|
390 |
list_remove(&dn->dn_dbufs, db); |
a0fdd5599c90
6397271 panic: mutex_enter: bad mutex, lp=f074ea3c owner=deadbee8 thread=d794fde0
maybee
parents:
1600
diff
changeset
|
391 |
list_insert_tail(&dn->dn_dbufs, db); |
12684
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
392 |
#ifdef DEBUG |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
393 |
DB_DNODE_ENTER(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
394 |
ASSERT3P(DB_DNODE(db), ==, dn); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
395 |
DB_DNODE_EXIT(db); |
397e44ebb8a9
6710343 dnode cache should register a dnode_move() callback to limit fragmentation
Tom Erickson <Tom.Erickson@Sun.COM>
parents:
12432
diff
changeset
|
396 |
#endif /* DEBUG */ |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
397 |
|
1544 | 398 |
mutex_enter(&db->db_mtx); |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
399 |
if (db->db_state == DB_EVICTING) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
400 |
progress = TRUE; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
401 |
evicting = TRUE; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
402 |
mutex_exit(&db->db_mtx); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
403 |
} else if (refcount_is_zero(&db->db_holds)) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
404 |
progress = TRUE; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
405 |
dbuf_clear(db); /* exits db_mtx for us */ |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
406 |
} else { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
407 |
mutex_exit(&db->db_mtx); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
408 |
} |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
409 |
|
1544 | 410 |
} |
1602
a0fdd5599c90
6397271 panic: mutex_enter: bad mutex, lp=f074ea3c owner=deadbee8 thread=d794fde0
maybee
parents:
1600
diff
changeset
|
411 |
list_remove(&dn->dn_dbufs, &marker); |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
412 |
/* |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
413 |
* NB: we need to drop dn_dbufs_mtx between passes so |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
414 |
* that any DB_EVICTING dbufs can make progress. |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
415 |
* Ideally, we would have some cv we could wait on, but |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
416 |
* since we don't, just wait a bit to give the other |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
417 |
* thread a chance to run. |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
418 |
*/ |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
419 |
mutex_exit(&dn->dn_dbufs_mtx); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
420 |
if (evicting) |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
421 |
delay(1); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
422 |
pass++; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
423 |
ASSERT(pass < 100); /* sanity check */ |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
424 |
} while (progress); |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
425 |
|
1544 | 426 |
rw_enter(&dn->dn_struct_rwlock, RW_WRITER); |
427 |
if (dn->dn_bonus && refcount_is_zero(&dn->dn_bonus->db_holds)) { |
|
428 |
mutex_enter(&dn->dn_bonus->db_mtx); |
|
429 |
dbuf_evict(dn->dn_bonus); |
|
430 |
dn->dn_bonus = NULL; |
|
431 |
} |
|
432 |
rw_exit(&dn->dn_struct_rwlock); |
|
433 |
} |
|
434 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
435 |
static void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
436 |
dnode_undirty_dbufs(list_t *list) |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
437 |
{ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
438 |
dbuf_dirty_record_t *dr; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
439 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
440 |
while (dr = list_head(list)) { |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
441 |
dmu_buf_impl_t *db = dr->dr_dbuf; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
442 |
uint64_t txg = dr->dr_txg; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
443 |
|
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
9396
diff
changeset
|
444 |
if (db->db_level != 0) |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
9396
diff
changeset
|
445 |
dnode_undirty_dbufs(&dr->dt.di.dr_children); |
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
9396
diff
changeset
|
446 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
447 |
mutex_enter(&db->db_mtx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
448 |
/* XXX - use dbuf_undirty()? */ |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
449 |
list_remove(list, dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
450 |
ASSERT(db->db_last_dirty == dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
451 |
db->db_last_dirty = NULL; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
452 |
db->db_dirtycnt -= 1; |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
453 |
if (db->db_level == 0) { |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
454 |
ASSERT(db->db_blkid == DMU_BONUS_BLKID || |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
455 |
dr->dt.dl.dr_data == db->db_buf); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
456 |
dbuf_unoverride(dr); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
457 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
458 |
kmem_free(dr, sizeof (dbuf_dirty_record_t)); |
10922
e2081f502306
PSARC 2009/571 ZFS Deduplication Properties
Jeff Bonwick <Jeff.Bonwick@Sun.COM>
parents:
9396
diff
changeset
|
459 |
dbuf_rele_and_unlock(db, (void *)(uintptr_t)txg); |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
460 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
461 |
} |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
462 |
|
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
463 |
static void |
789 | 464 |
dnode_sync_free(dnode_t *dn, dmu_tx_t *tx) |
465 |
{ |
|
466 |
int txgoff = tx->tx_txg & TXG_MASK; |
|
467 |
||
468 |
ASSERT(dmu_tx_is_syncing(tx)); |
|
469 |
||
6992 | 470 |
/* |
471 |
* Our contents should have been freed in dnode_sync() by the |
|
472 |
* free range record inserted by the caller of dnode_free(). |
|
473 |
*/ |
|
13805
e3a9ae14a119
3006 VERIFY[S,U,P] and ASSERT[S,U,P] frequently check if first argument is zero
Madhav Suresh <madhav.suresh@delphix.com>
parents:
13765
diff
changeset
|
474 |
ASSERT0(DN_USED_BYTES(dn->dn_phys)); |
6992 | 475 |
ASSERT(BP_IS_HOLE(dn->dn_phys->dn_blkptr)); |
476 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
477 |
dnode_undirty_dbufs(&dn->dn_dirty_records[txgoff]); |
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
478 |
dnode_evict_dbufs(dn); |
1544 | 479 |
ASSERT3P(list_head(&dn->dn_dbufs), ==, NULL); |
13973
4972ab336f54
3464 zfs synctask code needs restructuring
Matthew Ahrens <mahrens@delphix.com>
parents:
13805
diff
changeset
|
480 |
ASSERT3P(dn->dn_bonus, ==, NULL); |
1544 | 481 |
|
482 |
/* |
|
483 |
* XXX - It would be nice to assert this, but we may still |
|
484 |
* have residual holds from async evictions from the arc... |
|
485 |
* |
|
3444
dc160a70a50d
6410433 'zpool status -v' would be more useful with filenames
ek110237
parents:
3093
diff
changeset
|
486 |
* zfs_obj_to_path() also depends on this being |
dc160a70a50d
6410433 'zpool status -v' would be more useful with filenames
ek110237
parents:
3093
diff
changeset
|
487 |
* commented out. |
dc160a70a50d
6410433 'zpool status -v' would be more useful with filenames
ek110237
parents:
3093
diff
changeset
|
488 |
* |
1544 | 489 |
* ASSERT3U(refcount_count(&dn->dn_holds), ==, 1); |
490 |
*/ |
|
789 | 491 |
|
492 |
/* Undirty next bits */ |
|
493 |
dn->dn_next_nlevels[txgoff] = 0; |
|
494 |
dn->dn_next_indblkshift[txgoff] = 0; |
|
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
495 |
dn->dn_next_blksz[txgoff] = 0; |
789 | 496 |
|
497 |
/* ASSERT(blkptrs are zero); */ |
|
498 |
ASSERT(dn->dn_phys->dn_type != DMU_OT_NONE); |
|
499 |
ASSERT(dn->dn_type != DMU_OT_NONE); |
|
500 |
||
501 |
ASSERT(dn->dn_free_txg > 0); |
|
502 |
if (dn->dn_allocated_txg != dn->dn_free_txg) |
|
503 |
dbuf_will_dirty(dn->dn_dbuf, tx); |
|
504 |
bzero(dn->dn_phys, sizeof (dnode_phys_t)); |
|
505 |
||
506 |
mutex_enter(&dn->dn_mtx); |
|
507 |
dn->dn_type = DMU_OT_NONE; |
|
508 |
dn->dn_maxblkid = 0; |
|
509 |
dn->dn_allocated_txg = 0; |
|
4480
0976678e58c5
6544140 assertion failed: err == 0 (0x11 == 0x0), file: ../../common/fs/zfs/zfs_znode.c, line: 555
gw25295
parents:
4312
diff
changeset
|
510 |
dn->dn_free_txg = 0; |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
511 |
dn->dn_have_spill = B_FALSE; |
789 | 512 |
mutex_exit(&dn->dn_mtx); |
513 |
||
1544 | 514 |
ASSERT(dn->dn_object != DMU_META_DNODE_OBJECT); |
789 | 515 |
|
516 |
dnode_rele(dn, (void *)(uintptr_t)tx->tx_txg); |
|
517 |
/* |
|
518 |
* Now that we've released our hold, the dnode may |
|
519 |
* be evicted, so we musn't access it. |
|
520 |
*/ |
|
521 |
} |
|
522 |
||
523 |
/* |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
524 |
* Write out the dnode's dirty buffers. |
789 | 525 |
*/ |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
526 |
void |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
527 |
dnode_sync(dnode_t *dn, dmu_tx_t *tx) |
789 | 528 |
{ |
529 |
free_range_t *rp; |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
530 |
dnode_phys_t *dnp = dn->dn_phys; |
789 | 531 |
int txgoff = tx->tx_txg & TXG_MASK; |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
532 |
list_t *list = &dn->dn_dirty_records[txgoff]; |
9396
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
533 |
static const dnode_phys_t zerodn = { 0 }; |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
534 |
boolean_t kill_spill = B_FALSE; |
789 | 535 |
|
536 |
ASSERT(dmu_tx_is_syncing(tx)); |
|
537 |
ASSERT(dnp->dn_type != DMU_OT_NONE || dn->dn_allocated_txg); |
|
9396
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
538 |
ASSERT(dnp->dn_type != DMU_OT_NONE || |
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
539 |
bcmp(dnp, &zerodn, DNODE_SIZE) == 0); |
873
adefbfa5f42d
6347448 non ZFS_DEBUG kernels shouldn't call empty verify functions
ek110237
parents:
789
diff
changeset
|
540 |
DNODE_VERIFY(dn); |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
541 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
542 |
ASSERT(dn->dn_dbuf == NULL || arc_released(dn->dn_dbuf->db_buf)); |
789 | 543 |
|
9396
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
544 |
if (dmu_objset_userused_enabled(dn->dn_objset) && |
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
545 |
!DMU_OBJECT_IS_SPECIAL(dn->dn_object)) { |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
546 |
mutex_enter(&dn->dn_mtx); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
547 |
dn->dn_oldused = DN_USED_BYTES(dn->dn_phys); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
548 |
dn->dn_oldflags = dn->dn_phys->dn_flags; |
9396
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
549 |
dn->dn_phys->dn_flags |= DNODE_FLAG_USERUSED_ACCOUNTED; |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
550 |
mutex_exit(&dn->dn_mtx); |
12178
93ec1371f750
6939983 assertion failed: zap_count(os, DMU_USERUSED_OBJECT, &count) != 0||count == 0, in dsl_dataset.c
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11935
diff
changeset
|
551 |
dmu_objset_userquota_get_ids(dn, B_FALSE, tx); |
9396
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
552 |
} else { |
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
553 |
/* Once we account for it, we should always account for it. */ |
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
554 |
ASSERT(!(dn->dn_phys->dn_flags & |
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
555 |
DNODE_FLAG_USERUSED_ACCOUNTED)); |
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
556 |
} |
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
557 |
|
789 | 558 |
mutex_enter(&dn->dn_mtx); |
559 |
if (dn->dn_allocated_txg == tx->tx_txg) { |
|
560 |
/* The dnode is newly allocated or reallocated */ |
|
561 |
if (dnp->dn_type == DMU_OT_NONE) { |
|
562 |
/* this is a first alloc, not a realloc */ |
|
563 |
dnp->dn_nlevels = 1; |
|
8644
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
564 |
dnp->dn_nblkptr = dn->dn_nblkptr; |
789 | 565 |
} |
566 |
||
567 |
dnp->dn_type = dn->dn_type; |
|
568 |
dnp->dn_bonustype = dn->dn_bonustype; |
|
569 |
dnp->dn_bonuslen = dn->dn_bonuslen; |
|
570 |
} |
|
571 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
572 |
ASSERT(dnp->dn_nlevels > 1 || |
1599 | 573 |
BP_IS_HOLE(&dnp->dn_blkptr[0]) || |
574 |
BP_GET_LSIZE(&dnp->dn_blkptr[0]) == |
|
575 |
dnp->dn_datablkszsec << SPA_MINBLOCKSHIFT); |
|
576 |
||
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
577 |
if (dn->dn_next_blksz[txgoff]) { |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
578 |
ASSERT(P2PHASE(dn->dn_next_blksz[txgoff], |
789 | 579 |
SPA_MINBLOCKSIZE) == 0); |
1599 | 580 |
ASSERT(BP_IS_HOLE(&dnp->dn_blkptr[0]) || |
6992 | 581 |
dn->dn_maxblkid == 0 || list_head(list) != NULL || |
12178
93ec1371f750
6939983 assertion failed: zap_count(os, DMU_USERUSED_OBJECT, &count) != 0||count == 0, in dsl_dataset.c
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
11935
diff
changeset
|
582 |
avl_last(&dn->dn_ranges[txgoff]) || |
1600
d1ef3e135d5a
6397264 zfs-s10-0311:assertion failed:((&dnp->dn_blkptr[0])->blk_birth == 0)
ahrens
parents:
1599
diff
changeset
|
583 |
dn->dn_next_blksz[txgoff] >> SPA_MINBLOCKSHIFT == |
d1ef3e135d5a
6397264 zfs-s10-0311:assertion failed:((&dnp->dn_blkptr[0])->blk_birth == 0)
ahrens
parents:
1599
diff
changeset
|
584 |
dnp->dn_datablkszsec); |
789 | 585 |
dnp->dn_datablkszsec = |
1596
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
586 |
dn->dn_next_blksz[txgoff] >> SPA_MINBLOCKSHIFT; |
2e2377ccbf85
6395371 ASSERT in dmu_tx_count_free: blkid + i < dn->dn_phys->dn_nblkptr
ahrens
parents:
1544
diff
changeset
|
587 |
dn->dn_next_blksz[txgoff] = 0; |
789 | 588 |
} |
589 |
||
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
590 |
if (dn->dn_next_bonuslen[txgoff]) { |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
591 |
if (dn->dn_next_bonuslen[txgoff] == DN_ZERO_BONUSLEN) |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
592 |
dnp->dn_bonuslen = 0; |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
593 |
else |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
594 |
dnp->dn_bonuslen = dn->dn_next_bonuslen[txgoff]; |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
595 |
ASSERT(dnp->dn_bonuslen <= DN_MAX_BONUSLEN); |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
596 |
dn->dn_next_bonuslen[txgoff] = 0; |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
597 |
} |
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
598 |
|
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
599 |
if (dn->dn_next_bonustype[txgoff]) { |
13700
2889e2596bd6
2619 asynchronous destruction of ZFS file systems
Christopher Siden <chris.siden@delphix.com>
parents:
12684
diff
changeset
|
600 |
ASSERT(DMU_OT_IS_VALID(dn->dn_next_bonustype[txgoff])); |
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
601 |
dnp->dn_bonustype = dn->dn_next_bonustype[txgoff]; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
602 |
dn->dn_next_bonustype[txgoff] = 0; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
603 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
604 |
|
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
605 |
/* |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
606 |
* We will either remove a spill block when a file is being removed |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
607 |
* or we have been asked to remove it. |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
608 |
*/ |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
609 |
if (dn->dn_rm_spillblk[txgoff] || |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
610 |
((dnp->dn_flags & DNODE_FLAG_SPILL_BLKPTR) && |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
611 |
dn->dn_free_txg > 0 && dn->dn_free_txg <= tx->tx_txg)) { |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
612 |
if ((dnp->dn_flags & DNODE_FLAG_SPILL_BLKPTR)) |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
613 |
kill_spill = B_TRUE; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
614 |
dn->dn_rm_spillblk[txgoff] = 0; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
615 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
616 |
|
789 | 617 |
if (dn->dn_next_indblkshift[txgoff]) { |
618 |
ASSERT(dnp->dn_nlevels == 1); |
|
619 |
dnp->dn_indblkshift = dn->dn_next_indblkshift[txgoff]; |
|
620 |
dn->dn_next_indblkshift[txgoff] = 0; |
|
621 |
} |
|
622 |
||
623 |
/* |
|
624 |
* Just take the live (open-context) values for checksum and compress. |
|
625 |
* Strictly speaking it's a future leak, but nothing bad happens if we |
|
626 |
* start using the new checksum or compress algorithm a little early. |
|
627 |
*/ |
|
628 |
dnp->dn_checksum = dn->dn_checksum; |
|
629 |
dnp->dn_compress = dn->dn_compress; |
|
630 |
||
631 |
mutex_exit(&dn->dn_mtx); |
|
632 |
||
11935
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
633 |
if (kill_spill) { |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
634 |
(void) free_blocks(dn, &dn->dn_phys->dn_spill, 1, tx); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
635 |
mutex_enter(&dn->dn_mtx); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
636 |
dnp->dn_flags &= ~DNODE_FLAG_SPILL_BLKPTR; |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
637 |
mutex_exit(&dn->dn_mtx); |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
638 |
} |
538c866aaac6
6716117 ZFS needs native system attribute infrastructure
Mark Shellenbaum <Mark.Shellenbaum@Sun.COM>
parents:
10922
diff
changeset
|
639 |
|
789 | 640 |
/* process all the "freed" ranges in the file */ |
6992 | 641 |
while (rp = avl_last(&dn->dn_ranges[txgoff])) { |
642 |
dnode_sync_free_range(dn, rp->fr_blkid, rp->fr_nblks, tx); |
|
643 |
/* grab the mutex so we don't race with dnode_block_freed() */ |
|
644 |
mutex_enter(&dn->dn_mtx); |
|
645 |
avl_remove(&dn->dn_ranges[txgoff], rp); |
|
646 |
mutex_exit(&dn->dn_mtx); |
|
647 |
kmem_free(rp, sizeof (free_range_t)); |
|
789 | 648 |
} |
4944
96d96f8de974
6569719 panic dangling dbufs (dn=ffffffff28814d30, dbuf=ffffffff20756008)
maybee
parents:
4787
diff
changeset
|
649 |
|
789 | 650 |
if (dn->dn_free_txg > 0 && dn->dn_free_txg <= tx->tx_txg) { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
651 |
dnode_sync_free(dn, tx); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
652 |
return; |
789 | 653 |
} |
654 |
||
8644
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
655 |
if (dn->dn_next_nblkptr[txgoff]) { |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
656 |
/* this should only happen on a realloc */ |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
657 |
ASSERT(dn->dn_allocated_txg == tx->tx_txg); |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
658 |
if (dn->dn_next_nblkptr[txgoff] > dnp->dn_nblkptr) { |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
659 |
/* zero the new blkptrs we are gaining */ |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
660 |
bzero(dnp->dn_blkptr + dnp->dn_nblkptr, |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
661 |
sizeof (blkptr_t) * |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
662 |
(dn->dn_next_nblkptr[txgoff] - dnp->dn_nblkptr)); |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
663 |
#ifdef ZFS_DEBUG |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
664 |
} else { |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
665 |
int i; |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
666 |
ASSERT(dn->dn_next_nblkptr[txgoff] < dnp->dn_nblkptr); |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
667 |
/* the blkptrs we are losing better be unallocated */ |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
668 |
for (i = dn->dn_next_nblkptr[txgoff]; |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
669 |
i < dnp->dn_nblkptr; i++) |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
670 |
ASSERT(BP_IS_HOLE(&dnp->dn_blkptr[i])); |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
671 |
#endif |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
672 |
} |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
673 |
mutex_enter(&dn->dn_mtx); |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
674 |
dnp->dn_nblkptr = dn->dn_next_nblkptr[txgoff]; |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
675 |
dn->dn_next_nblkptr[txgoff] = 0; |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
676 |
mutex_exit(&dn->dn_mtx); |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
677 |
} |
10534c530688
6793373 assertion failed in dnode_reallocate: dn_nblkptr >= old_nblkptr
Mark Maybee <Mark.Maybee@Sun.COM>
parents:
7240
diff
changeset
|
678 |
|
789 | 679 |
if (dn->dn_next_nlevels[txgoff]) { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
680 |
dnode_increase_indirection(dn, tx); |
789 | 681 |
dn->dn_next_nlevels[txgoff] = 0; |
682 |
} |
|
683 |
||
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
684 |
dbuf_sync_list(list, tx); |
789 | 685 |
|
9396
f41cf682d0d3
PSARC/2009/204 ZFS user/group quotas & space accounting
Matthew Ahrens <Matthew.Ahrens@Sun.COM>
parents:
8644
diff
changeset
|
686 |
if (!DMU_OBJECT_IS_SPECIAL(dn->dn_object)) { |
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
687 |
ASSERT3P(list_head(list), ==, NULL); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
688 |
dnode_rele(dn, (void *)(uintptr_t)tx->tx_txg); |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
689 |
} |
1599 | 690 |
|
3547
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
691 |
/* |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
692 |
* Although we have dropped our reference to the dnode, it |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
693 |
* can't be evicted until its written, and we haven't yet |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
694 |
* initiated the IO for the dnode's dbuf. |
e396e0a440b1
6512391 DMU should leverage ZIO dependencies to achieve greater parallelism
maybee
parents:
3444
diff
changeset
|
695 |
*/ |
789 | 696 |
} |