Date: Fri, 6 Mar 2009 05:59:18 +0900
To: joerg.roedel@amd.com
Cc: mingo@redhat.com, iommu@lists.linux-foundation.org,
       linux-kernel@vger.kernel.org
Subject: Re: [RFC][GIT PULL] dma-api debugging facility
From: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
In-Reply-To: <20090305200338.GA31182@amd.com>
References: <20090305200338.GA31182@amd.com>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-Id: <20090306055944Q.fujita.tomonori@lab.ntt.co.jp>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2844
Lines: 62

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/