From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UKL0G-00089o-7G for kexec@lists.infradead.org; Tue, 26 Mar 2013 03:56:53 +0000 Message-ID: <51511C6A.1000001@redhat.com> Date: Tue, 26 Mar 2013 11:56:26 +0800 From: WANG Chao MIME-Version: 1.0 Subject: Re: [PATCH 1/1 v1] the recommended crash memory reservation is too small for x86_64. References: <5150350A.5050002@redhat.com> <1364209009-11592-1-git-send-email-zhouzhouyi@gmail.com> <1cd75ef.b411.13da1646593.Coremail.yizhouzhou@ict.ac.cn> In-Reply-To: <1cd75ef.b411.13da1646593.Coremail.yizhouzhou@ict.ac.cn> 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: =?UTF-8?B?5ZGo5rSy5Luq?= Cc: zhouzhouyi@gmail.com, kexec@lists.infradead.org, linux-kernel@vger.kernel.org, vgoyal@redhat.com, linux-doc@vger.kernel.org T24gMDMvMjUvMjAxMyAwNzo1MiBQTSwg5ZGo5rSy5LuqIHdyb3RlOgo+IFRoYW5rcyBmb3IgcmV2 aWV3aW5nCj4gICBJcyBpdCBzb3VuZHMgZ29vZCB0byBhZGQgZm9sbG93aW5nIGxpbmUgaW50byBz ZWN0aW9uICJCb290IGludG8gU3lzdGVtIEtlcm5lbCI6Cj4gCj4gICBUaGUgbWVtb3J5IHJlc2Vy dmVkIGZvciBjcmFzaCBrZXJuZWwgc2hvdWxkIGJlIG5vIGxlc3MgdGhhbiB0aGUgdW5wYWNrZWQg aW5pdCByYW0gZGlzayBzaXplCj4gdGhhdCBsb2FkZWQgd2l0aCBkdW1wLWNhcHR1cmUga2VybmVs IHBsdXMgd2lyZWQgbWVtb3J5IHVzZWQgYnkga2VybmVsIGl0c2VsZi4KClNvcnJ5LCB0aGF0IGlz IGp1c3QgYmFzaWMga25vd2xlZGdlIGZvciBldmVyeW9uZS4KCj4gCj4gICAjIyMjIyMjIyMjIyMj IyMjIwo+ICAgQWZ0ZXIgYWxsLCBJIGhhdmUgYmVlbiB0cmFwcGVkIGludG8gImhhbmcgYWZ0ZXIg Y2FwdHVyZSIgcHJvYmxlbSB3aGVuIHN0aWNraW5nIHN0cmljdGx5IHRvIHRoaXMgZG9jdW1lbnQu Cj4gQ2hlZXJzCj4gWmhvdXlpICAKPiAKPj4gIAo+PiAgT24gMDMvMjUvMjAxMyAwNjo1NiBQTSwg emhvdXpob3V5aUBnbWFpbC5jb20gd3JvdGU6Cj4+ICA+IEZyb206IHJvb3QgPHJvb3RAenp5LUxl bm92by4obm9uZSk+Cj4+ICA+IAo+PiAgPiAKPj4gID4gIE9uIERvY3VtZW50YXRpb24va2R1bXAv a2R1bXAudHh0LCBzZWN0aW9uIEJvb3QgaW50byAgU3lzdGVtIEtlcm5lbDogT24geDg2IGFuZCB4 ODZfNjQsIHVzZQo+PiAgPiAgImNyYXNoa2VybmVsPTY0TUAxNk0iLCBidXQgc29tZSBPU2VzIGxp a2UgdWJ1bnR1IDEyLjEwIHVzZSByYW0gZnMgbGFyZ2VyIHRoYW4gNjRNLCBzbyBpbiB0aGVzZSBj YXNlcyB0aGUKPj4gID4gIG1lbW9yeSByZXNlcnZlZCBmb3IgY3Jhc2hrZXJuZWwgc2hvdWxkIGJl IGF0IGxlYXN0IDEyOE0uCj4+ICAKPj4gIFBlb3BsZSB1c2UgZGlmZmVyZW50IGluaXRyYW1mcyBn ZW5lcmF0b3JzIGZvciBkaWZmZXJlbnQgcHVycG9zZS4gVGhhdCBtZWFucwo+PiAgdGhlIHNpemUg b2YgaW5pdHJhbWZzIGFuZCBhbHNvIGl0cyBtZW1vcnkgY29uc3VtaW5nIGNhbiB2YXJ5IHZlcnkg bXVjaCBmcm9tCj4+ICBlYWNoIG90aGVyLiBZb3UganVzdCBjYW4ndCBsaXN0IGFsbCB0aGVzZSBn ZW5lcmF0b3JzIGFuZCB0aGVpciByZWNvbW1lbmRlZAo+PiAgcmVzZXJ2ZWQgbWVtb3J5IGhlcmUu IFRob3VnaCBJIGhhdmUgdG8gc2F5IGNyYXNoa2VybmVsPTEyOE0gaXMgZ29vZCBjaG9pY2UuCj4+ ICAKPj4gIEkgdGhpbmsgaXQgd291bGQgYmUgYmV0dGVyIHRvIGxlYXZlIHRoaXMgdG8gdXNlciBv ciBkaXN0cmlidXRpb24gaXRzZWxmIHRvCj4+ICBkZXRlcm1pbmUgaG93IG11Y2ggbWVtb3J5IHNo b3VsZCBiZSByZXNlcnZlZCBmb3IgY3Jhc2gga2VybmVsLCB0aGVuIGV4cG9ydAo+PiAgdGhpcyB2 YWx1ZSB0byBrZXJuZWwgaW4gc29tZSB3YXlzLgo+PiAgCj4+ICBUaGFua3MsCj4+ICBXQU5HIENo YW8KPj4gIAo+PiAgPiAKPj4gID4gCj4+ICA+IFNpZ25lZC1vZmYtYnk6IFpob3V5aSBaaG91IDx5 aXpob3V6aG91QGljdC5hYy5jbj4KPj4gID4gLS0tCj4+ICA+ICBEb2N1bWVudGF0aW9uL2tkdW1w L2tkdW1wLnR4dCB8ICAgIDQgKysrLQo+PiAgPiAgMSBmaWxlIGNoYW5nZWQsIDMgaW5zZXJ0aW9u cygrKSwgMSBkZWxldGlvbigtKQo+PiAgPiAKPj4gID4gZGlmZiAtLWdpdCBhL0RvY3VtZW50YXRp b24va2R1bXAva2R1bXAudHh0IGIvRG9jdW1lbnRhdGlvbi9rZHVtcC9rZHVtcC50eHQKPj4gID4g aW5kZXggMTNmMWFhMC4uMWU4NTBlMCAxMDA2NDQKPj4gID4gLS0tIGEvRG9jdW1lbnRhdGlvbi9r ZHVtcC9rZHVtcC50eHQKPj4gID4gKysrIGIvRG9jdW1lbnRhdGlvbi9rZHVtcC9rZHVtcC50eHQK Pj4gID4gQEAgLTI5MCw3ICsyOTAsOSBAQCBCb290IGludG8gU3lzdGVtIEtlcm5lbAo+PiAgPiAg ICAgImNyYXNoa2VybmVsPTY0TUAxNk0iIHRlbGxzIHRoZSBzeXN0ZW0ga2VybmVsIHRvIHJlc2Vy dmUgNjQgTUIgb2YgbWVtb3J5Cj4+ICA+ICAgICBzdGFydGluZyBhdCBwaHlzaWNhbCBhZGRyZXNz IDB4MDEwMDAwMDAgKDE2TUIpIGZvciB0aGUgZHVtcC1jYXB0dXJlIGtlcm5lbC4KPj4gID4gIAo+ PiAgPiAtICAgT24geDg2IGFuZCB4ODZfNjQsIHVzZSAiY3Jhc2hrZXJuZWw9NjRNQDE2TSIuCj4+ ICA+ICsgICBPbiB4ODYgYW5kIHg4Nl82NCwgdXNlICJjcmFzaGtlcm5lbD02NE1AMTZNIiAoc29t ZSBPU2VzIHVzZSBpbml0IHJhbSBmcyBsYXJnZXIKPj4gID4gK3RoYW4gNjRNLCBmb3IgZXhhbXBs ZSB1YnVudHUtMTIuMTAsIHVzZSBjcmFzaGtlcm5lbD0xMjhNQDE2TSBpbnN0ZWFkLCBvciBkdW1w LWNhcHR1cmUKPj4gID4gK2tlcm5lbCB3aWxsIG91dCBvZiBtZW1vcnkpLgo+PiAgPiAgCj4+ICA+ ICAgICBPbiBwcGM2NCwgdXNlICJjcmFzaGtlcm5lbD0xMjhNQDMyTSIuCj4+ICA+ICAKPj4gID4g Cj4+ICAKPiAKPiAKPiAKPiAKCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwprZXhlYyBtYWlsaW5nIGxpc3QKa2V4ZWNAbGlzdHMuaW5mcmFkZWFkLm9yZwpo dHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2tleGVjCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759361Ab3CZD44 (ORCPT ); Mon, 25 Mar 2013 23:56:56 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42585 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753969Ab3CZD4y (ORCPT ); Mon, 25 Mar 2013 23:56:54 -0400 Message-ID: <51511C6A.1000001@redhat.com> Date: Tue, 26 Mar 2013 11:56:26 +0800 From: WANG Chao User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130311 Thunderbird/17.0.4 MIME-Version: 1.0 To: =?UTF-8?B?5ZGo5rSy5Luq?= CC: zhouzhouyi@gmail.com, kexec@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, vgoyal@redhat.com Subject: Re: [PATCH 1/1 v1] the recommended crash memory reservation is too small for x86_64. References: <5150350A.5050002@redhat.com> <1364209009-11592-1-git-send-email-zhouzhouyi@gmail.com> <1cd75ef.b411.13da1646593.Coremail.yizhouzhou@ict.ac.cn> In-Reply-To: <1cd75ef.b411.13da1646593.Coremail.yizhouzhou@ict.ac.cn> 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 03/25/2013 07:52 PM, 周洲仪 wrote: > Thanks for reviewing > Is it sounds good to add following line into section "Boot into System Kernel": > > The memory reserved for crash kernel should be no less than the unpacked init ram disk size > that loaded with dump-capture kernel plus wired memory used by kernel itself. Sorry, that is just basic knowledge for everyone. > > ################# > After all, I have been trapped into "hang after capture" problem when sticking strictly to this document. > Cheers > Zhouyi > >> >> On 03/25/2013 06:56 PM, zhouzhouyi@gmail.com wrote: >> > From: root >> > >> > >> > On Documentation/kdump/kdump.txt, section Boot into System Kernel: On x86 and x86_64, use >> > "crashkernel=64M@16M", but some OSes like ubuntu 12.10 use ram fs larger than 64M, so in these cases the >> > memory reserved for crashkernel should be at least 128M. >> >> People use different initramfs generators for different purpose. That means >> the size of initramfs and also its memory consuming can vary very much from >> each other. You just can't list all these generators and their recommended >> reserved memory here. Though I have to say crashkernel=128M is good choice. >> >> I think it would be better to leave this to user or distribution itself to >> determine how much memory should be reserved for crash kernel, then export >> this value to kernel in some ways. >> >> Thanks, >> WANG Chao >> >> > >> > >> > Signed-off-by: Zhouyi Zhou >> > --- >> > Documentation/kdump/kdump.txt | 4 +++- >> > 1 file changed, 3 insertions(+), 1 deletion(-) >> > >> > diff --git a/Documentation/kdump/kdump.txt b/Documentation/kdump/kdump.txt >> > index 13f1aa0..1e850e0 100644 >> > --- a/Documentation/kdump/kdump.txt >> > +++ b/Documentation/kdump/kdump.txt >> > @@ -290,7 +290,9 @@ Boot into System Kernel >> > "crashkernel=64M@16M" tells the system kernel to reserve 64 MB of memory >> > starting at physical address 0x01000000 (16MB) for the dump-capture kernel. >> > >> > - On x86 and x86_64, use "crashkernel=64M@16M". >> > + On x86 and x86_64, use "crashkernel=64M@16M" (some OSes use init ram fs larger >> > +than 64M, for example ubuntu-12.10, use crashkernel=128M@16M instead, or dump-capture >> > +kernel will out of memory). >> > >> > On ppc64, use "crashkernel=128M@32M". >> > >> > >> > > > >