diff mbox

ext4: remove a duplicate call in ext4_init_new_dir()

Message ID 1407856709-18612-1-git-send-email-wangshilong1991@gmail.com
State New
Headers show

Commit Message

Wang Shilong Aug. 12, 2014, 3:18 p.m. UTC
From: Wang Shilong <wshilong@ddn.com>

ext4_journal_get_write_access() has just been called in ext4_append()
calling it again here is duplicated.

Signed-off-by: Wang Shilong <wshilong@ddn.com>
---
 fs/ext4/namei.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Wang Shilong Aug. 19, 2014, 5:21 p.m. UTC | #1
Any comments on this cleanup patch?

> From: Wang Shilong <wshilong@ddn.com>
> 
> ext4_journal_get_write_access() has just been called in ext4_append()
> calling it again here is duplicated.
> 
> Signed-off-by: Wang Shilong <wshilong@ddn.com>
> ---
> fs/ext4/namei.c | 4 ----
> 1 file changed, 4 deletions(-)
> 
> diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c
> index 3520ab8..6d0d01f 100644
> --- a/fs/ext4/namei.c
> +++ b/fs/ext4/namei.c
> @@ -2390,10 +2390,6 @@ static int ext4_init_new_dir(handle_t *handle, struct inode *dir,
> 	dir_block = ext4_append(handle, inode, &block);
> 	if (IS_ERR(dir_block))
> 		return PTR_ERR(dir_block);
> -	BUFFER_TRACE(dir_block, "get_write_access");
> -	err = ext4_journal_get_write_access(handle, dir_block);
> -	if (err)
> -		goto out;
> 	de = (struct ext4_dir_entry_2 *)dir_block->b_data;
> 	ext4_init_dot_dotdot(inode, de, blocksize, csum_size, dir->i_ino, 0);
> 	set_nlink(inode, 2);
> -- 
> 1.9.3
> 

Best Regards,
Wang Shilong

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Theodore Ts'o Aug. 22, 2014, 11:55 p.m. UTC | #2
On Wed, Aug 20, 2014 at 01:21:55AM +0800, Wang Shilong wrote:
> Any comments on this cleanup patch?

Looks good.  This got sent just before the merge window opened, and
this week has been the Kernel Summit / LinuxCon North America.  So I
haven't started accepting patches to the ext4 git tree just yet.  Now
that the event is open, we'll start with regression fixes and high
priority bug fixes that need to go to Linus before 3.17 is released,
and then cleanup and patch fixups.

Cheers,

						- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Wang Shilong Aug. 23, 2014, 3:48 a.m. UTC | #3
Hi Ted,

> On Wed, Aug 20, 2014 at 01:21:55AM +0800, Wang Shilong wrote:
>> Any comments on this cleanup patch?
> 
> Looks good.  This got sent just before the merge window opened, and
> this week has been the Kernel Summit / LinuxCon North America.  So I
> haven't started accepting patches to the ext4 git tree just yet.  Now
> that the event is open, we'll start with regression fixes and high
> priority bug fixes that need to go to Linus before 3.17 is released,
> and then cleanup and patch mixups.

Thanks for detailed information!

> 
> Cheers,
> 
> 						- Ted

Best Regards,
Wang Shilong

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Theodore Ts'o Aug. 30, 2014, 3:23 a.m. UTC | #4
On Tue, Aug 12, 2014 at 11:18:29PM +0800, Wang Shilong wrote:
> From: Wang Shilong <wshilong@ddn.com>
> 
> ext4_journal_get_write_access() has just been called in ext4_append()
> calling it again here is duplicated.
> 
> Signed-off-by: Wang Shilong <wshilong@ddn.com>

Thanks, applied.

					- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c
index 3520ab8..6d0d01f 100644
--- a/fs/ext4/namei.c
+++ b/fs/ext4/namei.c
@@ -2390,10 +2390,6 @@  static int ext4_init_new_dir(handle_t *handle, struct inode *dir,
 	dir_block = ext4_append(handle, inode, &block);
 	if (IS_ERR(dir_block))
 		return PTR_ERR(dir_block);
-	BUFFER_TRACE(dir_block, "get_write_access");
-	err = ext4_journal_get_write_access(handle, dir_block);
-	if (err)
-		goto out;
 	de = (struct ext4_dir_entry_2 *)dir_block->b_data;
 	ext4_init_dot_dotdot(inode, de, blocksize, csum_size, dir->i_ino, 0);
 	set_nlink(inode, 2);