lkml.org 
[lkml]   [2019]   [Nov]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 064/211] PCI: vmd: Detach resources after stopping root bus
    Date
    From: Jon Derrick <jonathan.derrick@intel.com>

    [ Upstream commit dc8af3a827df6d4bb925d3b81b7ec94a7cce9482 ]

    The VMD removal path calls pci_stop_root_busi(), which tears down the pcie
    tree, including detaching all of the attached drivers. During driver
    detachment, devices may use pci_release_region() to release resources.
    This path relies on the resource being accessible in resource tree.

    By detaching the child domain from the parent resource domain prior to
    stopping the bus, we are preventing the list traversal from finding the
    resource to be freed. If we instead detach the resource after stopping
    the bus, we will have properly freed the resource and detaching is
    simply accounting at that point.

    Without this order, the resource is never freed and is orphaned on VMD
    removal, leading to a warning:

    [ 181.940162] Trying to free nonexistent resource <e5a10000-e5a13fff>

    Fixes: 2c2c5c5cd213 ("x86/PCI: VMD: Attach VMD resources to parent domain's resource tree")
    Signed-off-by: Jon Derrick <jonathan.derrick@intel.com>
    [lorenzo.pieralisi@arm.com: updated commit log]
    Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
    Reviewed-by: Keith Busch <keith.busch@intel.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/pci/host/vmd.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/drivers/pci/host/vmd.c b/drivers/pci/host/vmd.c
    index 2537b022f42d4..af6d5da10ea5f 100644
    --- a/drivers/pci/host/vmd.c
    +++ b/drivers/pci/host/vmd.c
    @@ -753,12 +753,12 @@ static void vmd_remove(struct pci_dev *dev)
    {
    struct vmd_dev *vmd = pci_get_drvdata(dev);

    - vmd_detach_resources(vmd);
    sysfs_remove_link(&vmd->dev->dev.kobj, "domain");
    pci_stop_root_bus(vmd->bus);
    pci_remove_root_bus(vmd->bus);
    vmd_cleanup_srcu(vmd);
    vmd_teardown_dma_ops(vmd);
    + vmd_detach_resources(vmd);
    irq_domain_remove(vmd->irq_domain);
    }

    --
    2.20.1


    \
     
     \ /
      Last update: 2019-11-27 22:36    [W:4.078 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site