diff for duplicates of <YnOvIuSRZX1cct4Y@MiWiFi-R3L-srv> diff --git a/a/1.txt b/N1/1.txt index b9c3641..0cef64a 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -11,19 +11,19 @@ On 05/05/22 at 03:29pm, Sourabh Jain wrote: > > > > > > > > > > +#if defined(CONFIG_MEMORY_HOTPLUG) > > > > > > > > +static int crash_memhp_notifier(struct notifier_block *nb, -> > > > > > > > +??? unsigned long val, void *v) +> > > > > > > > + unsigned long val, void *v) > > > > > > > > +{ -> > > > > > > > +??? struct memory_notify *mhp = v; +> > > > > > > > + struct memory_notify *mhp = v; > > > > > > > > + -> > > > > > > > +??? switch (val) { -> > > > > > > > +??? case MEM_ONLINE: -> > > > > > > > +??????? crash_hotplug_handler(KEXEC_CRASH_HP_ADD_MEMORY, -1U); +> > > > > > > > + switch (val) { +> > > > > > > > + case MEM_ONLINE: +> > > > > > > > + crash_hotplug_handler(KEXEC_CRASH_HP_ADD_MEMORY, -1U); > > > > > > > We don't differentiate the memory add/remove, cpu add, except of cpu > > > > > > > remove. Means the hp_action only differentiate cpu remove from the other > > > > > > > action. Maybe only making two types? > > > > > > > -> > > > > > > #define KEXEC_CRASH_HP_REMOVE_CPU?? 0 -> > > > > > > #define KEXEC_CRASH_HP_UPDATE_OTHER????? 1 +> > > > > > > #define KEXEC_CRASH_HP_REMOVE_CPU 0 +> > > > > > > #define KEXEC_CRASH_HP_UPDATE_OTHER 1 > > > > > > > > > > > > > Sourabh Jain's work with PPC uses REMOVE_CPU, REMOVE_MEMORY, and > > > > > > ADD_MEMORY. diff --git a/a/content_digest b/N1/content_digest index fd16ee8..51a07b8 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -8,9 +8,29 @@ "ref\0YnNkgfnHlUTky0lt@MiWiFi-R3L-srv\0" "ref\0e677f11b-32f9-0c89-9592-b987b00c4353@linux.ibm.com\0" "From\0Baoquan He <bhe@redhat.com>\0" - "Subject\0[PATCH v7 4/8] crash: add generic infrastructure for crash hotplug support\0" + "Subject\0Re: [PATCH v7 4/8] crash: add generic infrastructure for crash hotplug support\0" "Date\0Thu, 5 May 2022 19:04:02 +0800\0" - "To\0kexec@lists.infradead.org\0" + "To\0Sourabh Jain <sourabhjain@linux.ibm.com>" + " Eric DeVolder <eric.devolder@oracle.com>\0" + "Cc\0linux-kernel@vger.kernel.org" + x86@kernel.org + kexec@lists.infradead.org + ebiederm@xmission.com + dyoung@redhat.com + vgoyal@redhat.com + tglx@linutronix.de + mingo@redhat.com + bp@alien8.de + dave.hansen@linux.intel.com + hpa@zytor.com + nramas@linux.microsoft.com + thomas.lendacky@amd.com + robh@kernel.org + efault@gmx.de + rppt@kernel.org + david@redhat.com + konrad.wilk@oracle.com + " boris.ostrovsky@oracle.com\0" "\00:1\0" "b\0" "On 05/05/22 at 03:29pm, Sourabh Jain wrote:\n" @@ -26,19 +46,19 @@ "> > \n" "> > > > > > > > +#if defined(CONFIG_MEMORY_HOTPLUG)\n" "> > > > > > > > +static int crash_memhp_notifier(struct notifier_block *nb,\n" - "> > > > > > > > +??? unsigned long val, void *v)\n" + "> > > > > > > > +\302\240\302\240\302\240 unsigned long val, void *v)\n" "> > > > > > > > +{\n" - "> > > > > > > > +??? struct memory_notify *mhp = v;\n" + "> > > > > > > > +\302\240\302\240\302\240 struct memory_notify *mhp = v;\n" "> > > > > > > > +\n" - "> > > > > > > > +??? switch (val) {\n" - "> > > > > > > > +??? case MEM_ONLINE:\n" - "> > > > > > > > +??????? crash_hotplug_handler(KEXEC_CRASH_HP_ADD_MEMORY, -1U);\n" + "> > > > > > > > +\302\240\302\240\302\240 switch (val) {\n" + "> > > > > > > > +\302\240\302\240\302\240 case MEM_ONLINE:\n" + "> > > > > > > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240 crash_hotplug_handler(KEXEC_CRASH_HP_ADD_MEMORY, -1U);\n" "> > > > > > > We don't differentiate the memory add/remove, cpu add, except of cpu\n" "> > > > > > > remove. Means the hp_action only differentiate cpu remove from the other\n" "> > > > > > > action. Maybe only making two types?\n" "> > > > > > > \n" - "> > > > > > > #define KEXEC_CRASH_HP_REMOVE_CPU?? 0\n" - "> > > > > > > #define KEXEC_CRASH_HP_UPDATE_OTHER????? 1\n" + "> > > > > > > #define KEXEC_CRASH_HP_REMOVE_CPU\302\240\302\240 0\n" + "> > > > > > > #define KEXEC_CRASH_HP_UPDATE_OTHER\302\240\302\240\302\240\302\240\302\240 1\n" "> > > > > > > \n" "> > > > > > Sourabh Jain's work with PPC uses REMOVE_CPU, REMOVE_MEMORY, and\n" "> > > > > > ADD_MEMORY.\n" @@ -96,4 +116,4 @@ "\n" The consensus is reached, please proceed when it's convenient. -e9d4c223dc22badba298ea0b9577c7ce7d5f012dc28fe9d9c0a161423c168aaf +7cf11298250a3424e062a1d210625b0bb59e3f4af6880e9ea072264d5ed56318
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.