| Message ID | 20240108172040.178173-1-agruenba@redhat.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | fs: Wrong function name in comment | expand |
On Mon, Jan 08, 2024 at 06:20:40PM +0100, Andreas Gruenbacher wrote: > This comment refers to function mark_buffer_inode_dirty(), but the > function is actually called mark_buffer_dirty_inode(), so fix the > comment. > > Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com> Nobody noticed since 2002 (983b8a3c80b73 in mpe's fullhistory tree) ;-) Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
On Mon, 08 Jan 2024 18:20:40 +0100, Andreas Gruenbacher wrote: > This comment refers to function mark_buffer_inode_dirty(), but the > function is actually called mark_buffer_dirty_inode(), so fix the > comment. > > Applied to the vfs.misc branch of the vfs/vfs.git tree. Patches in the vfs.misc branch should appear in linux-next soon. Please report any outstanding bugs that were missed during review in a new review to the original patch series allowing us to drop it. It's encouraged to provide Acked-bys and Reviewed-bys even though the patch has now been applied. If possible patch trailers will be updated. Note that commit hashes shown below are subject to change due to rebase, trailer updates or similar. If in doubt, please check the listed branch. tree: https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git branch: vfs.misc [1/1] fs: Wrong function name in comment https://git.kernel.org/vfs/vfs/c/a9edccf32826
diff --git a/fs/buffer.c b/fs/buffer.c index 967f34b70aa8..6bdca819033f 100644 --- a/fs/buffer.c +++ b/fs/buffer.c @@ -465,7 +465,7 @@ EXPORT_SYMBOL(mark_buffer_async_write); * a successful fsync(). For example, ext2 indirect blocks need to be * written back and waited upon before fsync() returns. * - * The functions mark_buffer_inode_dirty(), fsync_inode_buffers(), + * The functions mark_buffer_dirty_inode(), fsync_inode_buffers(), * inode_has_buffers() and invalidate_inode_buffers() are provided for the * management of a list of dependent buffers at ->i_mapping->private_list. *
This comment refers to function mark_buffer_inode_dirty(), but the function is actually called mark_buffer_dirty_inode(), so fix the comment. Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com> --- fs/buffer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)