* [PATCH] PCI: use pr_warn_once for ACS parameter parse failure
@ 2026-03-12 11:54 Richard Cheng
2026-03-12 21:52 ` Bjorn Helgaas
0 siblings, 1 reply; 2+ messages in thread
From: Richard Cheng @ 2026-03-12 11:54 UTC (permalink / raw)
To: bhelgaas
Cc: linux-pci, linux-kernel, kristinc, newtonl, kaihengf, tdave,
Richard Cheng
When the ACS command line parameter cannot be parsed, the kernel skips
applying the requested ACS override. This indicates an invalid boot
parameter and should not be logged at informational level.
Use pr_warn_once() so the message is surfaced as a warning while still
avoiding repeated log spam during device enumeration.
Signed-off-by: Richard Cheng <icheng@nvidia.com>
Acked-by: Tushar Dave <tdave@nvidia.com>
---
drivers/pci/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index 13dbb405dc31..4e3b051c0f5f 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -953,7 +953,7 @@ static void __pci_config_acs(struct pci_dev *dev, struct pci_acs *caps,
ret = pci_dev_str_match(dev, p, &p);
if (ret < 0) {
- pr_info_once("PCI: Can't parse ACS command line parameter\n");
+ pr_warn_once("PCI: Can't parse ACS command line parameter\n");
break;
} else if (ret == 1) {
/* Found a match */
--
2.51.0
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] PCI: use pr_warn_once for ACS parameter parse failure
2026-03-12 11:54 [PATCH] PCI: use pr_warn_once for ACS parameter parse failure Richard Cheng
@ 2026-03-12 21:52 ` Bjorn Helgaas
0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Helgaas @ 2026-03-12 21:52 UTC (permalink / raw)
To: Richard Cheng
Cc: bhelgaas, linux-pci, linux-kernel, kristinc, newtonl, kaihengf,
tdave
On Thu, Mar 12, 2026 at 07:54:41PM +0800, Richard Cheng wrote:
> When the ACS command line parameter cannot be parsed, the kernel skips
> applying the requested ACS override. This indicates an invalid boot
> parameter and should not be logged at informational level.
>
> Use pr_warn_once() so the message is surfaced as a warning while still
> avoiding repeated log spam during device enumeration.
>
> Signed-off-by: Richard Cheng <icheng@nvidia.com>
> Acked-by: Tushar Dave <tdave@nvidia.com>
Applied to pci/misc for v7.1, thanks!
> ---
> drivers/pci/pci.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
> index 13dbb405dc31..4e3b051c0f5f 100644
> --- a/drivers/pci/pci.c
> +++ b/drivers/pci/pci.c
> @@ -953,7 +953,7 @@ static void __pci_config_acs(struct pci_dev *dev, struct pci_acs *caps,
>
> ret = pci_dev_str_match(dev, p, &p);
> if (ret < 0) {
> - pr_info_once("PCI: Can't parse ACS command line parameter\n");
> + pr_warn_once("PCI: Can't parse ACS command line parameter\n");
> break;
> } else if (ret == 1) {
> /* Found a match */
> --
> 2.51.0
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-03-12 21:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-12 11:54 [PATCH] PCI: use pr_warn_once for ACS parameter parse failure Richard Cheng
2026-03-12 21:52 ` Bjorn Helgaas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox