From: Bjorn Helgaas <helgaas@kernel.org>
To: linux-pci@vger.kernel.org
Cc: Thomas Gleixner <tglx@linutronix.de>,
Kevin Tian <kevin.tian@intel.com>, Marc Zyngier <maz@kernel.org>,
Reinette Chatre <reinette.chatre@intel.com>,
Jason Gunthorpe <jgg@nvidia.com>,
Alex Williamson <alex.williamson@redhat.com>,
Dave Jiang <dave.jiang@intel.com>,
Megha Dey <megha.dey@intel.com>,
linux-kernel@vger.kernel.org, Bjorn Helgaas <bhelgaas@google.com>
Subject: [PATCH 3/3] Revert "PCI/MSI: Provide IMS (Interrupt Message Store) support"
Date: Mon, 1 Apr 2024 18:23:26 -0500 [thread overview]
Message-ID: <20240401232326.1794707-4-helgaas@kernel.org> (raw)
In-Reply-To: <20240401232326.1794707-1-helgaas@kernel.org>
From: Bjorn Helgaas <bhelgaas@google.com>
This reverts commit 0194425af0c87acaad457989a2c6d90dba58e776.
IMS (Interrupt Message Store) support appeared in v6.2, but there are no
users yet.
Remove it for now. We can add it back when a user comes along. If this is
re-added later, the relevant part of 41efa431244f ("PCI/MSI: Provide stubs
for IMS functions") should be squashed into it.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
drivers/pci/msi/irqdomain.c | 59 -------------------------------------
include/linux/pci.h | 5 ----
2 files changed, 64 deletions(-)
diff --git a/drivers/pci/msi/irqdomain.c b/drivers/pci/msi/irqdomain.c
index cfd84a899c82..03d2dd25790d 100644
--- a/drivers/pci/msi/irqdomain.c
+++ b/drivers/pci/msi/irqdomain.c
@@ -355,65 +355,6 @@ bool pci_msi_domain_supports(struct pci_dev *pdev, unsigned int feature_mask,
return (supported & feature_mask) == feature_mask;
}
-/**
- * pci_create_ims_domain - Create a secondary IMS domain for a PCI device
- * @pdev: The PCI device to operate on
- * @template: The MSI info template which describes the domain
- * @hwsize: The size of the hardware entry table or 0 if the domain
- * is purely software managed
- * @data: Optional pointer to domain specific data to be stored
- * in msi_domain_info::data
- *
- * Return: True on success, false otherwise
- *
- * An IMS domain is expected to have the following constraints:
- * - The index space is managed by the core code
- *
- * - There is no requirement for consecutive index ranges
- *
- * - The interrupt chip must provide the following callbacks:
- * - irq_mask()
- * - irq_unmask()
- * - irq_write_msi_msg()
- *
- * - The interrupt chip must provide the following optional callbacks
- * when the irq_mask(), irq_unmask() and irq_write_msi_msg() callbacks
- * cannot operate directly on hardware, e.g. in the case that the
- * interrupt message store is in queue memory:
- * - irq_bus_lock()
- * - irq_bus_unlock()
- *
- * These callbacks are invoked from preemptible task context and are
- * allowed to sleep. In this case the mandatory callbacks above just
- * store the information. The irq_bus_unlock() callback is supposed
- * to make the change effective before returning.
- *
- * - Interrupt affinity setting is handled by the underlying parent
- * interrupt domain and communicated to the IMS domain via
- * irq_write_msi_msg().
- *
- * The domain is automatically destroyed when the PCI device is removed.
- */
-bool pci_create_ims_domain(struct pci_dev *pdev, const struct msi_domain_template *template,
- unsigned int hwsize, void *data)
-{
- struct irq_domain *domain = dev_get_msi_domain(&pdev->dev);
-
- if (!domain || !irq_domain_is_msi_parent(domain))
- return false;
-
- if (template->info.bus_token != DOMAIN_BUS_PCI_DEVICE_IMS ||
- !(template->info.flags & MSI_FLAG_ALLOC_SIMPLE_MSI_DESCS) ||
- !(template->info.flags & MSI_FLAG_FREE_MSI_DESCS) ||
- !template->chip.irq_mask || !template->chip.irq_unmask ||
- !template->chip.irq_write_msi_msg || template->chip.irq_set_affinity)
- return false;
-
- return msi_create_device_irq_domain(&pdev->dev, MSI_SECONDARY_DOMAIN, template,
- hwsize, data, NULL);
-}
-EXPORT_SYMBOL_GPL(pci_create_ims_domain);
-
/*
* Users of the generic MSI infrastructure expect a device to have a single ID,
* so with DMA aliases we have to pick the least-worst compromise. Devices with
diff --git a/include/linux/pci.h b/include/linux/pci.h
index 3deb3e42d990..98fb20bcd054 100644
--- a/include/linux/pci.h
+++ b/include/linux/pci.h
@@ -2664,11 +2664,6 @@ static inline bool pci_is_thunderbolt_attached(struct pci_dev *pdev)
void pci_uevent_ers(struct pci_dev *pdev, enum pci_ers_result err_type);
#endif
-struct msi_domain_template;
-
-bool pci_create_ims_domain(struct pci_dev *pdev, const struct msi_domain_template *template,
- unsigned int hwsize, void *data);
-
#include <linux/dma-mapping.h>
#define pci_printk(level, pdev, fmt, arg...) \
--
2.34.1
next prev parent reply other threads:[~2024-04-01 23:23 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-03 23:15 [PATCH 0/8] arch/ typo fixes Bjorn Helgaas
2024-01-03 23:15 ` [PATCH 1/8] alpha: Fix typos Bjorn Helgaas
2024-01-04 0:24 ` Randy Dunlap
2024-01-03 23:15 ` [PATCH 2/8] ARC: " Bjorn Helgaas
2024-01-04 0:31 ` Randy Dunlap
2024-01-04 19:02 ` Bjorn Helgaas
2024-03-29 19:26 ` Vineet Gupta
2024-03-29 22:14 ` [PATCH v2] " Bjorn Helgaas
2024-03-30 1:00 ` Vineet Gupta
2024-04-01 23:23 ` [PATCH 0/3] PCI/MSI: Remove IMS (Interrupt Message Store) support for now Bjorn Helgaas
2024-04-01 23:23 ` [PATCH 1/3] Revert "PCI/MSI: Provide stubs for IMS functions" Bjorn Helgaas
2024-04-02 2:46 ` Tian, Kevin
2024-04-01 23:23 ` [PATCH 2/3] Revert "PCI/MSI: Provide pci_ims_alloc/free_irq()" Bjorn Helgaas
2024-04-02 2:47 ` Tian, Kevin
2024-04-01 23:23 ` Bjorn Helgaas [this message]
2024-04-02 2:48 ` [PATCH 3/3] Revert "PCI/MSI: Provide IMS (Interrupt Message Store) support" Tian, Kevin
2024-04-02 2:45 ` [PATCH 0/3] PCI/MSI: Remove IMS (Interrupt Message Store) support for now Tian, Kevin
2024-04-02 16:42 ` Bjorn Helgaas
2024-01-03 23:16 ` [PATCH 3/8] arm64: Fix typos Bjorn Helgaas
2024-01-04 1:06 ` Randy Dunlap
2024-01-03 23:16 ` [PATCH 4/8] arm64: dts: " Bjorn Helgaas
2024-01-04 1:01 ` Randy Dunlap
2024-01-03 23:16 ` [PATCH 5/8] KVM: arm64: " Bjorn Helgaas
2024-01-04 0:59 ` Randy Dunlap
2024-01-15 12:01 ` Zenghui Yu
2024-02-24 4:39 ` Zenghui Yu
2024-02-24 9:12 ` Oliver Upton
2024-02-24 9:15 ` (subset) " Oliver Upton
2024-01-03 23:16 ` [PATCH 6/8] MIPS: " Bjorn Helgaas
2024-01-04 0:56 ` Randy Dunlap
2024-01-08 9:41 ` Thomas Bogendoerfer
2024-01-03 23:16 ` [PATCH 7/8] powerpc: " Bjorn Helgaas
2024-01-04 0:37 ` Randy Dunlap
2024-05-08 13:49 ` Michael Ellerman
2024-01-03 23:16 ` [PATCH 8/8] sparc: " Bjorn Helgaas
2024-01-04 0:41 ` Randy Dunlap
2024-02-16 16:19 ` Andreas Larsson
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=20240401232326.1794707-4-helgaas@kernel.org \
--to=helgaas@kernel.org \
--cc=alex.williamson@redhat.com \
--cc=bhelgaas@google.com \
--cc=dave.jiang@intel.com \
--cc=jgg@nvidia.com \
--cc=kevin.tian@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=maz@kernel.org \
--cc=megha.dey@intel.com \
--cc=reinette.chatre@intel.com \
--cc=tglx@linutronix.de \
/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