Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

[PATCH 1/1] intel-iommu: Disable DMA Remapping when intel_iommu=off

676 views
Skip to first unread message

Wei Hu

unread,
Apr 20, 2013, 3:00:02 AM4/20/13
to
On a VT-d capable machine Linux will enable IOMMU by default. If it
then kexec's a second kernel with intel_iommu=off, this second kernel
will leave the DMA remapping engine on with no code handling it. The
symptom is at least USB and SATA drives stop working. This patch fixes
the problem by always disabling DMA remapping when intel_iommu=off.

Signed-off-by: Wei Hu <w...@aristanetworks.com>
---
drivers/iommu/intel-iommu.c | 6 +++++-
1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index 0099667..0b8f8bb 100644
--- a/drivers/iommu/intel-iommu.c
+++ b/drivers/iommu/intel-iommu.c
@@ -3681,8 +3681,12 @@ int __init intel_iommu_init(void)
return -ENODEV;
}

- if (no_iommu || dmar_disabled)
+ if (no_iommu || dmar_disabled) {
+ struct dmar_drhd_unit *drhd;
+ for_each_drhd_unit(drhd)
+ iommu_disable_translation(drhd->iommu);
return -ENODEV;
+ }

if (iommu_init_mempool()) {
if (force_on)
--
1.7.4.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majo...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Takao Indoh

unread,
Apr 22, 2013, 1:40:01 AM4/22/13
to
(2013/04/20 15:58), Wei Hu wrote:
> On a VT-d capable machine Linux will enable IOMMU by default. If it
> then kexec's a second kernel with intel_iommu=off, this second kernel
> will leave the DMA remapping engine on with no code handling it. The
> symptom is at least USB and SATA drives stop working. This patch fixes
> the problem by always disabling DMA remapping when intel_iommu=off.

Even when second kernel boots up with intel_iommu=on, dma-remapping need
to be disabled as well before it is initialized and enabled again in
init_dmars(). So, how about something like this?

for_each_drhd_unit(drhd) {
struct dmar_drhd_unit *drhd;
if (drhd->ignored)
continue;

iommu = drhd->iommu;
if (iommu->gcmd & DMA_GCMD_TE)
iommu_disable_translation(iommu);
}

Note, if you agree above code and fix your patch like this, you need
additinal fix to set iommu->gcmd flag sinse gcmd is always zero here.
See first hunk of this patch.
https://lkml.org/lkml/2013/3/20/707

Thanks,
Takao Indoh


>
> Signed-off-by: Wei Hu <w...@aristanetworks.com>
> ---
> drivers/iommu/intel-iommu.c | 6 +++++-
> 1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
> index 0099667..0b8f8bb 100644
> --- a/drivers/iommu/intel-iommu.c
> +++ b/drivers/iommu/intel-iommu.c
> @@ -3681,8 +3681,12 @@ int __init intel_iommu_init(void)
> return -ENODEV;
> }
>
> - if (no_iommu || dmar_disabled)
> + if (no_iommu || dmar_disabled) {
> + struct dmar_drhd_unit *drhd;
> + for_each_drhd_unit(drhd)
> + iommu_disable_translation(drhd->iommu);
> return -ENODEV;
> + }
>
> if (iommu_init_mempool()) {
> if (force_on)
>

--

Wei Hu

unread,
Apr 22, 2013, 3:00:02 AM4/22/13
to
On Sun, Apr 21, 2013 at 10:31 PM, Takao Indoh
<indou...@jp.fujitsu.com> wrote:
> (2013/04/20 15:58), Wei Hu wrote:
>> On a VT-d capable machine Linux will enable IOMMU by default. If it
>> then kexec's a second kernel with intel_iommu=off, this second kernel
>> will leave the DMA remapping engine on with no code handling it. The
>> symptom is at least USB and SATA drives stop working. This patch fixes
>> the problem by always disabling DMA remapping when intel_iommu=off.
>
> Even when second kernel boots up with intel_iommu=on, dma-remapping need
> to be disabled as well before it is initialized and enabled again in
> init_dmars(). So, how about something like this?
>
> for_each_drhd_unit(drhd) {
> struct dmar_drhd_unit *drhd;
> if (drhd->ignored)
> continue;
>
> iommu = drhd->iommu;
> if (iommu->gcmd & DMA_GCMD_TE)
> iommu_disable_translation(iommu);
> }
>
> Note, if you agree above code and fix your patch like this, you need
> additinal fix to set iommu->gcmd flag sinse gcmd is always zero here.
> See first hunk of this patch.
> https://lkml.org/lkml/2013/3/20/707

Thanks for your reply. I reviewed your patch. Are you suggesting
keeping your change to dmar.c, and moving your change to intel-iommu.c
(i.e. the code you just showed) up after the call to
dmar_table_init()?

Takao Indoh

unread,
Apr 22, 2013, 3:10:01 AM4/22/13
to
Yes, exactly.

Thanks,
Takao Indoh

Wei Hu

unread,
Apr 22, 2013, 3:20:02 AM4/22/13
to
Sounds great. If you want to make the change and get your patch
accepted that's fine with me.
0 new messages