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

Dummy for spl 577 #5240

Closed
wants to merge 1 commit into from
Closed

Dummy for spl 577 #5240

wants to merge 1 commit into from

Conversation

tuxoko
Copy link
Contributor

@tuxoko tuxoko commented Oct 7, 2016

Signed-off-by: Chunwei Chen [email protected]
Requires-spl: refs/pull/577/head

@behlendorf
Copy link
Contributor

behlendorf commented Oct 7, 2016

It looks like there's been quite a lot of churn in the kernel's rw sem implementation recently. Can you point me at the kernel commit where this changed.

You can rebase openzfs/spl#577 on the latest spl/master to resolve the aarch64 and kernel.org build failures.

Signed-off-by: Chunwei Chen <[email protected]>
Requires-spl: refs/pull/577/head
@tuxoko
Copy link
Contributor Author

tuxoko commented Oct 7, 2016

Rebased. I link the kernel commit in the commit message in openzfs/spl#577

Copy link
Contributor

@behlendorf behlendorf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh, I missed that. Great. I don't see any problems with this. LGTM pending the test results.

@tuxoko
Copy link
Contributor Author

tuxoko commented Oct 7, 2016

You didn't miss that. I added that after your comment. 😃

@behlendorf behlendorf mentioned this pull request Oct 7, 2016
@behlendorf
Copy link
Contributor

@tuxoko that makes me feel a little better!

With this patch applied I was able to run all of the ZFS Test Suite with a mainline 4.8 kernel.

@behlendorf behlendorf closed this Oct 8, 2016
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