On 2024/7/30 19:33, Kemeng Shi wrote: > We will alloc journal_head with __GFP_NOFAIL anyway, test for failure > is pointless. > > Signed-off-by: Kemeng Shi <shikemeng@xxxxxxxxxxxxxxx> A nice cleanup. Reviewed-by: Zhang Yi <yi.zhang@xxxxxxxxxx> > --- > fs/jbd2/journal.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/jbd2/journal.c b/fs/jbd2/journal.c > index da5a56d700f1..b5d02de1ffff 100644 > --- a/fs/jbd2/journal.c > +++ b/fs/jbd2/journal.c > @@ -2866,8 +2866,7 @@ static struct journal_head *journal_alloc_journal_head(void) > ret = kmem_cache_zalloc(jbd2_journal_head_cache, > GFP_NOFS | __GFP_NOFAIL); > } > - if (ret) > - spin_lock_init(&ret->b_state_lock); > + spin_lock_init(&ret->b_state_lock); > return ret; > } > >
