mbox series

[v8,0/3] mm/hwpoison: fix race between hugetlb free/demotion and memory_failure_hugetlb()

Message ID 20220408135323.1559401-1-naoya.horiguchi@linux.dev (mailing list archive)
Headers show
Series mm/hwpoison: fix race between hugetlb free/demotion and memory_failure_hugetlb() | expand

Message

Naoya Horiguchi April 8, 2022, 1:53 p.m. UTC
Hi,

I updated v7 (https://lore.kernel.org/linux-mm/20220407112929.1344748-1-naoya.horiguchi@linux.dev/)
based on the comment from Miaohe Lin.  Now I have a few additional patches.
Any comments and suggestions are welcomed.

Thanks,
Naoya Horiguchi
---
Summary:

Naoya Horiguchi (3):
      mm/hwpoison: fix race between hugetlb free/demotion and memory_failure_hugetlb()
      mm/hwpoison: put page in already hwpoisoned case with MF_COUNT_INCREASED
      Revert "mm/memory-failure.c: fix race with changing page compound again"

 include/linux/hugetlb.h |   6 ++
 include/linux/mm.h      |   9 ++-
 include/ras/ras_event.h |   1 -
 mm/hugetlb.c            |  10 ++++
 mm/memory-failure.c     | 152 ++++++++++++++++++++++++++++++++----------------
 5 files changed, 126 insertions(+), 52 deletions(-)

Comments

Andrew Morton April 8, 2022, 11:47 p.m. UTC | #1
On Fri,  8 Apr 2022 22:53:20 +0900 Naoya Horiguchi <naoya.horiguchi@linux.dev> wrote:

> I updated v7 (https://lore.kernel.org/linux-mm/20220407112929.1344748-1-naoya.horiguchi@linux.dev/)
> based on the comment from Miaohe Lin.  Now I have a few additional patches.
> Any comments and suggestions are welcomed.

As [1/3] is cc:stable I queued that for 5.18-rcX and the other two
patches I queued for 5.19.

This plan means that [1/3] will be merged into mainline without the
other two patches yet present, which is a combination which you may not
have tested.  Please confirm that this will be OK?
Naoya Horiguchi April 9, 2022, 2:31 a.m. UTC | #2
On Fri, Apr 08, 2022 at 04:47:55PM -0700, Andrew Morton wrote:
> On Fri,  8 Apr 2022 22:53:20 +0900 Naoya Horiguchi <naoya.horiguchi@linux.dev> wrote:
> 
> > I updated v7 (https://lore.kernel.org/linux-mm/20220407112929.1344748-1-naoya.horiguchi@linux.dev/)
> > based on the comment from Miaohe Lin.  Now I have a few additional patches.
> > Any comments and suggestions are welcomed.
> 
> As [1/3] is cc:stable I queued that for 5.18-rcX and the other two
> patches I queued for 5.19.
> 
> This plan means that [1/3] will be merged into mainline without the
> other two patches yet present, which is a combination which you may not
> have tested.  Please confirm that this will be OK?

Yes, the plan is fine to me.
I've tested the kernel with all 3 patched, but 2/3 and 3/3 are simple and
the effect should be predictable.  I'll test only with 1/3 next week.

Thanks,
Naoya Horiguchi