Hi, On Thu, Aug 3, 2017 at 8:24 PM, Andrew Perepechko <anserper@xxxxxxxxx> wrote: >> >> I don't think this is right, as far as i understand, journal quota need go >> together with quota space change update inside same transaction, this will >> break consistency if power off or RO happen. >> > > Hello Wang! > > There is no transaction change in this case because all callers of this > function have open handles for the same transaction. > > If you enter that DQ_MOD_B check, you are guaranteed to reference > the SAME transaction as the thread that's in between of mark_dirty > and clear_dirty. > This change mean if this dquot is dirty we skip, this won't work because in this way, quota update is only kept in vfs dquota memory and newer update is not wrote to journal file and not wrapped into transaction too. This is not what journal quota means to do. Thanks, Shilong > Thank you, > Andrew
