Re: [PATCH v6 0/3] Generic IOMMU pooled allocator

2015-03-25 Thread Sowmini Varadhan
On (03/24/15 18:16), David Miller wrote: Generally this looks fine to me. But about patch #2, I see no reason to have multiple iommu_pool_hash tables. Even from a purely sparc perspective, we can always just do with just one of them. Furthermore, you can even probably move it down into

Re: [PATCH v6 0/3] Generic IOMMU pooled allocator

2015-03-24 Thread David Miller
From: Sowmini Varadhan sowmini.varad...@oracle.com Date: Tue, 24 Mar 2015 13:10:27 -0400 Deltas from patchv5: - removed iommu_tbl_ops, and instead pass the -flush_all as an indirection to iommu_tbl_pool_init() - only invoke -flush_all when there is no large_pool, based on the assumption

[PATCH v6 0/3] Generic IOMMU pooled allocator

2015-03-24 Thread Sowmini Varadhan
Deltas from patchv5: - removed iommu_tbl_ops, and instead pass the -flush_all as an indirection to iommu_tbl_pool_init() - only invoke -flush_all when there is no large_pool, based on the assumption that large-pool usage is infrequently encountered. Sowmini (2): Break up monolithic iommu