* [PATCH] mm: memory-failure: fix unexpected return value in soft_offline_page()
@ 2023-06-27 11:28 Miaohe Lin
2023-06-27 19:30 ` Andrew Morton
0 siblings, 1 reply; 5+ messages in thread
From: Miaohe Lin @ 2023-06-27 11:28 UTC (permalink / raw)
To: akpm, naoya.horiguchi; +Cc: linux-mm, linux-kernel, linmiaohe
When page_handle_poison() fails to handle the hugepage or free page in
retry path, soft_offline_page() will return 0 while -EBUSY is expected
in this case.
Fixes: b94e02822deb ("mm,hwpoison: try to narrow window race for free pages")
Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
---
mm/memory-failure.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/mm/memory-failure.c b/mm/memory-failure.c
index aada6ac72fe5..dc1572818b7d 100644
--- a/mm/memory-failure.c
+++ b/mm/memory-failure.c
@@ -2737,10 +2737,13 @@ int soft_offline_page(unsigned long pfn, int flags)
if (ret > 0) {
ret = soft_offline_in_use_page(page);
} else if (ret == 0) {
- if (!page_handle_poison(page, true, false) && try_again) {
- try_again = false;
- flags &= ~MF_COUNT_INCREASED;
- goto retry;
+ if (!page_handle_poison(page, true, false)) {
+ if (try_again) {
+ try_again = false;
+ flags &= ~MF_COUNT_INCREASED;
+ goto retry;
+ }
+ ret = -EBUSY;
}
}
--
2.27.0
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] mm: memory-failure: fix unexpected return value in soft_offline_page()
2023-06-27 11:28 [PATCH] mm: memory-failure: fix unexpected return value in soft_offline_page() Miaohe Lin
@ 2023-06-27 19:30 ` Andrew Morton
2023-06-28 1:56 ` Miaohe Lin
0 siblings, 1 reply; 5+ messages in thread
From: Andrew Morton @ 2023-06-27 19:30 UTC (permalink / raw)
To: Miaohe Lin; +Cc: naoya.horiguchi, linux-mm, linux-kernel
On Tue, 27 Jun 2023 19:28:08 +0800 Miaohe Lin <linmiaohe@huawei.com> wrote:
> When page_handle_poison() fails to handle the hugepage or free page in
> retry path, soft_offline_page() will return 0 while -EBUSY is expected
> in this case.
What are the user visible effects of the bug?
> Fixes: b94e02822deb ("mm,hwpoison: try to narrow window race for free pages")
>
> ...
>
> --- a/mm/memory-failure.c
> +++ b/mm/memory-failure.c
> @@ -2737,10 +2737,13 @@ int soft_offline_page(unsigned long pfn, int flags)
> if (ret > 0) {
> ret = soft_offline_in_use_page(page);
> } else if (ret == 0) {
> - if (!page_handle_poison(page, true, false) && try_again) {
> - try_again = false;
> - flags &= ~MF_COUNT_INCREASED;
> - goto retry;
> + if (!page_handle_poison(page, true, false)) {
> + if (try_again) {
> + try_again = false;
> + flags &= ~MF_COUNT_INCREASED;
> + goto retry;
> + }
> + ret = -EBUSY;
> }
> }
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] mm: memory-failure: fix unexpected return value in soft_offline_page()
2023-06-27 19:30 ` Andrew Morton
@ 2023-06-28 1:56 ` Miaohe Lin
2023-06-28 11:06 ` Naoya Horiguchi
0 siblings, 1 reply; 5+ messages in thread
From: Miaohe Lin @ 2023-06-28 1:56 UTC (permalink / raw)
To: Andrew Morton; +Cc: naoya.horiguchi, linux-mm, linux-kernel
On 2023/6/28 3:30, Andrew Morton wrote:
> On Tue, 27 Jun 2023 19:28:08 +0800 Miaohe Lin <linmiaohe@huawei.com> wrote:
>
>> When page_handle_poison() fails to handle the hugepage or free page in
>> retry path, soft_offline_page() will return 0 while -EBUSY is expected
>> in this case.
>
> What are the user visible effects of the bug?
The user will think soft_offline_page succeeds while it failed in fact. So user
will not try again later in this case.
>
>> Fixes: b94e02822deb ("mm,hwpoison: try to narrow window race for free pages")
>>
>> ...
>>
>> --- a/mm/memory-failure.c
>> +++ b/mm/memory-failure.c
>> @@ -2737,10 +2737,13 @@ int soft_offline_page(unsigned long pfn, int flags)
>> if (ret > 0) {
>> ret = soft_offline_in_use_page(page);
>> } else if (ret == 0) {
>> - if (!page_handle_poison(page, true, false) && try_again) {
>> - try_again = false;
>> - flags &= ~MF_COUNT_INCREASED;
>> - goto retry;
>> + if (!page_handle_poison(page, true, false)) {
>> + if (try_again) {
>> + try_again = false;
>> + flags &= ~MF_COUNT_INCREASED;
>> + goto retry;
>> + }
>> + ret = -EBUSY;
>> }
>> }
> .
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] mm: memory-failure: fix unexpected return value in soft_offline_page()
2023-06-28 1:56 ` Miaohe Lin
@ 2023-06-28 11:06 ` Naoya Horiguchi
2023-06-29 1:48 ` Miaohe Lin
0 siblings, 1 reply; 5+ messages in thread
From: Naoya Horiguchi @ 2023-06-28 11:06 UTC (permalink / raw)
To: Miaohe Lin; +Cc: Andrew Morton, naoya.horiguchi, linux-mm, linux-kernel
On Wed, Jun 28, 2023 at 09:56:38AM +0800, Miaohe Lin wrote:
> On 2023/6/28 3:30, Andrew Morton wrote:
> > On Tue, 27 Jun 2023 19:28:08 +0800 Miaohe Lin <linmiaohe@huawei.com> wrote:
> >
> >> When page_handle_poison() fails to handle the hugepage or free page in
> >> retry path, soft_offline_page() will return 0 while -EBUSY is expected
> >> in this case.
> >
> > What are the user visible effects of the bug?
>
> The user will think soft_offline_page succeeds while it failed in fact. So user
> will not try again later in this case.
I think that it's helpful to put this in patch descrition so that maintainers can
easily guess the impact of this patch.
Anyway, the patch looks good to me, thank you.
Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com>
>
> >
> >> Fixes: b94e02822deb ("mm,hwpoison: try to narrow window race for free pages")
> >>
> >> ...
> >>
> >> --- a/mm/memory-failure.c
> >> +++ b/mm/memory-failure.c
> >> @@ -2737,10 +2737,13 @@ int soft_offline_page(unsigned long pfn, int flags)
> >> if (ret > 0) {
> >> ret = soft_offline_in_use_page(page);
> >> } else if (ret == 0) {
> >> - if (!page_handle_poison(page, true, false) && try_again) {
> >> - try_again = false;
> >> - flags &= ~MF_COUNT_INCREASED;
> >> - goto retry;
> >> + if (!page_handle_poison(page, true, false)) {
> >> + if (try_again) {
> >> + try_again = false;
> >> + flags &= ~MF_COUNT_INCREASED;
> >> + goto retry;
> >> + }
> >> + ret = -EBUSY;
> >> }
> >> }
> > .
> >
>
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] mm: memory-failure: fix unexpected return value in soft_offline_page()
2023-06-28 11:06 ` Naoya Horiguchi
@ 2023-06-29 1:48 ` Miaohe Lin
0 siblings, 0 replies; 5+ messages in thread
From: Miaohe Lin @ 2023-06-29 1:48 UTC (permalink / raw)
To: Naoya Horiguchi; +Cc: Andrew Morton, naoya.horiguchi, linux-mm, linux-kernel
On 2023/6/28 19:06, Naoya Horiguchi wrote:
> On Wed, Jun 28, 2023 at 09:56:38AM +0800, Miaohe Lin wrote:
>> On 2023/6/28 3:30, Andrew Morton wrote:
>>> On Tue, 27 Jun 2023 19:28:08 +0800 Miaohe Lin <linmiaohe@huawei.com> wrote:
>>>
>>>> When page_handle_poison() fails to handle the hugepage or free page in
>>>> retry path, soft_offline_page() will return 0 while -EBUSY is expected
>>>> in this case.
>>>
>>> What are the user visible effects of the bug?
>>
>> The user will think soft_offline_page succeeds while it failed in fact. So user
>> will not try again later in this case.
>
> I think that it's helpful to put this in patch descrition so that maintainers can
> easily guess the impact of this patch.
Thanks for your review and advice. Will add it if v2 is needed.
>
> Anyway, the patch looks good to me, thank you.
>
> Acked-by: Naoya Horiguchi <naoya.horiguchi@nec.com>
Thanks Naoya.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-06-29 1:48 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-27 11:28 [PATCH] mm: memory-failure: fix unexpected return value in soft_offline_page() Miaohe Lin
2023-06-27 19:30 ` Andrew Morton
2023-06-28 1:56 ` Miaohe Lin
2023-06-28 11:06 ` Naoya Horiguchi
2023-06-29 1:48 ` Miaohe Lin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox