lkml.org 
[lkml]   [2008]   [Jul]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[patch 17/26] x64, x2apic/intr-remap: disable DMA-remapping if Interrupt-remapping is detected (temporary quirk)
    Interrupt-remapping enables queued invalidation. And once queued invalidation
    is enabled, IOTLB invalidation also needs to use the queued invalidation
    mechanism and the register based IOTLB invalidation doesn't work.

    For now, Support for IOTLB invalidation using queued invalidation is
    missing. Meanwhile, disable DMA-remapping, if Interrupt-remapping
    support is detected.

    For the meanwhile, if someone wants to really enable DMA-remapping, they
    can use nox2apic, which will disable interrupt-remapping and as such
    doesn't enable queued invalidation.

    And given that none of the release platforms support intr-remapping yet,
    we should be ok for this temporary hack.

    Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>
    ---

    Index: tree-x86/drivers/pci/dmar.c
    ===================================================================
    --- tree-x86.orig/drivers/pci/dmar.c 2008-07-10 09:52:03.000000000 -0700
    +++ tree-x86/drivers/pci/dmar.c 2008-07-10 09:52:19.000000000 -0700
    @@ -457,6 +457,31 @@

    #ifdef CONFIG_DMAR
    {
    + struct acpi_table_dmar *dmar;
    + /*
    + * for now we will disable dma-remapping when interrupt
    + * remapping is enabled.
    + * When support for queued invalidation for IOTLB invalidation
    + * is added, we will not need this any more.
    + */
    + dmar = (struct acpi_table_dmar *) dmar_tbl;
    + if (ret && cpu_has_x2apic && dmar->flags & 0x1) {
    + printk(KERN_INFO
    + "Queued invalidation will be enabled to support "
    + "x2apic and Intr-remapping.\n");
    + printk(KERN_INFO
    + "Disabling IOMMU detection, because of missing "
    + "queued invalidation support for IOTLB "
    + "invalidation\n");
    + printk(KERN_INFO
    + "Use \"nox2apic\", if you want to use Intel "
    + " IOMMU for DMA-remapping and don't care about "
    + " x2apic support\n");
    +
    + dmar_disabled = 1;
    + return;
    + }
    +
    if (ret && !no_iommu && !iommu_detected && !swiotlb &&
    !dmar_disabled)
    iommu_detected = 1;
    --



    \
     
     \ /
      Last update: 2008-07-10 20:57    [W:0.037 / U:32.564 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site