On Monday 14 January 2008 09:38:04 pm Matthew Hall wrote: > On Mon, Jan 14, 2008 at 05:02:26PM -0700, Bjorn Helgaas wrote: > > I think this is a bug in the BIOS description of the motherboard > > device. In 2.6.20, the PNP system driver reserved ioport resources > > but ignored mmio resources. In 2.6.23, the system driver reserves > > both ioport and mmio resources, which I think is more correct, but > > exposes this bug. > > Bjorn, > > I complained to Supermicro and mentioned that the beta BIOS they > provided to one of the customers in the Redhat Bugzilla threads did not > solve the issue at all. I also included links to both Bugzilla threads > as well as a link to the mail you sent back to me as snipped above which > contained your excellent analysis of the problem. My patch is based on the assumption that a PCI resource (like the SATA BAR here) is completely discoverable and configurable through the standard PCI mechanisms, and it therefore should not also be described as a resource of an ACPI device. If Supermicro went to the trouble of providing a beta BIOS, maybe they're interested enough to shed some light on why their BIOS does it this way. It's possible that my assumption is wrong, and Linux is just not doing the right thing. If you have a Supermicro contact, you might pass this by them. > > I posted the attached test patch to the redhat bugzilla above, but > > nobody's tested it yet. Can you try it? > > The patch applied, booted, ran, and solved the problem perfectly. Do you > want any output or configuration files to verify anything? Let me know > if there is anything I can do to help. Other than that if you think it > is safe to apply this, I am in favor of adding it to the mainline or > other appropriate tree. I'll clean up the patch and post it. With luck, it might show up in the -mm tree soon and then in 2.6.25. Bjorn - To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
