From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH 07/11] iommu/arm-smmu: Make use of the iommu_register interface Date: Fri, 10 Feb 2017 16:25:53 +0100 Message-ID: <20170210152553.GJ7339@8bytes.org> References: <1486639981-32368-1-git-send-email-joro@8bytes.org> <1486639981-32368-8-git-send-email-joro@8bytes.org> <9bba214f-4d83-1e37-2df8-2f7db11f7d1f@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <9bba214f-4d83-1e37-2df8-2f7db11f7d1f-5wv7dgnIgG8@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Robin Murphy Cc: Will Deacon , iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Joerg Roedel , David Woodhouse List-Id: iommu@lists.linux-foundation.org On Fri, Feb 10, 2017 at 02:20:34PM +0000, Robin Murphy wrote: > > @@ -1795,8 +1798,10 @@ static int arm_smmu_add_device(struct device *dev) > > } > > > > group = iommu_group_get_for_dev(dev); > > - if (!IS_ERR(group)) > > + if (!IS_ERR(group)) { > > iommu_group_put(group); > > + iommu_device_link(&smmu->iommu, dev); > > Given the coupling evident from this and the other patches, might it > work to simply do the linking/unlinking automatically in > iommu_group_{add,remove}_device()? Yes, this is one of the goals too. But currently we don't have a generic device->hw_iommu mapping in the iommu-code which would allow to call the link/unlink functions in generic code too. But changing this is one of the next things on my list :) Joerg