On 12/7/24 18:16, Zi Yan wrote:
> Some architectures have special handling after clearing user folios:
> architectures, which set cpu_dcache_is_aliasing() to true, require
> flushing dcache; arc, which sets cpu_icache_is_aliasing() to true, changes
> folio->flags to make icache coherent to dcache. So __GFP_ZERO using only
> clear_page() is not enough to zero user folios and clear_user_(high)page()
> must be used. Otherwise, user data will be corrupted.
>
> Fix it by always clearing user folios with clear_user_(high)page() when
> cpu_dcache_is_aliasing() is true or cpu_icache_is_aliasing() is true.
> Rename alloc_zeroed() to alloc_need_zeroing() and invert the logic to
> clarify its intend.
user_alloc_needs_zeroing() would be perhaps more descriptive?
> Fixes: 5708d96da20b ("mm: avoid zeroing user movable page twice with
> init_on_alloc=1")
> Reported-by: Geert Uytterhoeven <geert+rene...@glider.be>
> Closes:
> https://lore.kernel.org/linux-mm/CAMuHMdV1hRp_NtR5YnJo=hsfgkqeh91j537gh4gkk3pfzhs...@mail.gmail.com/
> Tested-by: Geert Uytterhoeven <geert+rene...@glider.be>
> Signed-off-by: Zi Yan <z...@nvidia.com>
Acked-by: Vlastimil Babka <vba...@suse.cz>
_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc
