[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[microblaze-uclinux] RE: [PATCH] Microblaze: implement dma-coherent API andrefactorcache flush code.
- To: "John Williams" <john.williams@xxxxxxxxxxxxx>
- Subject: [microblaze-uclinux] RE: [PATCH] Microblaze: implement dma-coherent API andrefactorcache flush code.
- From: "Stephen Neuendorffer" <stephen.neuendorffer@xxxxxxxxxx>
- Date: Mon, 5 May 2008 17:31:07 -0700
- Cc: <arnd@xxxxxxxx>, <linux-arch@xxxxxxxxxxxxxxx>, "John Linn" <linnj@xxxxxxxxxx>, <matthew@xxxxxx>, <will.newton@xxxxxxxxx>, <drepper@xxxxxxxxxx>, <microblaze-uclinux@xxxxxxxxxxxxxx>, <grant.likely@xxxxxxxxxxxx>, "Michal Simek" <monstr@xxxxxxxxx>, <linux-kernel@xxxxxxxxxxxxxxx>
- In-reply-to: <1210032899.5798.179.camel@localhost>
- References: <20080505223706.1236C1C8004E@xxxxxxxxxxxxxxxxxxxxxxx> <1210028237.5798.121.camel@localhost> <20080505231206.922A61C780C0@xxxxxxxxxxxxxxxxxxxxxx> <1210032899.5798.179.camel@localhost>
- Reply-to: microblaze-uclinux@xxxxxxxxxxxxxx
- Sender: owner-microblaze-uclinux@xxxxxxxxxxxxxx
- Thread-index: AcivDls/AtY1noHiSgCdVEtvoNvaEgAAe4pw
- Thread-topic: [PATCH] Microblaze: implement dma-coherent API andrefactorcache flush code.
> -----Original Message-----
> From: John Williams [mailto:john.williams@xxxxxxxxxxxxx]
> Sent: Monday, May 05, 2008 5:15 PM
> To: Stephen Neuendorffer
> Cc: arnd@xxxxxxxx; linux-arch@xxxxxxxxxxxxxxx; John Linn;
matthew@xxxxxx; will.newton@xxxxxxxxx;
> drepper@xxxxxxxxxx; microblaze-uclinux@xxxxxxxxxxxxxx;
grant.likely@xxxxxxxxxxxx; Michal Simek;
> linux-kernel@xxxxxxxxxxxxxxx
> Subject: RE: [PATCH] Microblaze: implement dma-coherent API
andrefactorcache flush code.
>
>
> > > Does the DMA API insist upon the dma_cache_sync call to guarantee
> > > sensible results? If so, your implementation looks fine. If not,
> > then
> > > the results will clearly be bogus as there's nothing magical about
the
> > > memory being allocated in dma_alloc.
> >
> > Yes, in fact this is one of the keys to getting the lltemac driver
to
> > work right. see Documentation/DMA-API.txt
> >
> > > To that end, can it just call kmalloc(), and similarly kfree() for
> > > dma_free?
> >
> > My understanding is that on other architectures (x86, for instance)
> > 'dma' memory ensures other things, like it's accessible in PCI
memory
> > space. On microblaze, there's nothing really special about dma
memory,
> > but you get the API as a chunk.
>
> Sure - what I meant is can dma_alloc just call kmalloc to do it's
work?
I scanned through Linux Device Drivers, and it appears that calling
get_free_pages is:
1) more efficient than kmalloc for large allocations
2) allocates physically contiguous memory, which kmalloc doesn't
necessarily do if there's an mmu.
Steve
___________________________
microblaze-uclinux mailing list
microblaze-uclinux@xxxxxxxxxxxxxx
Project Home Page : http://www.itee.uq.edu.au/~jwilliams/mblaze-uclinux
Mailing List Archive : http://www.itee.uq.edu.au/~listarch/microblaze-uclinux/