From: Ilpo Järvinen Date: Thu, 17 Oct 2024 09:55:45 +0000 (+0300) Subject: PCI: Improve pdev_sort_resources() warning message X-Git-Tag: v6.13-rc1~66^2~18^2 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=19f73e938df2b8edfa2e93e0280bd26fd4df5b92;p=thirdparty%2Flinux.git PCI: Improve pdev_sort_resources() warning message Use pci_resource_name() helper in pdev_sort_resources() to print resources in user-friendly format. Also replace the vague "bogus alignment" with a more precise explanation of the problem. Link: https://lore.kernel.org/r/20241017095545.1424-1-ilpo.jarvinen@linux.intel.com Signed-off-by: Ilpo Järvinen Signed-off-by: Bjorn Helgaas Reviewed-by: Philipp Stanner --- diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c index 27da27c59e899..5e00cecf1f1af 100644 --- a/drivers/pci/setup-bus.c +++ b/drivers/pci/setup-bus.c @@ -134,6 +134,7 @@ static void pdev_sort_resources(struct pci_dev *dev, struct list_head *head) int i; pci_dev_for_each_resource(dev, r, i) { + const char *r_name = pci_resource_name(dev, i); struct pci_dev_resource *dev_res, *tmp; resource_size_t r_align; struct list_head *n; @@ -146,8 +147,8 @@ static void pdev_sort_resources(struct pci_dev *dev, struct list_head *head) r_align = pci_resource_alignment(dev, r); if (!r_align) { - pci_warn(dev, "BAR %d: %pR has bogus alignment\n", - i, r); + pci_warn(dev, "%s %pR: alignment must not be zero\n", + r_name, r); continue; }