diff options
| author | Jeff Liu <jeff.liu@oracle.com> | 2012-07-18 12:12:41 +0800 |
|---|---|---|
| committer | Jan Kara <jack@suse.cz> | 2012-08-15 00:22:57 +0200 |
| commit | 6ea2eea1fa930b9308a06f77fce65c38931eeb13 (patch) | |
| tree | 39ee0487bdcbb9e1c1d9d6c250a4a3f1b36e80dd /fs | |
| parent | dc141a402b9dc03a4188cd978a4cf149c397172c (diff) | |
| download | op-kernel-dev-6ea2eea1fa930b9308a06f77fce65c38931eeb13.zip op-kernel-dev-6ea2eea1fa930b9308a06f77fce65c38931eeb13.tar.gz | |
quota: Move down dqptr_sem read after initializing default warn[] type at __dquot_alloc_space().
sb->s_dqopt->dqptr_sem is used to serialize ops using pointers from inode to
dquots. But for __dquot_alloc_space(), it could be safely moved down after the
default warn[] array got initialized.
Signed-off-by: Jie Liu <jeff.liu@oracle.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/quota/dquot.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c index 36a29b7..c495a30 100644 --- a/fs/quota/dquot.c +++ b/fs/quota/dquot.c @@ -1589,10 +1589,10 @@ int __dquot_alloc_space(struct inode *inode, qsize_t number, int flags) goto out; } - down_read(&sb_dqopt(inode->i_sb)->dqptr_sem); for (cnt = 0; cnt < MAXQUOTAS; cnt++) warn[cnt].w_type = QUOTA_NL_NOWARN; + down_read(&sb_dqopt(inode->i_sb)->dqptr_sem); spin_lock(&dq_data_lock); for (cnt = 0; cnt < MAXQUOTAS; cnt++) { if (!dquots[cnt]) |
