[lkml]   [2008]   [Sep]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Blacklist DMAR on Intel G31/G33 chipsets

    * David Woodhouse <> wrote:

    > Some BIOSes (the Intel DG33BU, for example) wrongly claim to have DMAR
    > when they don't. Avoid the resulting crashes when it doesn't work as
    > expected.
    > Signed-off-by: David Woodhouse <>
    > ---
    > This time, I build-tested it with CONFIG_DMAR actually enabled. Sorry.
    > I'd still be grateful if someone could test it on a DG33BU with the
    > old BIOS though, since I've killed mine. I tested the DMI version, but
    > not this one.

    ok - fixing this makes sense. I have two worries about this patch.

    Firstly, the quirk is keyed off an ACPI capability which is quite bad if
    someone boots with ACPI off. (which is still quite possible) The DMAR is
    PCI enumerated so there's nothing inherently ACPI about this. A DMI
    quirk (which will work even if ACPI is disabled) looks more robust.

    Secondly, keying off the PCI ID and assuming a BIOS bug based on the
    presence of an ACPI table can indeed get incoherent results as you
    suspect. It's better to single out the specific BIOS as long as the DMI
    info is specific enough. Even if that PCI ID is never supposed to be
    combined with VT-d (because, obviously, VT-d needs a different chipset),
    it's just a sloppy concept in general.

    The fact that you've already tested the DMI version and cannot test the
    new version is one more reason to favor the first patch.

    Could you please resend the initial DMI version of the
    drivers/pci/intel-iommu.c commit stand-alone, not embedded in a 'misc'
    pull request, and with Jesse Cc:-ed as well? Thanks,


     \ /
      Last update: 2008-09-05 20:37    [W:0.022 / U:4.156 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site