On Thu, Feb 27, 2014 at 05:15:38AM +0000, Ritesh Harjani wrote: > Hi Everyone,
Hi Ritesh, > I was going through some iommu code in arch/arm and of some other > archs code. I have some doubts on this for refactoring and may need > some suggestions from you guys. > > 1. So, looking at other arch code, looks like they have their > different way of implementation of iova management and buffer > allocation. So to refactor the iommu common code out from arch/arm/ to > lib/iommu-helper, do we need to take care across all arch ? Whilst the code should compile for all architectures, I don't think you need to go round porting them all to use it. That can happen in a piecemeal fashion as they get ported over to the generic code. Initially, I see arm and arm64 as the users. > 2. Should the approach be like take the common code(between arm/arm64) > and move it into lib/iommu-helper.c ? Well, something along those lines. We should put some thought into what the interfaces should look like, rather than blindly copying everything out as it stands. > Could someone give an example of what sort of code(will be better if > this is little more specific) we are talking here to be taken out to > lib/iommu-helper.c ? Earlier I was thinking of iova management can be > taken out but then I saw it might not be suited across all archs. > > I am ready to do this work, but need some guidance from the experts . Catalin can clarify here, but I think we'd basically want a generic equivalent to arm_iommu_create_mapping and its associated halpers (i.e. the dma_map_ops). The dma_map_ops will have generic parts (e.g. the interfaces to the iommu API) but also architecturally-specific parts (e.g. cache flushing for non-coherent devices), so that will need some thought. Once that's done, we can later look at hooking this into device-tree in order to set the correct ops each device. Will _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu