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

Revert ce90208cf9e04df966429f115d8831371ea9afce #710

Closed
ryao opened this issue Apr 30, 2012 · 2 comments
Closed

Revert ce90208cf9e04df966429f115d8831371ea9afce #710

ryao opened this issue Apr 30, 2012 · 2 comments
Milestone

Comments

@ryao
Copy link
Contributor

ryao commented Apr 30, 2012

I switched my server to Linux 2.6.32.59 and proceeded to recompile Gentoo inside a virtual machine on zvol. That led to the system entering a loop in which it output page allocation failures to the console with nothing available in ZONE_DMA.

Either I failed to stress the zvols enough with newer kernels or the allocation behavior changed sometime between 2.6.32.y and 3.3.y to prioritize page allocations ZONE_NORMAL when allocations from both zones are possible. Either way, this needs more attention and ce90208 should be reverted.

@behlendorf
Copy link
Contributor

Thanks for reporting this so quickly. I'll revert it from the tree right away and we can continue to iterate on this in a branch.

behlendorf added a commit to behlendorf/zfs that referenced this issue Apr 30, 2012
This reverts commit ce90208.  This
change was observed to cause problems when using a zvol to back a VM
under 2.6.32.59 kernels.  This issue was filed as openzfs#710.

Signed-off-by: Richard Yao <ryao@cs.stonybrook.edu>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Issue openzfs#342
Issue openzfs#710
@ryao
Copy link
Contributor Author

ryao commented May 17, 2012

I am having such good results with the patches in pull request #726 that I think discussion of this can move there.

@ryao ryao closed this as completed May 17, 2012
pcd1193182 added a commit to pcd1193182/zfs that referenced this issue Sep 26, 2023
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

No branches or pull requests

2 participants