From: John Blackwood <john.blackwood@ccur.com>
To: Vivek Goyal <vgoyal@redhat.com>
Cc: Tejun Heo <tj@kernel.org>,
"kexec@lists.infradead.org" <kexec@lists.infradead.org>
Subject: Re: Kdump issue with percpu_alloc=lpage
Date: Thu, 19 Nov 2009 09:20:16 -0600 [thread overview]
Message-ID: <4B056230.6000006@ccur.com> (raw)
In-Reply-To: <20091119150512.GC8741@redhat.com>
Vivek Goyal wrote:
> On Thu, Nov 19, 2009 at 11:45:25PM +0900, Tejun Heo wrote:
>> Hello,
>>
>> 11/19/2009 11:33 PM, Vivek Goyal wrote:
>>> I did load a kdump kernel on 32-rc7 and it worked fine. But I guess in
>>> this case memory might have come from linearly mapped region.
>>>
>>> If the default per cpu allocator can get memory from vmalloc region
>>> also, then I think we will need this function which can map virtual
>>> address to physical address.
>> I see.
>>
>>> Are there multiple allocators now? If yes, what are the command line
>>> options and I can try to use some other allocator and see if I can force
>>> the condition where memory comes from vmalloc region and I observe the
>>> crash.
>>>
>>> Once I can reproduce it, I can also send you the fix you suggested.
>> Now there are two allocators - embed (default) and page. You can
>> choose using percpu_alloc= parameter. Embed allocator will put the
>> first chunk in linear mapping area while page will put the first chunk
>> in vmalloc area too but regardless of the allocator from the second
>> chunk it will always be in the vmalloc area. So, either using
>> percpu_alloc=page or allocating some amount of percpu memory using
>> __alloc_percpu() - a thousand 4k blocks will always be enough - should
>> do it.
>
> Looks like with percpu_alloc=page, I am getting bogus physical addresses in
> /sys/devices/system/cpu<N>/crash_notes.
>
> With percpu_alloc=page
>
> cpu0 60fffff49800
> cpu1 60fffff60800
> cpu2 60fffff77800
>
> with percpu_alloc=embed
>
> cpu0 28219800
> cpu1 28259800
> cpu2 28299800
>
> I got 4G of RAM in my box. "embed" seems to be fine.
That sounds right to me. In previous kernels, 'embed' was o.k. also.
Thanks for looking into this Vivek.
I've been too busy right now to try this out.
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
next prev parent reply other threads:[~2009-11-19 15:20 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-28 21:22 Kdump issue with percpu_alloc=lpage John Blackwood
2009-11-18 9:25 ` Tejun Heo
2009-11-19 14:33 ` Vivek Goyal
2009-11-19 14:45 ` Tejun Heo
2009-11-19 15:05 ` Vivek Goyal
2009-11-19 15:20 ` John Blackwood [this message]
2009-11-19 16:23 ` Vivek Goyal
2009-11-19 16:31 ` Tejun Heo
2009-11-19 16:36 ` Vivek Goyal
-- strict thread matches above, loose matches on Subject: below --
2009-10-27 19:16 John Blackwood
2009-10-27 19:24 ` Vivek Goyal
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=4B056230.6000006@ccur.com \
--to=john.blackwood@ccur.com \
--cc=kexec@lists.infradead.org \
--cc=tj@kernel.org \
--cc=vgoyal@redhat.com \
/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