From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933322AbcIOAUU (ORCPT ); Wed, 14 Sep 2016 20:20:20 -0400 Received: from p3plsmtps2ded01.prod.phx3.secureserver.net ([208.109.80.58]:34340 "EHLO p3plsmtps2ded01.prod.phx3.secureserver.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1765900AbcIOAUS (ORCPT ); Wed, 14 Sep 2016 20:20:18 -0400 x-originating-ip: 72.167.245.219 From: Long Li To: "K. Y. Srinivasan" , Haiyang Zhang , Bjorn Helgaas Cc: devel@linuxdriverproject.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Long Li Subject: [PATCH 2/2 v2] pci-hyperv: lock pci bus on device eject Date: Wed, 14 Sep 2016 19:10:02 -0700 Message-Id: <1473905402-2375-2-git-send-email-longli@exchange.microsoft.com> X-Mailer: git-send-email 1.7.4.1 In-Reply-To: <1473905402-2375-1-git-send-email-longli@exchange.microsoft.com> References: <1473905402-2375-1-git-send-email-longli@exchange.microsoft.com> X-CMAE-Envelope: MS4wfNw1A2T1ph89LfSer737HxRJYHA8pa+pWAM4YjrhlzfJVc3UxnerbLPDb1o49AdlJ5zJoOqBoKyq8g3SAfAKwOgGxB4T08/RDNpcMaq7gPiQwK4sexB2 A/KLnDx+XYBm+4ojOmzNjddVK4KTHMaTB941YkNv/NTBr9hT698K/Mu/w+CJ1S8yvaRrClCNyTA6Ks4lL2TSMsUN5S40s1OWD0ZWfs8vTr/vvYkv9Lec61GY PIfXC7XpSFSpCWInx5VF5pc0FktIerOyBMy3cOSq5xq6fWcbFteyEg/AqflrQ2z24rXv6wwM6NB1xhy4sluM53BTpZ8FOyrgqCt/Ljcdh1eqt4fySrQGzntb tefj4ltk4M7sXfcEFHBW1CD0+J+1gEBMNUmcMXjgbYD7HoGo4qw= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Long Li A PCI_EJECT message can arrive at the same time we are calling pci_scan_child_bus in the workqueue for the previous PCI_BUS_RELATIONS message or in create_root_hv_pci_bus(), in this case we could potentailly modify the bus from multiple places. Properly lock the bus access. Thanks Dexuan Cui for pointing out the race condition in create_root_hv_pci_bus(). Signed-off-by: Long Li --- drivers/pci/host/pci-hyperv.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/pci/host/pci-hyperv.c b/drivers/pci/host/pci-hyperv.c index 4a37598..33c75c9 100644 --- a/drivers/pci/host/pci-hyperv.c +++ b/drivers/pci/host/pci-hyperv.c @@ -1198,9 +1198,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; } @@ -1590,8 +1592,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(); } memset(&ctxt, 0, sizeof(ctxt)); -- 1.8.5.6