From: Borislav Petkov <bp@alien8.de>
To: kexec@lists.infradead.org
Subject: [PATCH v7 2/8] x86/crash: Introduce new options to support cpu and memory hotplug
Date: Thu, 14 Apr 2022 15:59:19 +0200 [thread overview]
Message-ID: <Ylgot+LUDQl+G/5N@zn.tnic> (raw)
In-Reply-To: <20220413164237.20845-3-eric.devolder@oracle.com>
On Wed, Apr 13, 2022 at 12:42:31PM -0400, Eric DeVolder wrote:
> CRASH_HOTPLUG is to enable cpu and memory hotplug support of crash.
What for?
Why don't you put all that new code you're adding under an
MEMORY_HOTPLUG ifdef? It seems you would need to do that when memory
hotplug is enabled, anyway.
Also, looking further into your patchset, you have ugly ifdeffery.
Instead of that, pls add stubs for the !MEMORY_HOTPLUG case so that
everything is abstracted away in the headers.
Thx.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
next prev parent reply other threads:[~2022-04-14 13:59 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-13 16:42 [PATCH v7 0/8] crash: Kernel handling of CPU and memory hot un/plug Eric DeVolder
2022-04-13 16:42 ` [PATCH v7 1/8] x86/crash: fix minor typo/bug in debug message Eric DeVolder
2022-04-28 9:49 ` David Hildenbrand
2022-04-13 16:42 ` [PATCH v7 2/8] x86/crash: Introduce new options to support cpu and memory hotplug Eric DeVolder
2022-04-14 13:59 ` Borislav Petkov [this message]
2022-04-18 22:03 ` Eric DeVolder
2022-04-19 10:32 ` Borislav Petkov
2022-04-19 21:58 ` Eric DeVolder
2022-04-25 19:25 ` Borislav Petkov
2022-04-26 20:08 ` Eric DeVolder
2022-04-27 10:48 ` Borislav Petkov
2022-04-26 4:21 ` Sourabh Jain
2022-04-26 14:39 ` Eric DeVolder
2022-04-29 6:41 ` Sourabh Jain
2022-05-05 16:31 ` Eric DeVolder
2022-04-13 16:42 ` [PATCH v7 3/8] crash: prototype change for crash_prepare_elf64_headers Eric DeVolder
2022-04-13 16:42 ` [PATCH v7 4/8] crash: add generic infrastructure for crash hotplug support Eric DeVolder
2022-04-14 2:45 ` Baoquan He
2022-04-14 22:29 ` Eric DeVolder
2022-04-18 3:17 ` Baoquan He
2022-04-26 4:06 ` Sourabh Jain
2022-04-26 5:22 ` Baoquan He
2022-04-28 5:18 ` Sourabh Jain
2022-05-04 18:11 ` Eric DeVolder
2022-05-05 5:45 ` Baoquan He
2022-05-05 9:59 ` Sourabh Jain
2022-05-05 11:04 ` Baoquan He
2022-05-05 13:35 ` Eric DeVolder
2022-04-13 16:42 ` [PATCH v7 5/8] kexec: exclude elfcorehdr from the segment digest Eric DeVolder
2022-04-13 16:42 ` [PATCH v7 6/8] kexec: exclude hot remove cpu from elfcorehdr notes Eric DeVolder
2022-04-14 2:48 ` Baoquan He
2022-04-14 22:31 ` Eric DeVolder
2022-04-13 16:42 ` [PATCH v7 7/8] x86/crash: Add x86 crash hotplug support for kexec_file_load Eric DeVolder
2022-04-14 2:52 ` Baoquan He
2022-04-14 22:32 ` Eric DeVolder
2022-04-13 16:42 ` [PATCH v7 8/8] x86/crash: Add x86 crash hotplug support for kexec_load Eric DeVolder
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Ylgot+LUDQl+G/5N@zn.tnic \
--to=bp@alien8.de \
--cc=kexec@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox