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

Support 8KB pages #565

Closed
ryao opened this issue Feb 8, 2012 · 1 comment
Closed

Support 8KB pages #565

ryao opened this issue Feb 8, 2012 · 1 comment
Labels
Type: Feature Feature request or new feature
Milestone

Comments

@ryao
Copy link
Contributor

ryao commented Feb 8, 2012

The newest flash uses 8KB pages:

http://www.anandtech.com/show/4256/the-ocz-vertex-3-review-120gb/3

It might be a good idea to rework ca5fd24 to permit 8KB pages.

@behlendorf
Copy link
Contributor

Interesting, yeah we could probably safely bump the limit up for 8KB pages. However, I'm not sure we want to go much past that since then we'll only be keeping the last 16 uberblocks we doesn't allow for much rollback.

behlendorf pushed a commit to behlendorf/zfs that referenced this issue May 21, 2018
Signed-off-by: Nikolay Borisov <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Issue openzfs#565
behlendorf pushed a commit to behlendorf/zfs that referenced this issue May 21, 2018
Kernel 4.7 added the option to trim the unused exported symbols. In
my testing this showed to be problematic since the PDE_DATA function
was considered unused and as such was trimmed. This in turn caused the
respective test during spl's configure stage to falsely detect that
PDE_DATA is not defined, which in turn caused build failures later.

Handle this situation by adding detection whether CONFIG_TRIM_UNUSED_KSYMS
is enabled and refuse to build against a kernel which has it enabled

Signed-off-by: Nikolay Borisov <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#565
pcd1193182 pushed 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
Type: Feature Feature request or new feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants