Re: [PATCH v3 0/9] bugfix for memory hotplug

From: Yasuaki Ishimatsu
Date: Fri Oct 19 2012 - 05:40:08 EST


2012/10/19 17:45, Wen Congyang wrote:
> At 10/19/2012 04:19 PM, Yasuaki Ishimatsu Wrote:
>> 2012/10/19 17:06, Yasuaki Ishimatsu wrote:
>>> Hi Wen,
>>>
>>> Some bug fix patches have been merged into linux-next.
>>> So the patches confuse me.
>
> Sorry, I don't check linux-next tree.
>
>>
>> The following patches have been already merged into linux-next
>> and mm-tree as long as I know.
>>
>>>> Wen Congyang (6):
>>>> clear the memory to store struct page
>>
>>
>>>> memory-hotplug: skip HWPoisoned page when offlining pages
>>
>> mm-tree
>
> Hmm, I don't find this patch in this URL:
> http://www.ozlabs.org/~akpm/mmotm/broken-out/
>
> Do I miss something?

But Andrew announced that the patch was merged in mm-tree.
And you received the announcement.

>>
>>>> memory-hotplug: update mce_bad_pages when removing the memory
>>
>>>> memory-hotplug: auto offline page_cgroup when onlining memory block
>>>> failed
>>
>> mm-tree
>>
>>>> memory-hotplug: fix NR_FREE_PAGES mismatch
>>
>> mm-tree
>>
>>>> memory-hotplug: allocate zone's pcp before onlining pages
>>
>> mm-tree
>>
>>>>
>>>> Yasuaki Ishimatsu (3):
>>>> suppress "Device memoryX does not have a release() function" warning
>>
>> linux-next
>>
>>>> suppress "Device nodeX does not have a release() function" warning
>>>> memory-hotplug: flush the work for the node when the node is offlined
>>
>> linux-next
>
> I split this patch to two patches according to kosaki's comment.

Yeah, I know. But is the patch really need now?

Thanks,
Yasuaki Ishimatsu

>
> Thanks
> Wen Congyang
>
>>
>> Thanks,
>> Yasuaki Ishimatsu
>>
>>> Why did you send same patches again?
>>>
>>> Thanks,
>>> Yasuaki Ishimatsu
>>>
>>> 2012/10/19 15:46, wency@xxxxxxxxxxxxxx wrote:
>>>> From: Wen Congyang <wency@xxxxxxxxxxxxxx>
>>>>
>>>> Changes from v2 to v3:
>>>> Merge the bug fix from ishimatsu to this patchset(Patch 1-3)
>>>> Patch 3: split it from patch as it fixes another bug.
>>>> Patch 4: new patch, and fix bad-page state when hotadding a memory
>>>> device after hotremoving it. I forgot to post this patch in v2.
>>>> Patch 6: update it according to Dave Hansen's comment.
>>>>
>>>> Changes from v1 to v2:
>>>> Patch 1: updated according to kosaki's suggestion
>>>>
>>>> Patch 2: new patch, and update mce_bad_pages when removing memory.
>>>>
>>>> Patch 4: new patch, and fix a NR_FREE_PAGES mismatch, and this bug
>>>> cause oom in my test.
>>>>
>>>> Patch 5: new patch, and fix a new bug. When repeating to online/offline
>>>> pages, the free pages will continue to decrease.
>>>>
>>>> Wen Congyang (6):
>>>> clear the memory to store struct page
>>>> memory-hotplug: skip HWPoisoned page when offlining pages
>>>> memory-hotplug: update mce_bad_pages when removing the memory
>>>> memory-hotplug: auto offline page_cgroup when onlining memory block
>>>> failed
>>>> memory-hotplug: fix NR_FREE_PAGES mismatch
>>>> memory-hotplug: allocate zone's pcp before onlining pages
>>>>
>>>> Yasuaki Ishimatsu (3):
>>>> suppress "Device memoryX does not have a release() function" warning
>>>> suppress "Device nodeX does not have a release() function" warning
>>>> memory-hotplug: flush the work for the node when the node is offlined
>>>>
>>>> drivers/base/memory.c | 9 ++++++++-
>>>> drivers/base/node.c | 11 +++++++++++
>>>> include/linux/page-isolation.h | 10 ++++++----
>>>> mm/memory-failure.c | 2 +-
>>>> mm/memory_hotplug.c | 14 ++++++++------
>>>> mm/page_alloc.c | 37 ++++++++++++++++++++++++++++---------
>>>> mm/page_cgroup.c | 3 +++
>>>> mm/page_isolation.c | 27 ++++++++++++++++++++-------
>>>> mm/sparse.c | 22 +++++++++++++++++++++-
>>>> 9 files changed, 106 insertions(+), 29 deletions(-)
>>>>
>>>
>>>
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>>> the body of a message to majordomo@xxxxxxxxxxxxxxx
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>> Please read the FAQ at http://www.tux.org/lkml/
>>>
>>
>>
>>
>


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/