| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 16 Mar 2010 22:22:05 -0600
From: Bjorn Helgaas <bjorn.helgaas@...com>
To: Kenji Kaneshige <kaneshige.kenji@...fujitsu.com>
Cc: Jesse Barnes <jbarnes@...tuousgeek.org>, linux-pci@...r.kernel.org,
linux-kernel@...r.kernel.org, "Rafael J. Wysocki" <rjw@...k.pl>,
Yanko Kaneti <yaneti@...lera.com>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Thomas Renninger <trenn@...e.de>, maciej.rutecki@...il.com
Subject: Re: [PATCH v1 2/3] x86/PCI: trim _CRS windows when they conflict
with previous reservations
On Wed, 2010-03-17 at 12:25 +0900, Kenji Kaneshige wrote:
> Bjorn Helgaas wrote:
> > Yanko's GA-MA78GM-S2H (BIOS F11) reports a host bridge window that overlaps
> > system memory:
> >
> > PCI window: [mem 0xcff00000-0x10ed0ffff]
> > System RAM: [mem 0x100000000-0x22fffffff]
> >
> > We can be pretty confident that the System RAM region is correct (if it
> > were wrong, we'd crash as soon as we tried to use any memory in that area),
> > so this patch tries to correct the PCI window by trimming it so it doesn't
> > conflict with any previous reservations.
>
> Though I might misunderstand something, it looks Yanko's machine specific
> workaround. I'm wondering if trimming _CRS is a generic workaround for
> broken _CRS machine.
>
> How about doing this when GA-MA78GM-S2H (BIOS F11) (and known machines
> that have the same problem) is detected? Or how about switching nocrs
> mode if the problem (resource conflict) is detected?
It's certainly a possibility to do this only for specific machines, but
I'd like to avoid tripping over issues one-by-one.
I think there are three ways to address BIOS _CRS defects:
1) Ship an OEM-specific host bridge driver
2) Put a platform- or BIOS-specific quirk into Windows
3) Change the BIOS
The first two sound like such a hassle to me that I doubt they would be
practical.
But it's clear that there are systems like this with what appear to be
_CRS defects. It's quite possible that it's not really a defect, and we
just don't understand how to parse _CRS correctly yet. Or, Windows
might have a few heuristics to clean up obvious errors.
For example, I think Windows aligns host bridge windows, as documented
here: http://bugzilla.kernel.org/show_bug.cgi?id=14337
I think Windows also knows to ignore the Consumer/Producer bit in
Address Space Descriptors, and assume that all resources on bridges are
Producers.
Hmm, what we really need is a way to run Windows in a virtualized
environment where we could manipulate the _CRS method and see what
Windows does with it...
Anyway, I'd like to make Linux behave as much like Windows as possible
in this area so we can take advantage of all the testing that's done
with Windows.
Bjorn
> > http://bugzilla.kernel.org/show_bug.cgi?id=15480
> >
> > Signed-off-by: Bjorn Helgaas <bjorn.helgaas@...com>
> > Reported-by: Yanko Kaneti <yaneti@...lera.com>
> > ---
> >
> > arch/x86/pci/acpi.c | 48 +++++++++++++++++++++++++++++++++++-------------
> > 1 files changed, 35 insertions(+), 13 deletions(-)
> >
> >
> > diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c
> > index 6e22454..d255ce8 100644
> > --- a/arch/x86/pci/acpi.c
> > +++ b/arch/x86/pci/acpi.c
> > @@ -114,11 +114,19 @@ align_resource(struct acpi_device *bridge, struct resource *res)
> > }
> > }
> >
> > +static bool
> > +resource_contains(struct resource *res, resource_size_t n)
> > +{
> > + if (n < res->start || n > res->end)
> > + return false;
> > + return true;
> > +}
> > +
> > static acpi_status
> > setup_resource(struct acpi_resource *acpi_res, void *data)
> > {
> > struct pci_root_info *info = data;
> > - struct resource *res;
> > + struct resource *res, *conflict;
> > struct acpi_resource_address64 addr;
> > acpi_status status;
> > unsigned long flags;
> > @@ -157,21 +165,35 @@ setup_resource(struct acpi_resource *acpi_res, void *data)
> > return AE_OK;
> > }
> >
> > - if (insert_resource(root, res)) {
> > + conflict = insert_resource_conflict(root, res);
> > + while (conflict) {
> > dev_err(&info->bridge->dev,
> > - "can't allocate host bridge window %pR\n", res);
> > - } else {
> > - pci_bus_add_resource(info->bus, res, 0);
> > - info->res_num++;
> > - if (addr.translation_offset)
> > - dev_info(&info->bridge->dev, "host bridge window %pR "
> > - "(PCI address [%#llx-%#llx])\n",
> > - res, res->start - addr.translation_offset,
> > - res->end - addr.translation_offset);
> > + "host bridge window %pR conflicts with %s %pR\n",
> > + res, conflict->name, conflict);
> > +
> > + if (resource_contains(res, conflict->end))
> > + res->start = conflict->end + 1;
> > + else if (resource_contains(res, conflict->start))
> > + res->end = conflict->start - 1;
> > else
> > - dev_info(&info->bridge->dev,
> > - "host bridge window %pR\n", res);
> > + return AE_OK;
> > +
> > + if (res->start >= res->end)
> > + return AE_OK;
> > +
> > + conflict = insert_resource_conflict(root, res);
> > }
> > +
> > + pci_bus_add_resource(info->bus, res, 0);
> > + info->res_num++;
> > + if (addr.translation_offset)
> > + dev_info(&info->bridge->dev, "host bridge window %pR "
> > + "(PCI address [%#llx-%#llx])\n",
> > + res, res->start - addr.translation_offset,
> > + res->end - addr.translation_offset);
> > + else
> > + dev_info(&info->bridge->dev,
> > + "host bridge window %pR\n", res);
> > return AE_OK;
> > }
> >
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> > the body of a message to majordomo@...r.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
> >
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists