From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx177.postini.com [74.125.245.177]) by kanga.kvack.org (Postfix) with SMTP id C467F6B005D for ; Fri, 19 Oct 2012 05:40:17 -0400 (EDT) Received: from m1.gw.fujitsu.co.jp (unknown [10.0.50.71]) by fgwmail5.fujitsu.co.jp (Postfix) with ESMTP id 1358B3EE0C3 for ; Fri, 19 Oct 2012 18:40:16 +0900 (JST) Received: from smail (m1 [127.0.0.1]) by outgoing.m1.gw.fujitsu.co.jp (Postfix) with ESMTP id ECD6645DE5E for ; Fri, 19 Oct 2012 18:40:15 +0900 (JST) Received: from s1.gw.fujitsu.co.jp (s1.gw.fujitsu.co.jp [10.0.50.91]) by m1.gw.fujitsu.co.jp (Postfix) with ESMTP id D20A045DE56 for ; Fri, 19 Oct 2012 18:40:15 +0900 (JST) Received: from s1.gw.fujitsu.co.jp (localhost.localdomain [127.0.0.1]) by s1.gw.fujitsu.co.jp (Postfix) with ESMTP id BFAA41DB8055 for ; Fri, 19 Oct 2012 18:40:15 +0900 (JST) Received: from g01jpexchyt07.g01.fujitsu.local (g01jpexchyt07.g01.fujitsu.local [10.128.194.46]) by s1.gw.fujitsu.co.jp (Postfix) with ESMTP id 6A47E1DB804A for ; Fri, 19 Oct 2012 18:40:15 +0900 (JST) Message-ID: <50811FE1.4080606@jp.fujitsu.com> Date: Fri, 19 Oct 2012 18:39:45 +0900 From: Yasuaki Ishimatsu MIME-Version: 1.0 Subject: Re: [PATCH v3 0/9] bugfix for memory hotplug References: <1350629202-9664-1-git-send-email-wency@cn.fujitsu.com> <508109F2.1080402@jp.fujitsu.com> <50810D14.8020609@jp.fujitsu.com> <50811336.7070704@cn.fujitsu.com> In-Reply-To: <50811336.7070704@cn.fujitsu.com> Content-Type: text/plain; charset="ISO-2022-JP" Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Wen Congyang Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, rientjes@google.com, liuj97@gmail.com, len.brown@intel.com, benh@kernel.crashing.org, paulus@samba.org, minchan.kim@gmail.com, akpm@linux-foundation.org, kosaki.motohiro@jp.fujitsu.com 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@cn.fujitsu.com wrote: >>>> From: Wen Congyang >>>> >>>> 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@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> Please read the FAQ at http://www.tux.org/lkml/ >>> >> >> >> > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758003Ab2JSJkR (ORCPT ); Fri, 19 Oct 2012 05:40:17 -0400 Received: from fgwmail6.fujitsu.co.jp ([192.51.44.36]:34438 "EHLO fgwmail6.fujitsu.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757806Ab2JSJkO (ORCPT ); Fri, 19 Oct 2012 05:40:14 -0400 X-SecurityPolicyCheck: OK by SHieldMailChecker v1.7.4 Message-ID: <50811FE1.4080606@jp.fujitsu.com> Date: Fri, 19 Oct 2012 18:39:45 +0900 From: Yasuaki Ishimatsu User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:16.0) Gecko/20121010 Thunderbird/16.0.1 MIME-Version: 1.0 To: Wen Congyang CC: , , , , , , , , , Subject: Re: [PATCH v3 0/9] bugfix for memory hotplug References: <1350629202-9664-1-git-send-email-wency@cn.fujitsu.com> <508109F2.1080402@jp.fujitsu.com> <50810D14.8020609@jp.fujitsu.com> <50811336.7070704@cn.fujitsu.com> In-Reply-To: <50811336.7070704@cn.fujitsu.com> Content-Type: text/plain; charset="ISO-2022-JP" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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@cn.fujitsu.com wrote: >>>> From: Wen Congyang >>>> >>>> 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@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> Please read the FAQ at http://www.tux.org/lkml/ >>> >> >> >> >