From: Jesse Barnes on
On Thu, 03 Jun 2010 13:43:03 -0700
Yinghai Lu <yinghai.lu(a)oracle.com> wrote:

>
> Make sure We can reject wrong size from BIOS.
>
> https://bugzilla.kernel.org/show_bug.cgi?id=16009
> Yannick found that video does not work with 2.6.34
>
> the root cause:
> BIOS assigned wrong range to pci bridge. and before 2.6.34 kernel will
> just get range that is needed.
> for 2.6.34.
> | d65245c PCI: don't shrink bridge resources
> will try to range size is bigger than old one.
> (used by boot stage multi-try to get big BAR size for pci bridge,
> and pcie hotplug to get big range)
>
> So try to 0 for old size for pci bridge in this boot stage case.
>
> Reported-by: Yannick <yannick.roehlly(a)free.fr>
> Analyzed-by: Bjorn Helgaas <bjorn.helgaas(a)hp.com>
> Signed-off-by: Yinghai Lu <yinghai(a)kernel.org>
>

Bjorn, are you ok with this version? It would probably help to have
some comments here too (I can add them), indicating that we'll try to
reassign the resource later, rather than just ignoring it as the
existing comment implies.

--
Jesse Barnes, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Yinghai Lu on
On 06/08/2010 03:48 PM, Bjorn Helgaas wrote:
> On Tuesday, June 08, 2010 03:43:50 pm Jesse Barnes wrote:

>
> In this case, the aperture *size* from the BIOS is actually OK, but
> the beginning of the aperture overlaps system memory. With Yinghai's
> patch, we reduce the size and move the start. Windows was able to
> just move the start of the aperture and preserve the original
> 0x20000000 size (but I think it had to move something else out of
> the way).

are you sure? that looks more aggressive.

Yinghai
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/