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: <6f5881df-0e0b-4278-808b-7c0cffa12a30@suse.cz>
Date: Tue, 3 Feb 2026 17:55:27 +0100
From: Vlastimil Babka <vbabka@...e.cz>
To: Christoph Hellwig <hch@...radead.org>, Harry Yoo <harry.yoo@...cle.com>
Cc: Vernon Yang <vernon2gm@...il.com>, 李龙兴
 <coregee2000@...il.com>, syzkaller@...glegroups.com,
 akpm@...ux-foundation.org, cl@...two.org, rientjes@...gle.com,
 roman.gushchin@...ux.dev, linux-mm@...ck.org, linux-kernel@...r.kernel.org
Subject: Re: [Kernel Bug] WARNING in mempool_alloc_noprof

On 2/3/26 17:27, Christoph Hellwig wrote:
> On Tue, Feb 03, 2026 at 06:52:39PM +0900, Harry Yoo wrote:
>> Maybe the changelog could be rephrased a bit,
>> but overall LGTM, thanks!
> 
> 
> No, that does not make sense.  If mempool is used with __GFP_RECLAIM in
> the flags it won't fail, and if it isn't, GFP_NOFAIL can't work.

So that means as long as there's __GFP_RECLAIM, __GFP_NOFAIL isn't wrong,
just redundant.

> So if there is any work in mempool_alloc it's that it should warn about
> GFP_NOFAIL.  The fix is in the callers to not pass the flag, assuming
> current kernels still do this.

We could just tolerate the redundant. The code would be simpler too. But I
don't feel too strongly about it.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ