diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2016-01-14 19:03:57 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-01-14 19:03:57 -0500 |
commit | 63f729cb4aa9a224cfd6bb35eab6b4556c29115d (patch) | |
tree | 4bfd85dc758d93d82d84a591cb9c61f4796f5cb8 /Documentation/filesystems | |
parent | 10a0c0f0595b20efa127a1816670c64a3d0e4965 (diff) | |
parent | e8ecde25f5e08f89b61d86c32bbb56b405e90c32 (diff) |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs
Pull vfs fix from Al Viro:
"Don't put symlink bodies in pagecache into highmem"
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs:
Make sure that highmem pages are not added to symlink page cache
Diffstat (limited to 'Documentation/filesystems')
-rw-r--r-- | Documentation/filesystems/porting | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/Documentation/filesystems/porting b/Documentation/filesystems/porting index 0f88e6020487..f1b87d8aa2da 100644 --- a/Documentation/filesystems/porting +++ b/Documentation/filesystems/porting | |||
@@ -508,7 +508,11 @@ in your dentry operations instead. | |||
508 | [mandatory] | 508 | [mandatory] |
509 | any symlink that might use page_follow_link_light/page_put_link() must | 509 | any symlink that might use page_follow_link_light/page_put_link() must |
510 | have inode_nohighmem(inode) called before anything might start playing with | 510 | have inode_nohighmem(inode) called before anything might start playing with |
511 | its pagecache. | 511 | its pagecache. No highmem pages should end up in the pagecache of such |
512 | symlinks. That includes any preseeding that might be done during symlink | ||
513 | creation. __page_symlink() will honour the mapping gfp flags, so once | ||
514 | you've done inode_nohighmem() it's safe to use, but if you allocate and | ||
515 | insert the page manually, make sure to use the right gfp flags. | ||
512 | -- | 516 | -- |
513 | [mandatory] | 517 | [mandatory] |
514 | ->follow_link() is replaced with ->get_link(); same API, except that | 518 | ->follow_link() is replaced with ->get_link(); same API, except that |