2008-02-16 21:49:26

by Daniel Drake

[permalink] [raw]
Subject: pci-disable-decoding-during-sizing-of-bars.patch no longer needed

Hi,

The patch titled pci-disable-decoding-during-sizing-of-bars.patch in -mm
was previously used as a candidate to fix a boot hang with Intel's Q35
chipset: https://bugs.gentoo.org/show_bug.cgi?id=198810

However, that particular issue is solved by commit a0ca9909609 in Linus
tree:
PCI x86: always use conf1 to access config space below 256 bytes

So unless there are other reasons for keeping
pci-disable-decoding-during-sizing-of-bars around, I think we can drop it.

Thanks,
Daniel


2008-02-16 22:00:36

by Matthew Wilcox

[permalink] [raw]
Subject: Re: pci-disable-decoding-during-sizing-of-bars.patch no longer needed

On Sat, Feb 16, 2008 at 09:50:51PM +0000, Daniel Drake wrote:
> Hi,
>
> The patch titled pci-disable-decoding-during-sizing-of-bars.patch in -mm
> was previously used as a candidate to fix a boot hang with Intel's Q35
> chipset: https://bugs.gentoo.org/show_bug.cgi?id=198810
>
> However, that particular issue is solved by commit a0ca9909609 in Linus
> tree:
> PCI x86: always use conf1 to access config space below 256 bytes
>
> So unless there are other reasons for keeping
> pci-disable-decoding-during-sizing-of-bars around, I think we can drop it.

I agree it should be dropped.

--
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."

2008-02-16 22:06:22

by Andrew Morton

[permalink] [raw]
Subject: Re: pci-disable-decoding-during-sizing-of-bars.patch no longer needed

On Sat, 16 Feb 2008 21:50:51 +0000 Daniel Drake <[email protected]> wrote:

> Hi,
>
> The patch titled pci-disable-decoding-during-sizing-of-bars.patch in -mm
> was previously used as a candidate to fix a boot hang with Intel's Q35
> chipset: https://bugs.gentoo.org/show_bug.cgi?id=198810
>
> However, that particular issue is solved by commit a0ca9909609 in Linus
> tree:
> PCI x86: always use conf1 to access config space below 256 bytes
>
> So unless there are other reasons for keeping
> pci-disable-decoding-during-sizing-of-bars around, I think we can drop it.
>

Great, thanks for letting me know.