From: Nick Piggin <nickpiggin@yahoo.com.au>
To: Takashi Iwai <tiwai@suse.de>
Cc: Russell King <rmk+lkml@arm.linux.org.uk>,
Marc Singer <elf@buici.com>,
Linux-Kernel <linux-kernel@vger.kernel.org>
Subject: Re: DMA memory, split_page, BUG_ON(PageCompound()), sound
Date: Thu, 13 Jul 2006 23:38:17 +1000 [thread overview]
Message-ID: <44B64CC9.7070307@yahoo.com.au> (raw)
In-Reply-To: <s5hu05lvdgw.wl%tiwai@suse.de>
Takashi Iwai wrote:
> At Wed, 12 Jul 2006 11:32:41 +0100,
>>In which case should ALSA be passing __GFP_COMP to the dma allocator ?
Yes, sorry I didn't answer your question: ALSA basically wants to be
able to have that memory mmapable by userspace programs, and it doesn't
care whether this is via compound pages or split pages.
> I would be willing to remove __GFP_COMP if it's not needed :)
> Passing this flag is really confusing. The driver doesn't use the
> compound pages at all but it was added just to enable mmap support.
> So, Nick's proposal appears reasonable to me.
I don't think it would be hard. Just need a bit of thinking about how
to go about composing some sort of GFP_USERMAP that can easily be
handled by all allocators, without costing performance or being too
intrusive.
I suspect I won't have time to wade into such an exercise until after
OLS. But I could give a quick review for anyone who does ;)
--
SUSE Labs, Novell Inc.
Send instant messages to your online friends http://au.messenger.yahoo.com
prev parent reply other threads:[~2006-07-13 20:22 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-09 0:07 DMA memory, split_page, BUG_ON(PageCompound()), sound Marc Singer
2006-07-09 3:26 ` Nick Piggin
2006-07-10 2:51 ` Marc Singer
2006-07-10 6:59 ` Nick Piggin
2006-07-10 16:26 ` Russell King
2006-07-10 17:34 ` Nick Piggin
2006-07-10 22:27 ` Marc Singer
2006-07-11 2:51 ` Marc Singer
2006-07-12 10:32 ` Russell King
2006-07-13 13:30 ` Takashi Iwai
2006-07-13 13:38 ` Nick Piggin [this message]
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=44B64CC9.7070307@yahoo.com.au \
--to=nickpiggin@yahoo.com.au \
--cc=elf@buici.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rmk+lkml@arm.linux.org.uk \
--cc=tiwai@suse.de \
/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.