[v2,0/5] HWpoison: further fixes and cleanups

Message

Oscar Salvador Sept. 8, 2020, 7:56 a.m. UTC

The important bit of this patchset is patch#1, which is a fix to take off
HWPoison pages off a buddy freelist since it can lead us to having HWPoison
pages back in the game without no one noticing it.
So fix it (we did that already for soft_offline_page [1]).

The other patches are clean-ups and not that important, so if anything,
consider patch#1 for inclusion.

[1] https://patchwork.kernel.org/cover/11704083/

Thanks

Oscar Salvador (5):
  mm,hwpoison: Take free pages off the buddy freelists
  mm,hwpoison: Refactor madvise_inject_error
  mm,hwpoison: Drain pcplists before bailing out for non-buddy
    zero-refcount page
  mm,hwpoison: Drop unneeded pcplist draining
  mm,hwpoison: Remove stale code

 mm/madvise.c        | 34 ++++++++++++------------------
 mm/memory-failure.c | 50 +++++++++++++++++++++++++++++++++------------
 2 files changed, 50 insertions(+), 34 deletions(-)

Comments

Oscar Salvador Sept. 10, 2020, 8:07 p.m. UTC | #1

On 2020-09-08 09:56, Oscar Salvador wrote:
> The important bit of this patchset is patch#1, which is a fix to take 
> off
> HWPoison pages off a buddy freelist since it can lead us to having 
> HWPoison
> pages back in the game without no one noticing it.
> So fix it (we did that already for soft_offline_page [1]).
> 
> The other patches are clean-ups and not that important, so if anything,
> consider patch#1 for inclusion.
> 
> [1] https://patchwork.kernel.org/cover/11704083/
> 
> Thanks
> 
> Oscar Salvador (5):
>   mm,hwpoison: Take free pages off the buddy freelists
>   mm,hwpoison: Refactor madvise_inject_error
>   mm,hwpoison: Drain pcplists before bailing out for non-buddy
>     zero-refcount page
>   mm,hwpoison: Drop unneeded pcplist draining
>   mm,hwpoison: Remove stale code

Naoya, any insight?

Thanks