lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 03 Aug 2017 16:41:38 +0300 From: Andrew Perepechko <anserper@...dex.ru> To: Wang Shilong <wangshilong1991@...il.com> Cc: Shuichi Ihara <sihara@....com>, Wang Shilong <wshilong@....com>, Li Xi <lixi@....com>, Ext4 Developers List <linux-ext4@...r.kernel.org>, Jan Kara <jack@...e.cz>, linux-fsdevel@...r.kernel.org Subject: Re: quota: dqio_mutex design > > 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
Powered by blists - more mailing lists