From: Joerg Roedel <joro@8bytes.org>
To: Will Deacon <will.deacon@arm.com>
Cc: "iommu@lists.linux-foundation.org"
<iommu@lists.linux-foundation.org>, Kukjin Kim <kgene@kernel.org>,
David Woodhouse <dwmw2@infradead.org>,
Heiko Stuebner <heiko@sntech.de>, Hiroshi Doyu <hdoyu@nvidia.com>,
Stephen Warren <swarren@wwwdotorg.org>,
Thierry Reding <thierry.reding@gmail.com>,
Alexandre Courbot <gnurou@gmail.com>,
Alex Williamson <alex.williamson@redhat.com>,
Arnd Bergmann <arnd@arndb.de>,
Yingjoe Chen <yingjoe.chen@mediatek.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-samsung-soc@vger.kernel.org"
<linux-samsung-soc@vger.kernel.org>,
"linux-rockchip@lists.infradead.org"
<linux-rockchip@lists.infradead.org>,
"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
"jroedel@suse.de" <jroedel@suse.de>
Subject: Re: [PATCH 00/16 v2] iommu: Move domain allocation into drivers
Date: Tue, 31 Mar 2015 15:18:34 +0200 [thread overview]
Message-ID: <20150331131834.GE4441@8bytes.org> (raw)
In-Reply-To: <20150330174315.GD15655@arm.com>
Hi Will,
On Mon, Mar 30, 2015 at 06:43:15PM +0100, Will Deacon wrote:
> Whilst I'm still a bit worried about the ability of the IOMMU driver to do
> very much in the ->domain_alloc callback (since we don't know which IOMMU
> instance the domain is for), it doesn't leave us any worse off than we
> currently are.
The IOMMU driver should just allocate the data-structures for a domain
in the domain_alloc call-back. It should not be necessary to care about
any relations of the domain with specific iommu instances.
> So, for the arm-smmu patch and the core IOMMU changes:
>
> Acked-by: Will Deacon <will.deacon@arm.com>
Thanks a lot!
Joerg
WARNING: multiple messages have this Message-ID (diff)
From: joro@8bytes.org (Joerg Roedel)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/16 v2] iommu: Move domain allocation into drivers
Date: Tue, 31 Mar 2015 15:18:34 +0200 [thread overview]
Message-ID: <20150331131834.GE4441@8bytes.org> (raw)
In-Reply-To: <20150330174315.GD15655@arm.com>
Hi Will,
On Mon, Mar 30, 2015 at 06:43:15PM +0100, Will Deacon wrote:
> Whilst I'm still a bit worried about the ability of the IOMMU driver to do
> very much in the ->domain_alloc callback (since we don't know which IOMMU
> instance the domain is for), it doesn't leave us any worse off than we
> currently are.
The IOMMU driver should just allocate the data-structures for a domain
in the domain_alloc call-back. It should not be necessary to care about
any relations of the domain with specific iommu instances.
> So, for the arm-smmu patch and the core IOMMU changes:
>
> Acked-by: Will Deacon <will.deacon@arm.com>
Thanks a lot!
Joerg
next prev parent reply other threads:[~2015-03-31 13:18 UTC|newest]
Thread overview: 79+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-26 12:43 [PATCH 00/16 v2] iommu: Move domain allocation into drivers Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 15/16] iommu/fsl: Make use of domain_alloc and domain_free Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
[not found] ` <1427373799-18662-16-git-send-email-joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-04-12 14:25 ` Varun Sethi
2015-04-12 14:25 ` Varun Sethi
2015-04-12 14:25 ` Varun Sethi
2015-03-26 12:43 ` [PATCH 16/16] iommu: Remove domain_init and domain_free iommu_ops Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-27 11:03 ` [PATCH 00/16 v2] iommu: Move domain allocation into drivers Thierry Reding
2015-03-27 11:03 ` Thierry Reding
[not found] ` <1427373799-18662-1-git-send-email-joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-03-26 12:43 ` [PATCH 01/16] iommu: Introduce domain_alloc and domain_free iommu_ops Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 02/16] iommu: Introduce iommu domain types Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:52 ` Swapna Nannapaneni
2015-03-26 12:52 ` Swapna Nannapaneni
2015-03-26 12:52 ` Swapna Nannapaneni
2015-03-26 12:43 ` [PATCH 03/16] iommu: Only allow iommu_map/unmap for paging domains Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 04/16] iommu/amd: Make use of domain_alloc and domain_free Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 05/16] iommu/vt-d: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 06/16] iommu/omap: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 07/16] iommu/arm-smmu: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 08/16] iommu/exynos: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 09/16] iommu/tegra-smmu: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
[not found] ` <1427373799-18662-10-git-send-email-joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-03-27 10:59 ` Thierry Reding
2015-03-27 10:59 ` Thierry Reding
2015-03-27 10:59 ` Thierry Reding
2015-03-26 12:43 ` [PATCH 10/16] iommu/tegra-gart: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
[not found] ` <1427373799-18662-11-git-send-email-joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>
2015-03-27 11:00 ` Thierry Reding
2015-03-27 11:00 ` Thierry Reding
2015-03-27 11:00 ` Thierry Reding
2015-03-26 12:43 ` [PATCH 11/16] iommu/msm: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 12/16] iommu/shmobile: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 13/16] iommu/ipmmu-vmsa: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` [PATCH 14/16] iommu/rockchip: " Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-26 12:43 ` Joerg Roedel
2015-03-27 16:39 ` [PATCH 00/16 v2] iommu: Move domain allocation into drivers Heiko Stuebner
2015-03-27 16:39 ` Heiko Stuebner
2015-03-27 16:39 ` Heiko Stuebner
2015-03-28 4:24 ` Alex Williamson
2015-03-28 4:24 ` Alex Williamson
2015-03-28 4:24 ` Alex Williamson
[not found] ` <1427516662.3643.906.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-03-31 13:14 ` Joerg Roedel
2015-03-31 13:14 ` Joerg Roedel
2015-03-31 13:14 ` Joerg Roedel
2015-03-30 17:43 ` Will Deacon
2015-03-30 17:43 ` Will Deacon
2015-03-30 17:43 ` Will Deacon
2015-03-31 13:18 ` Joerg Roedel [this message]
2015-03-31 13:18 ` Joerg Roedel
2015-03-31 13:33 ` Joerg Roedel
2015-03-31 13:33 ` Joerg Roedel
2015-03-31 13:33 ` Joerg Roedel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150331131834.GE4441@8bytes.org \
--to=joro@8bytes.org \
--cc=alex.williamson@redhat.com \
--cc=arnd@arndb.de \
--cc=dwmw2@infradead.org \
--cc=gnurou@gmail.com \
--cc=hdoyu@nvidia.com \
--cc=heiko@sntech.de \
--cc=iommu@lists.linux-foundation.org \
--cc=jroedel@suse.de \
--cc=kgene@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=swarren@wwwdotorg.org \
--cc=thierry.reding@gmail.com \
--cc=will.deacon@arm.com \
--cc=yingjoe.chen@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.