* [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB
@ 2018-05-23 20:18 dmeyer
2018-05-23 20:18 ` [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header dmeyer
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: dmeyer @ 2018-05-23 20:18 UTC (permalink / raw)
To: logang, kurt.schwemmer, linux-pci, linux-ntb
Cc: bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel, Doug Meyer
From: Doug Meyer <dmeyer@gigaio.com>
This is a resend of the patch series to enable Microsemi Switchtec
NTB configurations to run with the IOMMU in the hosts turned on.
Because of the nature PCI Quirk implementation, it was preferable
to migrate the Microsemi PCI vendor and device definitions to the
Linux canonical location. Logan Gunthorpe requested that this
migration be done as a separate patch in a set, and so this patch
series was created as shown here.
The first patch encapsulates the movement of constants from
switchtec.h to pci_ids.h, with commensurate changes to the source
files. This patch is not dependent on any other work.
The second patch is the PCI quirk implementation itself, and is
completely dependent upon the first patch in this series.
Testing of the quirk was done on with a 2-host x86-64 system
with all combinations of IOMMU off/on. The ntb_perf module was
used as test stimulus.
Blessings,
Doug Meyer
Changes since v1:
- Call pci_device_disable() at return points to clean up properly.
- Changed all dev_* print macros to pci_* macros.
- Removed superfluous variable initializations.
Doug Meyer (2):
NTB: Migrate PCI Constants to Cannonical PCI Header
NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On
drivers/ntb/hw/mscc/ntb_hw_switchtec.c | 3 +-
drivers/pci/quirks.c | 197 +++++++++++++++++++++++++++++++++
drivers/pci/switch/switchtec.c | 15 ++-
include/linux/pci_ids.h | 32 ++++++
include/linux/switchtec.h | 4 -
5 files changed, 238 insertions(+), 13 deletions(-)
--
1.8.3.1
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header 2018-05-23 20:18 [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB dmeyer @ 2018-05-23 20:18 ` dmeyer 2018-05-23 20:26 ` Logan Gunthorpe 2018-05-23 20:18 ` [PATCH v2 2/2] NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On dmeyer 2018-06-30 1:50 ` [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB Bjorn Helgaas 2 siblings, 1 reply; 9+ messages in thread From: dmeyer @ 2018-05-23 20:18 UTC (permalink / raw) To: logang, kurt.schwemmer, linux-pci, linux-ntb Cc: bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel, Doug Meyer From: Doug Meyer <dmeyer@gigaio.com> This is the first of two patches to implement a PCI quirk which will allow the Switchtec NTB code to work with the IOMMU turned on. Here, the Microsemi Switchtec PCI vendor and device ID constants are moved to the canonical location in pci_ids.h. Also, Microsemi class constants are replaced with the standard PCI usage. Signed-off-by: Doug Meyer <dmeyer@gigaio.com> --- drivers/ntb/hw/mscc/ntb_hw_switchtec.c | 3 ++- drivers/pci/switch/switchtec.c | 15 +++++++-------- include/linux/pci_ids.h | 32 ++++++++++++++++++++++++++++++++ include/linux/switchtec.h | 4 ---- 4 files changed, 41 insertions(+), 13 deletions(-) diff --git a/drivers/ntb/hw/mscc/ntb_hw_switchtec.c b/drivers/ntb/hw/mscc/ntb_hw_switchtec.c index f624ae2..5ee5f40 100644 --- a/drivers/ntb/hw/mscc/ntb_hw_switchtec.c +++ b/drivers/ntb/hw/mscc/ntb_hw_switchtec.c @@ -19,6 +19,7 @@ #include <linux/kthread.h> #include <linux/interrupt.h> #include <linux/ntb.h> +#include <linux/pci.h> MODULE_DESCRIPTION("Microsemi Switchtec(tm) NTB Driver"); MODULE_VERSION("0.1"); @@ -1487,7 +1488,7 @@ static int switchtec_ntb_add(struct device *dev, stdev->sndev = NULL; - if (stdev->pdev->class != MICROSEMI_NTB_CLASSCODE) + if (stdev->pdev->class != (PCI_CLASS_BRIDGE_OTHER << 8)) return -ENODEV; sndev = kzalloc_node(sizeof(*sndev), GFP_KERNEL, dev_to_node(dev)); diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c index 47cd0c0..07a03b9 100644 --- a/drivers/pci/switch/switchtec.c +++ b/drivers/pci/switch/switchtec.c @@ -1,4 +1,3 @@ -// SPDX-License-Identifier: GPL-2.0 /* * Microsemi Switchtec(tm) PCIe Management Driver * Copyright (c) 2017, Microsemi Corporation @@ -641,7 +640,7 @@ static int ioctl_event_summary(struct switchtec_dev *stdev, for (i = 0; i < SWITCHTEC_MAX_PFF_CSR; i++) { reg = ioread16(&stdev->mmio_pff_csr[i].vendor_id); - if (reg != MICROSEMI_VENDOR_ID) + if (reg != PCI_VENDOR_ID_MICROSEMI) break; reg = ioread32(&stdev->mmio_pff_csr[i].pff_event_summary); @@ -1203,7 +1202,7 @@ static void init_pff(struct switchtec_dev *stdev) for (i = 0; i < SWITCHTEC_MAX_PFF_CSR; i++) { reg = ioread16(&stdev->mmio_pff_csr[i].vendor_id); - if (reg != MICROSEMI_VENDOR_ID) + if (reg != PCI_VENDOR_ID_MICROSEMI) break; } @@ -1267,7 +1266,7 @@ static int switchtec_pci_probe(struct pci_dev *pdev, struct switchtec_dev *stdev; int rc; - if (pdev->class == MICROSEMI_NTB_CLASSCODE) + if (pdev->class == (PCI_CLASS_BRIDGE_OTHER << 8)) request_module_nowait("ntb_hw_switchtec"); stdev = stdev_create(pdev); @@ -1321,19 +1320,19 @@ static void switchtec_pci_remove(struct pci_dev *pdev) #define SWITCHTEC_PCI_DEVICE(device_id) \ { \ - .vendor = MICROSEMI_VENDOR_ID, \ + .vendor = PCI_VENDOR_ID_MICROSEMI, \ .device = device_id, \ .subvendor = PCI_ANY_ID, \ .subdevice = PCI_ANY_ID, \ - .class = MICROSEMI_MGMT_CLASSCODE, \ + .class = (PCI_CLASS_MEMORY_OTHER << 8), \ .class_mask = 0xFFFFFFFF, \ }, \ { \ - .vendor = MICROSEMI_VENDOR_ID, \ + .vendor = PCI_VENDOR_ID_MICROSEMI, \ .device = device_id, \ .subvendor = PCI_ANY_ID, \ .subdevice = PCI_ANY_ID, \ - .class = MICROSEMI_NTB_CLASSCODE, \ + .class = (PCI_CLASS_BRIDGE_OTHER << 8), \ .class_mask = 0xFFFFFFFF, \ } diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h index cc608fc5..7b04ca95 100644 --- a/include/linux/pci_ids.h +++ b/include/linux/pci_ids.h @@ -3073,4 +3073,36 @@ #define PCI_VENDOR_ID_OCZ 0x1b85 +#define PCI_VENDOR_ID_MICROSEMI 0x11f8 +#define PCI_DEVICE_ID_MICROSEMI_PFX24XG3 0x8531 +#define PCI_DEVICE_ID_MICROSEMI_PFX32XG3 0x8532 +#define PCI_DEVICE_ID_MICROSEMI_PFX48XG3 0x8533 +#define PCI_DEVICE_ID_MICROSEMI_PFX64XG3 0x8534 +#define PCI_DEVICE_ID_MICROSEMI_PFX80XG3 0x8535 +#define PCI_DEVICE_ID_MICROSEMI_PFX96XG3 0x8536 +#define PCI_DEVICE_ID_MICROSEMI_PSX24XG3 0x8541 +#define PCI_DEVICE_ID_MICROSEMI_PSX32XG3 0x8542 +#define PCI_DEVICE_ID_MICROSEMI_PSX48XG3 0x8543 +#define PCI_DEVICE_ID_MICROSEMI_PSX64XG3 0x8544 +#define PCI_DEVICE_ID_MICROSEMI_PSX80XG3 0x8545 +#define PCI_DEVICE_ID_MICROSEMI_PSX96XG3 0x8546 +#define PCI_DEVICE_ID_MICROSEMI_PAX24XG3 0x8551 +#define PCI_DEVICE_ID_MICROSEMI_PAX32XG3 0x8552 +#define PCI_DEVICE_ID_MICROSEMI_PAX48XG3 0x8553 +#define PCI_DEVICE_ID_MICROSEMI_PAX64XG3 0x8554 +#define PCI_DEVICE_ID_MICROSEMI_PAX80XG3 0x8555 +#define PCI_DEVICE_ID_MICROSEMI_PAX96XG3 0x8556 +#define PCI_DEVICE_ID_MICROSEMI_PFXL24XG3 0x8561 +#define PCI_DEVICE_ID_MICROSEMI_PFXL32XG3 0x8562 +#define PCI_DEVICE_ID_MICROSEMI_PFXL48XG3 0x8563 +#define PCI_DEVICE_ID_MICROSEMI_PFXL64XG3 0x8564 +#define PCI_DEVICE_ID_MICROSEMI_PFXL80XG3 0x8565 +#define PCI_DEVICE_ID_MICROSEMI_PFXL96XG3 0x8566 +#define PCI_DEVICE_ID_MICROSEMI_PFXI24XG3 0x8571 +#define PCI_DEVICE_ID_MICROSEMI_PFXI32XG3 0x8572 +#define PCI_DEVICE_ID_MICROSEMI_PFXI48XG3 0x8573 +#define PCI_DEVICE_ID_MICROSEMI_PFXI64XG3 0x8574 +#define PCI_DEVICE_ID_MICROSEMI_PFXI80XG3 0x8575 +#define PCI_DEVICE_ID_MICROSEMI_PFXI96XG3 0x8576 + #endif /* _LINUX_PCI_IDS_H */ diff --git a/include/linux/switchtec.h b/include/linux/switchtec.h index ec93e93..ab400af 100644 --- a/include/linux/switchtec.h +++ b/include/linux/switchtec.h @@ -19,10 +19,6 @@ #include <linux/pci.h> #include <linux/cdev.h> -#define MICROSEMI_VENDOR_ID 0x11f8 -#define MICROSEMI_NTB_CLASSCODE 0x068000 -#define MICROSEMI_MGMT_CLASSCODE 0x058000 - #define SWITCHTEC_MRPC_PAYLOAD_SIZE 1024 #define SWITCHTEC_MAX_PFF_CSR 48 -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header 2018-05-23 20:18 ` [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header dmeyer @ 2018-05-23 20:26 ` Logan Gunthorpe 2018-05-23 20:39 ` Doug Meyer 0 siblings, 1 reply; 9+ messages in thread From: Logan Gunthorpe @ 2018-05-23 20:26 UTC (permalink / raw) To: dmeyer, kurt.schwemmer, linux-pci, linux-ntb Cc: bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel On 23/05/18 02:18 PM, dmeyer@gigaio.com wrote: > +#define PCI_DEVICE_ID_MICROSEMI_PFX24XG3 0x8531 I think Bjorn asked not to add the device IDs to the pci_ids.h and to just use the raw IDs in quirks.h. switchtec.c already uses the raw values. Besides that, it looks good to me. Reviewed-by: Logan Gunthorpe <logang@deltatee.com> ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header 2018-05-23 20:26 ` Logan Gunthorpe @ 2018-05-23 20:39 ` Doug Meyer 0 siblings, 0 replies; 9+ messages in thread From: Doug Meyer @ 2018-05-23 20:39 UTC (permalink / raw) To: Logan Gunthorpe Cc: kurt.schwemmer, linux-pci, linux-ntb, Bjorn Helgaas, Jon Mason, Jiang, Dave, Allen Hubbe, linux-kernel Dear Logan, Thank you for the comment. My gmail filters got me and I never saw Bjorn's comments on pci_ids.h. I'll start work on v3 when I get another window of time at work. Blessings, Doug On Wed, May 23, 2018 at 1:26 PM, Logan Gunthorpe <logang@deltatee.com> wrote: > > > On 23/05/18 02:18 PM, dmeyer@gigaio.com wrote: >> +#define PCI_DEVICE_ID_MICROSEMI_PFX24XG3 0x8531 > > I think Bjorn asked not to add the device IDs to the pci_ids.h and to > just use the raw IDs in quirks.h. switchtec.c already uses the raw values. > > Besides that, it looks good to me. > > Reviewed-by: Logan Gunthorpe <logang@deltatee.com> > > > -- > You received this message because you are subscribed to the Google Groups "linux-ntb" group. > To unsubscribe from this group and stop receiving emails from it, send an email to linux-ntb+unsubscribe@googlegroups.com. > To post to this group, send email to linux-ntb@googlegroups.com. > To view this discussion on the web visit https://groups.google.com/d/msgid/linux-ntb/553308ee-9c6f-2e1e-1ce8-03b739d1acb0%40deltatee.com. > For more options, visit https://groups.google.com/d/optout. ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 2/2] NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On 2018-05-23 20:18 [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB dmeyer 2018-05-23 20:18 ` [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header dmeyer @ 2018-05-23 20:18 ` dmeyer 2018-05-23 20:32 ` Logan Gunthorpe 2018-06-30 1:50 ` [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB Bjorn Helgaas 2 siblings, 1 reply; 9+ messages in thread From: dmeyer @ 2018-05-23 20:18 UTC (permalink / raw) To: logang, kurt.schwemmer, linux-pci, linux-ntb Cc: bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel, Doug Meyer From: Doug Meyer <dmeyer@gigaio.com> Here we add the PCI quirk for the Microsemi Switchtec parts to allow DMA access via non-transparent bridging to work when the IOMMU is turned on. This exclusively addresses the ability of a remote NT endpoint to perform DMA accesses through the locally enumerated NT endpoint. Other aspects of the Switchtec NTB functionality, such as interrupts for doorbells and messages are independent of this quirk, and will work whether the IOMMU is on or off. When a requestor on one NT endpoint accesses memory on another NT endpoint, it does this via a devfn proxy ID. Proxy IDs are statically assigned to each NT endpoint by the NTB hardware as part of the release-from-reset sequence prior to PCI enumeration. These proxy IDs cannot be modified dynamically, and are not visible to the host during enumeration. When the Switchtec NTB driver loads it will map local requestor IDs, such as the root complex and transparent bridge DMA engines, to proxy IDs by populating those requestor IDs in hardware mapping table table entries. This establishes a fixed relationship between a requestor ID and a proxy ID. When a peer on a remote NT endpoint performs an access within a particular translation window in it's NT endpoint BAR address space, that access is translated to a DMA request on the local endpoint's bus. As part of the translation process, the original requestor ID has its devfn replaced with the proxy ID, and the bus portion of the BDF is replaced with the bus of the local NT endpoint. Thus, the DMA access from a remote NT endpoint will appear on the local bus to have come from the unknown devfn which the IOMMU will reject. The quirk introduced here interrogates NTB hardware registers for each remote NT endpoint to obtain the proxy IDs that have been assigned to it, and aliases them to the local (enumerated) NT endpoint's device. The IOMMU then accepts the remote proxy IDs as if they were requests coming directly from the enumerated endpoint, giving remote requestors access to memory resources which the local host has made available. Note that the aliasing of the proxy IDs cannot be performed at the driver level given the current IOMMU architecture. Superficially this is because pci_add_dma_alias() symbol is not exported. Functionally, the current IOMMU design requires the aliasing to be performed prior to the creation of IOMMU groups. If a driver were to attempt to use pci_add_dma_alias() in its probe routine it would fail since the IOMMU groups have been set up by that time. If the Switchtec hardware supported dynamic proxy ID (re-)assignment this would be an issue, but it does not. To further clarify static proxy ID assignment: While the requester ID to proxy ID mapping can be dynamically changed, the number and value of proxy IDs given to an NT EP cannot, even for dynamic reconfiguration such as hot-add. Therefore, the chip configuration must account a priori for the proxy IDs needs, considering both static and dynamic system configurations. For example, a port on the chip may not having anything plugged into it at start of day; but it must have a sufficient number of proxy IDs assigned to accommodate the supported devices which may be hot-added. Switchtec NTB functionality with the IOMMU off is unchanged by this quirk. Signed-off-by: Doug Meyer <dmeyer@gigaio.com> --- drivers/pci/quirks.c | 197 +++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 197 insertions(+) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 2990ad1..4456165 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -27,6 +27,7 @@ #include <linux/mm.h> #include <linux/platform_data/x86/apple.h> #include <linux/pm_runtime.h> +#include <linux/switchtec.h> #include <asm/dma.h> /* isa_dma_bridge_buggy */ #include "pci.h" @@ -4741,3 +4742,199 @@ static void quirk_gpu_hda(struct pci_dev *hda) PCI_CLASS_MULTIMEDIA_HD_AUDIO, 8, quirk_gpu_hda); DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_NVIDIA, PCI_ANY_ID, PCI_CLASS_MULTIMEDIA_HD_AUDIO, 8, quirk_gpu_hda); + +/* + * Microsemi Switchtec NTB uses devfn proxy IDs to move TLPs between + * NT endpoints via the internal switch fabric. These IDs replace the + * originating requestor ID TLPs which access host memory on peer NTB + * ports. Therefore, all proxy IDs must be aliased to the NTB device + * to permit access when the IOMMU is turned on. + */ +static void quirk_switchtec_ntb_dma_alias(struct pci_dev *pdev) +{ + void __iomem *mmio; + struct ntb_info_regs __iomem *mmio_ntb; + struct ntb_ctrl_regs __iomem *mmio_ctrl; + struct sys_info_regs __iomem *mmio_sys_info; + u64 partition_map; + u8 partition; + int pp; + + if (pci_enable_device(pdev)) { + pci_err(pdev, "Cannot enable Switchtec device\n"); + return; + } + + mmio = pci_iomap(pdev, 0, 0); + if (mmio == NULL) { + pci_disable_device(pdev); + pci_err(pdev, "Cannot iomap Switchtec device\n"); + return; + } + + pci_info(pdev, "Setting Switchtec proxy ID aliases\n"); + + mmio_ntb = mmio + SWITCHTEC_GAS_NTB_OFFSET; + mmio_ctrl = (void * __iomem) mmio_ntb + SWITCHTEC_NTB_REG_CTRL_OFFSET; + mmio_sys_info = mmio + SWITCHTEC_GAS_SYS_INFO_OFFSET; + + partition = ioread8(&mmio_ntb->partition_id); + + partition_map = (u64) ioread32((void * __iomem) &mmio_ntb->ep_map); + partition_map |= + ((u64) ioread32((void * __iomem) &mmio_ntb->ep_map + 4)) << 32; + partition_map &= ~(1ULL << partition); + + for (pp = 0; pp < (sizeof(partition_map) * 8); pp++) { + struct ntb_ctrl_regs __iomem *mmio_peer_ctrl; + u32 table_sz = 0; + int te; + + if (!(partition_map & (1ULL << pp))) + continue; + + pci_dbg(pdev, "Processing partition %d\n", pp); + + mmio_peer_ctrl = &mmio_ctrl[pp]; + + table_sz = ioread16(&mmio_peer_ctrl->req_id_table_size); + if (!table_sz) { + pci_warn(pdev, "Partition %d table_sz 0\n", pp); + continue; + } + + if (table_sz > 512) { + pci_warn(pdev, + "Invalid Switchtec partition %d table_sz %d\n", + pp, table_sz); + continue; + } + + for (te = 0; te < table_sz; te++) { + u32 rid_entry; + u8 devfn; + + rid_entry = ioread32(&mmio_peer_ctrl->req_id_table[te]); + devfn = (rid_entry >> 1) & 0xFF; + pci_dbg(pdev, + "Aliasing Partition %d Proxy ID %02d.%d\n", + pp, PCI_SLOT(devfn), PCI_FUNC(devfn)); + pci_add_dma_alias(pdev, devfn); + } + } + + pci_iounmap(pdev, mmio); + pci_disable_device(pdev); +} +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFX24XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFX32XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFX48XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFX64XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFX80XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFX96XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PSX48XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PSX64XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PSX80XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PSX96XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PAX24XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PAX32XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PAX48XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PAX64XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PAX80XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PAX96XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXL24XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXL32XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXL48XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXL64XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXL80XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXL96XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXI24XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXI32XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXI48XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXI64XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXI80XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, + PCI_DEVICE_ID_MICROSEMI_PFXI96XG3, + PCI_CLASS_BRIDGE_OTHER, 8, + quirk_switchtec_ntb_dma_alias); -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On 2018-05-23 20:18 ` [PATCH v2 2/2] NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On dmeyer @ 2018-05-23 20:32 ` Logan Gunthorpe 0 siblings, 0 replies; 9+ messages in thread From: Logan Gunthorpe @ 2018-05-23 20:32 UTC (permalink / raw) To: dmeyer, kurt.schwemmer, linux-pci, linux-ntb Cc: bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel On 23/05/18 02:18 PM, dmeyer@gigaio.com wrote: > Signed-off-by: Doug Meyer <dmeyer@gigaio.com> Modulo the device ID issue: Reviewed-by: Logan Gunthorpe <logang@deltatee.com> Thanks for your work on this Doug! Logan ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB 2018-05-23 20:18 [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB dmeyer 2018-05-23 20:18 ` [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header dmeyer 2018-05-23 20:18 ` [PATCH v2 2/2] NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On dmeyer @ 2018-06-30 1:50 ` Bjorn Helgaas 2018-06-30 18:57 ` Bjorn Helgaas 2018-07-05 16:56 ` Logan Gunthorpe 2 siblings, 2 replies; 9+ messages in thread From: Bjorn Helgaas @ 2018-06-30 1:50 UTC (permalink / raw) To: dmeyer Cc: logang, kurt.schwemmer, linux-pci, linux-ntb, bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel On Wed, May 23, 2018 at 01:18:04PM -0700, dmeyer@gigaio.com wrote: > From: Doug Meyer <dmeyer@gigaio.com> > > This is a resend of the patch series to enable Microsemi Switchtec > NTB configurations to run with the IOMMU in the hosts turned on. > Because of the nature PCI Quirk implementation, it was preferable > to migrate the Microsemi PCI vendor and device definitions to the > Linux canonical location. Logan Gunthorpe requested that this > migration be done as a separate patch in a set, and so this patch > series was created as shown here. > > The first patch encapsulates the movement of constants from > switchtec.h to pci_ids.h, with commensurate changes to the source > files. This patch is not dependent on any other work. > > The second patch is the PCI quirk implementation itself, and is > completely dependent upon the first patch in this series. > > Testing of the quirk was done on with a 2-host x86-64 system > with all combinations of IOMMU off/on. The ntb_perf module was > used as test stimulus. > > Blessings, > Doug Meyer > > Changes since v1: > - Call pci_device_disable() at return points to clean up properly. > - Changed all dev_* print macros to pci_* macros. > - Removed superfluous variable initializations. > > Doug Meyer (2): > NTB: Migrate PCI Constants to Cannonical PCI Header > NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On > > drivers/ntb/hw/mscc/ntb_hw_switchtec.c | 3 +- > drivers/pci/quirks.c | 197 +++++++++++++++++++++++++++++++++ > drivers/pci/switch/switchtec.c | 15 ++- > include/linux/pci_ids.h | 32 ++++++ > include/linux/switchtec.h | 4 - > 5 files changed, 238 insertions(+), 13 deletions(-) I applied these with Logan's reviewed-by to my pci/switchtec branch for v4.19 with the following updates: - You removed the SPDX tag from drivers/pci/switch/switchtec.c. I assume that was a mistake, so I restored it. - I moved the PCI_VENDOR_ID_MICROSEMI definition to keep the file sorted. - I dropped the Device ID definitions per the policy at the top of the file (which I mentioned on your v1 posting). - I converted all the Device ID definition uses to raw hex constants. I noticed that the following were defined by your patch, but not used: PCI_DEVICE_ID_MICROSEMI_PSX24XG3 PCI_DEVICE_ID_MICROSEMI_PSX32XG3 I can't tell whether the quirk is supposed to apply to them or not. Please review and holler if I broke something. This touches drivers/ntb/hw/..., which isn't my area. Let me know if you'd rather take these through a different tree. Bjorn ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB 2018-06-30 1:50 ` [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB Bjorn Helgaas @ 2018-06-30 18:57 ` Bjorn Helgaas 2018-07-05 16:56 ` Logan Gunthorpe 1 sibling, 0 replies; 9+ messages in thread From: Bjorn Helgaas @ 2018-06-30 18:57 UTC (permalink / raw) To: dmeyer Cc: logang, kurt.schwemmer, linux-pci, linux-ntb, bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel On Fri, Jun 29, 2018 at 08:50:02PM -0500, Bjorn Helgaas wrote: > On Wed, May 23, 2018 at 01:18:04PM -0700, dmeyer@gigaio.com wrote: > > From: Doug Meyer <dmeyer@gigaio.com> > > > > This is a resend of the patch series to enable Microsemi Switchtec > > NTB configurations to run with the IOMMU in the hosts turned on. > > Because of the nature PCI Quirk implementation, it was preferable > > to migrate the Microsemi PCI vendor and device definitions to the > > Linux canonical location. Logan Gunthorpe requested that this > > migration be done as a separate patch in a set, and so this patch > > series was created as shown here. > > > > The first patch encapsulates the movement of constants from > > switchtec.h to pci_ids.h, with commensurate changes to the source > > files. This patch is not dependent on any other work. > > > > The second patch is the PCI quirk implementation itself, and is > > completely dependent upon the first patch in this series. > > > > Testing of the quirk was done on with a 2-host x86-64 system > > with all combinations of IOMMU off/on. The ntb_perf module was > > used as test stimulus. > > > > Blessings, > > Doug Meyer > > > > Changes since v1: > > - Call pci_device_disable() at return points to clean up properly. > > - Changed all dev_* print macros to pci_* macros. > > - Removed superfluous variable initializations. > > > > Doug Meyer (2): > > NTB: Migrate PCI Constants to Cannonical PCI Header > > NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On > > > > drivers/ntb/hw/mscc/ntb_hw_switchtec.c | 3 +- > > drivers/pci/quirks.c | 197 +++++++++++++++++++++++++++++++++ > > drivers/pci/switch/switchtec.c | 15 ++- > > include/linux/pci_ids.h | 32 ++++++ > > include/linux/switchtec.h | 4 - > > 5 files changed, 238 insertions(+), 13 deletions(-) > > I applied these with Logan's reviewed-by to my pci/switchtec branch for > v4.19 with the following updates: > > - You removed the SPDX tag from drivers/pci/switch/switchtec.c. I assume > that was a mistake, so I restored it. > > - I moved the PCI_VENDOR_ID_MICROSEMI definition to keep the file sorted. > > - I dropped the Device ID definitions per the policy at the top of the > file (which I mentioned on your v1 posting). > > - I converted all the Device ID definition uses to raw hex constants. I > noticed that the following were defined by your patch, but not used: > > PCI_DEVICE_ID_MICROSEMI_PSX24XG3 > PCI_DEVICE_ID_MICROSEMI_PSX32XG3 > > I can't tell whether the quirk is supposed to apply to them or not. > > Please review and holler if I broke something. > > This touches drivers/ntb/hw/..., which isn't my area. Let me know if you'd > rather take these through a different tree. I also: - Fixed the sparse warnings reported by the kbuild test robot. - Removed some (void * __iomem) casts in the ioread32() parameters and the (u64) cast on the return value. I noticed that switchtec_ntb_init_sndev() does an ioread64() of ep_map instead of two ioread32() calls. I suspect both places could and should do it the same way. - Used %02x.%d instead of %02d.%d when printing the DMA alias, so it matches the other places that print PCI device addresses. - Replaced DECLARE_PCI_FIXUP_CLASS_FINAL with DECLARE_PCI_FIXUP_FINAL, since it should be sufficient to filter on Vendor and Device ID, without worrying about the class. Here's the diff from what you originally posted to what's currently on my pci/switchtec branch: diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index f178cecdc001..d54a182a09cf 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -4787,14 +4787,13 @@ static void quirk_switchtec_ntb_dma_alias(struct pci_dev *pdev) pci_info(pdev, "Setting Switchtec proxy ID aliases\n"); mmio_ntb = mmio + SWITCHTEC_GAS_NTB_OFFSET; - mmio_ctrl = (void * __iomem) mmio_ntb + SWITCHTEC_NTB_REG_CTRL_OFFSET; + mmio_ctrl = (void __iomem *) mmio_ntb + SWITCHTEC_NTB_REG_CTRL_OFFSET; mmio_sys_info = mmio + SWITCHTEC_GAS_SYS_INFO_OFFSET; partition = ioread8(&mmio_ntb->partition_id); - partition_map = (u64) ioread32((void * __iomem) &mmio_ntb->ep_map); - partition_map |= - ((u64) ioread32((void * __iomem) &mmio_ntb->ep_map + 4)) << 32; + partition_map = ioread32(&mmio_ntb->ep_map); + partition_map |= ((u64) ioread32(&mmio_ntb->ep_map + 4)) << 32; partition_map &= ~(1ULL << partition); for (pp = 0; pp < (sizeof(partition_map) * 8); pp++) { @@ -4829,7 +4828,7 @@ static void quirk_switchtec_ntb_dma_alias(struct pci_dev *pdev) rid_entry = ioread32(&mmio_peer_ctrl->req_id_table[te]); devfn = (rid_entry >> 1) & 0xFF; pci_dbg(pdev, - "Aliasing Partition %d Proxy ID %02d.%d\n", + "Aliasing Partition %d Proxy ID %02x.%d\n", pp, PCI_SLOT(devfn), PCI_FUNC(devfn)); pci_add_dma_alias(pdev, devfn); } @@ -4838,115 +4837,59 @@ static void quirk_switchtec_ntb_dma_alias(struct pci_dev *pdev) pci_iounmap(pdev, mmio); pci_disable_device(pdev); } -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFX24XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFX32XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFX48XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFX64XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFX80XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFX96XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PSX48XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PSX64XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PSX80XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PSX96XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PAX24XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PAX32XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PAX48XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PAX64XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PAX80XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PAX96XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXL24XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXL32XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXL48XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXL64XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXL80XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXL96XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXI24XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXI32XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXI48XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXI64XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXI80XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); -DECLARE_PCI_FIXUP_CLASS_FINAL(PCI_VENDOR_ID_MICROSEMI, - PCI_DEVICE_ID_MICROSEMI_PFXI96XG3, - PCI_CLASS_BRIDGE_OTHER, 8, - quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8531, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8532, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8533, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8534, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8535, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8536, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8543, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8544, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8545, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8546, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8551, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8552, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8553, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8554, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8555, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8556, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8561, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8562, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8563, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8564, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8565, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8566, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8571, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8572, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8573, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8574, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8575, + quirk_switchtec_ntb_dma_alias); +DECLARE_PCI_FIXUP_FINAL(PCI_VENDOR_ID_MICROSEMI, 0x8576, + quirk_switchtec_ntb_dma_alias); diff --git a/drivers/pci/switch/switchtec.c b/drivers/pci/switch/switchtec.c index 07a03b90498e..9940cc70f38b 100644 --- a/drivers/pci/switch/switchtec.c +++ b/drivers/pci/switch/switchtec.c @@ -1,3 +1,4 @@ +// SPDX-License-Identifier: GPL-2.0 /* * Microsemi Switchtec(tm) PCIe Management Driver * Copyright (c) 2017, Microsemi Corporation diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h index 7bc61402d28b..80aec5b9a6c1 100644 --- a/include/linux/pci_ids.h +++ b/include/linux/pci_ids.h @@ -1668,6 +1668,7 @@ #define PCI_DEVICE_ID_COMPEX_ENET100VG4 0x0112 #define PCI_VENDOR_ID_PMC_Sierra 0x11f8 +#define PCI_VENDOR_ID_MICROSEMI 0x11f8 #define PCI_VENDOR_ID_RP 0x11fe #define PCI_DEVICE_ID_RP32INTF 0x0001 @@ -3082,36 +3083,4 @@ #define PCI_VENDOR_ID_OCZ 0x1b85 -#define PCI_VENDOR_ID_MICROSEMI 0x11f8 -#define PCI_DEVICE_ID_MICROSEMI_PFX24XG3 0x8531 -#define PCI_DEVICE_ID_MICROSEMI_PFX32XG3 0x8532 -#define PCI_DEVICE_ID_MICROSEMI_PFX48XG3 0x8533 -#define PCI_DEVICE_ID_MICROSEMI_PFX64XG3 0x8534 -#define PCI_DEVICE_ID_MICROSEMI_PFX80XG3 0x8535 -#define PCI_DEVICE_ID_MICROSEMI_PFX96XG3 0x8536 -#define PCI_DEVICE_ID_MICROSEMI_PSX24XG3 0x8541 -#define PCI_DEVICE_ID_MICROSEMI_PSX32XG3 0x8542 -#define PCI_DEVICE_ID_MICROSEMI_PSX48XG3 0x8543 -#define PCI_DEVICE_ID_MICROSEMI_PSX64XG3 0x8544 -#define PCI_DEVICE_ID_MICROSEMI_PSX80XG3 0x8545 -#define PCI_DEVICE_ID_MICROSEMI_PSX96XG3 0x8546 -#define PCI_DEVICE_ID_MICROSEMI_PAX24XG3 0x8551 -#define PCI_DEVICE_ID_MICROSEMI_PAX32XG3 0x8552 -#define PCI_DEVICE_ID_MICROSEMI_PAX48XG3 0x8553 -#define PCI_DEVICE_ID_MICROSEMI_PAX64XG3 0x8554 -#define PCI_DEVICE_ID_MICROSEMI_PAX80XG3 0x8555 -#define PCI_DEVICE_ID_MICROSEMI_PAX96XG3 0x8556 -#define PCI_DEVICE_ID_MICROSEMI_PFXL24XG3 0x8561 -#define PCI_DEVICE_ID_MICROSEMI_PFXL32XG3 0x8562 -#define PCI_DEVICE_ID_MICROSEMI_PFXL48XG3 0x8563 -#define PCI_DEVICE_ID_MICROSEMI_PFXL64XG3 0x8564 -#define PCI_DEVICE_ID_MICROSEMI_PFXL80XG3 0x8565 -#define PCI_DEVICE_ID_MICROSEMI_PFXL96XG3 0x8566 -#define PCI_DEVICE_ID_MICROSEMI_PFXI24XG3 0x8571 -#define PCI_DEVICE_ID_MICROSEMI_PFXI32XG3 0x8572 -#define PCI_DEVICE_ID_MICROSEMI_PFXI48XG3 0x8573 -#define PCI_DEVICE_ID_MICROSEMI_PFXI64XG3 0x8574 -#define PCI_DEVICE_ID_MICROSEMI_PFXI80XG3 0x8575 -#define PCI_DEVICE_ID_MICROSEMI_PFXI96XG3 0x8576 - #endif /* _LINUX_PCI_IDS_H */ ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB 2018-06-30 1:50 ` [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB Bjorn Helgaas 2018-06-30 18:57 ` Bjorn Helgaas @ 2018-07-05 16:56 ` Logan Gunthorpe 1 sibling, 0 replies; 9+ messages in thread From: Logan Gunthorpe @ 2018-07-05 16:56 UTC (permalink / raw) To: Bjorn Helgaas, dmeyer Cc: kurt.schwemmer, linux-pci, linux-ntb, bhelgaas, jdmason, dave.jiang, allenbh, linux-kernel Thanks for cleaning this up Bjorn. On 29/06/18 07:50 PM, Bjorn Helgaas wrote: > Please review and holler if I broke something. I pulled your branch, reviewed it and lightly tested it. Everything looks good to me. > This touches drivers/ntb/hw/..., which isn't my area. Let me know if you'd > rather take these through a different tree. The NTB change is very minor, and there's nothing this will conflict with so I think it's fine to go through the PCI tree. Logan ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-07-05 16:56 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-05-23 20:18 [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB dmeyer 2018-05-23 20:18 ` [PATCH v2 1/2] NTB: Migrate PCI Constants to Cannonical PCI Header dmeyer 2018-05-23 20:26 ` Logan Gunthorpe 2018-05-23 20:39 ` Doug Meyer 2018-05-23 20:18 ` [PATCH v2 2/2] NTB: PCI Quirk to Enable Switchtec NT Functionality with IOMMU On dmeyer 2018-05-23 20:32 ` Logan Gunthorpe 2018-06-30 1:50 ` [PATCH v2 0/2] PCI Quirk Patchset for Microsemi Switchtec NTB Bjorn Helgaas 2018-06-30 18:57 ` Bjorn Helgaas 2018-07-05 16:56 ` Logan Gunthorpe
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).