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

Ensure dsl_destroy_head() decrypts objsets

Browse Source 
This patch corrects a small issue where the dsl_destroy_head()
code that runs when the async_destroy feature is disabled would
not properly decrypt the dataset before beginning processing.
If the dataset is not able to be decrypted, the optimization
code now simply does not run and the dataset is completely
destroyed in the DSL sync task.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Tom Caputi <tcaputi@datto.com>
Closes #9021
This commit is contained in:
Tom Caputi 2019-07-15 16:08:42 -07:00 committed by Tony Hutter
parent 0a223246e1
commit 78831d4290
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
module/zfs/dsl_destroy.c
View File

@ -1059,9 +1059,10 @@ dsl_destroy_head(const char *name)
/* /*
* Head deletion is processed in one txg on old pools; * Head deletion is processed in one txg on old pools;
* remove the objects from open context so that the txg sync * remove the objects from open context so that the txg sync
* is not too long. * is not too long. This optimization can only work for
* encrypted datasets if the wrapping key is loaded.
*/ */
error = dmu_objset_own(name, DMU_OST_ANY, B_FALSE, B_FALSE, error = dmu_objset_own(name, DMU_OST_ANY, B_FALSE, B_TRUE,
FTAG, &os); FTAG, &os);
if (error == 0) { if (error == 0) {
uint64_t prev_snap_txg = uint64_t prev_snap_txg =
@ -1073,7 +1074,7 @@ dsl_destroy_head(const char *name)
(void) dmu_free_long_object(os, obj); (void) dmu_free_long_object(os, obj);
/* sync out all frees */ /* sync out all frees */
txg_wait_synced(dmu_objset_pool(os), 0); txg_wait_synced(dmu_objset_pool(os), 0);
dmu_objset_disown(os, B_FALSE, FTAG); dmu_objset_disown(os, B_TRUE, FTAG);
} }
} }