From: Kai-Heng Feng <kai.heng.feng@canonical.com>
To: Bjorn Helgaas <bhelgaas@google.com>
Cc: "open list:PCI SUBSYSTEM" <linux-pci@vger.kernel.org>,
open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2] PCI: Coalesce contiguous regions for host bridges
Date: Tue, 11 May 2021 11:33:34 +0800 [thread overview]
Message-ID: <CAAd53p52C9TQz_zOPE-ySqfKbRNFo0MPzq7E3p_8mAwF_g0VjQ@mail.gmail.com> (raw)
In-Reply-To: <CAAd53p5rtZW_yqV2S77g34Dv9m9941yoBM6a_6fAvKpEuzXJ9g@mail.gmail.com>
On Thu, Apr 22, 2021 at 4:57 PM Kai-Heng Feng
<kai.heng.feng@canonical.com> wrote:
>
> On Thu, Apr 1, 2021 at 9:12 PM Kai-Heng Feng
> <kai.heng.feng@canonical.com> wrote:
> >
> > Built-in graphics on HP EliteDesk 805 G6 doesn't work because graphics
> > can't get the BAR it needs:
> > [ 0.611504] pci_bus 0000:00: root bus resource [mem 0x10020200000-0x100303fffff window]
> > [ 0.611505] pci_bus 0000:00: root bus resource [mem 0x10030400000-0x100401fffff window]
> > ...
> > [ 0.638083] pci 0000:00:08.1: bridge window [mem 0xd2000000-0xd23fffff]
> > [ 0.638086] pci 0000:00:08.1: bridge window [mem 0x10030000000-0x100401fffff 64bit pref]
> > [ 0.962086] pci 0000:00:08.1: can't claim BAR 15 [mem 0x10030000000-0x100401fffff 64bit pref]: no compatible bridge window
> > [ 0.962086] pci 0000:00:08.1: [mem 0x10030000000-0x100401fffff 64bit pref] clipped to [mem 0x10030000000-0x100303fffff 64bit pref]
> > [ 0.962086] pci 0000:00:08.1: bridge window [mem 0x10030000000-0x100303fffff 64bit pref]
> > [ 0.962086] pci 0000:07:00.0: can't claim BAR 0 [mem 0x10030000000-0x1003fffffff 64bit pref]: no compatible bridge window
> > [ 0.962086] pci 0000:07:00.0: can't claim BAR 2 [mem 0x10040000000-0x100401fffff 64bit pref]: no compatible bridge window
> >
> > However, the root bus has two contiguous regions that can contain the
> > child resource requested.
> >
> > Bjorn Helgaas pointed out that we can simply coalesce contiguous regions
> > for host bridges, since host bridge don't have _SRS. So do that
> > accordingly to make child resource can be contained. This change makes
> > the graphics works on the system in question.
> >
> > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=212013
> > Suggested-by: Bjorn Helgaas <bhelgaas@google.com>
> > Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
>
> A gentle ping...
Another gentle ping...
>
> > ---
> > v2:
> > - Coalesce all contiguous regresion in pci_register_host_bridge(), if
> > conditions are met.
> >
> > drivers/pci/probe.c | 49 +++++++++++++++++++++++++++++++++++++++++----
> > 1 file changed, 45 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
> > index 953f15abc850..3607ce7402b4 100644
> > --- a/drivers/pci/probe.c
> > +++ b/drivers/pci/probe.c
> > @@ -19,6 +19,7 @@
> > #include <linux/hypervisor.h>
> > #include <linux/irqdomain.h>
> > #include <linux/pm_runtime.h>
> > +#include <linux/list_sort.h>
> > #include "pci.h"
> >
> > #define CARDBUS_LATENCY_TIMER 176 /* secondary latency timer */
> > @@ -874,14 +875,30 @@ static void pci_set_bus_msi_domain(struct pci_bus *bus)
> > dev_set_msi_domain(&bus->dev, d);
> > }
> >
> > +static int res_cmp(void *priv, struct list_head *a, struct list_head *b)
> > +{
> > + struct resource_entry *entry1, *entry2;
> > +
> > + entry1 = container_of(a, struct resource_entry, node);
> > + entry2 = container_of(b, struct resource_entry, node);
> > +
> > + if (entry1->res->flags != entry2->res->flags)
> > + return entry1->res->flags > entry2->res->flags;
> > +
> > + if (entry1->offset != entry2->offset)
> > + return entry1->offset > entry2->offset;
> > +
> > + return entry1->res->start > entry2->res->start;
> > +}
> > +
> > static int pci_register_host_bridge(struct pci_host_bridge *bridge)
> > {
> > struct device *parent = bridge->dev.parent;
> > - struct resource_entry *window, *n;
> > + struct resource_entry *window, *next, *n;
> > struct pci_bus *bus, *b;
> > - resource_size_t offset;
> > + resource_size_t offset, next_offset;
> > LIST_HEAD(resources);
> > - struct resource *res;
> > + struct resource *res, *next_res;
> > char addr[64], *fmt;
> > const char *name;
> > int err;
> > @@ -959,11 +976,35 @@ static int pci_register_host_bridge(struct pci_host_bridge *bridge)
> > if (nr_node_ids > 1 && pcibus_to_node(bus) == NUMA_NO_NODE)
> > dev_warn(&bus->dev, "Unknown NUMA node; performance will be reduced\n");
> >
> > + /* Sort and coalesce contiguous windows */
> > + list_sort(NULL, &resources, res_cmp);
> > + resource_list_for_each_entry_safe(window, n, &resources) {
> > + if (list_is_last(&window->node, &resources))
> > + break;
> > +
> > + next = list_next_entry(window, node);
> > + offset = window->offset;
> > + res = window->res;
> > + next_offset = next->offset;
> > + next_res = next->res;
> > +
> > + if (res->flags != next_res->flags || offset != next_offset)
> > + continue;
> > +
> > + if (res->end + 1 == next_res->start) {
> > + next_res->start = res->start;
> > + res->flags = res->start = res->end = 0;
> > + }
> > + }
> > +
> > /* Add initial resources to the bus */
> > resource_list_for_each_entry_safe(window, n, &resources) {
> > - list_move_tail(&window->node, &bridge->windows);
> > offset = window->offset;
> > res = window->res;
> > + if (!res->end)
> > + continue;
> > +
> > + list_move_tail(&window->node, &bridge->windows);
> >
> > if (res->flags & IORESOURCE_BUS)
> > pci_bus_insert_busn_res(bus, bus->number, res->end);
> > --
> > 2.30.2
> >
next prev parent reply other threads:[~2021-05-11 3:33 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-01 13:12 [PATCH v2] PCI: Coalesce contiguous regions for host bridges Kai-Heng Feng
2021-04-22 8:57 ` Kai-Heng Feng
2021-05-11 3:33 ` Kai-Heng Feng [this message]
2021-05-27 22:13 ` Bjorn Helgaas
2021-07-09 23:15 ` Guenter Roeck
2021-07-09 23:43 ` Bjorn Helgaas
2021-07-12 3:50 ` Kai-Heng Feng
2021-07-12 4:50 ` Guenter Roeck
2021-07-12 4:52 ` Kai-Heng Feng
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=CAAd53p52C9TQz_zOPE-ySqfKbRNFo0MPzq7E3p_8mAwF_g0VjQ@mail.gmail.com \
--to=kai.heng.feng@canonical.com \
--cc=bhelgaas@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).