linux-kernel - Re: [PATCH v3 1/2] Documentation: kdump: remind user of nr

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160803012341.GA3637@x1.redhat.com>
Date:	Wed, 3 Aug 2016 09:23:41 +0800
From:	Baoquan He <bhe@...hat.com>
To:	Zhou Wenjian <zhouwj-fnst@...fujitsu.com>,
	akpm@...ux-foundation.org
Cc:	linux-kernel@...r.kernel.org, kexec@...ts.infradead.org,
	dyoung@...hat.com, d.hatayama@...fujitsu.com
Subject: Re: [PATCH v3 1/2] Documentation: kdump: remind user of nr_cpus

On 08/03/16 at 08:59am, Zhou Wenjian wrote:
> v2->v3: add description of nr_cpus.
> v1->v2: change nr_cpus to maxcpus
> 
> nr_cpus can help to save memory. So we should remind user of it.
> 
> Signed-off-by: Zhou Wenjian <zhouwj-fnst@...fujitsu.com>
> ---
>  Documentation/kdump/kdump.txt | 2 ++
>  1 file changed, 2 insertions(+)

Looks good to me! Thanks, Wenjian.

Ack this series
Acked-by: Baoquan He <bhe@...hat.com>


Hi Andrew,

Could you please help merge this patchset?

Thanks
Baoquan

> 
> diff --git a/Documentation/kdump/kdump.txt b/Documentation/kdump/kdump.txt
> index 88ff63d..4aa194e 100644
> --- a/Documentation/kdump/kdump.txt
> +++ b/Documentation/kdump/kdump.txt
> @@ -393,6 +393,8 @@ Notes on loading the dump-capture kernel:
>  * We generally don' have to bring up a SMP kernel just to capture the
>    dump. Hence generally it is useful either to build a UP dump-capture
>    kernel or specify maxcpus=1 option while loading dump-capture kernel.
> +  Note, though maxcpus always works, we should replace it by nr_cpus to
> +  save memory if supported by the current ARCH, such as x86. 
>  
>  * For s390x there are two kdump modes: If a ELF header is specified with
>    the elfcorehdr= kernel parameter, it is used by the kdump kernel as it
> -- 
> 1.8.3.1
> 
> 
>