[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-iommu
Subject: [PATCH 6/7] x86: cleanup dma_*_coherent functions
From: joerg.roedel () amd ! com (Joerg Roedel)
Date: 2008-08-13 12:51:14
Message-ID: 20080813125114.GJ14993 () amd ! com
[Download RAW message or body]
On Wed, Aug 13, 2008 at 09:45:51AM +0900, FUJITA Tomonori wrote:
> On Tue, 12 Aug 2008 17:24:16 +0200
> Joerg Roedel <joerg.roedel at amd.com> wrote:
>
> > All dma_ops implementations support the alloc_coherent and free_coherent
> > callbacks now. This allows a big simplification of the dma_alloc_coherent
> > function which is done with this patch. The dma_free_coherent functions is also
> > cleaned up and calls now the free_coherent callback of the dma_ops
> > implementation.
> >
> > Signed-off-by: Joerg Roedel <joerg.roedel at amd.com>
> > ---
> > arch/x86/kernel/pci-dma.c | 116 ++++-----------------------------------------
> > 1 files changed, 10 insertions(+), 106 deletions(-)
> >
> > diff --git a/arch/x86/kernel/pci-dma.c b/arch/x86/kernel/pci-dma.c
> > index f704cb5..60fa80d 100644
> > --- a/arch/x86/kernel/pci-dma.c
> > +++ b/arch/x86/kernel/pci-dma.c
>
> How about moving dma_free_coherent and dma_alloc_coherent to
> asm-x86/dma-mapping.h? It would be nice to have all the dma operations
> in one place.
Actually I thought about the other direction. Having these functions not
inlined should not add a big overhead but reduces kernel code size.
Joerg
--
| AMD Saxony Limited Liability Company & Co. KG
Operating | Wilschdorfer Landstr. 101, 01109 Dresden, Germany
System | Register Court Dresden: HRA 4896
Research | General Partner authorized to represent:
Center | AMD Saxony LLC (Wilmington, Delaware, US)
| General Manager of AMD Saxony LLC: Dr. Hans-R. Deppe, Thomas McCoy
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic