Re: [RFC][GIT PULL] dma-api debugging facility

[Joerg Roedel <joro@8bytes.org>]

On Fri, Mar 06, 2009 at 05:59:18AM +0900, FUJITA Tomonori wrote:
> On Thu, 5 Mar 2009 21:03:38 +0100
> Joerg Roedel <joerg.roedel@amd.com> wrote:
> 
> > Hi Ingo,
> > 
> > The following changes since commit 0bd5c4f7c874cf48ff7904dcf8a59988c8fea0e8:
> >   Ingo Molnar (1):
> >         Merge branch 'iommu/fixes-2.6.29' of git://git.kernel.org/.../joro/linux-2.6-iommu into core/iommu
> > 
> > are available in the git repository at:
> > 
> >   git://git.kernel.org/pub/scm/linux/kernel/git/joro/linux-2.6-iommu.git dma-api/debug
> > 
> > David Woodhouse (2):
> >       dma-debug: add function to dump dma mappings
> >       dma-debug: print stacktrace of mapping path on unmap error
> > 
> > Joerg Roedel (16):
> >       dma-debug: add Kconfig entry
> >       dma-debug: add header file and core data structures
> >       dma-debug: add hash functions for dma_debug_entries
> >       dma-debug: add allocator code
> >       dma-debug: add initialization code
> >       dma-debug: add kernel command line parameters
> >       dma-debug: add debugfs interface
> >       dma-debug: add core checking functions
> >       dma-debug: add checking for map/unmap_page/single
> >       dma-debug: add add checking for map/unmap_sg
> >       dma-debug: add checking for [alloc|free]_coherent
> >       dma-debug: add checks for sync_single_*
> >       dma-debug: add checks for sync_single_range_*
> >       dma-debug: add checks for sync_single_sg_*
> >       dma-debug: x86 architecture bindings
> >       dma-debug: Documentation update
> > 
> >  Documentation/DMA-API.txt           |  106 +++++
> >  Documentation/kernel-parameters.txt |   10 +
> >  arch/Kconfig                        |    2 +
> >  arch/x86/Kconfig                    |    1 +
> >  arch/x86/include/asm/dma-mapping.h  |   45 ++-
> >  arch/x86/kernel/pci-dma.c           |    6 +
> >  include/linux/dma-debug.h           |  167 +++++++
> >  lib/Kconfig.debug                   |   11 +
> >  lib/Makefile                        |    2 +
> >  lib/dma-debug.c                     |  870 +++++++++++++++++++++++++++++++++++
> >  10 files changed, 1214 insertions(+), 6 deletions(-)
> >  create mode 100644 include/linux/dma-debug.h
> >  create mode 100644 lib/dma-debug.c
> > 
> > A few changes were made since the v3 post of this code:
> > 
> > * new checks were added
> > * stack trace of mapping path is printed
> > * default number of preallocated entries has been increased to 32768 on
> >   x86 - in the latest tests I had more than 26000 dma-api allocations in
> >   the hash
> > * therefore the hash size has been increased to 1024
> 
> You have not posted patches that addressed some reviewed comments. For
> example, dma-debug's dma_map_sg support is just wrong:
> 
> http://marc.info/?l=linux-kernel&m=123556055325566&w=2

Thats fixed in the tree above. I will send out the patches tomorrow
seperatly for easier reviewing and commenting.

Joerg