All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <ak@suse.de>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Christoph Lameter <clameter@sgi.com>,
	Martin Bligh <mbligh@mbligh.org>,
	akpm@google.com, linux-kernel@vger.kernel.org,
	Christoph Hellwig <hch@infradead.org>,
	James Bottomley <James.Bottomley@steeleye.com>,
	linux-mm@kvack.org
Subject: Re: [RFC] Initial alpha-0 for new page allocator API
Date: Fri, 22 Sep 2006 22:02:41 +0200	[thread overview]
Message-ID: <200609222202.41692.ak@suse.de> (raw)
In-Reply-To: <1158955850.24572.37.camel@localhost.localdomain>

On Friday 22 September 2006 22:10, Alan Cox wrote:
> Ar Gwe, 2006-09-22 am 21:10 +0200, ysgrifennodd Andi Kleen:
> > We already have that scheme. Any existing driver should be already converted
> > away from GFP_DMA towards dma_*/pci_*. dma_* knows all the magic
> > how to get memory for the various ranges. No need to mess up the 
> > main allocator.
> 
> Add an isa_device class and that'll fall into place nicely. isa_alloc_*
> will end up asking for 20bit DMA and it will work nicely.


The old school way is to pass NULL to pci_alloc_coherent()

> > that basically goes through the buddy lists freeing in >O(1) 
> > and does some directed reclaim, but that would likely be a separate
> > path anyways and not need your new structure to impact the O(1)
> > allocator.
> 
> Just search within the candidate 4MB (or whatever it is these days)
> chunks.
> 

What chunks?

> Ok the examples I know about are
> - ESS Maestro series audio - PCI, common on 32bit boxes a few years ago,
> no longer shipped and unlikely to be met on 64bit. Also slow allocations
> is fine.

And is fine with 16MB anyways I think.

> - Some aacraid, mostly only for control structures. Those found on 64bit
> are probably fine with slow alloc.

That is the only case where there are rumours they are not fine with 16MB.

> - Broadcom stuff - not sure if 30 or 31bit, around today and on 64bit

b44 is 30bit. That's true. I even got one here.

But it doesn't count really because we can handle it fine with existing 
16MB GFP_DMA

> - Floppy controller

That one only needs one page or so. In the worst case memory could be preallocated
in .bss for it. 

-Andi

WARNING: multiple messages have this Message-ID (diff)
From: Andi Kleen <ak@suse.de>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Christoph Lameter <clameter@sgi.com>,
	Martin Bligh <mbligh@mbligh.org>,
	akpm@google.com, linux-kernel@vger.kernel.org,
	Christoph Hellwig <hch@infradead.org>,
	James Bottomley <James.Bottomley@steeleye.com>,
	linux-mm@kvack.org
Subject: Re: [RFC] Initial alpha-0 for new page allocator API
Date: Fri, 22 Sep 2006 22:02:41 +0200	[thread overview]
Message-ID: <200609222202.41692.ak@suse.de> (raw)
In-Reply-To: <1158955850.24572.37.camel@localhost.localdomain>

On Friday 22 September 2006 22:10, Alan Cox wrote:
> Ar Gwe, 2006-09-22 am 21:10 +0200, ysgrifennodd Andi Kleen:
> > We already have that scheme. Any existing driver should be already converted
> > away from GFP_DMA towards dma_*/pci_*. dma_* knows all the magic
> > how to get memory for the various ranges. No need to mess up the 
> > main allocator.
> 
> Add an isa_device class and that'll fall into place nicely. isa_alloc_*
> will end up asking for 20bit DMA and it will work nicely.


The old school way is to pass NULL to pci_alloc_coherent()

> > that basically goes through the buddy lists freeing in >O(1) 
> > and does some directed reclaim, but that would likely be a separate
> > path anyways and not need your new structure to impact the O(1)
> > allocator.
> 
> Just search within the candidate 4MB (or whatever it is these days)
> chunks.
> 

What chunks?

> Ok the examples I know about are
> - ESS Maestro series audio - PCI, common on 32bit boxes a few years ago,
> no longer shipped and unlikely to be met on 64bit. Also slow allocations
> is fine.

And is fine with 16MB anyways I think.

> - Some aacraid, mostly only for control structures. Those found on 64bit
> are probably fine with slow alloc.

That is the only case where there are rumours they are not fine with 16MB.

> - Broadcom stuff - not sure if 30 or 31bit, around today and on 64bit

b44 is 30bit. That's true. I even got one here.

But it doesn't count really because we can handle it fine with existing 
16MB GFP_DMA

> - Floppy controller

That one only needs one page or so. In the worst case memory could be preallocated
in .bss for it. 

-Andi

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2006-09-22 20:02 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-09-22  4:02 [RFC] Initial alpha-0 for new page allocator API Christoph Lameter
2006-09-22  4:02 ` Christoph Lameter
2006-09-22  6:17 ` Andi Kleen
2006-09-22  6:17   ` Andi Kleen
2006-09-22 16:35   ` Christoph Lameter
2006-09-22 16:35     ` Christoph Lameter
2006-09-22 19:10     ` Andi Kleen
2006-09-22 19:10       ` Andi Kleen
2006-09-22 19:17       ` Christoph Lameter
2006-09-22 19:17         ` Christoph Lameter
2006-09-22 19:24       ` Martin Bligh
2006-09-22 19:24         ` Martin Bligh
2006-09-22 20:10       ` Alan Cox
2006-09-22 20:10         ` Alan Cox
2006-09-22 20:02         ` Andi Kleen [this message]
2006-09-22 20:02           ` Andi Kleen
2006-09-22 20:14           ` Martin Bligh
2006-09-22 20:14             ` Martin Bligh
2006-09-22 20:23             ` Christoph Lameter
2006-09-22 20:23               ` Christoph Lameter
2006-09-22 20:41               ` Jesse Barnes
2006-09-22 20:41                 ` Jesse Barnes
2006-09-22 21:01                 ` Christoph Lameter
2006-09-22 21:01                   ` Christoph Lameter
2006-09-22 21:14                   ` Jesse Barnes
2006-09-22 21:14                     ` Jesse Barnes
2006-09-22 21:21                     ` Christoph Lameter
2006-09-22 21:21                       ` Christoph Lameter
2006-09-22 20:48               ` Andi Kleen
2006-09-22 20:48                 ` Andi Kleen
2006-09-22 21:13                 ` Christoph Lameter
2006-09-22 21:13                   ` Christoph Lameter
2006-09-22 21:32                 ` Christoph Lameter
2006-09-22 21:32                   ` Christoph Lameter
2006-09-22 23:34               ` More thoughts on getting rid of ZONE_DMA Andi Kleen
2006-09-22 23:34                 ` Andi Kleen
2006-09-23  0:23                 ` Christoph Lameter
2006-09-23  0:23                   ` Christoph Lameter
2006-09-23  0:39                   ` Andi Kleen
2006-09-23  0:39                     ` Andi Kleen
2006-09-23  0:25                 ` Christoph Lameter
2006-09-23  0:25                   ` Christoph Lameter
2006-09-23  0:37                   ` Andi Kleen
2006-09-23  0:37                     ` Andi Kleen
2006-09-24  2:13                 ` Christoph Lameter
2006-09-24  2:13                   ` Christoph Lameter
2006-09-24  2:36                   ` Martin J. Bligh
2006-09-24  2:36                     ` Martin J. Bligh
2006-09-24  7:26                     ` Andi Kleen
2006-09-24  7:26                       ` Andi Kleen
2006-09-24  7:19                   ` Andi Kleen
2006-09-24  7:19                     ` Andi Kleen
2006-09-22 17:36   ` [RFC] Initial alpha-0 for new page allocator API Christoph Lameter
2006-09-22 17:36     ` Christoph Lameter

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=200609222202.41692.ak@suse.de \
    --to=ak@suse.de \
    --cc=James.Bottomley@steeleye.com \
    --cc=akpm@google.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=clameter@sgi.com \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mbligh@mbligh.org \
    /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.