Message ID | 20221208190341.1560157-5-helgaas@kernel.org (mailing list archive) |
---|---|
State | Accepted |
Commit | 567ae8230107b79658ad1d577521818418d41199 |
Headers | show |
Series | PCI: Continue E820 vs host bridge window saga | expand |
On Thu, Dec 08, 2022 at 01:03:41PM -0600, Bjorn Helgaas wrote: > From: Bjorn Helgaas <bhelgaas@google.com> > > Add missing word in the log message: > > - ... so future kernels can this automatically > + ... so future kernels can do this automatically ... > printk(KERN_INFO "PCI: %s E820 reservations for host bridge windows\n", > pci_use_e820 ? "Using" : "Ignoring"); > if (pci_probe & (PCI_NO_E820 | PCI_USE_E820)) > - printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can this automatically\n"); > + printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can do this automatically\n"); Wondering if we can change printk(KERN_LVL) to pr_lvl() in this file.
On Fri, Dec 09, 2022 at 08:43:06PM +0200, Andy Shevchenko wrote: > On Thu, Dec 08, 2022 at 01:03:41PM -0600, Bjorn Helgaas wrote: > > From: Bjorn Helgaas <bhelgaas@google.com> > > > > Add missing word in the log message: > > > > - ... so future kernels can this automatically > > + ... so future kernels can do this automatically > > ... > > > printk(KERN_INFO "PCI: %s E820 reservations for host bridge windows\n", > > pci_use_e820 ? "Using" : "Ignoring"); > > if (pci_probe & (PCI_NO_E820 | PCI_USE_E820)) > > - printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can this automatically\n"); > > + printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can do this automatically\n"); > > Wondering if we can change printk(KERN_LVL) to pr_lvl() in this file. Sure! How about this? commit 7058cdb558d5 ("x86/PCI: Use pr_info() when possible") Author: Bjorn Helgaas <bhelgaas@google.com> Date: Fri Dec 9 14:41:27 2022 -0600 x86/PCI: Use pr_info() when possible Use pr_info() and similar when possible. No functional change intended. Suggested-by: Andy Shevchenko <andriy.shevchenko@intel.com> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c index 83dfea9e9894..ea2eb2ec90e2 100644 --- a/arch/x86/pci/acpi.c +++ b/arch/x86/pci/acpi.c @@ -1,4 +1,7 @@ // SPDX-License-Identifier: GPL-2.0 + +#define pr_fmt(fmt) "PCI: " fmt + #include <linux/pci.h> #include <linux/acpi.h> #include <linux/init.h> @@ -37,15 +40,15 @@ static int __init set_nouse_crs(const struct dmi_system_id *id) static int __init set_ignore_seg(const struct dmi_system_id *id) { - printk(KERN_INFO "PCI: %s detected: ignoring ACPI _SEG\n", id->ident); + pr_info("%s detected: ignoring ACPI _SEG\n", id->ident); pci_ignore_seg = true; return 0; } static int __init set_no_e820(const struct dmi_system_id *id) { - printk(KERN_INFO "PCI: %s detected: not clipping E820 regions from _CRS\n", - id->ident); + pr_info("%s detected: not clipping E820 regions from _CRS\n", + id->ident); pci_use_e820 = false; return 0; } @@ -231,10 +234,9 @@ void __init pci_acpi_crs_quirks(void) else if (pci_probe & PCI_USE__CRS) pci_use_crs = true; - printk(KERN_INFO "PCI: %s host bridge windows from ACPI; " - "if necessary, use \"pci=%s\" and report a bug\n", - pci_use_crs ? "Using" : "Ignoring", - pci_use_crs ? "nocrs" : "use_crs"); + pr_info("%s host bridge windows from ACPI; if necessary, use \"pci=%s\" and report a bug\n", + pci_use_crs ? "Using" : "Ignoring", + pci_use_crs ? "nocrs" : "use_crs"); /* "pci=use_e820"/"pci=no_e820" on the kernel cmdline takes precedence */ if (pci_probe & PCI_NO_E820) @@ -242,19 +244,17 @@ void __init pci_acpi_crs_quirks(void) else if (pci_probe & PCI_USE_E820) pci_use_e820 = true; - printk(KERN_INFO "PCI: %s E820 reservations for host bridge windows\n", - pci_use_e820 ? "Using" : "Ignoring"); + pr_info("%s E820 reservations for host bridge windows\n", + pci_use_e820 ? "Using" : "Ignoring"); if (pci_probe & (PCI_NO_E820 | PCI_USE_E820)) - printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can do this automatically\n"); + pr_info("Please notify linux-pci@vger.kernel.org so future kernels can do this automatically\n"); } #ifdef CONFIG_PCI_MMCONFIG static int check_segment(u16 seg, struct device *dev, char *estr) { if (seg) { - dev_err(dev, - "%s can't access PCI configuration " - "space under this host bridge.\n", + dev_err(dev, "%s can't access configuration space under this host bridge\n", estr); return -EIO; } @@ -264,9 +264,7 @@ static int check_segment(u16 seg, struct device *dev, char *estr) * just can't access extended configuration space of * devices under this host bridge. */ - dev_warn(dev, - "%s can't access extended PCI configuration " - "space under this bridge.\n", + dev_warn(dev, "%s can't access extended configuration space under this bridge\n", estr); return 0; @@ -421,9 +419,8 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root) root->segment = domain = 0; if (domain && !pci_domains_supported) { - printk(KERN_WARNING "pci_bus %04x:%02x: " - "ignored (multiple domains not supported)\n", - domain, busnum); + pr_warn("pci_bus %04x:%02x: ignored (multiple domains not supported)\n", + domain, busnum); return NULL; } @@ -491,7 +488,7 @@ int __init pci_acpi_init(void) if (acpi_noirq) return -ENODEV; - printk(KERN_INFO "PCI: Using ACPI for IRQ routing\n"); + pr_info("Using ACPI for IRQ routing\n"); acpi_irq_penalty_init(); pcibios_enable_irq = acpi_pci_irq_enable; pcibios_disable_irq = acpi_pci_irq_disable; @@ -503,7 +500,7 @@ int __init pci_acpi_init(void) * also do it here in case there are still broken drivers that * don't use pci_enable_device(). */ - printk(KERN_INFO "PCI: Routing PCI interrupts for all devices because \"pci=routeirq\" specified\n"); + pr_info("Routing PCI interrupts for all devices because \"pci=routeirq\" specified\n"); for_each_pci_dev(dev) acpi_pci_irq_enable(dev); }
On Fri, Dec 09, 2022 at 02:51:31PM -0600, Bjorn Helgaas wrote: > On Fri, Dec 09, 2022 at 08:43:06PM +0200, Andy Shevchenko wrote: > > On Thu, Dec 08, 2022 at 01:03:41PM -0600, Bjorn Helgaas wrote: ... > > Wondering if we can change printk(KERN_LVL) to pr_lvl() in this file. > > Sure! How about this? LGTM, you can add Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> to it if you wish. > commit 7058cdb558d5 ("x86/PCI: Use pr_info() when possible") > Author: Bjorn Helgaas <bhelgaas@google.com> > Date: Fri Dec 9 14:41:27 2022 -0600 > > x86/PCI: Use pr_info() when possible > > Use pr_info() and similar when possible. No functional change intended. > > Suggested-by: Andy Shevchenko <andriy.shevchenko@intel.com> I prefer @linux.intel.com. > Signed-off-by: Bjorn Helgaas <bhelgaas@google.com> > diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c > index 83dfea9e9894..ea2eb2ec90e2 100644 > --- a/arch/x86/pci/acpi.c > +++ b/arch/x86/pci/acpi.c > @@ -1,4 +1,7 @@ > // SPDX-License-Identifier: GPL-2.0 > + > +#define pr_fmt(fmt) "PCI: " fmt > + > #include <linux/pci.h> > #include <linux/acpi.h> > #include <linux/init.h> > @@ -37,15 +40,15 @@ static int __init set_nouse_crs(const struct dmi_system_id *id) > > static int __init set_ignore_seg(const struct dmi_system_id *id) > { > - printk(KERN_INFO "PCI: %s detected: ignoring ACPI _SEG\n", id->ident); > + pr_info("%s detected: ignoring ACPI _SEG\n", id->ident); > pci_ignore_seg = true; > return 0; > } > > static int __init set_no_e820(const struct dmi_system_id *id) > { > - printk(KERN_INFO "PCI: %s detected: not clipping E820 regions from _CRS\n", > - id->ident); > + pr_info("%s detected: not clipping E820 regions from _CRS\n", > + id->ident); > pci_use_e820 = false; > return 0; > } > @@ -231,10 +234,9 @@ void __init pci_acpi_crs_quirks(void) > else if (pci_probe & PCI_USE__CRS) > pci_use_crs = true; > > - printk(KERN_INFO "PCI: %s host bridge windows from ACPI; " > - "if necessary, use \"pci=%s\" and report a bug\n", > - pci_use_crs ? "Using" : "Ignoring", > - pci_use_crs ? "nocrs" : "use_crs"); > + pr_info("%s host bridge windows from ACPI; if necessary, use \"pci=%s\" and report a bug\n", > + pci_use_crs ? "Using" : "Ignoring", > + pci_use_crs ? "nocrs" : "use_crs"); > > /* "pci=use_e820"/"pci=no_e820" on the kernel cmdline takes precedence */ > if (pci_probe & PCI_NO_E820) > @@ -242,19 +244,17 @@ void __init pci_acpi_crs_quirks(void) > else if (pci_probe & PCI_USE_E820) > pci_use_e820 = true; > > - printk(KERN_INFO "PCI: %s E820 reservations for host bridge windows\n", > - pci_use_e820 ? "Using" : "Ignoring"); > + pr_info("%s E820 reservations for host bridge windows\n", > + pci_use_e820 ? "Using" : "Ignoring"); > if (pci_probe & (PCI_NO_E820 | PCI_USE_E820)) > - printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can do this automatically\n"); > + pr_info("Please notify linux-pci@vger.kernel.org so future kernels can do this automatically\n"); > } > > #ifdef CONFIG_PCI_MMCONFIG > static int check_segment(u16 seg, struct device *dev, char *estr) > { > if (seg) { > - dev_err(dev, > - "%s can't access PCI configuration " > - "space under this host bridge.\n", > + dev_err(dev, "%s can't access configuration space under this host bridge\n", > estr); > return -EIO; > } > @@ -264,9 +264,7 @@ static int check_segment(u16 seg, struct device *dev, char *estr) > * just can't access extended configuration space of > * devices under this host bridge. > */ > - dev_warn(dev, > - "%s can't access extended PCI configuration " > - "space under this bridge.\n", > + dev_warn(dev, "%s can't access extended configuration space under this bridge\n", > estr); > > return 0; > @@ -421,9 +419,8 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root) > root->segment = domain = 0; > > if (domain && !pci_domains_supported) { > - printk(KERN_WARNING "pci_bus %04x:%02x: " > - "ignored (multiple domains not supported)\n", > - domain, busnum); > + pr_warn("pci_bus %04x:%02x: ignored (multiple domains not supported)\n", > + domain, busnum); > return NULL; > } > > @@ -491,7 +488,7 @@ int __init pci_acpi_init(void) > if (acpi_noirq) > return -ENODEV; > > - printk(KERN_INFO "PCI: Using ACPI for IRQ routing\n"); > + pr_info("Using ACPI for IRQ routing\n"); > acpi_irq_penalty_init(); > pcibios_enable_irq = acpi_pci_irq_enable; > pcibios_disable_irq = acpi_pci_irq_disable; > @@ -503,7 +500,7 @@ int __init pci_acpi_init(void) > * also do it here in case there are still broken drivers that > * don't use pci_enable_device(). > */ > - printk(KERN_INFO "PCI: Routing PCI interrupts for all devices because \"pci=routeirq\" specified\n"); > + pr_info("Routing PCI interrupts for all devices because \"pci=routeirq\" specified\n"); > for_each_pci_dev(dev) > acpi_pci_irq_enable(dev); > }
On Fri, Dec 09, 2022 at 11:35:57PM +0200, Andy Shevchenko wrote: > On Fri, Dec 09, 2022 at 02:51:31PM -0600, Bjorn Helgaas wrote: > > On Fri, Dec 09, 2022 at 08:43:06PM +0200, Andy Shevchenko wrote: > > > On Thu, Dec 08, 2022 at 01:03:41PM -0600, Bjorn Helgaas wrote: > > ... > > > > Wondering if we can change printk(KERN_LVL) to pr_lvl() in this file. > > > > Sure! How about this? > > LGTM, you can add > Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > to it if you wish. Done, thanks! > > Suggested-by: Andy Shevchenko <andriy.shevchenko@intel.com> > > I prefer @linux.intel.com. Oops, sorry, I should have known that. I had copied that from the From: line of your email (https://lore.kernel.org/r/Y5OBupWBghHfvG/h@smile.fi.intel.com) Bjorn
On Fri, Dec 09, 2022 at 03:52:11PM -0600, Bjorn Helgaas wrote: > On Fri, Dec 09, 2022 at 11:35:57PM +0200, Andy Shevchenko wrote: > > On Fri, Dec 09, 2022 at 02:51:31PM -0600, Bjorn Helgaas wrote: > > > On Fri, Dec 09, 2022 at 08:43:06PM +0200, Andy Shevchenko wrote: ... > > > Suggested-by: Andy Shevchenko <andriy.shevchenko@intel.com> > > > > I prefer @linux.intel.com. > > Oops, sorry, I should have known that. I had copied that from > the From: line of your email > (https://lore.kernel.org/r/Y5OBupWBghHfvG/h@smile.fi.intel.com) I understand. It's not your fault.
diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c index 2f82480fd430..83dfea9e9894 100644 --- a/arch/x86/pci/acpi.c +++ b/arch/x86/pci/acpi.c @@ -245,7 +245,7 @@ void __init pci_acpi_crs_quirks(void) printk(KERN_INFO "PCI: %s E820 reservations for host bridge windows\n", pci_use_e820 ? "Using" : "Ignoring"); if (pci_probe & (PCI_NO_E820 | PCI_USE_E820)) - printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can this automatically\n"); + printk(KERN_INFO "PCI: Please notify linux-pci@vger.kernel.org so future kernels can do this automatically\n"); } #ifdef CONFIG_PCI_MMCONFIG