Hi, On Tue, Nov 19, 2019 at 02:10:56AM -0500, Namjae Jeon wrote: > +static void exfat_put_super(struct super_block *sb) > +{ > + struct exfat_sb_info *sbi = EXFAT_SB(sb); > + > + mutex_lock(&sbi->s_lock); > + if (READ_ONCE(sbi->s_dirt)) { > + WRITE_ONCE(sbi->s_dirt, true); No idea what the code does. But I was just skimming over and find the above pattern somehow strange. Shouldn't this be something like if (!READ_ONCE(sbi->s_dirt)) { WRITE_ONCE(sbi->s_dirt, true); ? Thanks, Daniel