Re: [PATCH v2] mm: use clear_user_(high)page() for arch with special user folio handling

[Mathieu Desnoyers]

On 2024-12-07 11:20, Zi Yan wrote:
> On 7 Dec 2024, at 10:31, Mathieu Desnoyers wrote:
[...]
> > I'm not entirely sure if we want to go for the wording "is_aliasing"
> > or "is_incoherent" when talking about icache vs dcache, so I'm open
> > to ideas here.
>
> Let me know if the code below looks good to you. I will use
> (cpu_icache_is_aliasing() || cpu_dcache_is_aliasing())
> instead of
> (cpu_dcache_is_aliasing() || IS_ENABLED(CONFIG_ARC))
> in my next version.

Yes, this is in line with what I have in mind.

And I prefer your second version with the generic
cpu_icache_is_aliasing rather than defining it for
each architecture.

Thanks,

Mathieu

--
Mathieu Desnoyers
EfficiOS Inc.
https://www.efficios.com