On Wed, Sep 13, 2023 at 08:10:07AM -0300, Christoph Hellwig wrote: > -static void affs_kill_sb(struct super_block *sb) > +static void affs_free_sb(struct super_block *sb) > { > struct affs_sb_info *sbi = AFFS_SB(sb); > - kill_block_super(sb); > + > + block_free_sb(sb); > if (sbi) { > affs_free_bitmap(sb); > affs_brelse(sbi->s_root_bh); <checks> Yep, that's printk + brelse()... Could we have that block_free_sb() (an awful name aside) done after the if (sbi) { ... } there?