Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix destroy snapshots and race on zfs diff #562

Closed
wants to merge 2 commits into from

Conversation

b333z
Copy link
Contributor

@b333z b333z commented Feb 6, 2012

Fixes "dataset not found" error on zfs destory see #173.
Fixes race in dsl_dataset_user_release_tmp() when the temp snapshot from zfs diff dataset@snap command is used see #481.
Fixes a snapshot name problem zfs_umount_snap().

Fixed comment at top of zfs_umount_snap()
Corrected construction of fullname.
Fixes "dataset not found" error on zfs destory <snapshot> see openzfs#173.
Fixes race in dsl_dataset_user_release_tmp() when the temp snapshot
from zfs diff dataset@snap command is used see openzfs#481.
@FransUrbo
Copy link
Contributor

Works like a charm, thanx!

@behlendorf
Copy link
Contributor

This fix was merged in to the refreshed .zfs snapshot patch in behlendorf/zfs@f34cd0a .

@behlendorf behlendorf closed this Feb 9, 2012
@b333z
Copy link
Contributor Author

b333z commented Feb 10, 2012

Thanks Brian, checked out your branch and all looks good.

behlendorf pushed a commit to behlendorf/zfs that referenced this pull request May 21, 2018
Signed-off-by: Tom Caputi <tcaputi@datto.com>
Signed-off-by: Tony Hutter <hutter2@llnl.gov>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes openzfs#562
pcd1193182 pushed a commit to pcd1193182/zfs that referenced this pull request Sep 26, 2023
…aster

Merge remote-tracking branch '6.0/stage' into 'master'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants