linux-kernel - Re: [PATCH] mm/damon/reclaim: Use resource

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 9 May 2022 10:13:48 +0000
From:   "Boehme, Markus" <markubo@...zon.de>
To:     "jiapeng.chong@...ux.alibaba.com" <jiapeng.chong@...ux.alibaba.com>,
        "sj@...nel.org" <sj@...nel.org>
CC:     "linux-mm@...ck.org" <linux-mm@...ck.org>,
        "damon@...ts.linux.dev" <damon@...ts.linux.dev>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "abaci@...ux.alibaba.com" <abaci@...ux.alibaba.com>,
        "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>
Subject: Re: [PATCH] mm/damon/reclaim: Use resource_size function on resource object

(Keeping SJ's mail for context, since original patch seems to not have
gone to the DAMON list.)

On Sat, 2022-05-07 at 15:38 +0000, SeongJae Park wrote:
> On Sat, 7 May 2022 11:25:12 +0800 Jiapeng Chong <jiapeng.chong@...ux.alibaba.com> wrote:
> 
> > Fix the following coccicheck warnings:
> > 
> > ./mm/damon/reclaim.c:241:30-33: WARNING: Suspicious code. resource_size
> > is maybe missing with res.
> 
> Nit.  I'd prefer having this kind of program outputs in commit message be
> indented and not broken, like below:
> 
>     ./mm/damon/reclaim.c:241:30-33: WARNING: Suspicious code. resource_size is maybe missing with res.
> 
> > Reported-by: Abaci Robot <abaci@...ux.alibaba.com>
> > Signed-off-by: Jiapeng Chong <jiapeng.chong@...ux.alibaba.com>
> 
> Other than the nit,
> 
> Reviewed-by: SeongJae Park <sj@...nel.org>
> 
> 
> Thanks,
> SJ
> 
> > ---
> >  mm/damon/reclaim.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/mm/damon/reclaim.c b/mm/damon/reclaim.c
> > index f37c5d4b27fa..8efbfb24f3a1 100644
> > --- a/mm/damon/reclaim.c
> > +++ b/mm/damon/reclaim.c
> > @@ -238,7 +238,7 @@ static int walk_system_ram(struct resource *res, void *arg)
> >  {
> >       struct damon_reclaim_ram_walk_arg *a = arg;
> > 
> > -     if (a->end - a->start < res->end - res->start) {
> > +     if (a->end - a->start < resource_size(res)) {
> >               a->start = res->start;
> >               a->end = res->end;
> >       }
> > --
> > 2.20.1.7.g153144c

This is not just fixing the warning, but changing the calculation
slightly to correctly determine size (res->end not pointing one beyond
the end, but to the last address in the region). For finding the
biggest region it doesn't matter being off by one consistently, but
just fixing one side of the comparison changes behavior in the presence
of two equal regions. When fixing both sides, the "struct
damon_reclaim_ram_walk_arg" might just be replaced with a "struct
resource", too.

Markus



Amazon Development Center Germany GmbH
Krausenstr. 38
10117 Berlin
Geschaeftsfuehrung: Christian Schlaeger, Jonathan Weiss
Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B
Sitz: Berlin
Ust-ID: DE 289 237 879