Re: [PATCH v10 8/8] x86/crash: Add x86 crash hotplug support

2022-08-25 Thread Baoquan He
On 08/16/22 at 10:23am, Eric DeVolder wrote: > > > On 8/12/22 19:34, Baoquan He wrote: > > On 07/21/22 at 02:17pm, Eric DeVolder wrote: > > ...snip > > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > > > index e58798f636d4..bb59596c8bea 100644 > > > --- a/arch/x86/Kconfig > > > +++

Re: [PATCH RFC 1/2] coding-style.rst: document BUG() and WARN() rules ("do not crash the kernel")

2022-08-25 Thread Dave Young
Hi David, [Added more people in cc] On Thu, 25 Aug 2022 at 20:13, David Hildenbrand wrote: > > On 24.08.22 23:59, John Hubbard wrote: > > On 8/24/22 09:30, David Hildenbrand wrote: > >> diff --git a/Documentation/process/coding-style.rst > >> b/Documentation/process/coding-style.rst > >> index

Re: [PATCH] vmcoreinfo: add kallsyms_num_syms symbol

2022-08-25 Thread Andrew Morton
On Thu, 25 Aug 2022 10:27:04 -0700 Stephen Brennan wrote: > I just wanted to check on the status of this. I got your email about it > being added to mm-hotfixes-unstable, and I do still see it in the quilt > patches set: > >

Re: [PATCH v10 8/8] x86/crash: Add x86 crash hotplug support

2022-08-25 Thread Eric DeVolder
Hi Baoquan, I've v11 ready to go, but I did raise some questions below which would be good to resolve before posting. Thanks! eric On 8/16/22 10:23, Eric DeVolder wrote: On 8/12/22 19:34, Baoquan He wrote: On 07/21/22 at 02:17pm, Eric DeVolder wrote: ...snip diff --git

Re: [PATCH] vmcoreinfo: add kallsyms_num_syms symbol

2022-08-25 Thread Stephen Brennan
Hi Andrew, I just wanted to check on the status of this. I got your email about it being added to mm-hotfixes-unstable, and I do still see it in the quilt patches set: https://git.kernel.org/pub/scm/linux/kernel/git/akpm/25-new.git/tree/patches/vmcoreinfo-add-kallsyms_num_syms-symbol.patch

Re: [PATCH RFC 1/2] coding-style.rst: document BUG() and WARN() rules ("do not crash the kernel")

2022-08-25 Thread David Hildenbrand
On 24.08.22 23:59, John Hubbard wrote: > On 8/24/22 09:30, David Hildenbrand wrote: >> diff --git a/Documentation/process/coding-style.rst >> b/Documentation/process/coding-style.rst >> index 03eb53fd029a..a6d81ff578fe 100644 >> --- a/Documentation/process/coding-style.rst >> +++

Re: [PATCH RFC 2/2] checkpatch: warn on usage of VM_BUG_ON() and friends

2022-08-25 Thread David Hildenbrand
On 25.08.22 13:43, Jani Nikula wrote: > On Thu, 25 Aug 2022, David Hildenbrand wrote: >> On 24.08.22 18:52, Joe Perches wrote: >>> On Wed, 2022-08-24 at 18:31 +0200, David Hildenbrand wrote: checkpatch does not point out that VM_BUG_ON() and friends should be avoided, however, Linus

Re: [PATCH RFC 2/2] checkpatch: warn on usage of VM_BUG_ON() and friends

2022-08-25 Thread Jani Nikula
On Thu, 25 Aug 2022, David Hildenbrand wrote: > On 24.08.22 18:52, Joe Perches wrote: >> On Wed, 2022-08-24 at 18:31 +0200, David Hildenbrand wrote: >>> checkpatch does not point out that VM_BUG_ON() and friends should be >>> avoided, however, Linus notes: >>> >>> VM_BUG_ON() has the exact

Re: [PATCH RFC 2/2] checkpatch: warn on usage of VM_BUG_ON() and friends

2022-08-25 Thread David Hildenbrand
On 24.08.22 18:52, Joe Perches wrote: > On Wed, 2022-08-24 at 18:31 +0200, David Hildenbrand wrote: >> checkpatch does not point out that VM_BUG_ON() and friends should be >> avoided, however, Linus notes: >> >> VM_BUG_ON() has the exact same semantics as BUG_ON. It is literally >> no

Re: [PATCH 0/5] arm64/mm: remap crash kernel with base pages even if rodata_full disabled

2022-08-25 Thread Mike Rapoport
Hi Baoquan, On Thu, Aug 25, 2022 at 03:35:04PM +0800, Baoquan He wrote: > Add kexec list in CC > > On 08/19/22 at 07:11am, Mike Rapoport wrote: > > From: Mike Rapoport > > > > Hi, > > > > There were several rounds of discussion how to remap with base pages only > > the crash kernel area, the

Re: [PATCH 0/5] arm64/mm: remap crash kernel with base pages even if rodata_full disabled

2022-08-25 Thread Baoquan He
Add kexec list in CC On 08/19/22 at 07:11am, Mike Rapoport wrote: > From: Mike Rapoport > > Hi, > > There were several rounds of discussion how to remap with base pages only > the crash kernel area, the latest one here: > >