> > 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. That's not true. As I explained earlier, having DQ_MOD_B set at this point means another thread is going to write dquot but hasn't yet started doing so. This thread does not care whether it updates the ondisk dquot with its own data or with fresher data which came from another thread. In-core dquot has no indication of whose data in contains. As I also explained earlier, the update cannot happen in the context of another transaction because thread A which sees DQ_MOD_B set and thread B which is running dquot_commit() both have journal handles to the same transaction. There's only one running transaction at a time and thread B does not switch to another transaction. Please read the code carefully. > > This is not what journal quota means to do. > > > Thanks, > Shilong > > > Thank you, > > Andrew
