Merge tag 'pci-v4.12-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci

Pull PCI updates from Bjorn Helgaas:

 - add framework for supporting PCIe devices in Endpoint mode (Kishon
   Vijay Abraham I)

 - use non-postable PCI config space mappings when possible (Lorenzo
   Pieralisi)

 - clean up and unify mmap of PCI BARs (David Woodhouse)

 - export and unify Function Level Reset support (Christoph Hellwig)

 - avoid FLR for Intel 82579 NICs (Sasha Neftin)

 - add pci_request_irq() and pci_free_irq() helpers (Christoph Hellwig)

 - short-circuit config access failures for disconnected devices (Keith
   Busch)

 - remove D3 sleep delay when possible (Adrian Hunter)

 - freeze PME scan before suspending devices (Lukas Wunner)

 - stop disabling MSI/MSI-X in pci_device_shutdown() (Prarit Bhargava)

 - disable boot interrupt quirk for ASUS M2N-LR (Stefan Assmann)

 - add arch-specific alignment control to improve device passthrough by
   avoiding multiple BARs in a page (Yongji Xie)

 - add sysfs sriov_drivers_autoprobe to control VF driver binding
   (Bodong Wang)

 - allow slots below PCI-to-PCIe "reverse bridges" (Bjorn Helgaas)

 - fix crashes when unbinding host controllers that don't support
   removal (Brian Norris)

 - add driver for MicroSemi Switchtec management interface (Logan
   Gunthorpe)

 - add driver for Faraday Technology FTPCI100 host bridge (Linus
   Walleij)

 - add i.MX7D support (Andrey Smirnov)

 - use generic MSI support for Aardvark (Thomas Petazzoni)

 - make Rockchip driver modular (Brian Norris)

 - advertise 128-byte Read Completion Boundary support for Rockchip
   (Shawn Lin)

 - advertise PCI_EXP_LNKSTA_SLC for Rockchip root port (Shawn Lin)

 - convert atomic_t to refcount_t in HV driver (Elena Reshetova)

 - add CPU IRQ affinity in HV driver (K. Y. Srinivasan)

 - fix PCI bus removal in HV driver (Long Li)

 - add support for ThunderX2 DMA alias topology (Jayachandran C)

 - add ThunderX pass2.x 2nd node MCFG quirk (Tomasz Nowicki)

 - add ITE 8893 bridge DMA alias quirk (Jarod Wilson)

 - restrict Cavium ACS quirk only to CN81xx/CN83xx/CN88xx devices
   (Manish Jaggi)

* tag 'pci-v4.12-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci: (146 commits)
  PCI: Don't allow unbinding host controllers that aren't prepared
  ARM: DRA7: clockdomain: Change the CLKTRCTRL of CM_PCIE_CLKSTCTRL to SW_WKUP
  MAINTAINERS: Add PCI Endpoint maintainer
  Documentation: PCI: Add userguide for PCI endpoint test function
  tools: PCI: Add sample test script to invoke pcitest
  tools: PCI: Add a userspace tool to test PCI endpoint
  Documentation: misc-devices: Add Documentation for pci-endpoint-test driver
  misc: Add host side PCI driver for PCI test function device
  PCI: Add device IDs for DRA74x and DRA72x
  dt-bindings: PCI: dra7xx: Add DT bindings to enable unaligned access
  PCI: dwc: dra7xx: Workaround for errata id i870
  dt-bindings: PCI: dra7xx: Add DT bindings for PCI dra7xx EP mode
  PCI: dwc: dra7xx: Add EP mode support
  PCI: dwc: dra7xx: Facilitate wrapper and MSI interrupts to be enabled independently
  dt-bindings: PCI: Add DT bindings for PCI designware EP mode
  PCI: dwc: designware: Add EP mode support
  Documentation: PCI: Add binding documentation for pci-test endpoint function
  ixgbe: Use pcie_flr() instead of duplicating it
  IB/hfi1: Use pcie_flr() instead of duplicating it
  PCI: imx6: Fix spelling mistake: "contol" -> "control"
  ...

1 files changed, 35 insertions, 11 deletions

diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyperv.c
index ada98569b78e..84936383e269 100644
--- a/drivers/pci/host/pci-hyperv.c
+++ b/drivers/pci/host/pci-hyperv.c
@@ -56,6 +56,7 @@
 #include <asm/apic.h>
 #include <linux/msi.h>
 #include <linux/hyperv.h>
+#include <linux/refcount.h>
 #include <asm/mshyperv.h>
 
 /*
@@ -72,6 +73,7 @@ enum {
         PCI_PROTOCOL_VERSION_CURRENT = PCI_PROTOCOL_VERSION_1_1
 };
 
+#define CPU_AFFINITY_ALL        -1ULL
 #define PCI_CONFIG_MMIO_LENGTH  0x2000
 #define CFG_PAGE_OFFSET 0x1000
 #define CFG_PAGE_SIZE (PCI_CONFIG_MMIO_LENGTH - CFG_PAGE_OFFSET)
@@ -350,6 +352,7 @@ enum hv_pcibus_state {
         hv_pcibus_init = 0,
         hv_pcibus_probed,
         hv_pcibus_installed,
+        hv_pcibus_removed,
         hv_pcibus_maximum
 };
 
@@ -421,7 +424,7 @@ enum hv_pcidev_ref_reason {
 struct hv_pci_dev {
         /* List protected by pci_rescan_remove_lock */
         struct list_head list_entry;
-        atomic_t refs;
+        refcount_t refs;
         enum hv_pcichild_state state;
         struct pci_function_description desc;
         bool reported_missing;
@@ -876,7 +879,7 @@ static void hv_compose_msi_msg(struct irq_data *data, struct msi_msg *msg)
                 hv_int_desc_free(hpdev, int_desc);
         }
 
-        int_desc = kzalloc(sizeof(*int_desc), GFP_KERNEL);
+        int_desc = kzalloc(sizeof(*int_desc), GFP_ATOMIC);
         if (!int_desc)
                 goto drop_reference;
 
@@ -897,9 +900,13 @@ static void hv_compose_msi_msg(struct irq_data *data, struct msi_msg *msg)
          * processors because Hyper-V only supports 64 in a guest.
          */
         affinity = irq_data_get_affinity_mask(data);
-        for_each_cpu_and(cpu, affinity, cpu_online_mask) {
-                int_pkt->int_desc.cpu_mask |=
-                        (1ULL << vmbus_cpu_number_to_vp_number(cpu));
+        if (cpumask_weight(affinity) >= 32) {
+                int_pkt->int_desc.cpu_mask = CPU_AFFINITY_ALL;
+        } else {
+                for_each_cpu_and(cpu, affinity, cpu_online_mask) {
+                        int_pkt->int_desc.cpu_mask |=
+                                (1ULL << vmbus_cpu_number_to_vp_number(cpu));
+                }
         }
 
         ret = vmbus_sendpacket(hpdev->hbus->hdev->channel, int_pkt,
@@ -1208,9 +1215,11 @@ static int create_root_hv_pci_bus(struct hv_pcibus_device *hbus)
         hbus->pci_bus->msi = &hbus->msi_chip;
         hbus->pci_bus->msi->dev = &hbus->hdev->device;
 
+        pci_lock_rescan_remove();
         pci_scan_child_bus(hbus->pci_bus);
         pci_bus_assign_resources(hbus->pci_bus);
         pci_bus_add_devices(hbus->pci_bus);
+        pci_unlock_rescan_remove();
         hbus->state = hv_pcibus_installed;
         return 0;
 }
@@ -1254,13 +1263,13 @@ static void q_resource_requirements(void *context, struct pci_response *resp,
 static void get_pcichild(struct hv_pci_dev *hpdev,
                             enum hv_pcidev_ref_reason reason)
 {
-        atomic_inc(&hpdev->refs);
+        refcount_inc(&hpdev->refs);
 }
 
 static void put_pcichild(struct hv_pci_dev *hpdev,
                             enum hv_pcidev_ref_reason reason)
 {
-        if (atomic_dec_and_test(&hpdev->refs))
+        if (refcount_dec_and_test(&hpdev->refs))
                 kfree(hpdev);
 }
 
@@ -1314,7 +1323,7 @@ static struct hv_pci_dev *new_pcichild_device(struct hv_pcibus_device *hbus,
         wait_for_completion(&comp_pkt.host_event);
 
         hpdev->desc = *desc;
-        get_pcichild(hpdev, hv_pcidev_ref_initial);
+        refcount_set(&hpdev->refs, 1);
         get_pcichild(hpdev, hv_pcidev_ref_childlist);
         spin_lock_irqsave(&hbus->device_list_lock, flags);
 
@@ -1504,13 +1513,24 @@ static void pci_devices_present_work(struct work_struct *work)
                 put_pcichild(hpdev, hv_pcidev_ref_initial);
         }
 
-        /* Tell the core to rescan bus because there may have been changes. */
-        if (hbus->state == hv_pcibus_installed) {
+        switch(hbus->state) {
+        case hv_pcibus_installed:
+                /*
+                * Tell the core to rescan bus
+                * because there may have been changes.
+                */
                 pci_lock_rescan_remove();
                 pci_scan_child_bus(hbus->pci_bus);
                 pci_unlock_rescan_remove();
-        } else {
+                break;
+
+        case hv_pcibus_init:
+        case hv_pcibus_probed:
                 survey_child_resources(hbus);
+                break;
+
+        default:
+                break;
         }
 
         up(&hbus->enum_sem);
@@ -1600,8 +1620,10 @@ static void hv_eject_device_work(struct work_struct *work)
         pdev = pci_get_domain_bus_and_slot(hpdev->hbus->sysdata.domain, 0,
                                            wslot);
         if (pdev) {
+                pci_lock_rescan_remove();
                 pci_stop_and_remove_bus_device(pdev);
                 pci_dev_put(pdev);
+                pci_unlock_rescan_remove();
         }
 
         spin_lock_irqsave(&hpdev->hbus->device_list_lock, flags);
@@ -2185,6 +2207,7 @@ static int hv_pci_probe(struct hv_device *hdev,
         hbus = kzalloc(sizeof(*hbus), GFP_KERNEL);
         if (!hbus)
                 return -ENOMEM;
+        hbus->state = hv_pcibus_init;
 
         /*
          * The PCI bus "domain" is what is called "segment" in ACPI and
@@ -2348,6 +2371,7 @@ static int hv_pci_remove(struct hv_device *hdev)
                 pci_stop_root_bus(hbus->pci_bus);
                 pci_remove_root_bus(hbus->pci_bus);
                 pci_unlock_rescan_remove();
+                hbus->state = hv_pcibus_removed;
         }
 
         hv_pci_bus_exit(hdev);