From: Bjorn Helgaas <bjorn.helgaas@hp.com>
To: linux-pci@atrey.karlin.mff.cuni.cz
Cc: Matthew Hall <mhall@mhcomputing.net>,
linux-acpi@vger.kernel.org, Jeff Garzik <jeff@garzik.org>
Subject: Re: sata_nv does not function in kernel > 2.6.20.21... possible ACPI or PCI involvement?
Date: Tue, 15 Jan 2008 09:47:07 -0700 [thread overview]
Message-ID: <200801150947.07998.bjorn.helgaas@hp.com> (raw)
In-Reply-To: <20080115043804.GA17040@mhcomputing.net>
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
next prev parent reply other threads:[~2008-01-15 16:46 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-12 20:59 sata_nv does not function in kernel > 2.6.20.21... possible ACPI or PCI involvement? Matthew Hall
2008-01-15 0:02 ` Bjorn Helgaas
2008-01-15 4:38 ` Matthew Hall
2008-01-15 16:47 ` Bjorn Helgaas [this message]
2008-01-16 21:07 ` Bjorn Helgaas
2008-01-17 19:04 ` Matthew Hall
2008-01-15 22:02 ` Bjorn Helgaas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200801150947.07998.bjorn.helgaas@hp.com \
--to=bjorn.helgaas@hp.com \
--cc=jeff@garzik.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-pci@atrey.karlin.mff.cuni.cz \
--cc=mhall@mhcomputing.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox