Re: [v2 1/1] iommu/tegra: smmu: debugfs for TLB/PTC statistics

2012-08-03 Thread Joerg Roedel
On Thu, Aug 02, 2012 at 11:46:40AM +0300, Hiroshi Doyu wrote: > Add debugfs entries to collect TLB/PTC statistics. > > $ echo "reset" > /sys/kernel/debug/smmu/mc/{tlb,ptc} > $ echo "on" > /sys/kernel/debug/smmu/mc/{tlb,ptc} > $ echo "off" > /sys/kernel/debug/smmu/mc/{tlb,ptc} > $ cat /sys/

Re: [PATCH 2/2] iommu: Include linux/notifier.h

2012-08-03 Thread Thierry Reding
On Fri, Aug 03, 2012 at 04:08:26PM +0200, Joerg Roedel wrote: > On Wed, Jul 25, 2012 at 04:24:50PM +0200, Thierry Reding wrote: > > The linux/iommu.h header uses types defined in linux/notifier.h but > > doesn't include it. > > > > Signed-off-by: Thierry Reding > > Better than adding the header-

Re: [PATCH 1/2] iommu/tegra: smmu: Cleanup with lesser nest

2012-08-03 Thread Joerg Roedel
On Mon, Jul 30, 2012 at 08:39:18AM +0300, Hiroshi Doyu wrote: > Small clean up with lesser nest for readability. > > Signed-off-by: Hiroshi Doyu Applied both to iommu/fixes, thanks Hiroshi. -- AMD Operating System Research Center Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach Gener

Re: [PATCH 2/2] iommu: Include linux/notifier.h

2012-08-03 Thread Joerg Roedel
On Wed, Jul 25, 2012 at 04:24:50PM +0200, Thierry Reding wrote: > The linux/iommu.h header uses types defined in linux/notifier.h but > doesn't include it. > > Signed-off-by: Thierry Reding Better than adding the header-file is to add a forward declaration for struct notifier_block. I added a pa

Re: [PATCH 1/2] iommu: Include linux/types.h

2012-08-03 Thread Joerg Roedel
On Wed, Jul 25, 2012 at 04:24:49PM +0200, Thierry Reding wrote: > The linux/iommu.h header uses types defined in linux/types.h but doesn't > include it. > > Signed-off-by: Thierry Reding Applied to iommu/fixes, thanks Thierry. -- AMD Operating System Research Center Advanced Micro Devices Gmb

Re: [PATCH v2] drivers/iommu/intel-iommu.c: add missing free_domain_mem

2012-08-03 Thread Joerg Roedel
On Tue, Jul 24, 2012 at 04:18:14PM +0200, Julia Lawall wrote: > From: Julia Lawall > > Add missing free_domain_mem on failure path after alloc_domain. > > Signed-off-by: Julia Lawall Applied to iommu/fixes, thanks Julia. -- AMD Operating System Research Center Advanced Micro Devices GmbH Ei