From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0F83AC432BE for ; Tue, 17 Aug 2021 00:31:23 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E665460F58 for ; Tue, 17 Aug 2021 00:31:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235109AbhHQAby (ORCPT ); Mon, 16 Aug 2021 20:31:54 -0400 Received: from mail.kernel.org ([198.145.29.99]:60746 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233009AbhHQAbw (ORCPT ); Mon, 16 Aug 2021 20:31:52 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 4874D60F39; Tue, 17 Aug 2021 00:31:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1629160280; bh=PhrDqW+iL1rUOz0aN2eeBsoZrADOX0NtAbiLIl1UPPY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=piRqxQkzx2vsJU3zIhmTVgQc9+rnFHsl+/V2oMp0Q7oZ65uxiLIwYIKn8JxVcSQEj U8yz/YhPvBC4FSjGNkMmmXxjS1ZTt++b7mYD/RQ1k1+2tGrqnnTZZasEMvOAuY1VHa F6eFpQoSqJxfNOSpT+D1aPkFmhJNJBnv8UqKCSYmkxJj/zbICj1JUBP3rd+k8x7/AW 54HT07yv5VOo5N4KAovylyPMEfA/YMOgK9DAjPaKZS83bihWfk3iKCB1+0N3IsFoZi mgqgw01XSVgDxuhqxwcpKJhqYGmnxViH8GRn/jIJB55EP3LS0kIZQOPWnT7RJRNXf4 O/VNjZFXH4Qbw== Date: Mon, 16 Aug 2021 17:31:18 -0700 From: Jaegeuk Kim To: Yangtao Li Cc: chao@kernel.org, linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org Subject: Re: [PATCH] f2fs: Use f2fs_bug_on instead of if condition followed by f2fs_bug_on Message-ID: References: <20210816102207.277804-1-frank.li@vivo.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210816102207.277804-1-frank.li@vivo.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > --- > 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