* [PATCH] f2fs: Use f2fs_bug_on instead of if condition followed by f2fs_bug_on
@ 2021-08-16 10:22 ` Yangtao Li
0 siblings, 0 replies; 4+ messages in thread
From: Yangtao Li @ 2021-08-16 10:22 UTC (permalink / raw)
To: jaegeuk, chao; +Cc: linux-f2fs-devel, linux-kernel, Yangtao Li
Similar:
WARNING: Use BUG_ON instead of if condition followed by BUG.
Let's replace if (cond) f2fs_bug_on() with f2fs_bug_on().
Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
fs/f2fs/checkpoint.c | 6 ++----
fs/f2fs/compress.c | 3 +--
fs/f2fs/data.c | 6 ++----
fs/f2fs/node.h | 4 +---
fs/f2fs/segment.c | 5 ++---
fs/f2fs/segment.h | 10 ++++------
6 files changed, 12 insertions(+), 22 deletions(-)
diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index 6c208108d69c..eebe5a80b842 100644
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
@@ -475,8 +475,7 @@ static void __add_ino_entry(struct f2fs_sb_info *sbi, nid_t ino,
e = radix_tree_lookup(&im->ino_root, ino);
if (!e) {
e = tmp;
- if (unlikely(radix_tree_insert(&im->ino_root, ino, e)))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, radix_tree_insert(&im->ino_root, ino, e));
memset(e, 0, sizeof(struct ino_entry));
e->ino = ino;
@@ -1369,8 +1368,7 @@ static void commit_checkpoint(struct f2fs_sb_info *sbi,
memcpy(page_address(page), src, PAGE_SIZE);
set_page_dirty(page);
- if (unlikely(!clear_page_dirty_for_io(page)))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, !clear_page_dirty_for_io(page));
/* writeout cp pack 2 page */
err = __f2fs_write_meta_page(page, &wbc, FS_CP_META_IO);
diff --git a/fs/f2fs/compress.c b/fs/f2fs/compress.c
index 455561826c7d..43362f43c3d2 100644
--- a/fs/f2fs/compress.c
+++ b/fs/f2fs/compress.c
@@ -160,8 +160,7 @@ void f2fs_compress_ctx_add_page(struct compress_ctx *cc, struct page *page)
{
unsigned int cluster_ofs;
- if (!f2fs_cluster_can_merge_page(cc, page->index))
- f2fs_bug_on(F2FS_I_SB(cc->inode), 1);
+ f2fs_bug_on(F2FS_I_SB(cc->inode), !f2fs_cluster_can_merge_page(cc, page->index));
cluster_ofs = offset_in_cluster(cc, page->index);
cc->rpages[cluster_ofs] = page;
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index d2cf48c5a2e4..4e85d5bcf1bc 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -464,8 +464,7 @@ static inline void __submit_bio(struct f2fs_sb_info *sbi,
zero_user_segment(page, 0, PAGE_SIZE);
set_page_private_dummy(page);
- if (bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE)
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE);
}
/*
* In the NODE case, we lose next block address chain. So, we
@@ -727,8 +726,7 @@ static void add_bio_entry(struct f2fs_sb_info *sbi, struct bio *bio,
be->bio = bio;
bio_get(bio);
- if (bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE)
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE);
down_write(&io->bio_list_lock);
list_add_tail(&be->list, &io->bio_list);
diff --git a/fs/f2fs/node.h b/fs/f2fs/node.h
index ff14a6e5ac1c..4acf365771f4 100644
--- a/fs/f2fs/node.h
+++ b/fs/f2fs/node.h
@@ -191,9 +191,7 @@ static inline void get_nat_bitmap(struct f2fs_sb_info *sbi, void *addr)
struct f2fs_nm_info *nm_i = NM_I(sbi);
#ifdef CONFIG_F2FS_CHECK_FS
- if (memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir,
- nm_i->bitmap_size))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir, nm_i->bitmap_size));
#endif
memcpy(addr, nm_i->nat_bitmap, nm_i->bitmap_size);
}
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index 15cc89eef28d..12f2a9bc15d7 100644
--- a/fs/f2fs/segment.c
+++ b/fs/f2fs/segment.c
@@ -4216,9 +4216,8 @@ void f2fs_flush_sit_entries(struct f2fs_sb_info *sbi, struct cp_control *cpc)
se = get_seg_entry(sbi, segno);
#ifdef CONFIG_F2FS_CHECK_FS
- if (memcmp(se->cur_valid_map, se->cur_valid_map_mir,
- SIT_VBLOCK_MAP_SIZE))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, memcmp(se->cur_valid_map,
+ se->cur_valid_map_mir, SIT_VBLOCK_MAP_SIZE));
#endif
/* add discard candidates */
diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h
index 050230c70a53..c985b889572a 100644
--- a/fs/f2fs/segment.h
+++ b/fs/f2fs/segment.h
@@ -518,9 +518,8 @@ static inline void get_sit_bitmap(struct f2fs_sb_info *sbi,
struct sit_info *sit_i = SIT_I(sbi);
#ifdef CONFIG_F2FS_CHECK_FS
- if (memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
- sit_i->bitmap_size))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
+ sit_i->bitmap_size));
#endif
memcpy(dst_addr, sit_i->sit_bitmap, sit_i->bitmap_size);
}
@@ -762,9 +761,8 @@ static inline pgoff_t current_sit_addr(struct f2fs_sb_info *sbi,
check_seg_range(sbi, start);
#ifdef CONFIG_F2FS_CHECK_FS
- if (f2fs_test_bit(offset, sit_i->sit_bitmap) !=
- f2fs_test_bit(offset, sit_i->sit_bitmap_mir))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, f2fs_test_bit(offset, sit_i->sit_bitmap) !=
+ f2fs_test_bit(offset, sit_i->sit_bitmap_mir));
#endif
/* calculate sit block address */
--
2.32.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [f2fs-dev] [PATCH] f2fs: Use f2fs_bug_on instead of if condition followed by f2fs_bug_on
@ 2021-08-16 10:22 ` Yangtao Li
0 siblings, 0 replies; 4+ messages in thread
From: Yangtao Li @ 2021-08-16 10:22 UTC (permalink / raw)
To: jaegeuk, chao; +Cc: Yangtao Li, linux-kernel, linux-f2fs-devel
Similar:
WARNING: Use BUG_ON instead of if condition followed by BUG.
Let's replace if (cond) f2fs_bug_on() with f2fs_bug_on().
Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
fs/f2fs/checkpoint.c | 6 ++----
fs/f2fs/compress.c | 3 +--
fs/f2fs/data.c | 6 ++----
fs/f2fs/node.h | 4 +---
fs/f2fs/segment.c | 5 ++---
fs/f2fs/segment.h | 10 ++++------
6 files changed, 12 insertions(+), 22 deletions(-)
diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
index 6c208108d69c..eebe5a80b842 100644
--- a/fs/f2fs/checkpoint.c
+++ b/fs/f2fs/checkpoint.c
@@ -475,8 +475,7 @@ static void __add_ino_entry(struct f2fs_sb_info *sbi, nid_t ino,
e = radix_tree_lookup(&im->ino_root, ino);
if (!e) {
e = tmp;
- if (unlikely(radix_tree_insert(&im->ino_root, ino, e)))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, radix_tree_insert(&im->ino_root, ino, e));
memset(e, 0, sizeof(struct ino_entry));
e->ino = ino;
@@ -1369,8 +1368,7 @@ static void commit_checkpoint(struct f2fs_sb_info *sbi,
memcpy(page_address(page), src, PAGE_SIZE);
set_page_dirty(page);
- if (unlikely(!clear_page_dirty_for_io(page)))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, !clear_page_dirty_for_io(page));
/* writeout cp pack 2 page */
err = __f2fs_write_meta_page(page, &wbc, FS_CP_META_IO);
diff --git a/fs/f2fs/compress.c b/fs/f2fs/compress.c
index 455561826c7d..43362f43c3d2 100644
--- a/fs/f2fs/compress.c
+++ b/fs/f2fs/compress.c
@@ -160,8 +160,7 @@ void f2fs_compress_ctx_add_page(struct compress_ctx *cc, struct page *page)
{
unsigned int cluster_ofs;
- if (!f2fs_cluster_can_merge_page(cc, page->index))
- f2fs_bug_on(F2FS_I_SB(cc->inode), 1);
+ f2fs_bug_on(F2FS_I_SB(cc->inode), !f2fs_cluster_can_merge_page(cc, page->index));
cluster_ofs = offset_in_cluster(cc, page->index);
cc->rpages[cluster_ofs] = page;
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index d2cf48c5a2e4..4e85d5bcf1bc 100644
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -464,8 +464,7 @@ static inline void __submit_bio(struct f2fs_sb_info *sbi,
zero_user_segment(page, 0, PAGE_SIZE);
set_page_private_dummy(page);
- if (bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE)
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE);
}
/*
* In the NODE case, we lose next block address chain. So, we
@@ -727,8 +726,7 @@ static void add_bio_entry(struct f2fs_sb_info *sbi, struct bio *bio,
be->bio = bio;
bio_get(bio);
- if (bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE)
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE);
down_write(&io->bio_list_lock);
list_add_tail(&be->list, &io->bio_list);
diff --git a/fs/f2fs/node.h b/fs/f2fs/node.h
index ff14a6e5ac1c..4acf365771f4 100644
--- a/fs/f2fs/node.h
+++ b/fs/f2fs/node.h
@@ -191,9 +191,7 @@ static inline void get_nat_bitmap(struct f2fs_sb_info *sbi, void *addr)
struct f2fs_nm_info *nm_i = NM_I(sbi);
#ifdef CONFIG_F2FS_CHECK_FS
- if (memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir,
- nm_i->bitmap_size))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir, nm_i->bitmap_size));
#endif
memcpy(addr, nm_i->nat_bitmap, nm_i->bitmap_size);
}
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index 15cc89eef28d..12f2a9bc15d7 100644
--- a/fs/f2fs/segment.c
+++ b/fs/f2fs/segment.c
@@ -4216,9 +4216,8 @@ void f2fs_flush_sit_entries(struct f2fs_sb_info *sbi, struct cp_control *cpc)
se = get_seg_entry(sbi, segno);
#ifdef CONFIG_F2FS_CHECK_FS
- if (memcmp(se->cur_valid_map, se->cur_valid_map_mir,
- SIT_VBLOCK_MAP_SIZE))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, memcmp(se->cur_valid_map,
+ se->cur_valid_map_mir, SIT_VBLOCK_MAP_SIZE));
#endif
/* add discard candidates */
diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h
index 050230c70a53..c985b889572a 100644
--- a/fs/f2fs/segment.h
+++ b/fs/f2fs/segment.h
@@ -518,9 +518,8 @@ static inline void get_sit_bitmap(struct f2fs_sb_info *sbi,
struct sit_info *sit_i = SIT_I(sbi);
#ifdef CONFIG_F2FS_CHECK_FS
- if (memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
- sit_i->bitmap_size))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
+ sit_i->bitmap_size));
#endif
memcpy(dst_addr, sit_i->sit_bitmap, sit_i->bitmap_size);
}
@@ -762,9 +761,8 @@ static inline pgoff_t current_sit_addr(struct f2fs_sb_info *sbi,
check_seg_range(sbi, start);
#ifdef CONFIG_F2FS_CHECK_FS
- if (f2fs_test_bit(offset, sit_i->sit_bitmap) !=
- f2fs_test_bit(offset, sit_i->sit_bitmap_mir))
- f2fs_bug_on(sbi, 1);
+ f2fs_bug_on(sbi, f2fs_test_bit(offset, sit_i->sit_bitmap) !=
+ f2fs_test_bit(offset, sit_i->sit_bitmap_mir));
#endif
/* calculate sit block address */
--
2.32.0
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] f2fs: Use f2fs_bug_on instead of if condition followed by f2fs_bug_on
2021-08-16 10:22 ` [f2fs-dev] " Yangtao Li
@ 2021-08-17 0:31 ` Jaegeuk Kim
-1 siblings, 0 replies; 4+ messages in thread
From: Jaegeuk Kim @ 2021-08-17 0:31 UTC (permalink / raw)
To: Yangtao Li; +Cc: chao, linux-f2fs-devel, linux-kernel
No, I don't think this is a right way.
On 08/16, Yangtao Li wrote:
> Similar:
> WARNING: Use BUG_ON instead of if condition followed by BUG.
>
> Let's replace if (cond) f2fs_bug_on() with f2fs_bug_on().
>
> Signed-off-by: Yangtao Li <frank.li@vivo.com>
> ---
> fs/f2fs/checkpoint.c | 6 ++----
> fs/f2fs/compress.c | 3 +--
> fs/f2fs/data.c | 6 ++----
> fs/f2fs/node.h | 4 +---
> fs/f2fs/segment.c | 5 ++---
> fs/f2fs/segment.h | 10 ++++------
> 6 files changed, 12 insertions(+), 22 deletions(-)
>
> diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
> index 6c208108d69c..eebe5a80b842 100644
> --- a/fs/f2fs/checkpoint.c
> +++ b/fs/f2fs/checkpoint.c
> @@ -475,8 +475,7 @@ static void __add_ino_entry(struct f2fs_sb_info *sbi, nid_t ino,
> e = radix_tree_lookup(&im->ino_root, ino);
> if (!e) {
> e = tmp;
> - if (unlikely(radix_tree_insert(&im->ino_root, ino, e)))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, radix_tree_insert(&im->ino_root, ino, e));
>
> memset(e, 0, sizeof(struct ino_entry));
> e->ino = ino;
> @@ -1369,8 +1368,7 @@ static void commit_checkpoint(struct f2fs_sb_info *sbi,
> memcpy(page_address(page), src, PAGE_SIZE);
>
> set_page_dirty(page);
> - if (unlikely(!clear_page_dirty_for_io(page)))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, !clear_page_dirty_for_io(page));
>
> /* writeout cp pack 2 page */
> err = __f2fs_write_meta_page(page, &wbc, FS_CP_META_IO);
> diff --git a/fs/f2fs/compress.c b/fs/f2fs/compress.c
> index 455561826c7d..43362f43c3d2 100644
> --- a/fs/f2fs/compress.c
> +++ b/fs/f2fs/compress.c
> @@ -160,8 +160,7 @@ void f2fs_compress_ctx_add_page(struct compress_ctx *cc, struct page *page)
> {
> unsigned int cluster_ofs;
>
> - if (!f2fs_cluster_can_merge_page(cc, page->index))
> - f2fs_bug_on(F2FS_I_SB(cc->inode), 1);
> + f2fs_bug_on(F2FS_I_SB(cc->inode), !f2fs_cluster_can_merge_page(cc, page->index));
>
> cluster_ofs = offset_in_cluster(cc, page->index);
> cc->rpages[cluster_ofs] = page;
> diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
> index d2cf48c5a2e4..4e85d5bcf1bc 100644
> --- a/fs/f2fs/data.c
> +++ b/fs/f2fs/data.c
> @@ -464,8 +464,7 @@ static inline void __submit_bio(struct f2fs_sb_info *sbi,
> zero_user_segment(page, 0, PAGE_SIZE);
> set_page_private_dummy(page);
>
> - if (bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE)
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE);
> }
> /*
> * In the NODE case, we lose next block address chain. So, we
> @@ -727,8 +726,7 @@ static void add_bio_entry(struct f2fs_sb_info *sbi, struct bio *bio,
> be->bio = bio;
> bio_get(bio);
>
> - if (bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE)
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE);
>
> down_write(&io->bio_list_lock);
> list_add_tail(&be->list, &io->bio_list);
> diff --git a/fs/f2fs/node.h b/fs/f2fs/node.h
> index ff14a6e5ac1c..4acf365771f4 100644
> --- a/fs/f2fs/node.h
> +++ b/fs/f2fs/node.h
> @@ -191,9 +191,7 @@ static inline void get_nat_bitmap(struct f2fs_sb_info *sbi, void *addr)
> struct f2fs_nm_info *nm_i = NM_I(sbi);
>
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir,
> - nm_i->bitmap_size))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir, nm_i->bitmap_size));
> #endif
> memcpy(addr, nm_i->nat_bitmap, nm_i->bitmap_size);
> }
> diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
> index 15cc89eef28d..12f2a9bc15d7 100644
> --- a/fs/f2fs/segment.c
> +++ b/fs/f2fs/segment.c
> @@ -4216,9 +4216,8 @@ void f2fs_flush_sit_entries(struct f2fs_sb_info *sbi, struct cp_control *cpc)
>
> se = get_seg_entry(sbi, segno);
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (memcmp(se->cur_valid_map, se->cur_valid_map_mir,
> - SIT_VBLOCK_MAP_SIZE))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, memcmp(se->cur_valid_map,
> + se->cur_valid_map_mir, SIT_VBLOCK_MAP_SIZE));
> #endif
>
> /* add discard candidates */
> diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h
> index 050230c70a53..c985b889572a 100644
> --- a/fs/f2fs/segment.h
> +++ b/fs/f2fs/segment.h
> @@ -518,9 +518,8 @@ static inline void get_sit_bitmap(struct f2fs_sb_info *sbi,
> struct sit_info *sit_i = SIT_I(sbi);
>
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
> - sit_i->bitmap_size))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
> + sit_i->bitmap_size));
> #endif
> memcpy(dst_addr, sit_i->sit_bitmap, sit_i->bitmap_size);
> }
> @@ -762,9 +761,8 @@ static inline pgoff_t current_sit_addr(struct f2fs_sb_info *sbi,
> check_seg_range(sbi, start);
>
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (f2fs_test_bit(offset, sit_i->sit_bitmap) !=
> - f2fs_test_bit(offset, sit_i->sit_bitmap_mir))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, f2fs_test_bit(offset, sit_i->sit_bitmap) !=
> + f2fs_test_bit(offset, sit_i->sit_bitmap_mir));
> #endif
>
> /* calculate sit block address */
> --
> 2.32.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [f2fs-dev] [PATCH] f2fs: Use f2fs_bug_on instead of if condition followed by f2fs_bug_on
@ 2021-08-17 0:31 ` Jaegeuk Kim
0 siblings, 0 replies; 4+ messages in thread
From: Jaegeuk Kim @ 2021-08-17 0:31 UTC (permalink / raw)
To: Yangtao Li; +Cc: linux-kernel, linux-f2fs-devel
No, I don't think this is a right way.
On 08/16, Yangtao Li wrote:
> Similar:
> WARNING: Use BUG_ON instead of if condition followed by BUG.
>
> Let's replace if (cond) f2fs_bug_on() with f2fs_bug_on().
>
> Signed-off-by: Yangtao Li <frank.li@vivo.com>
> ---
> fs/f2fs/checkpoint.c | 6 ++----
> fs/f2fs/compress.c | 3 +--
> fs/f2fs/data.c | 6 ++----
> fs/f2fs/node.h | 4 +---
> fs/f2fs/segment.c | 5 ++---
> fs/f2fs/segment.h | 10 ++++------
> 6 files changed, 12 insertions(+), 22 deletions(-)
>
> diff --git a/fs/f2fs/checkpoint.c b/fs/f2fs/checkpoint.c
> index 6c208108d69c..eebe5a80b842 100644
> --- a/fs/f2fs/checkpoint.c
> +++ b/fs/f2fs/checkpoint.c
> @@ -475,8 +475,7 @@ static void __add_ino_entry(struct f2fs_sb_info *sbi, nid_t ino,
> e = radix_tree_lookup(&im->ino_root, ino);
> if (!e) {
> e = tmp;
> - if (unlikely(radix_tree_insert(&im->ino_root, ino, e)))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, radix_tree_insert(&im->ino_root, ino, e));
>
> memset(e, 0, sizeof(struct ino_entry));
> e->ino = ino;
> @@ -1369,8 +1368,7 @@ static void commit_checkpoint(struct f2fs_sb_info *sbi,
> memcpy(page_address(page), src, PAGE_SIZE);
>
> set_page_dirty(page);
> - if (unlikely(!clear_page_dirty_for_io(page)))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, !clear_page_dirty_for_io(page));
>
> /* writeout cp pack 2 page */
> err = __f2fs_write_meta_page(page, &wbc, FS_CP_META_IO);
> diff --git a/fs/f2fs/compress.c b/fs/f2fs/compress.c
> index 455561826c7d..43362f43c3d2 100644
> --- a/fs/f2fs/compress.c
> +++ b/fs/f2fs/compress.c
> @@ -160,8 +160,7 @@ void f2fs_compress_ctx_add_page(struct compress_ctx *cc, struct page *page)
> {
> unsigned int cluster_ofs;
>
> - if (!f2fs_cluster_can_merge_page(cc, page->index))
> - f2fs_bug_on(F2FS_I_SB(cc->inode), 1);
> + f2fs_bug_on(F2FS_I_SB(cc->inode), !f2fs_cluster_can_merge_page(cc, page->index));
>
> cluster_ofs = offset_in_cluster(cc, page->index);
> cc->rpages[cluster_ofs] = page;
> diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
> index d2cf48c5a2e4..4e85d5bcf1bc 100644
> --- a/fs/f2fs/data.c
> +++ b/fs/f2fs/data.c
> @@ -464,8 +464,7 @@ static inline void __submit_bio(struct f2fs_sb_info *sbi,
> zero_user_segment(page, 0, PAGE_SIZE);
> set_page_private_dummy(page);
>
> - if (bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE)
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) < PAGE_SIZE);
> }
> /*
> * In the NODE case, we lose next block address chain. So, we
> @@ -727,8 +726,7 @@ static void add_bio_entry(struct f2fs_sb_info *sbi, struct bio *bio,
> be->bio = bio;
> bio_get(bio);
>
> - if (bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE)
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, bio_add_page(bio, page, PAGE_SIZE, 0) != PAGE_SIZE);
>
> down_write(&io->bio_list_lock);
> list_add_tail(&be->list, &io->bio_list);
> diff --git a/fs/f2fs/node.h b/fs/f2fs/node.h
> index ff14a6e5ac1c..4acf365771f4 100644
> --- a/fs/f2fs/node.h
> +++ b/fs/f2fs/node.h
> @@ -191,9 +191,7 @@ static inline void get_nat_bitmap(struct f2fs_sb_info *sbi, void *addr)
> struct f2fs_nm_info *nm_i = NM_I(sbi);
>
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir,
> - nm_i->bitmap_size))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, memcmp(nm_i->nat_bitmap, nm_i->nat_bitmap_mir, nm_i->bitmap_size));
> #endif
> memcpy(addr, nm_i->nat_bitmap, nm_i->bitmap_size);
> }
> diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
> index 15cc89eef28d..12f2a9bc15d7 100644
> --- a/fs/f2fs/segment.c
> +++ b/fs/f2fs/segment.c
> @@ -4216,9 +4216,8 @@ void f2fs_flush_sit_entries(struct f2fs_sb_info *sbi, struct cp_control *cpc)
>
> se = get_seg_entry(sbi, segno);
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (memcmp(se->cur_valid_map, se->cur_valid_map_mir,
> - SIT_VBLOCK_MAP_SIZE))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, memcmp(se->cur_valid_map,
> + se->cur_valid_map_mir, SIT_VBLOCK_MAP_SIZE));
> #endif
>
> /* add discard candidates */
> diff --git a/fs/f2fs/segment.h b/fs/f2fs/segment.h
> index 050230c70a53..c985b889572a 100644
> --- a/fs/f2fs/segment.h
> +++ b/fs/f2fs/segment.h
> @@ -518,9 +518,8 @@ static inline void get_sit_bitmap(struct f2fs_sb_info *sbi,
> struct sit_info *sit_i = SIT_I(sbi);
>
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
> - sit_i->bitmap_size))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, memcmp(sit_i->sit_bitmap, sit_i->sit_bitmap_mir,
> + sit_i->bitmap_size));
> #endif
> memcpy(dst_addr, sit_i->sit_bitmap, sit_i->bitmap_size);
> }
> @@ -762,9 +761,8 @@ static inline pgoff_t current_sit_addr(struct f2fs_sb_info *sbi,
> check_seg_range(sbi, start);
>
> #ifdef CONFIG_F2FS_CHECK_FS
> - if (f2fs_test_bit(offset, sit_i->sit_bitmap) !=
> - f2fs_test_bit(offset, sit_i->sit_bitmap_mir))
> - f2fs_bug_on(sbi, 1);
> + f2fs_bug_on(sbi, f2fs_test_bit(offset, sit_i->sit_bitmap) !=
> + f2fs_test_bit(offset, sit_i->sit_bitmap_mir));
> #endif
>
> /* calculate sit block address */
> --
> 2.32.0
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-08-17 0:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-16 10:22 [PATCH] f2fs: Use f2fs_bug_on instead of if condition followed by f2fs_bug_on Yangtao Li
2021-08-16 10:22 ` [f2fs-dev] " Yangtao Li
2021-08-17 0:31 ` Jaegeuk Kim
2021-08-17 0:31 ` [f2fs-dev] " Jaegeuk Kim
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.