On Mon, Dec 16, 2024 at 07:56:07PM +0200, Ilpo Järvinen wrote: > Hi all, > > This series focuses on PCI resource fitting and assignment algorithms. > I've further changes in works to enable handling resizable BARs better > during resource fitting built on top of these, but that's still WIP and > this series seems way too large as is to have more stuff included. > > First there are small tweaks and fixes to the relaxed tail alignment > code and applying the lessons learned to other similar cases. They are > sort of independent of the rest. Then a large set of pure cleanups and > refactoring that are not intended to make any functional changes. > Finally, starting from "PCI: Extend enable to check for any optional > resource" are again patches that aim to make behavioral changes to fix > bridge window sizing to consider expansion ROM as an optional resource > (to fix a remove/rescan cycle issue) and improve resource fitting > algorithm in general. > > The series includes one of the change from Michał Winiarski > <michal.winiarski@xxxxxxxxx> as these changes also touch the same IOV > checks. > > Please let me know if you'd prefer me to order the changes differently > or split it into smaller chunks. > > > I've extensively tested this series over the hosts in our lab which > have quite heterogeneous PCI setup each. There were no losses of any > important resource. Without pci=realloc, there's some churn in which of > the disabled expansion ROMs gets a scarce memory space assigned (with > pci=realloc, they are all assigned large enough bridge window). > > > Ilpo Järvinen (24): > PCI: Remove add_align overwrite unrelated to size0 > PCI: size0 is unrelated to add_align > PCI: Simplify size1 assignment logic > PCI: Optional bridge window size too may need relaxing > PCI: Fix old_size lower bound in calculate_iosize() too > PCI: Use SZ_* instead of literals in setup-bus.c > PCI: resource_set_range/size() conversions > PCI: Check resource_size() separately > PCI: Add pci_resource_num() helper > PCI: Add dev & res local variables to resource assignment funcs > PCI: Converge return paths in __assign_resources_sorted() > PCI: Refactor pdev_sort_resources() & __dev_sort_resources() > PCI: Use while loop and break instead of gotos > PCI: Rename retval to ret > PCI: Consolidate assignment loop next round preparation > PCI: Remove wrong comment from pci_reassign_resource() > PCI: Add restore_dev_resource() > PCI: Extend enable to check for any optional resource > PCI: Always have realloc_head in __assign_resources_sorted() > PCI: Indicate optional resource assignment failures > PCI: Add debug print when releasing resources before retry > PCI: Use res->parent to check is resource is assigned > PCI: Perform reset_resource() and build fail list in sync > PCI: Rework optional resource handling > > Michał Winiarski (1): > PCI: Add a helper to identify IOV resources > > drivers/pci/pci.h | 44 +++- > drivers/pci/setup-bus.c | 566 +++++++++++++++++++++++----------------- > drivers/pci/setup-res.c | 8 +- > 3 files changed, 364 insertions(+), 254 deletions(-) Applied to pci/resource for v6.15, thanks, Ilpo!
