linux-kernel - RE: [PATCH v4] page_alloc: consider highatomic reserve in watermark fast

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <1891546521.01663549682346.JavaMail.epsvc@epcpadp4>
Date:   Sun, 18 Sep 2022 10:41:40 +0900
From:   Jaewon Kim <jaewon31.kim@...sung.com>
To:     Greg KH <gregkh@...uxfoundation.org>, yong w <yongw.pur@...il.com>
CC:     Jaewon Kim <jaewon31.kim@...sung.com>,
        "mhocko@...nel.org" <mhocko@...nel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "stable@...r.kernel.org" <stable@...r.kernel.org>,
        "wang.yong12@....com.cn" <wang.yong12@....com.cn>,
        YongTaek Lee <ytk.lee@...sung.com>
Subject: RE: [PATCH v4] page_alloc: consider highatomic reserve in watermark
 fast

>On Wed, Sep 14, 2022 at 08:46:15AM +0800, yong w wrote:
>> Greg KH <gregkh@...uxfoundation.org> 于2022年9月13日周二 21:54?道：
>> 
>> >
>> > On Tue, Sep 13, 2022 at 09:09:47PM +0800, yong wrote:
>> > > Hello,
>> > > This patch is required to be patched in linux-5.4.y and linux-4.19.y.
>> >
>> > What is "this patch"?  There is no context here :(
>> >
>> Sorry, I forgot to quote the original patch. the patch is as follows
>> 
>>     f27ce0e page_alloc: consider highatomic reserve in watermark fast
>> 
>> > > In addition to that, the following two patches are somewhat related:
>> > >
>> > >       3334a45 mm/page_alloc: use ac->high_zoneidx for classzone_idx
>> > >       9282012 page_alloc: fix invalid watermark check on a negative value
>> >
>> > In what way?  What should be done here by us?
>> >
>> 
>> I think these two patches should also be merged.
>> 
>>     The classzone_idx  parameter is used in the zone_watermark_fast
>> functionzone, and 3334a45 use ac->high_zoneidx for classzone_idx.
>>     "9282012 page_alloc: fix invalid watermark check on a negative
>> value"  fix f27ce0e introduced issues
>
>Ok, I need an ack by all the developers involved in those commits, as
>well as the subsystem maintainer so that I know it's ok to take them.
>
>Can you provide a series of backported and tested patches so that they
>are easy to review?
>
>thanks,
>
>greg k-h

Hello I didn't know my Act is needed to merge it.

Acked-by: Jaewon Kim <jaewon31.kim@...sung.com>

I don't understand well why the commit f27ce0e has dependency on 3334a45, though.

Thank you
Jaewon Kim