From: Tejun Heo <tj@kernel.org>
To: David Miller <davem@davemloft.net>
Cc: linux-kernel@vger.kernel.org, rusty@rustcorp.com.au,
cl@linux-foundation.org, mingo@elte.hu, hpa@zytor.com
Subject: Re: [PATCH 2/3] sparc64: implement page mapping percpu first chunk allocator
Date: Tue, 29 Sep 2009 09:14:58 +0900 [thread overview]
Message-ID: <4AC15182.4020603@kernel.org> (raw)
In-Reply-To: <20090928.143656.208349447.davem@davemloft.net>
Hello, David.
David Miller wrote:
> From: Tejun Heo <tj@kernel.org>
> Date: Thu, 24 Sep 2009 21:56:32 +0900
>
>> Implement page mapping percpu first chunk allocator as a fallback to
>> the embedding allocator. The next patch will make the embedding
>> allocator check distances between units to determine whether it fits
>> within the vmalloc area so that this fallback can be used on such
>> cases.
>>
>> sparc64 currently has relatively small vmalloc area which makes it
>> impossible to create any dynamic chunks on certain configurations
>> leading to percpu allocation failures. This and the next patch should
>> allow those configurations to keep working until proper solution is
>> found.
>>
>> Signed-off-by: Tejun Heo <tj@kernel.org>
>> ---
>> David, can you please ack this after reviewing?
>
> Tejun I am testing out the following patch which will make these
> patches of your's basically unnecessary:
Ah... great but unless you object, I think it would be better to push
it out anyway just to make things a bit more robust and ease tracking
and debugging when something goes wrong. The added code is small and
ditched once boot is complete. What do you think?
Thanks.
--
tejun
next prev parent reply other threads:[~2009-09-29 0:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-24 12:55 [PATCH 1/3] percpu: make pcpu_build_alloc_info() clear static buffers Tejun Heo
2009-09-24 12:56 ` [PATCH 2/3] sparc64: implement page mapping percpu first chunk allocator Tejun Heo
2009-09-24 12:57 ` [PATCH 3/3] percpu: make embedding first chunk allocator check vmalloc space size Tejun Heo
2009-09-24 22:51 ` [PATCH 2/3] sparc64: implement page mapping percpu first chunk allocator David Miller
2009-09-28 21:36 ` David Miller
2009-09-29 0:14 ` Tejun Heo [this message]
2009-09-29 0:16 ` David Miller
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=4AC15182.4020603@kernel.org \
--to=tj@kernel.org \
--cc=cl@linux-foundation.org \
--cc=davem@davemloft.net \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=rusty@rustcorp.com.au \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.