linux-kernel - Re: [PATCH] filemap: call filemap_get_folios_tag() from filemap_get

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]

Message-ID: <ZSAcmFaZC9d4VJNF@casper.infradead.org>
Date:   Fri, 6 Oct 2023 15:41:28 +0100
From:   Matthew Wilcox <willy@...radead.org>
To:     Pankaj Raghav <kernel@...kajraghav.com>
Cc:     akpm@...ux-foundation.org, linux-fsdevel@...r.kernel.org,
        linux-mm@...ck.org, gost.dev@...sung.com,
        linux-kernel@...r.kernel.org, Pankaj Raghav <p.raghav@...sung.com>
Subject: Re: [PATCH] filemap: call filemap_get_folios_tag() from
 filemap_get_folios()

On Fri, Oct 06, 2023 at 01:01:20PM +0200, Pankaj Raghav wrote:
> From: Pankaj Raghav <p.raghav@...sung.com>
> 
> filemap_get_folios() is filemap_get_folios_tag() with XA_PRESENT as the
> tag that is being matched. Return filemap_get_folios_tag() with
> XA_PRESENT as the tag instead of duplicating the code in
> filemap_get_folios().

Yes, I think this makes sense.  I was consciously trying to make them
as similar to each other as possible.  I hadn't realised I'd succeeded
to such an extent.

Reviewed-by: Matthew Wilcox (Oracle) <willy@...radead.org>