On Thu, Jul 30, 2026 at 02:08:56PM -0400, Johannes Weiner wrote: > On Thu, Jul 30, 2026 at 10:40:32AM -0700, Frank van der Linden wrote: > > On Thu, Jul 30, 2026 at 10:30 AM Gregory Price <[email protected]> wrote: > > > arm64 already seems to use stores with NT hints by default. > > > > In general, though, I agree that for ASYNC requests, just always using > > NT seems fine. > > > > Maybe the mode and reason should be folded in to one variable, so > > that, further down the stack, a decision can be made as to what type > > of copy to use? E.g. if the mode is !MIGRATE_ASYNC and the reason is > > MR_DEMOTION, then non-temporal is still a good choice. If the mode is > > MIGRATE_ASYNC, NT is still a good idea. If you wanted to get fancy, > > any mode with folio_test_waiters(folio) == true should not use NT, > > since a task is waiting to use the data, so caching it is better. > > Maybe that's overthinking it. > > Agree. `reason' seems like a much stronger signal.
I think this ends up cleaner as well, it would be a nice improvement. ~Gregory
