linux-kernel - Clear tags in dmar.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <CAEk7JKYFESRP2wSTzS34d5e2c5FuaW2KhzJaZYwn43j7TruFWg@mail.gmail.com>
Date:	Mon, 24 Dec 2012 16:59:13 +0800
From:	"Li, Zhen-Hua" <lizhenhua.dev@...il.com>
To:	Linux Kernel <linux-kernel@...r.kernel.org>
Subject: Clear tags in dmar.c

In dmar.c: qi_check_fault, I think the tags should be clear when
finished processing.
See this patch:

--- drivers/iommu/dmar.c.orig   2012-12-25 00:56:16.000000000 +0800
+++ drivers/iommu/dmar.c.new    2012-12-25 00:56:37.000000000 +0800
@@ -759,7 +759,7 @@ static int qi_check_fault(struct intel_i
                                        sizeof(struct qi_desc));
                        __iommu_flush_cache(iommu, &qi->desc[index],
                                        sizeof(struct qi_desc));
-                       writel(DMA_FSTS_IQE, iommu->reg + DMAR_FSTS_REG);
+                       writel(~DMA_FSTS_IQE, iommu->reg + DMAR_FSTS_REG);
                        return -EINVAL;
                }
        }
@@ -775,7 +775,7 @@ static int qi_check_fault(struct intel_i
                tail = readl(iommu->reg + DMAR_IQT_REG);
                tail = ((tail >> DMAR_IQ_SHIFT) - 1 + QI_LENGTH) % QI_LENGTH;

-               writel(DMA_FSTS_ITE, iommu->reg + DMAR_FSTS_REG);
+               writel(~DMA_FSTS_ITE, iommu->reg + DMAR_FSTS_REG);

                do {
                        if (qi->desc_status[head] == QI_IN_USE)
@@ -788,7 +788,7 @@ static int qi_check_fault(struct intel_i
        }

        if (fault & DMA_FSTS_ICE)
-               writel(DMA_FSTS_ICE, iommu->reg + DMAR_FSTS_REG);
+               writel(~DMA_FSTS_ICE, iommu->reg + DMAR_FSTS_REG);

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