diff options
author | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-05-16 13:11:23 -0400 |
---|---|---|
committer | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-05-16 13:11:23 -0400 |
commit | a6aae4dd0ffad299a33d122f8a339b399bee5381 (patch) | |
tree | 958b015be27d97d079a3c14694576571ca916600 /fs/ubifs/recovery.c | |
parent | f70b7e52aa23c9aea5346b9730b402fb55f9079b (diff) |
UBIFS: get rid of dbg_err
This patch removes the 'dbg_err()' macro and we now use 'ubifs_err()' instead.
The idea of 'dbg_err()' was to compile out some error message to make the
binary a bit smaller - but I think it was a bad idea.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Diffstat (limited to 'fs/ubifs/recovery.c')
-rw-r--r-- | fs/ubifs/recovery.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/fs/ubifs/recovery.c b/fs/ubifs/recovery.c index bbeab617960b..01a348dd4587 100644 --- a/fs/ubifs/recovery.c +++ b/fs/ubifs/recovery.c | |||
@@ -362,11 +362,11 @@ out_err: | |||
362 | out_free: | 362 | out_free: |
363 | ubifs_err("failed to recover master node"); | 363 | ubifs_err("failed to recover master node"); |
364 | if (mst1) { | 364 | if (mst1) { |
365 | dbg_err("dumping first master node"); | 365 | ubifs_err("dumping first master node"); |
366 | ubifs_dump_node(c, mst1); | 366 | ubifs_dump_node(c, mst1); |
367 | } | 367 | } |
368 | if (mst2) { | 368 | if (mst2) { |
369 | dbg_err("dumping second master node"); | 369 | ubifs_err("dumping second master node"); |
370 | ubifs_dump_node(c, mst2); | 370 | ubifs_dump_node(c, mst2); |
371 | } | 371 | } |
372 | vfree(buf2); | 372 | vfree(buf2); |
@@ -683,7 +683,7 @@ struct ubifs_scan_leb *ubifs_recover_leb(struct ubifs_info *c, int lnum, | |||
683 | ret, lnum, offs); | 683 | ret, lnum, offs); |
684 | break; | 684 | break; |
685 | } else { | 685 | } else { |
686 | dbg_err("unexpected return value %d", ret); | 686 | ubifs_err("unexpected return value %d", ret); |
687 | err = -EINVAL; | 687 | err = -EINVAL; |
688 | goto error; | 688 | goto error; |
689 | } | 689 | } |
@@ -789,7 +789,7 @@ struct ubifs_scan_leb *ubifs_recover_leb(struct ubifs_info *c, int lnum, | |||
789 | 789 | ||
790 | corrupted_rescan: | 790 | corrupted_rescan: |
791 | /* Re-scan the corrupted data with verbose messages */ | 791 | /* Re-scan the corrupted data with verbose messages */ |
792 | dbg_err("corruptio %d", ret); | 792 | ubifs_err("corruptio %d", ret); |
793 | ubifs_scan_a_node(c, buf, len, lnum, offs, 1); | 793 | ubifs_scan_a_node(c, buf, len, lnum, offs, 1); |
794 | corrupted: | 794 | corrupted: |
795 | ubifs_scanned_corruption(c, lnum, offs, buf); | 795 | ubifs_scanned_corruption(c, lnum, offs, buf); |
@@ -827,17 +827,17 @@ static int get_cs_sqnum(struct ubifs_info *c, int lnum, int offs, | |||
827 | goto out_free; | 827 | goto out_free; |
828 | ret = ubifs_scan_a_node(c, cs_node, UBIFS_CS_NODE_SZ, lnum, offs, 0); | 828 | ret = ubifs_scan_a_node(c, cs_node, UBIFS_CS_NODE_SZ, lnum, offs, 0); |
829 | if (ret != SCANNED_A_NODE) { | 829 | if (ret != SCANNED_A_NODE) { |
830 | dbg_err("Not a valid node"); | 830 | ubifs_err("Not a valid node"); |
831 | goto out_err; | 831 | goto out_err; |
832 | } | 832 | } |
833 | if (cs_node->ch.node_type != UBIFS_CS_NODE) { | 833 | if (cs_node->ch.node_type != UBIFS_CS_NODE) { |
834 | dbg_err("Node a CS node, type is %d", cs_node->ch.node_type); | 834 | ubifs_err("Node a CS node, type is %d", cs_node->ch.node_type); |
835 | goto out_err; | 835 | goto out_err; |
836 | } | 836 | } |
837 | if (le64_to_cpu(cs_node->cmt_no) != c->cmt_no) { | 837 | if (le64_to_cpu(cs_node->cmt_no) != c->cmt_no) { |
838 | dbg_err("CS node cmt_no %llu != current cmt_no %llu", | 838 | ubifs_err("CS node cmt_no %llu != current cmt_no %llu", |
839 | (unsigned long long)le64_to_cpu(cs_node->cmt_no), | 839 | (unsigned long long)le64_to_cpu(cs_node->cmt_no), |
840 | c->cmt_no); | 840 | c->cmt_no); |
841 | goto out_err; | 841 | goto out_err; |
842 | } | 842 | } |
843 | *cs_sqnum = le64_to_cpu(cs_node->ch.sqnum); | 843 | *cs_sqnum = le64_to_cpu(cs_node->ch.sqnum); |
@@ -1138,7 +1138,7 @@ static int grab_empty_leb(struct ubifs_info *c) | |||
1138 | */ | 1138 | */ |
1139 | lnum = ubifs_find_free_leb_for_idx(c); | 1139 | lnum = ubifs_find_free_leb_for_idx(c); |
1140 | if (lnum < 0) { | 1140 | if (lnum < 0) { |
1141 | dbg_err("could not find an empty LEB"); | 1141 | ubifs_err("could not find an empty LEB"); |
1142 | ubifs_dump_lprops(c); | 1142 | ubifs_dump_lprops(c); |
1143 | ubifs_dump_budg(c, &c->bi); | 1143 | ubifs_dump_budg(c, &c->bi); |
1144 | return lnum; | 1144 | return lnum; |
@@ -1218,7 +1218,7 @@ int ubifs_rcvry_gc_commit(struct ubifs_info *c) | |||
1218 | } | 1218 | } |
1219 | mutex_unlock(&wbuf->io_mutex); | 1219 | mutex_unlock(&wbuf->io_mutex); |
1220 | if (err < 0) { | 1220 | if (err < 0) { |
1221 | dbg_err("GC failed, error %d", err); | 1221 | ubifs_err("GC failed, error %d", err); |
1222 | if (err == -EAGAIN) | 1222 | if (err == -EAGAIN) |
1223 | err = -EINVAL; | 1223 | err = -EINVAL; |
1224 | return err; | 1224 | return err; |