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 rollback of mounted filesystems as broken by Illumos #3875 patch. #1851

Closed
wants to merge 1 commit into from

Conversation

dweeezil
Copy link
Contributor

The Illumos #3875 patch reverted a part of ZoL's 7b3e34b which added
special-case error handling for zfs_rezget(). The error handling dealt
with the case in which an all-ones object number ended up being passed
to dnode_hold() and causing an EINVAL to be returned from zfs_rezget().

…patch.

The Illumos openzfs#3875 patch reverted a part of ZoL's 7b3e34b which added
special-case error handling for zfs_rezget().  The error handling dealt
with the case  in which an all-ones object number ended up being passed
to dnode_hold() and causing an EINVAL to be returned from zfs_rezget().
@dweeezil
Copy link
Contributor Author

This problem also affects "on-line" receive operations because they also require a remount.

@behlendorf
Copy link
Contributor

Nice catch.

@behlendorf
Copy link
Contributor

Merged as:

2008e92 Fix rollback of mounted filesystem regression

@behlendorf behlendorf closed this Nov 15, 2013
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.

2 participants