linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Alex Williamson <alex.williamson@redhat.com>
To: bhelgaas@google.com
Cc: linux-pci@vger.kernel.org, joro@8bytes.org,
	iommu@lists.linux-foundation.org, acooks@gmail.com,
	ddutile@redhat.com, dwmw2@infradead.org
Subject: [RFC PATCH v2 0/2] pci/iommu: PCIe requester ID interface
Date: Thu, 11 Jul 2013 15:03:18 -0600	[thread overview]
Message-ID: <20130711204439.1701.90503.stgit@bling.home> (raw)

v2: second pass at refining the interface.  Rolled pci_requester_id
into pci_get_visible_pcie_requester.  Fixed some bus walk issues.

Another obscure issue that I forgot to mention that this should fix
is gaps in requester IDs.  If you had:

 ---pcie--->[pcie2pci]---pci--->[pci2pcie]---pcie--->[endpoint]

I think the current code would try to use the endpoint requester id and
fail.  This code will use the pcie2pci->secondary as the requester id.
This may not even be that obscure of a bug if a PCIe ASIC is soldered
to a pci2pcie bridge to make a compatible conventional PCI card.

The diffstat for pci improves a bit when we remove:
pci_find_upstream_pci_bridge:

 drivers/iommu/intel-iommu.c         |  164 ++++++++++-----------------
 drivers/iommu/intel_irq_remapping.c |    2 
 drivers/pci/search.c                |  215 +++++++++++++++++++++++++++++++-----
 include/linux/pci.h                 |   16 +-
 4 files changed, 260 insertions(+), 137 deletions(-)

Comments?  Thanks,

Alex

---

Alex Williamson (2):
      pci: Create PCIe requester ID interface
      iommu/intel: Make use of PCIe requester ID interface


 drivers/iommu/intel-iommu.c         |  164 +++++++++++------------------
 drivers/iommu/intel_irq_remapping.c |    2 
 drivers/pci/search.c                |  198 +++++++++++++++++++++++++++++++++++
 include/linux/pci.h                 |    7 +
 4 files changed, 270 insertions(+), 101 deletions(-)

             reply	other threads:[~2013-07-11 21:03 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-11 21:03 Alex Williamson [this message]
2013-07-11 21:03 ` [RFC PATCH v2 1/2] pci: Create PCIe requester ID interface Alex Williamson
2013-07-23 22:35   ` Bjorn Helgaas
2013-07-23 23:21     ` Alex Williamson
2013-07-24 15:03       ` Andrew Cooks
2013-07-24 15:50         ` Alex Williamson
2013-07-24 16:47       ` Bjorn Helgaas
2013-07-24 18:12         ` Alex Williamson
2013-07-24 23:24           ` Bjorn Helgaas
2013-07-25 17:56             ` Alex Williamson
2013-07-26 21:54               ` Bjorn Helgaas
2013-07-29 16:06                 ` Alex Williamson
2013-07-29 21:02                   ` Bjorn Helgaas
2013-07-29 22:32                     ` Alex Williamson
2013-08-01 22:08                       ` Bjorn Helgaas
2013-08-02 16:59                         ` Alex Williamson
2014-04-03 21:48                           ` Bjorn Helgaas
2014-04-04  2:51                             ` Alex Williamson
2014-04-04 15:00                               ` Bjorn Helgaas
2013-07-29 21:03                   ` Don Dutile
2013-07-29 22:55                     ` Alex Williamson
2013-07-24 20:42     ` Don Dutile
2013-07-24 21:22       ` Alex Williamson
2013-07-25 18:38         ` Don Dutile
2013-07-25 17:19       ` Bjorn Helgaas
2013-07-25 18:25         ` Don Dutile
2013-07-26 19:48           ` Bjorn Helgaas
2013-07-26 20:04             ` Don Dutile
2013-07-11 21:03 ` [RFC PATCH v2 2/2] iommu/intel: Make use of " Alex Williamson

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=20130711204439.1701.90503.stgit@bling.home \
    --to=alex.williamson@redhat.com \
    --cc=acooks@gmail.com \
    --cc=bhelgaas@google.com \
    --cc=ddutile@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=iommu@lists.linux-foundation.org \
    --cc=joro@8bytes.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).