diff options
| author | Matthew Wilcox (Oracle) <willy@infradead.org> | 2025-06-24 14:21:27 +0100 |
|---|---|---|
| committer | Christian Brauner <brauner@kernel.org> | 2025-06-24 15:53:40 +0200 |
| commit | b39f7d75dc41b5f5d028192cd5d66cff71179f35 (patch) | |
| tree | 5b7f4f6e62ee183c49c8646b95108430aeb1e7ca /fs/buffer.c | |
| parent | 06a705356d7543b4eb2f41bef1ee0d5951700f7a (diff) | |
fs: Remove three arguments from block_write_end()
block_write_end() looks like it can be used as a ->write_end()
implementation. However, it can't as it does not unlock nor put
the folio. Since it does not use the 'file', 'mapping' nor 'fsdata'
arguments, remove them.
Signed-off-by: "Matthew Wilcox (Oracle)" <willy@infradead.org>
Link: https://lore.kernel.org/20250624132130.1590285-1-willy@infradead.org
Signed-off-by: Christian Brauner <brauner@kernel.org>
Diffstat (limited to 'fs/buffer.c')
| -rw-r--r-- | fs/buffer.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/fs/buffer.c b/fs/buffer.c index 1d34200f69c8..d61073143127 100644 --- a/fs/buffer.c +++ b/fs/buffer.c @@ -2270,9 +2270,8 @@ int block_write_begin(struct address_space *mapping, loff_t pos, unsigned len, } EXPORT_SYMBOL(block_write_begin); -int block_write_end(struct file *file, struct address_space *mapping, - loff_t pos, unsigned len, unsigned copied, - struct folio *folio, void *fsdata) +int block_write_end(loff_t pos, unsigned len, unsigned copied, + struct folio *folio) { size_t start = pos - folio_pos(folio); @@ -2311,7 +2310,7 @@ int generic_write_end(struct file *file, struct address_space *mapping, loff_t old_size = inode->i_size; bool i_size_changed = false; - copied = block_write_end(file, mapping, pos, len, copied, folio, fsdata); + copied = block_write_end(pos, len, copied, folio); /* * No need to use i_size_read() here, the i_size cannot change under us |