[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-changelog] [xen-unstable] x86: cleanup after tboot fix (c/s 23101:dd386a4b6595)

# HG changeset patch
# User Jan Beulich <jbeulich@xxxxxxxxxx>
# Date 1301310472 -3600
# Node ID bbf4b6dd9c3214637476972e96cd0d3cd9cb3844
# Parent  c4576aafb21ebfc16b438684ebb1148c13e80085
x86: cleanup after tboot fix (c/s 23101:dd386a4b6595)

When submitting the patch that became 23101:dd386a4b6595 I forgot that
I actually intended to remove the pointless IRQ disabling and
restoring (and the then pointless comment). Pointless because the
code, other than its pre-23013:65d26504e843 original, now runs before
interrupts get enabled for the first time.

Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>

diff -r c4576aafb21e -r bbf4b6dd9c32 xen/drivers/passthrough/vtd/dmar.c
--- a/xen/drivers/passthrough/vtd/dmar.c        Sun Mar 27 17:04:02 2011 +0100
+++ b/xen/drivers/passthrough/vtd/dmar.c        Mon Mar 28 12:07:52 2011 +0100
@@ -761,12 +761,7 @@
 int __init acpi_dmar_init(void)
-    unsigned long flags;
-    /* Disabling IRQs avoids cross-CPU TLB flush in map_pages_to_xen(). */
-    local_irq_save(flags);
     acpi_get_table(ACPI_SIG_DMAR, 0, &dmar_table);
-    local_irq_restore(flags);
     return parse_dmar_table(acpi_parse_dmar);

Xen-changelog mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.