Re: [PATCH v3 7/8] PCI: Fix composite resource sizing

[sashiko-bot@kernel.org]

> The bridge window sizing algorithm aims to pack child resources back to
> back. With multiple composite child resources whose sizes do not align
> to the calculated minimal alignment for the bridge window, back-to-back
> placement may not be possible. The non-aligning remainder placement is
> limited because it must be adjacent to the rest of the composite
> resource.
> 
> Effectively, two remainder parts may be placed into the same align
> sized block, but sum of their size might not match align. In such case,
> a gap is required to meet the alignment requirement of both resources.
> 
> Add bridge window gap size calculator. Basic rules:
> 
> 1) Gaps are only necessary if there is more than one non-aligning
>    composite resource within a single bridge window.
> [ ... ]
> Reported-by: Nikolas Joshua Britton <nbritton@exabit.io>
> Tested-by: Nikolas Joshua Britton <nbritton@exabit.io>
> Link: https://lore.kernel.org/linux-pci/20260903063124.9316-1-nbritton@exabit.io/
> Reported-by: Maciej Grochowski <Maciej.Grochowski@sony.com>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

Sashiko has reviewed this patch and found no issues. It looks great!

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