Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Sat, 04 Jan 2020 09:50:43 +0800
From:   Chengguang Xu <cgxu519@mykernel.net>
Reply-To: cgxu519@mykernel.net
To:     "Pavel Machek" <pavel@denx.de>
Cc:     "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
        "linux-kernel" <linux-kernel@vger.kernel.org>,
        "stable" <stable@vger.kernel.org>, "Chao Yu" <yuchao0@huawei.com>,
        "Jaegeuk Kim" <jaegeuk@kernel.org>,
        "Sasha Levin" <sashal@kernel.org>
Message-ID: <16f6e3f5bbe.d291a05d38838.5222280714928609391@mykernel.net>
In-Reply-To: <20200103171213.GC14328@amd>
References: <20200102220029.183913184@linuxfoundation.org>
 <20200102220035.294585461@linuxfoundation.org> <20200103171213.GC14328@amd>
Subject: Re: [PATCH 4.19 062/114] f2fs: choose hardlimit when softlimit is
 larger than hardlimit in f2fs_statfs_project()
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: ZohoCN Mail
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

 ---- =E5=9C=A8 =E6=98=9F=E6=9C=9F=E5=85=AD, 2020-01-04 01:12:13 Pavel Mach=
ek <pavel@denx.de> =E6=92=B0=E5=86=99 ----
 > Hi!
 >=20
 > > From: Chengguang Xu <cgxu519@mykernel.net>
 > >=20
 > > [ Upstream commit 909110c060f22e65756659ec6fa957ae75777e00 ]
 > >=20
 > > Setting softlimit larger than hardlimit seems meaningless
 > > for disk quota but currently it is allowed. In this case,
 > > there may be a bit of comfusion for users when they run
 > > df comamnd to directory which has project quota.
 > >=20
 > > For example, we set 20M softlimit and 10M hardlimit of
 > > block usage limit for project quota of test_dir(project id 123).
 >=20
 > > Signed-off-by: Chengguang Xu <cgxu519@mykernel.net>
 > > Reviewed-by: Chao Yu <yuchao0@huawei.com>
 > > Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
 > > Signed-off-by: Sasha Levin <sashal@kernel.org>
 > > ---
 > >  fs/f2fs/super.c | 20 ++++++++++++++------
 > >  1 file changed, 14 insertions(+), 6 deletions(-)
 > >=20
 > > diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
 > > index 7a9cc64f5ca3..662c7de58b99 100644
 > > --- a/fs/f2fs/super.c
 > > +++ b/fs/f2fs/super.c
 > > @@ -1148,9 +1148,13 @@ static int f2fs_statfs_project(struct super_blo=
ck *sb,
 > >          return PTR_ERR(dquot);
 > >      spin_lock(&dquot->dq_dqb_lock);
 > > =20
 > > -    limit =3D (dquot->dq_dqb.dqb_bsoftlimit ?
 > > -         dquot->dq_dqb.dqb_bsoftlimit :
 > > -         dquot->dq_dqb.dqb_bhardlimit) >> sb->s_blocksize_bits;
 > > +    limit =3D 0;
 > > +    if (dquot->dq_dqb.dqb_bsoftlimit)
 > > +        limit =3D dquot->dq_dqb.dqb_bsoftlimit;
 > > +    if (dquot->dq_dqb.dqb_bhardlimit &&
 > > +            (!limit || dquot->dq_dqb.dqb_bhardlimit < limit))
 > > +        limit =3D dquot->dq_dqb.dqb_bhardlimit;
 > > +
 > >      if (limit && buf->f_blocks > limit) {
 >=20
 > >> blocksize disappeared here. That can't be right.
 >=20
 > Plus, is this just obfuscated way of saying
 >=20
 > limit =3D min_not_zero(dquot->dq_dqb.dqb_bsoftlimit, dquot->dq_dqb.dqb_b=
hardlimit)?
 >=20

Please  skip this patch from  stable list,  I'll send  a revised patch to u=
pstream.

Thanks.