On Aug 16, 2017, at 9:41 AM, Jan Kara <jack@xxxxxxx> wrote: > > dquot_get_next_id() needs dqio_sem only for reading to protect against > racing with modification of quota file structure. > > Signed-off-by: Jan Kara <jack@xxxxxxx> Reviewed-by: Andreas Dilger <adilger@xxxxxxxxx> > --- > fs/quota/dquot.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/quota/dquot.c b/fs/quota/dquot.c > index 21358f31923d..8d5ccad3bf3e 100644 > --- a/fs/quota/dquot.c > +++ b/fs/quota/dquot.c > @@ -2079,9 +2079,9 @@ int dquot_get_next_id(struct super_block *sb, struct kqid *qid) > return -ESRCH; > if (!dqopt->ops[qid->type]->get_next_id) > return -ENOSYS; > - down_write(&dqopt->dqio_sem); > + down_read(&dqopt->dqio_sem); > err = dqopt->ops[qid->type]->get_next_id(sb, qid); > - up_write(&dqopt->dqio_sem); > + up_read(&dqopt->dqio_sem); > return err; > } > EXPORT_SYMBOL(dquot_get_next_id); > -- > 2.12.3 > Cheers, Andreas
Attachment:
signature.asc
Description: Message signed with OpenPGP
