diff options
| author | Christoph Hellwig <hch@lst.de> | 2019-07-24 09:15:25 +0200 |
|---|---|---|
| committer | Richard Weinberger <richard@nod.at> | 2019-09-15 22:42:33 +0200 |
| commit | 61b875e88a04fd626e68c657400f237af8afa95e (patch) | |
| tree | 649e05ab656c518c00688ee5fbf4ebfeb49d11e1 /fs/jffs2/fs.c | |
| parent | jffs2: Fix possible null-pointer dereferences in jffs2_add_frag_to_fragtree() (diff) | |
| download | linux-61b875e88a04fd626e68c657400f237af8afa95e.tar.xz linux-61b875e88a04fd626e68c657400f237af8afa95e.zip | |
jffs2: Remove jffs2_gc_fetch_page and jffs2_gc_release_page
Merge these two helpers into the only callers to get rid of some
amazingly bad calling conventions.
Suggested-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Richard Weinberger <richard@nod.at>
Diffstat (limited to 'fs/jffs2/fs.c')
| -rw-r--r-- | fs/jffs2/fs.c | 27 |
1 files changed, 0 insertions, 27 deletions
diff --git a/fs/jffs2/fs.c b/fs/jffs2/fs.c index 8a20ddd25f2d..a3193c0f3a9b 100644 --- a/fs/jffs2/fs.c +++ b/fs/jffs2/fs.c @@ -678,33 +678,6 @@ struct jffs2_inode_info *jffs2_gc_fetch_inode(struct jffs2_sb_info *c, return JFFS2_INODE_INFO(inode); } -unsigned char *jffs2_gc_fetch_page(struct jffs2_sb_info *c, - struct jffs2_inode_info *f, - unsigned long offset, - unsigned long *priv) -{ - struct inode *inode = OFNI_EDONI_2SFFJ(f); - struct page *pg; - - pg = read_cache_page(inode->i_mapping, offset >> PAGE_SHIFT, - jffs2_do_readpage_unlock, inode); - if (IS_ERR(pg)) - return (void *)pg; - - *priv = (unsigned long)pg; - return kmap(pg); -} - -void jffs2_gc_release_page(struct jffs2_sb_info *c, - unsigned char *ptr, - unsigned long *priv) -{ - struct page *pg = (void *)*priv; - - kunmap(pg); - put_page(pg); -} - static int jffs2_flash_setup(struct jffs2_sb_info *c) { int ret = 0; |