diff options
| author | Christoph Hellwig <hch@lst.de> | 2023-01-18 18:30:21 +0100 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2023-01-19 19:26:24 -0500 |
| commit | 35bb6a092cfcab895ea7e9c31a2cc6b80b25c134 (patch) | |
| tree | 1ae34dd0fa0e83f8763d916d5856e4f7e2ac9ff5 /fs/minix/dir.c | |
| parent | 4a29a1262a47acd8e81a3824492a5aa8f8c5fc95 (diff) | |
minix: move releasing pages into unlink and rename
Instead of consuming the page reference and kmap in the low-level
minix_delete_entry and minix_set_link helpers, do it in the callers
where that code can be shared with the error cleanup path.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/minix/dir.c')
| -rw-r--r-- | fs/minix/dir.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/minix/dir.c b/fs/minix/dir.c index dcfe5b25378b..ec462330e749 100644 --- a/fs/minix/dir.c +++ b/fs/minix/dir.c @@ -306,7 +306,6 @@ int minix_delete_entry(struct minix_dir_entry *de, struct page *page) } else { unlock_page(page); } - dir_put_page(page); inode->i_ctime = inode->i_mtime = current_time(inode); mark_inode_dirty(inode); return err; @@ -430,7 +429,6 @@ void minix_set_link(struct minix_dir_entry *de, struct page *page, } else { unlock_page(page); } - dir_put_page(page); dir->i_mtime = dir->i_ctime = current_time(dir); mark_inode_dirty(dir); } |