Re: [PATCH] calgary: remove IOMMU_DEBUG

[Muli Ben-Yehuda]

On Tue, Apr 14, 2009 at 12:12:29PM +0900, FUJITA Tomonori wrote:
> CONFIG_IOMMU_DEBUG has depends on CONFIG_GART_IOMMU:
> 
> config IOMMU_DEBUG
> 	bool "Enable IOMMU debugging"
> 	depends on GART_IOMMU && DEBUG_KERNEL
> 	depends on X86_64
> 
> So it's not useful to have CONFIG_IOMMU_DEBUG in Calgary IOMMU code,
> which does the extra checking of the bitmap space management.
> 
> And Calgary uses the iommu helper for the bitmap space management now
> so it would be better to have the extra checking feature in the iommu
> helper rather than Calgary code (if necessary).
> 
> Signed-off-by: FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>

Acked-by: Muli Ben-Yehuda <muli@xxxxxxxxxx>

Cheers,
Muli
-- 
Muli Ben-Yehuda | muli@xxxxxxxxxx | +972-4-8281080
Manager, Virtualization and Systems Architecture
Master Inventor, IBM Haifa Research Laboratory

SYSTOR 2009---The Israeli Experimental Systems Conference
http://www.haifa.il.ibm.com/conferences/systor2009/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/