The patch titled Subject: fs/fs-writeback.c: remove unneccesary parameter of __writeback_single_inode() has been added to the -mm tree. Its filename is fs-fs-writebackc-remove-unneccesary-parameter-of-__writeback_single_inode.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Yan Hong <clouds.yan@xxxxxxxxx> Subject: fs/fs-writeback.c: remove unneccesary parameter of __writeback_single_inode() The parameter 'wb' is never used in this function. Signed-off-by: Yan Hong <clouds.yan@xxxxxxxxx> Acked-by: Wu Fengguang <fengguang.wu@xxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/fs-writeback.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff -puN fs/fs-writeback.c~fs-fs-writebackc-remove-unneccesary-parameter-of-__writeback_single_inode fs/fs-writeback.c --- a/fs/fs-writeback.c~fs-fs-writebackc-remove-unneccesary-parameter-of-__writeback_single_inode +++ a/fs/fs-writeback.c @@ -439,8 +439,7 @@ static void requeue_inode(struct inode * * setting I_SYNC flag and calling inode_sync_complete() to clear it. */ static int -__writeback_single_inode(struct inode *inode, struct bdi_writeback *wb, - struct writeback_control *wbc) +__writeback_single_inode(struct inode *inode, struct writeback_control *wbc) { struct address_space *mapping = inode->i_mapping; long nr_to_write = wbc->nr_to_write; @@ -527,7 +526,7 @@ writeback_single_inode(struct inode *ino inode->i_state |= I_SYNC; spin_unlock(&inode->i_lock); - ret = __writeback_single_inode(inode, wb, wbc); + ret = __writeback_single_inode(inode, wbc); spin_lock(&wb->list_lock); spin_lock(&inode->i_lock); @@ -670,7 +669,7 @@ static long writeback_sb_inodes(struct s * We use I_SYNC to pin the inode in memory. While it is set * evict_inode() will wait so the inode cannot be freed. */ - __writeback_single_inode(inode, wb, &wbc); + __writeback_single_inode(inode, &wbc); work->nr_pages -= write_chunk - wbc.nr_to_write; wrote += write_chunk - wbc.nr_to_write; _ Patches currently in -mm which might be from clouds.yan@xxxxxxxxx are fs-debugsfs-remove-unnecessary-inode-i_private-initialization.patch fs-block_devc-need-not-to-check-inode-i_bdev-in-bd_forget.patch fs-fs-writebackc-remove-unneccesary-parameter-of-__writeback_single_inode.patch proc-return-enomem-when-inode-allocation-failed.patch proc-no-need-to-initialize-proc_inode-fd-in-proc_get_inode.patch proc-use-kzalloc-instead-of-kmalloc-and-memset.patch proc-use-kzalloc-instead-of-kmalloc-and-memset-fix.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html