From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932235AbcHSP7D (ORCPT ); Fri, 19 Aug 2016 11:59:03 -0400 Received: from tex.lwn.net ([70.33.254.29]:37277 "EHLO vena.lwn.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750889AbcHSP7B (ORCPT ); Fri, 19 Aug 2016 11:59:01 -0400 Date: Fri, 19 Aug 2016 09:57:40 -0600 From: Jonathan Corbet To: "Zhou, =?UTF-8?B?V2Vuamlhbi/lkajmlofliZE=?=" Cc: , , , , , , , , Subject: Re: [PATCH v9 1/2] Documentation: kdump: remind user of nr_cpus Message-ID: <20160819095740.1cccc073@lwn.net> In-Reply-To: <57B653D1.8060106@cn.fujitsu.com> References: <1471489907-27737-1-git-send-email-zhouwj-fnst@cn.fujitsu.com> <1471489907-27737-2-git-send-email-zhouwj-fnst@cn.fujitsu.com> <20160818111854.362bb972@lwn.net> <57B653D1.8060106@cn.fujitsu.com> Organization: LWN.net MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 19 Aug 2016 08:33:21 +0800 "Zhou, Wenjian/周文剑" wrote: > I was also confused by maxcpus and nr_cpus before writing this patch. > I think it is a good choice to describe it in kernel-parameters.txt. > > Then, only two things need to be done I think. > One is move the above description to maxcpus= in kernel-parameters.txt. > And the other is replace maxcpus with maxcpus/nr_cpus in kdump.txt. > > How do you think? That is not quite what I had in mind, sorry. What I would really like to see in kernel-parameters.txt is an explanation of how those two parameters differ - what do they do differently and how should a user choose one over the other? What we have now offers no guidance in that matter. I suspect that may be a bit more than you had signed up to do. As an intermediate step, how about this: rather than tacking on those lines in kdump.txt, rewrite that paragraph to simply say what the reader should use. If nr_cpus is good for everybody, just say that, but your previous patch suggests that the situation isn't quite that simple? Thanks, jon From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from tex.lwn.net ([70.33.254.29] helo=vena.lwn.net) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bamBH-00087p-EX for kexec@lists.infradead.org; Fri, 19 Aug 2016 15:58:04 +0000 Date: Fri, 19 Aug 2016 09:57:40 -0600 From: Jonathan Corbet Subject: Re: [PATCH v9 1/2] Documentation: kdump: remind user of nr_cpus Message-ID: <20160819095740.1cccc073@lwn.net> In-Reply-To: <57B653D1.8060106@cn.fujitsu.com> References: <1471489907-27737-1-git-send-email-zhouwj-fnst@cn.fujitsu.com> <1471489907-27737-2-git-send-email-zhouwj-fnst@cn.fujitsu.com> <20160818111854.362bb972@lwn.net> <57B653D1.8060106@cn.fujitsu.com> MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "kexec" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: "Zhou, =?UTF-8?B?V2Vuamlhbi/lkajmlofliZE=?=" Cc: bhe@redhat.com, linux-doc@vger.kernel.org, xlpang@redhat.com, kexec@lists.infradead.org, linux-kernel@vger.kernel.org, joe@perches.com, akpm@linux-foundation.org, dyoung@redhat.com, vgoyal@redhat.com T24gRnJpLCAxOSBBdWcgMjAxNiAwODozMzoyMSArMDgwMAoiWmhvdSwgV2Vuamlhbi/lkajmlofl iZEiIDx6aG91d2otZm5zdEBjbi5mdWppdHN1LmNvbT4gd3JvdGU6Cgo+IEkgd2FzIGFsc28gY29u ZnVzZWQgYnkgbWF4Y3B1cyBhbmQgbnJfY3B1cyBiZWZvcmUgd3JpdGluZyB0aGlzIHBhdGNoLgo+ IEkgdGhpbmsgaXQgaXMgYSBnb29kIGNob2ljZSB0byBkZXNjcmliZSBpdCBpbiBrZXJuZWwtcGFy YW1ldGVycy50eHQuCj4gCj4gVGhlbiwgb25seSB0d28gdGhpbmdzIG5lZWQgdG8gYmUgZG9uZSBJ IHRoaW5rLgo+IE9uZSBpcyBtb3ZlIHRoZSBhYm92ZSBkZXNjcmlwdGlvbiB0byBtYXhjcHVzPSBp biBrZXJuZWwtcGFyYW1ldGVycy50eHQuCj4gQW5kIHRoZSBvdGhlciBpcyByZXBsYWNlIG1heGNw dXMgd2l0aCBtYXhjcHVzL25yX2NwdXMgaW4ga2R1bXAudHh0Lgo+IAo+IEhvdyBkbyB5b3UgdGhp bms/CgpUaGF0IGlzIG5vdCBxdWl0ZSB3aGF0IEkgaGFkIGluIG1pbmQsIHNvcnJ5LiAgV2hhdCBJ IHdvdWxkIHJlYWxseSBsaWtlIHRvCnNlZSBpbiBrZXJuZWwtcGFyYW1ldGVycy50eHQgaXMgYW4g ZXhwbGFuYXRpb24gb2YgaG93IHRob3NlIHR3byBwYXJhbWV0ZXJzCmRpZmZlciAtIHdoYXQgZG8g dGhleSBkbyBkaWZmZXJlbnRseSBhbmQgaG93IHNob3VsZCBhIHVzZXIgY2hvb3NlIG9uZSBvdmVy CnRoZSBvdGhlcj8gIFdoYXQgd2UgaGF2ZSBub3cgb2ZmZXJzIG5vIGd1aWRhbmNlIGluIHRoYXQg bWF0dGVyLgoKSSBzdXNwZWN0IHRoYXQgbWF5IGJlIGEgYml0IG1vcmUgdGhhbiB5b3UgaGFkIHNp Z25lZCB1cCB0byBkby4gIEFzIGFuCmludGVybWVkaWF0ZSBzdGVwLCBob3cgYWJvdXQgdGhpczog cmF0aGVyIHRoYW4gdGFja2luZyBvbiB0aG9zZSBsaW5lcyBpbgprZHVtcC50eHQsIHJld3JpdGUg dGhhdCBwYXJhZ3JhcGggdG8gc2ltcGx5IHNheSB3aGF0IHRoZSByZWFkZXIgc2hvdWxkCnVzZS4g IElmIG5yX2NwdXMgaXMgZ29vZCBmb3IgZXZlcnlib2R5LCBqdXN0IHNheSB0aGF0LCBidXQgeW91 ciBwcmV2aW91cwpwYXRjaCBzdWdnZXN0cyB0aGF0IHRoZSBzaXR1YXRpb24gaXNuJ3QgcXVpdGUg dGhhdCBzaW1wbGU/CgpUaGFua3MsCgpqb24KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fCmtleGVjIG1haWxpbmcgbGlzdAprZXhlY0BsaXN0cy5pbmZyYWRl YWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8va2V4ZWMK