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

Prevent reclaim in send_traverse_thread() #4998

Closed
wants to merge 1 commit into from

Conversation

dweeezil
Copy link
Contributor

As is the case with traverse_prefetch_thread(), the deep stacks caused
by traversal require disabling reclaim in the send traverse thread.

Fixes: #4912

@behlendorf
Copy link
Contributor

Thanks for posting the patch, I happened to observed the same issue in the receive_writer_thread(). Could you extend the patch to cover that case as well.

As is the case with traverse_prefetch_thread(), the deep stacks caused
by traversal require disabling reclaim in the send traverse thread.

Also, do the same for receive_writer_thread() in which similar problems
have been observed.

Fixes: openzfs#4912
@behlendorf
Copy link
Contributor

LGTM. Thanks!

DeHackEd pushed a commit to DeHackEd/zfs that referenced this pull request Oct 19, 2016
As is the case with traverse_prefetch_thread(), the deep stacks caused
by traversal require disabling reclaim in the send traverse thread.

Also, do the same for receive_writer_thread() in which similar problems
have been observed.

Signed-off-by: Tim Chase <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#4912
Closes openzfs#4998
DeHackEd pushed a commit to DeHackEd/zfs that referenced this pull request Oct 29, 2016
As is the case with traverse_prefetch_thread(), the deep stacks caused
by traversal require disabling reclaim in the send traverse thread.

Also, do the same for receive_writer_thread() in which similar problems
have been observed.

Signed-off-by: Tim Chase <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#4912
Closes openzfs#4998
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