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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 9 Nov 2023 01:12:15 +0100 From: Andreas Grünbacher <andreas.gruenbacher@...il.com> To: Andrew Morton <akpm@...ux-foundation.org> Cc: "Matthew Wilcox (Oracle)" <willy@...radead.org>, linux-ext4 <linux-ext4@...r.kernel.org>, gfs2@...ts.linux.dev, Linux FS-devel Mailing List <linux-fsdevel@...r.kernel.org>, linux-xfs <linux-xfs@...r.kernel.org>, "Darrick J . Wong" <djwong@...nel.org>, linux-erofs@...ts.ozlabs.org, "Theodore Ts'o" <tytso@....edu>, Andreas Dilger <adilger.kernel@...ger.ca>, Andreas Gruenbacher <agruenba@...hat.com> Subject: Re: [PATCH 1/3] mm: Add folio_zero_tail() and use it in ext4 Andrew, Andrew Morton <akpm@...ux-foundation.org> schrieb am Do., 9. Nov. 2023, 00:06: > > + > > + if (folio_test_highmem(folio)) { > > + size_t max = PAGE_SIZE - offset_in_page(offset); > > + > > + while (len > max) { > > Shouldn't this be `while (len)'? AFAICT this code can fail to clear > the final page. not sure what you're seeing there, but this looks fine to me. Thanks, Andreas
Powered by blists - more mailing lists