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_info(fs_info);
+       set_fs_error(fs_info);

The helper is too trivial, can you please drop it and replace by the
it's code here

Dave,

In fact I was thinking that too, but comments below stop me doing that,
potentially this function might provision persistent FS state.

----
        /*
         * today we only save the error info into ram.  Long term we'll
         * also send it down to the disk
         */
-----

Thanks, Anand


        if (sb->s_flags & MS_BORN)
                btrfs_handle_error(fs_info);
  }
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to