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

Linux 4.6 compat: PAGE_CACHE_SIZE removal #4489

Closed
wants to merge 1 commit into from

Conversation

behlendorf
Copy link
Contributor

As described in torvalds/linux@4a2d057e the macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were originally introduced
to make it possible to add bigger chunks to the page cache. This
never panned out and it has therefore been removed from the kernel.

ZFS has been updated to use the PAGE_{SIZE,SHIFT,MASK,ALIGN} macros
and calls to page_cache_release() have been replaced with put_page().
There was no need to introduce an configure check for this because
these interfaces have existed for a very long time.

Signed-off-by: Brian Behlendorf [email protected]

As described in torvalds/linux/commit/4a2d057e the macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were originally introduced
to make it possible to add bigger chunks to the page cache.  This
never panned out and it has therefore been removed from the kernel.

ZFS has been updated to use the PAGE_{SIZE,SHIFT,MASK,ALIGN} macros
and calls to page_cache_release() have been replaced with put_page().
There was no need to introduce an configure check for this because
these interfaces have existed for a very long time.

Signed-off-by: Brian Behlendorf <[email protected]>
@tuxoko
Copy link
Contributor

tuxoko commented Apr 5, 2016

Oops, you beat me...

@behlendorf
Copy link
Contributor Author

@tuxoko only by a few minutes! It looks like our patches are identical which should make it easy for you to review this. :)

@tuxoko
Copy link
Contributor

tuxoko commented Apr 5, 2016

@behlendorf
LGTM. I just ran sed on those file, so mine has an unaligned comment.

@behlendorf behlendorf closed this in 8b1899d Apr 6, 2016
@behlendorf behlendorf added this to the 0.6.5.7 milestone Apr 29, 2016
nedbass pushed a commit to nedbass/zfs that referenced this pull request May 6, 2016
As described in torvalds/linux@4a2d057e the macros
PAGE_CACHE_{SIZE,SHIFT,MASK,ALIGN} macros were originally introduced
to make it possible to add bigger chunks to the page cache.  This
never panned out and it has therefore been removed from the kernel.

ZFS has been updated to use the PAGE_{SIZE,SHIFT,MASK,ALIGN} macros
and calls to page_cache_release() have been replaced with put_page().

There was no need to introduce a configure check for this because
these interfaces have existed for a very long time.

Signed-off-by: Brian Behlendorf <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Closes openzfs#4489
@behlendorf behlendorf deleted the linux-4.6-compat branch May 1, 2017 20:52
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