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

* David Woodhouse <dwmw2@infradead.org> 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 <David.Woodhouse@intel.com>
> ---
>
> 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,

Ingo


\
 
 \ /
  Last update: 2008-09-05 20:37    [from the cache]
©2003-2011 Jasper Spaans. Advertise on this site