Re: [PATCH 3/3] PCI: Honor window granularity during resource distribution

[sashiko-bot@kernel.org]

> When a bus below a hotplug bridge has BARs smaller than the window
> granularity, removing them leaves available[i].start unaligned.
> pci_bus_distribute_available_resources() then aligns the start up for the
> downstream bridge but keeps the end, so the size passed to
> adjust_bridge_window() has a sub-granularity tail. The bridge window ends
> up not being a multiple of the window granularity.
> 
> Align the size down to the window granularity after aligning the start.
> 
> Consider a hotplug Root Port (bus 01) with a Switch Upstream Port (bus 02)
> below it that has an e1000 and an ivshmem with a 64k pref BAR as its other
> Functions. When the switch is hot-added (emulated with a remove and
> rescan), /proc/iomem before and after this change is:
> 
>    faa00000-fe9fffff : PCI Bus 0000:01
> [ ... ]
> of 01:00.1 and 01:00.2.
> 
> Fixes: 9db0b9b6a142 ("PCI: Take other bus devices into account when distributing resources")
> Reported-by: Sashiko <sashiko-bot@kernel.org>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261109144553.2516-1-ilpo.jarvinen@linux.intel.com?part=3