diff options
author | Bjorn Helgaas <bhelgaas@google.com> | 2023-10-28 13:30:59 -0500 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2023-10-28 13:30:59 -0500 |
commit | 209491885f827f04302c6abad587f45f744e9d90 (patch) | |
tree | ce99319acaa3f289dc2293e6e77e8782ee9a4135 /drivers/pci | |
parent | adfe8d727dc5720da787725e6cf9a80591e5cd78 (diff) | |
parent | 805b196fb3bceda87f785ac34824fff9246c800f (diff) | |
download | lwn-209491885f827f04302c6abad587f45f744e9d90.tar.gz lwn-209491885f827f04302c6abad587f45f744e9d90.zip |
Merge branch 'pci/p2pdma'
- Move struct dev_pagemap (a flexible structure) to end of struct
pci_p2pdma_pagemap to avoid overwriting things after dev_pagemap
(Gustavo A. R. Silva)
* pci/p2pdma:
PCI/P2PDMA: Remove redundant goto
PCI/P2PDMA: Fix undefined behavior bug in struct pci_p2pdma_pagemap
Diffstat (limited to 'drivers/pci')
-rw-r--r-- | drivers/pci/p2pdma.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c index fa7370f9561a..0c361561b855 100644 --- a/drivers/pci/p2pdma.c +++ b/drivers/pci/p2pdma.c @@ -28,9 +28,9 @@ struct pci_p2pdma { }; struct pci_p2pdma_pagemap { - struct dev_pagemap pgmap; struct pci_dev *provider; u64 bus_offset; + struct dev_pagemap pgmap; }; static struct pci_p2pdma_pagemap *to_p2p_pgmap(struct dev_pagemap *pgmap) @@ -837,7 +837,6 @@ void *pci_alloc_p2pmem(struct pci_dev *pdev, size_t size) if (unlikely(!percpu_ref_tryget_live_rcu(ref))) { gen_pool_free(p2pdma->pool, (unsigned long) ret, size); ret = NULL; - goto out; } out: rcu_read_unlock(); |