Re: [PATCH v1 0/6] P2PDMA Cleanup

From: Bjorn Helgaas
Date: Thu Jun 10 2021 - 19:04:34 EST


On Thu, Jun 10, 2021 at 10:06:03AM -0600, Logan Gunthorpe wrote:
> Hi Bjorn,
>
> This patch series consists of the P2PDMA cleanup and prep patches based
> on feedback from my P2PDMA mapping operations series (most recently
> posted at [1]). I've reduced the recipient list of this series to those
> that I thought would be interested or have provided the feedback that
> inspired these patches.
>
> Please consider taking these patches in the near term ahead of my mapping
> ops series. These patches are largely cleanup and other minor fixes. The only
> functional change is Patch 4 which adds a new warning that was suggested by
> Don.
>
> Patch 6 arguably isn't necessary yet as we don't care about sleeping
> yet -- but it'd be a nice to have to reduce the number of prep patches for my
> other series. However, if you don't want to take this patch now, I can
> carry it in my other series.
>
> I'm happy to make further fixes and update this series if anyone finds any
> additional issues on review.
>
> Thanks,
>
> Logan
>
> [1] https://lore.kernel.org/linux-block/20210513223203.5542-1-logang@xxxxxxxxxxxx/
>
> --
>
> Logan Gunthorpe (6):
> PCI/P2PDMA: Rename upstream_bridge_distance() and rework documentation
> PCI/P2PDMA: Use a buffer on the stack for collecting the acs list
> PCI/P2PDMA: Cleanup type for return value of calc_map_type_and_dist()
> PCI/P2PDMA: Print a warning if the host bridge is not in the whitelist
> PCI/P2PDMA: Refactor pci_p2pdma_map_type() to take pagemap and device
> PCI/P2PDMA: Avoid pci_get_slot() which sleeps
>
> drivers/pci/p2pdma.c | 157 +++++++++++++++++++++++++------------------
> 1 file changed, 92 insertions(+), 65 deletions(-)

Applied all 6 to pci/p2pdma for v5.14, thanks!

6389d4374522 ("PCI/P2PDMA: Rename upstream_bridge_distance() and rework doc")
e4ece59abd70 ("PCI/P2PDMA: Collect acs list in stack buffer to avoid sleeping")
f9c125b9eb30 ("PCI/P2PDMA: Use correct calc_map_type_and_dist() return type")
cf201bfe8cdc ("PCI/P2PDMA: Warn if host bridge not in whitelist")
7e2faa1710c4 ("PCI/P2PDMA: Refactor pci_p2pdma_map_type()")
3ec0c3ec2d92 ("PCI/P2PDMA: Avoid pci_get_slot(), which may sleep")