* [GIT PULL] AMD IOMMU fixes for 2.6.27-rc6
@ 2008-09-17 14:23 Joerg Roedel
2008-09-18 7:27 ` Ingo Molnar
0 siblings, 1 reply; 2+ messages in thread
From: Joerg Roedel @ 2008-09-17 14:23 UTC (permalink / raw)
To: Ingo Molnar; +Cc: linux-kernel
Hi Ingo,
please pull two fixes for possible race conditions in the AMD IOMMU
command buffer handling code from
git://git.kernel.org/pub/scm/linux/kernel/git/joro/linux-2.6-iommu.git iommu-2.6.27
These patches should be merged into 2.6.27 because they are important
bugfixes. The diff is small enough so I attached it to this email.
Joerg Roedel (2):
AMD IOMMU: set iommu sunc flag after command queuing
AMD IOMMU: protect comletion wait loop with iommu lock
arch/x86/kernel/amd_iommu.c | 22 ++++++++++++++++------
1 files changed, 16 insertions(+), 6 deletions(-)
diff --git a/arch/x86/kernel/amd_iommu.c b/arch/x86/kernel/amd_iommu.c
index 01c68c3..40509eb 100644
--- a/arch/x86/kernel/amd_iommu.c
+++ b/arch/x86/kernel/amd_iommu.c
@@ -101,10 +101,10 @@ static int iommu_queue_command(struct amd_iommu *iommu, struct iommu_cmd *cmd)
*/
static int iommu_completion_wait(struct amd_iommu *iommu)
{
- int ret, ready = 0;
+ int ret = 0, ready = 0;
unsigned status = 0;
struct iommu_cmd cmd;
- unsigned long i = 0;
+ unsigned long flags, i = 0;
memset(&cmd, 0, sizeof(cmd));
cmd.data[0] = CMD_COMPL_WAIT_INT_MASK;
@@ -112,10 +112,12 @@ static int iommu_completion_wait(struct amd_iommu *iommu)
iommu->need_sync = 0;
- ret = iommu_queue_command(iommu, &cmd);
+ spin_lock_irqsave(&iommu->lock, flags);
+
+ ret = __iommu_queue_command(iommu, &cmd);
if (ret)
- return ret;
+ goto out;
while (!ready && (i < EXIT_LOOP_COUNT)) {
++i;
@@ -130,6 +132,8 @@ static int iommu_completion_wait(struct amd_iommu *iommu)
if (unlikely((i == EXIT_LOOP_COUNT) && printk_ratelimit()))
printk(KERN_WARNING "AMD IOMMU: Completion wait loop failed\n");
+out:
+ spin_unlock_irqrestore(&iommu->lock, flags);
return 0;
}
@@ -140,6 +144,7 @@ static int iommu_completion_wait(struct amd_iommu *iommu)
static int iommu_queue_inv_dev_entry(struct amd_iommu *iommu, u16 devid)
{
struct iommu_cmd cmd;
+ int ret;
BUG_ON(iommu == NULL);
@@ -147,9 +152,11 @@ static int iommu_queue_inv_dev_entry(struct amd_iommu *iommu, u16 devid)
CMD_SET_TYPE(&cmd, CMD_INV_DEV_ENTRY);
cmd.data[0] = devid;
+ ret = iommu_queue_command(iommu, &cmd);
+
iommu->need_sync = 1;
- return iommu_queue_command(iommu, &cmd);
+ return ret;
}
/*
@@ -159,6 +166,7 @@ static int iommu_queue_inv_iommu_pages(struct amd_iommu *iommu,
u64 address, u16 domid, int pde, int s)
{
struct iommu_cmd cmd;
+ int ret;
memset(&cmd, 0, sizeof(cmd));
address &= PAGE_MASK;
@@ -171,9 +179,11 @@ static int iommu_queue_inv_iommu_pages(struct amd_iommu *iommu,
if (pde) /* PDE bit - we wan't flush everything not only the PTEs */
cmd.data[2] |= CMD_INV_IOMMU_PAGES_PDE_MASK;
+ ret = iommu_queue_command(iommu, &cmd);
+
iommu->need_sync = 1;
- return iommu_queue_command(iommu, &cmd);
+ return ret;
}
/*
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [GIT PULL] AMD IOMMU fixes for 2.6.27-rc6
2008-09-17 14:23 [GIT PULL] AMD IOMMU fixes for 2.6.27-rc6 Joerg Roedel
@ 2008-09-18 7:27 ` Ingo Molnar
0 siblings, 0 replies; 2+ messages in thread
From: Ingo Molnar @ 2008-09-18 7:27 UTC (permalink / raw)
To: Joerg Roedel; +Cc: linux-kernel, Thomas Gleixner, H. Peter Anvin
* Joerg Roedel <joerg.roedel@amd.com> wrote:
> Hi Ingo,
>
> please pull two fixes for possible race conditions in the AMD IOMMU
> command buffer handling code from
>
> git://git.kernel.org/pub/scm/linux/kernel/git/joro/linux-2.6-iommu.git iommu-2.6.27
>
> These patches should be merged into 2.6.27 because they are important
> bugfixes. The diff is small enough so I attached it to this email.
pulled these two commits into tip/x86/urgent, thanks Joerg!
> Joerg Roedel (2):
> AMD IOMMU: set iommu sunc flag after command queuing
> AMD IOMMU: protect comletion wait loop with iommu lock
(FYI, i respun the commits, to fix the s/comletion/completion typo, and
to rebase the fixes to -git.)
Ingo
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-09-18 7:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-09-17 14:23 [GIT PULL] AMD IOMMU fixes for 2.6.27-rc6 Joerg Roedel
2008-09-18 7:27 ` Ingo Molnar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).