Message ID | 20171123152101.5864-1-chao@kernel.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Thu 23-11-17 23:21:01, Chao Yu wrote: > @@ -2364,10 +2370,11 @@ static int vfs_load_quota_inode(struct inode *inode, int type, int format_id, > dqopt->flags |= dquot_state_flag(flags, type); > spin_unlock(&dq_state_lock); > > - add_dquot_ref(sb, type); > - > - return 0; > + error = add_dquot_ref(sb, type); > + if (error) > + dquot_disable(sb, type, dqopt->flags); Why dqopt->flags? If anything these flags are shifted by user/group/project shift and combined together for all the types. You need to just pass 'flags' here. Other than that the patch looks good. Honza
On 2017/11/27 23:56, Jan Kara wrote: > On Thu 23-11-17 23:21:01, Chao Yu wrote: >> @@ -2364,10 +2370,11 @@ static int vfs_load_quota_inode(struct inode *inode, int type, int format_id, >> dqopt->flags |= dquot_state_flag(flags, type); >> spin_unlock(&dq_state_lock); >> >> - add_dquot_ref(sb, type); >> - >> - return 0; >> + error = add_dquot_ref(sb, type); >> + if (error) >> + dquot_disable(sb, type, dqopt->flags); > > Why dqopt->flags? If anything these flags are shifted by user/group/project Oh, correct, user/group/project type quota flags are shifted and merged into dqopt->flags.... just notice that, sorry. Let me update the patch. :) Thanks, > shift and combined together for all the types. You need to just pass > 'flags' here. Other than that the patch looks good. > > Honza >
diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c index 8381db9db6d9..7ea02fc315cd 100644 --- a/fs/quota/dquot.c +++ b/fs/quota/dquot.c @@ -933,12 +933,13 @@ static int dqinit_needed(struct inode *inode, int type) } /* This routine is guarded by s_umount semaphore */ -static void add_dquot_ref(struct super_block *sb, int type) +static int add_dquot_ref(struct super_block *sb, int type) { struct inode *inode, *old_inode = NULL; #ifdef CONFIG_QUOTA_DEBUG int reserved = 0; #endif + int err = 0; spin_lock(&sb->s_inode_list_lock); list_for_each_entry(inode, &sb->s_inodes, i_sb_list) { @@ -958,7 +959,11 @@ static void add_dquot_ref(struct super_block *sb, int type) reserved = 1; #endif iput(old_inode); - __dquot_initialize(inode, type); + err = __dquot_initialize(inode, type); + if (err) { + iput(inode); + goto out; + } /* * We hold a reference to 'inode' so it couldn't have been @@ -973,7 +978,7 @@ static void add_dquot_ref(struct super_block *sb, int type) } spin_unlock(&sb->s_inode_list_lock); iput(old_inode); - +out: #ifdef CONFIG_QUOTA_DEBUG if (reserved) { quota_error(sb, "Writes happened before quota was turned on " @@ -981,6 +986,7 @@ static void add_dquot_ref(struct super_block *sb, int type) "Please run quotacheck(8)"); } #endif + return err; } /* @@ -2364,10 +2370,11 @@ static int vfs_load_quota_inode(struct inode *inode, int type, int format_id, dqopt->flags |= dquot_state_flag(flags, type); spin_unlock(&dq_state_lock); - add_dquot_ref(sb, type); - - return 0; + error = add_dquot_ref(sb, type); + if (error) + dquot_disable(sb, type, dqopt->flags); + return error; out_file_init: dqopt->files[type] = NULL; iput(inode);