From: Ben Hutchings <ben@decadent.org.uk>
To: "Bjorn Helgaas" <helgaas@kernel.org>, "Marek Behún" <kabel@kernel.org>
Cc: linux-pci@vger.kernel.org, "Pali Rohár" <pali@kernel.org>,
Rötti <espressobinboardarmbiantempmailaddress@posteo.de>,
"Zachary Zhang" <zhangzg@marvell.com>,
"Edward Cree" <ecree.xilinx@gmail.com>,
"Martin Habets" <habetsm.xilinx@gmail.com>,
"Keith Busch" <kbusch@kernel.org>
Subject: Re: [PATCH 1/2] PCI: Call MPS fixup quirks early
Date: Fri, 02 Jul 2021 17:39:43 +0200 [thread overview]
Message-ID: <237a49d8a113f44b55e537f6f2f99b7db9d97485.camel@decadent.org.uk> (raw)
In-Reply-To: <20210701152512.GA55520@bjorn-Precision-5520>
[-- Attachment #1: Type: text/plain, Size: 1116 bytes --]
On Thu, 2021-07-01 at 10:25 -0500, Bjorn Helgaas wrote:
[...]
> After 27d868b5e6cfa, pci_configure_device() did actually call
> pcie_set_mps(), which updates the Device Control register (possibly
> restricted by dev->pcie_mpss, which is set by this quirk).
>
> The fixup_mpss_256() quirk was added in 2011 by a94d072b2023 ("PCI:
> Add quirk for known incorrect MPSS"). Interesting that 27d868b5e6cfa
> was merged in 2015 but apparently nobody noticed until now. I guess
> those Solarflare devices aren't widely used?
[...]
The key thing is that this quirk was working around an issue with
legacy interrupts, while the sfc and sfc-falcon drivers have always
preferred to use MSIs if available. (But I also don't think many
SFC4000-based NICs were sold, and they were EOL'd about 10 years ago.)
Ben.
>
--
Ben Hutchings
[W]e found...that it wasn't as easy to get programs right as we had
thought. I realized that a large part of my life from then on was going
to be spent in finding mistakes in my own programs.
- Maurice Wilkes, 1949
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2021-07-02 15:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-24 17:14 [PATCH 1/2] PCI: Call MPS fixup quirks early Marek Behún
2021-06-24 17:14 ` [PATCH 2/2] PCI: Add Max Payload Size quirk for ASMedia ASM1062 SATA controller Marek Behún
2021-07-24 11:14 ` Pali Rohár
2021-07-26 17:24 ` Bjorn Helgaas
2021-07-26 21:13 ` Pali Rohár
2021-07-01 15:25 ` [PATCH 1/2] PCI: Call MPS fixup quirks early Bjorn Helgaas
2021-07-02 15:39 ` Ben Hutchings [this message]
2021-07-02 16:24 ` Bjorn Helgaas
2021-07-02 21:53 ` Ben Hutchings
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=237a49d8a113f44b55e537f6f2f99b7db9d97485.camel@decadent.org.uk \
--to=ben@decadent.org.uk \
--cc=ecree.xilinx@gmail.com \
--cc=espressobinboardarmbiantempmailaddress@posteo.de \
--cc=habetsm.xilinx@gmail.com \
--cc=helgaas@kernel.org \
--cc=kabel@kernel.org \
--cc=kbusch@kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=pali@kernel.org \
--cc=zhangzg@marvell.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.