Re: [PATCH] mm/huge_memory: fix swap entry values of tail pages of THP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Closes:
https://lore.kernel.org/linux-mm/69cb784f-578d-ded1-cd9f-c6db04696336@xxxxxxxxxxx/
Fixes: 3417013e0d18 ("mm/migrate: Add folio_migrate_mapping()")
Cc: <stable@xxxxxxxxxxxxxxx> # see patch description, applicable to <=6.1
Signed-off-by: Charan Teja Kalla <quic_charante@xxxxxxxxxxx>

3417013e0d18 went into 5.16.

cfeed8ffe55b3 went into 6.6.

So only 6.1 is affected.


I tried to dig why the older kernels don't have this issue. The issue
persists on the older kernels too, AFAICS. See,
migrate_page_move_mapping().

So, The Fixes: tag too is unique for 6.1 kernel. And older LTS kernel
requires different tag it seems..

We really have to identify which commit it actually broke, and whether it was repeatedly fixed and broken again.

Because backporting this to 6.1 might be feasible. Backporting to much older kernels possibly not.



Isn't there a way to bite the bullet and backport that series to 6.1
instead?

My worry is that, because of merge conflicts, not sure If It can end up
in inducing some other issues.

I can have a look this/next week. I don't recall if there was any particular dependency.


Although we didn't test THP on older kernels, from the code walk, it
seems issue persists to me on older to 6.1 kernel, unless I am missing
something here. So back porting of this series to all those LTS kernels,
may not be a straight forward?

So, I am really not sure of what is the way forward here...

Again, if we want to fix this properly, we should first identify the commit that actually broke it.

If it predates folios, we'd need different fixes for different stable kernels most likely.

The big question are:

1) Is it broken in 5.15? Did you actually try to reproduce or is this
   just a guess?

2) How did you come up with 417013e0d18 ("mm/migrate: Add
   folio_migrate_mapping()")

--
Cheers,

David / dhildenb





[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux