Re: [PATCH] btrfs: rename save_error_info to set_fs_error

2016-03-19 Thread Anand Jain
On 03/16/2016 06:05 PM, David Sterba wrote: On Wed, Mar 16, 2016 at 04:43:05PM +0800, Anand Jain wrote: @@ -171,7 +171,7 @@ void __btrfs_std_error(struct btrfs_fs_info *fs_info, const char *function, #endif /* Don't go through full error handling during mount */ - save_error

Re: [PATCH] btrfs: rename save_error_info to set_fs_error

2016-03-19 Thread Anand Jain
On 03/16/2016 11:49 PM, David Sterba wrote: On Wed, Mar 16, 2016 at 09:42:11PM +0800, Anand Jain wrote: On 03/16/2016 06:05 PM, David Sterba wrote: On Wed, Mar 16, 2016 at 04:43:05PM +0800, Anand Jain wrote: @@ -171,7 +171,7 @@ void __btrfs_std_error(struct btrfs_fs_info *fs_info, const cha

Re: [PATCH] btrfs: rename save_error_info to set_fs_error

2016-03-19 Thread David Sterba
On Wed, Mar 16, 2016 at 09:42:11PM +0800, Anand Jain wrote: > On 03/16/2016 06:05 PM, David Sterba wrote: > > On Wed, Mar 16, 2016 at 04:43:05PM +0800, Anand Jain wrote: > >> @@ -171,7 +171,7 @@ void __btrfs_std_error(struct btrfs_fs_info *fs_info, > >> const char *function, > >> #endif > >> > >

Re: [PATCH] btrfs: rename save_error_info to set_fs_error

2016-03-16 Thread David Sterba
On Wed, Mar 16, 2016 at 04:43:05PM +0800, Anand Jain wrote: > @@ -171,7 +171,7 @@ void __btrfs_std_error(struct btrfs_fs_info *fs_info, > const char *function, > #endif > > /* Don't go through full error handling during mount */ > - save_error_info(fs_info); > + set_fs_error(fs_in

[PATCH] btrfs: rename save_error_info to set_fs_error

2016-03-16 Thread Anand Jain
Actually save_error_info() is setting the FS state to error. So rename is more appropriate. Signed-off-by: Anand Jain --- fs/btrfs/super.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 63a74303da9e..a11823b317ac 100644 --- a/fs/