btrfs: rename errno identifiers to error
authorDavid Sterba <dsterba@suse.com>
Fri, 8 Sep 2023 19:05:47 +0000 (21:05 +0200)
committerDavid Sterba <dsterba@suse.com>
Thu, 12 Oct 2023 14:44:07 +0000 (16:44 +0200)
We sync the kernel files to userspace and the 'errno' symbol is defined
by standard library, which does not matter in kernel but the parameters
or local variables could clash. Rename them all.

Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/backref.h
fs/btrfs/compression.c
fs/btrfs/ctree.c
fs/btrfs/messages.c
fs/btrfs/messages.h
fs/btrfs/transaction.c
fs/btrfs/transaction.h

index 79742935399f4f0612fced5b775e042b301cb757..3b077d10bbc0d5f75701c70a9a2d41b8f714856d 100644 (file)
@@ -533,9 +533,9 @@ void btrfs_backref_cleanup_node(struct btrfs_backref_cache *cache,
 void btrfs_backref_release_cache(struct btrfs_backref_cache *cache);
 
 static inline void btrfs_backref_panic(struct btrfs_fs_info *fs_info,
-                                      u64 bytenr, int errno)
+                                      u64 bytenr, int error)
 {
-       btrfs_panic(fs_info, errno,
+       btrfs_panic(fs_info, error,
                    "Inconsistency in backref cache found at offset %llu",
                    bytenr);
 }
index 8818ed5c390faa1774a8ee8dbb73c48f5508f449..19b22b4653c83b4fb3694f54e6d26a9d621d3e95 100644 (file)
@@ -193,12 +193,12 @@ static noinline void end_compressed_writeback(const struct compressed_bio *cb)
        unsigned long index = cb->start >> PAGE_SHIFT;
        unsigned long end_index = (cb->start + cb->len - 1) >> PAGE_SHIFT;
        struct folio_batch fbatch;
-       const int errno = blk_status_to_errno(cb->bbio.bio.bi_status);
+       const int error = blk_status_to_errno(cb->bbio.bio.bi_status);
        int i;
        int ret;
 
-       if (errno)
-               mapping_set_error(inode->i_mapping, errno);
+       if (error)
+               mapping_set_error(inode->i_mapping, error);
 
        folio_batch_init(&fbatch);
        while (index <= end_index) {
index c9d44bbf2ffa617a10cbcece1940f9a12c6ef3b4..986212d0b1f550c1c65d40e65e3710312d4d53f8 100644 (file)
@@ -230,9 +230,9 @@ noinline void btrfs_release_path(struct btrfs_path *p)
  * cause could be a bug, eg. due to ENOSPC, and not for common errors that are
  * caused by external factors.
  */
-bool __cold abort_should_print_stack(int errno)
+bool __cold abort_should_print_stack(int error)
 {
-       switch (errno) {
+       switch (error) {
        case -EIO:
        case -EROFS:
        case -ENOMEM:
index 5be060cb6ef5bdfc85af9c8ee27f622a476d3f9f..b8f9c9e56c8c36573133b49247dbae251584171a 100644 (file)
@@ -72,11 +72,11 @@ static void btrfs_state_to_string(const struct btrfs_fs_info *info, char *buf)
  *        over the error.  Each subsequent error that doesn't have any context
  *        of the original error should use EROFS when handling BTRFS_FS_STATE_ERROR.
  */
-const char * __attribute_const__ btrfs_decode_error(int errno)
+const char * __attribute_const__ btrfs_decode_error(int error)
 {
        char *errstr = "unknown";
 
-       switch (errno) {
+       switch (error) {
        case -ENOENT:           /* -2 */
                errstr = "No such entry";
                break;
@@ -115,7 +115,7 @@ const char * __attribute_const__ btrfs_decode_error(int errno)
  */
 __cold
 void __btrfs_handle_fs_error(struct btrfs_fs_info *fs_info, const char *function,
-                      unsigned int line, int errno, const char *fmt, ...)
+                      unsigned int line, int error, const char *fmt, ...)
 {
        struct super_block *sb = fs_info->sb;
 #ifdef CONFIG_PRINTK
@@ -132,11 +132,11 @@ void __btrfs_handle_fs_error(struct btrfs_fs_info *fs_info, const char *function
         * Special case: if the error is EROFS, and we're already under
         * SB_RDONLY, then it is safe here.
         */
-       if (errno == -EROFS && sb_rdonly(sb))
+       if (error == -EROFS && sb_rdonly(sb))
                return;
 
 #ifdef CONFIG_PRINTK
-       errstr = btrfs_decode_error(errno);
+       errstr = btrfs_decode_error(error);
        btrfs_state_to_string(fs_info, statestr);
        if (fmt) {
                struct va_format vaf;
@@ -147,11 +147,11 @@ void __btrfs_handle_fs_error(struct btrfs_fs_info *fs_info, const char *function
                vaf.va = &args;
 
                pr_crit("BTRFS: error (device %s%s) in %s:%d: errno=%d %s (%pV)\n",
-                       sb->s_id, statestr, function, line, errno, errstr, &vaf);
+                       sb->s_id, statestr, function, line, error, errstr, &vaf);
                va_end(args);
        } else {
                pr_crit("BTRFS: error (device %s%s) in %s:%d: errno=%d %s\n",
-                       sb->s_id, statestr, function, line, errno, errstr);
+                       sb->s_id, statestr, function, line, error, errstr);
        }
 #endif
 
@@ -159,7 +159,7 @@ void __btrfs_handle_fs_error(struct btrfs_fs_info *fs_info, const char *function
         * Today we only save the error info to memory.  Long term we'll also
         * send it down to the disk.
         */
-       WRITE_ONCE(fs_info->fs_error, errno);
+       WRITE_ONCE(fs_info->fs_error, error);
 
        /* Don't go through full error handling during mount. */
        if (!(sb->s_flags & SB_BORN))
@@ -288,7 +288,7 @@ void __cold btrfs_err_32bit_limit(struct btrfs_fs_info *fs_info)
  */
 __cold
 void __btrfs_panic(struct btrfs_fs_info *fs_info, const char *function,
-                  unsigned int line, int errno, const char *fmt, ...)
+                  unsigned int line, int error, const char *fmt, ...)
 {
        char *s_id = "<unknown>";
        const char *errstr;
@@ -301,13 +301,13 @@ void __btrfs_panic(struct btrfs_fs_info *fs_info, const char *function,
        va_start(args, fmt);
        vaf.va = &args;
 
-       errstr = btrfs_decode_error(errno);
+       errstr = btrfs_decode_error(error);
        if (fs_info && (btrfs_test_opt(fs_info, PANIC_ON_FATAL_ERROR)))
                panic(KERN_CRIT "BTRFS panic (device %s) in %s:%d: %pV (errno=%d %s)\n",
-                       s_id, function, line, &vaf, errno, errstr);
+                       s_id, function, line, &vaf, error, errstr);
 
        btrfs_crit(fs_info, "panic in %s:%d: %pV (errno=%d %s)",
-                  function, line, &vaf, errno, errstr);
+                  function, line, &vaf, error, errstr);
        va_end(args);
        /* Caller calls BUG() */
 }
index 1ae6f8e23e071c3b5b6e9325e9cd91588b03de89..4d04c1fa5899acfd12fca82e0aaf366de04c2bf8 100644 (file)
@@ -184,25 +184,25 @@ do {                                                              \
 __printf(5, 6)
 __cold
 void __btrfs_handle_fs_error(struct btrfs_fs_info *fs_info, const char *function,
-                    unsigned int line, int errno, const char *fmt, ...);
+                    unsigned int line, int error, const char *fmt, ...);
 
-const char * __attribute_const__ btrfs_decode_error(int errno);
+const char * __attribute_const__ btrfs_decode_error(int error);
 
-#define btrfs_handle_fs_error(fs_info, errno, fmt, args...)            \
+#define btrfs_handle_fs_error(fs_info, error, fmt, args...)            \
        __btrfs_handle_fs_error((fs_info), __func__, __LINE__,          \
-                               (errno), fmt, ##args)
+                               (error), fmt, ##args)
 
 __printf(5, 6)
 __cold
 void __btrfs_panic(struct btrfs_fs_info *fs_info, const char *function,
-                  unsigned int line, int errno, const char *fmt, ...);
+                  unsigned int line, int error, const char *fmt, ...);
 /*
  * If BTRFS_MOUNT_PANIC_ON_FATAL_ERROR is in mount_opt, __btrfs_panic
  * will panic().  Otherwise we BUG() here.
  */
-#define btrfs_panic(fs_info, errno, fmt, args...)                      \
+#define btrfs_panic(fs_info, error, fmt, args...)                      \
 do {                                                                   \
-       __btrfs_panic(fs_info, __func__, __LINE__, errno, fmt, ##args); \
+       __btrfs_panic(fs_info, __func__, __LINE__, error, fmt, ##args); \
        BUG();                                                          \
 } while (0)
 
index 74b0c3b9dc032ab8e1a37ea015b721bcd49d3a57..3f9f933039c624dda7d49e708ea0f4c4ad971537 100644 (file)
@@ -2735,18 +2735,18 @@ int btrfs_clean_one_deleted_snapshot(struct btrfs_fs_info *fs_info)
  */
 void __cold __btrfs_abort_transaction(struct btrfs_trans_handle *trans,
                                      const char *function,
-                                     unsigned int line, int errno, bool first_hit)
+                                     unsigned int line, int error, bool first_hit)
 {
        struct btrfs_fs_info *fs_info = trans->fs_info;
 
-       WRITE_ONCE(trans->aborted, errno);
-       WRITE_ONCE(trans->transaction->aborted, errno);
-       if (first_hit && errno == -ENOSPC)
+       WRITE_ONCE(trans->aborted, error);
+       WRITE_ONCE(trans->transaction->aborted, error);
+       if (first_hit && error == -ENOSPC)
                btrfs_dump_space_info_for_trans_abort(fs_info);
        /* Wake up anybody who may be waiting on this transaction */
        wake_up(&fs_info->transaction_wait);
        wake_up(&fs_info->transaction_blocked_wait);
-       __btrfs_handle_fs_error(fs_info, function, line, errno, NULL);
+       __btrfs_handle_fs_error(fs_info, function, line, error, NULL);
 }
 
 int __init btrfs_transaction_init(void)
index 44c5cdcd500f148b876b0e2af40b4c763ec389e1..934cadf9d7feb9720a1c78aa0209a7a1243395cd 100644 (file)
@@ -203,32 +203,32 @@ static inline void btrfs_clear_skip_qgroup(struct btrfs_trans_handle *trans)
        delayed_refs->qgroup_to_skip = 0;
 }
 
-bool __cold abort_should_print_stack(int errno);
+bool __cold abort_should_print_stack(int error);
 
 /*
  * Call btrfs_abort_transaction as early as possible when an error condition is
  * detected, that way the exact stack trace is reported for some errors.
  */
-#define btrfs_abort_transaction(trans, errno)          \
+#define btrfs_abort_transaction(trans, error)          \
 do {                                                           \
        bool first = false;                                     \
        /* Report first abort since mount */                    \
        if (!test_and_set_bit(BTRFS_FS_STATE_TRANS_ABORTED,     \
                        &((trans)->fs_info->fs_state))) {       \
                first = true;                                   \
-               if (WARN(abort_should_print_stack(errno),       \
+               if (WARN(abort_should_print_stack(error),       \
                        KERN_ERR                                \
                        "BTRFS: Transaction aborted (error %d)\n",      \
-                       (errno))) {                                     \
+                       (error))) {                                     \
                        /* Stack trace printed. */                      \
                } else {                                                \
                        btrfs_err((trans)->fs_info,                     \
                                  "Transaction aborted (error %d)",     \
-                                 (errno));                     \
+                                 (error));                     \
                }                                               \
        }                                                       \
        __btrfs_abort_transaction((trans), __func__,            \
-                                 __LINE__, (errno), first);    \
+                                 __LINE__, (error), first);    \
 } while (0)
 
 int btrfs_end_transaction(struct btrfs_trans_handle *trans);
@@ -267,7 +267,7 @@ void btrfs_add_dropped_root(struct btrfs_trans_handle *trans,
 void btrfs_trans_release_chunk_metadata(struct btrfs_trans_handle *trans);
 void __cold __btrfs_abort_transaction(struct btrfs_trans_handle *trans,
                                      const char *function,
-                                     unsigned int line, int errno, bool first_hit);
+                                     unsigned int line, int error, bool first_hit);
 
 int __init btrfs_transaction_init(void);
 void __cold btrfs_transaction_exit(void);