Panic in bookmark deletion
|Assignee:||Serapheim Dimitropoulos||% Done:|
|Category:||zfs - Zettabyte File System|
The problem is that when dsl_bookmark_destroy_check() is executed from open context (the pre-check), it fills in dbda_success based on the existence of the bookmark.
But the bookmark (or containing filesystem as in this case) can be destroyed before we get to syncing context. When we re-run dsl_bookmark_destroy_check() in syncing
context, it will not add the deleted bookmark to dbda_success, intending for dsl_bookmark_destroy_sync() to not process it. But because the bookmark is still in dbda_success
from the open-context call, we do try to destroy it.
The fix is that dsl_bookmark_destroy_check() should not modify dbda_success when called from open context.
#1 Updated by Electric Monk about 2 months ago
- % Done changed from 0 to 100
- Status changed from New to Closed
commit 42418f9e73f0d007aa87675ecc206c26fc8e073e Author: Matthew Ahrens <email@example.com> Date: 2017-06-29T23:10:42.000Z 8377 Panic in bookmark deletion Reviewed by: Paul Dagnelie <firstname.lastname@example.org> Reviewed by: Pavel Zakharov <email@example.com> Reviewed by: George Wilson <firstname.lastname@example.org> Approved by: Robert Mustacchi <email@example.com>
Also available in: Atom