From: Bjorn Helgaas <helgaas@kernel.org> To: "Rafael J. Wysocki" <rjw@rjwysocki.net> Cc: Imre Deak <imre.deak@intel.com>, Bjorn Helgaas <bhelgaas@google.com>, Jani Nikula <jani.nikula@intel.com>, Daniel Vetter <daniel.vetter@ffwll.ch>, Linux PM <linux-pm@vger.kernel.org>, intel-gfx@lists.freedesktop.org, Linux PCI <linux-pci@vger.kernel.org> Subject: Re: [PATCH v5] PCI / PM: Tune down retryable runtime suspend error messages Date: Tue, 1 Dec 2015 22:43:48 -0600 [thread overview] Message-ID: <20151202044348.GA11971@localhost> (raw) In-Reply-To: <4357316.BsOyM9M4tL@vostro.rjw.lan> On Wed, Dec 02, 2015 at 02:54:45AM +0100, Rafael J. Wysocki wrote: > On Monday, November 30, 2015 09:02:55 PM Imre Deak wrote: > > The runtime PM core doesn't treat EBUSY and EAGAIN retvals from the driver > > suspend hooks as errors, but they still show up as errors in dmesg. Tune > > them down. See rpm_suspend() for details of handling these return values. > > > > Note that we use dev_dbg() for the retryable retvals, so after this > > change you'll need either CONFIG_DYNAMIC_DEBUG or CONFIG_PCI_DEBUG > > for them to show up in the log. > > > > One problem caused by this was noticed by Daniel: the i915 driver > > returns EAGAIN to signal a temporary failure to suspend and as a request > > towards the RPM core for scheduling a suspend again. This is a normal > > event, but the resulting error message flags a breakage during the > > driver's automated testing which parses dmesg and picks up the error. > > > > Reported-by: Daniel Vetter <daniel.vetter@intel.com> > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92992 > > CC: Bjorn Helgaas <bhelgaas@google.com> > > CC: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Signed-off-by: Imre Deak <imre.deak@intel.com> > > Hi Bjorn, > > Are you going to handle this one or should I take care of it? Why don't you take it, since you're the PM guru :) Acked-by: Bjorn Helgaas <bhelgaas@google.com> Thanks for all your work, Imre! > > --- > > > > v2: > > - fix compile breake when CONFIG_PM_SLEEP=n (0-day builder) > > v3: > > - instead of modifying the suspend_report_result() helper to disinguish > > between the runtime and system suspend case, inline the error > > printing, it's not used anywhere else (Rafael) > > v4: > > - don't refer to log levels as flags in code comment (Rafael) > > - use pr_debug(), pr_err() instead of the corresponding printk() (Rafael) > > v5: > > - clarify commit message (Bjorn) > > - use dev_dbg, dev_err instead of pr_debug, pr_err (Bjorn) > > - use %pf in printk format instead of %pF (Bjorn) > > - make the debug/error messages more meaningful (Bjorn) > > --- > > drivers/pci/pci-driver.c | 16 ++++++++++++++-- > > 1 file changed, 14 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c > > index 4446fcb..d7ffd66 100644 > > --- a/drivers/pci/pci-driver.c > > +++ b/drivers/pci/pci-driver.c > > @@ -1146,9 +1146,21 @@ static int pci_pm_runtime_suspend(struct device *dev) > > pci_dev->state_saved = false; > > pci_dev->no_d3cold = false; > > error = pm->runtime_suspend(dev); > > - suspend_report_result(pm->runtime_suspend, error); > > - if (error) > > + if (error) { > > + /* > > + * -EBUSY and -EAGAIN is used to request the runtime PM core > > + * to schedule a new suspend, so log the event only with debug > > + * log level. > > + */ > > + if (error == -EBUSY || error == -EAGAIN) > > + dev_dbg(dev, "can't suspend now (%pf returned %d)\n", > > + pm->runtime_suspend, error); > > + else > > + dev_err(dev, "can't suspend (%pf returned %d)\n", > > + pm->runtime_suspend, error); > > + > > return error; > > + } > > if (!pci_dev->d3cold_allowed) > > pci_dev->no_d3cold = true; > > > > > > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center. > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Bjorn Helgaas <helgaas@kernel.org> To: "Rafael J. Wysocki" <rjw@rjwysocki.net> Cc: Linux PM <linux-pm@vger.kernel.org>, Jani Nikula <jani.nikula@intel.com>, Daniel Vetter <daniel.vetter@ffwll.ch>, Linux PCI <linux-pci@vger.kernel.org>, Bjorn Helgaas <bhelgaas@google.com>, intel-gfx@lists.freedesktop.org Subject: Re: [PATCH v5] PCI / PM: Tune down retryable runtime suspend error messages Date: Tue, 1 Dec 2015 22:43:48 -0600 [thread overview] Message-ID: <20151202044348.GA11971@localhost> (raw) In-Reply-To: <4357316.BsOyM9M4tL@vostro.rjw.lan> On Wed, Dec 02, 2015 at 02:54:45AM +0100, Rafael J. Wysocki wrote: > On Monday, November 30, 2015 09:02:55 PM Imre Deak wrote: > > The runtime PM core doesn't treat EBUSY and EAGAIN retvals from the driver > > suspend hooks as errors, but they still show up as errors in dmesg. Tune > > them down. See rpm_suspend() for details of handling these return values. > > > > Note that we use dev_dbg() for the retryable retvals, so after this > > change you'll need either CONFIG_DYNAMIC_DEBUG or CONFIG_PCI_DEBUG > > for them to show up in the log. > > > > One problem caused by this was noticed by Daniel: the i915 driver > > returns EAGAIN to signal a temporary failure to suspend and as a request > > towards the RPM core for scheduling a suspend again. This is a normal > > event, but the resulting error message flags a breakage during the > > driver's automated testing which parses dmesg and picks up the error. > > > > Reported-by: Daniel Vetter <daniel.vetter@intel.com> > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92992 > > CC: Bjorn Helgaas <bhelgaas@google.com> > > CC: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > Signed-off-by: Imre Deak <imre.deak@intel.com> > > Hi Bjorn, > > Are you going to handle this one or should I take care of it? Why don't you take it, since you're the PM guru :) Acked-by: Bjorn Helgaas <bhelgaas@google.com> Thanks for all your work, Imre! > > --- > > > > v2: > > - fix compile breake when CONFIG_PM_SLEEP=n (0-day builder) > > v3: > > - instead of modifying the suspend_report_result() helper to disinguish > > between the runtime and system suspend case, inline the error > > printing, it's not used anywhere else (Rafael) > > v4: > > - don't refer to log levels as flags in code comment (Rafael) > > - use pr_debug(), pr_err() instead of the corresponding printk() (Rafael) > > v5: > > - clarify commit message (Bjorn) > > - use dev_dbg, dev_err instead of pr_debug, pr_err (Bjorn) > > - use %pf in printk format instead of %pF (Bjorn) > > - make the debug/error messages more meaningful (Bjorn) > > --- > > drivers/pci/pci-driver.c | 16 ++++++++++++++-- > > 1 file changed, 14 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c > > index 4446fcb..d7ffd66 100644 > > --- a/drivers/pci/pci-driver.c > > +++ b/drivers/pci/pci-driver.c > > @@ -1146,9 +1146,21 @@ static int pci_pm_runtime_suspend(struct device *dev) > > pci_dev->state_saved = false; > > pci_dev->no_d3cold = false; > > error = pm->runtime_suspend(dev); > > - suspend_report_result(pm->runtime_suspend, error); > > - if (error) > > + if (error) { > > + /* > > + * -EBUSY and -EAGAIN is used to request the runtime PM core > > + * to schedule a new suspend, so log the event only with debug > > + * log level. > > + */ > > + if (error == -EBUSY || error == -EAGAIN) > > + dev_dbg(dev, "can't suspend now (%pf returned %d)\n", > > + pm->runtime_suspend, error); > > + else > > + dev_err(dev, "can't suspend (%pf returned %d)\n", > > + pm->runtime_suspend, error); > > + > > return error; > > + } > > if (!pci_dev->d3cold_allowed) > > pci_dev->no_d3cold = true; > > > > > > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center. > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2015-12-02 4:43 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-18 9:16 [PATCH] PCI / PM: tune down RPM suspend error message with EBUSY and EAGAIN retval Imre Deak 2015-11-18 10:16 ` kbuild test robot 2015-11-18 10:56 ` [PATCH v2] " Imre Deak 2015-11-18 13:28 ` Imre Deak 2015-11-18 14:19 ` Daniel Vetter 2015-11-27 11:39 ` Jani Nikula 2015-11-27 14:44 ` [Intel-gfx] " Rafael J. Wysocki 2015-11-27 14:44 ` Rafael J. Wysocki 2015-11-27 14:56 ` [Intel-gfx] " Imre Deak 2015-11-27 18:17 ` [PATCH v3] " Imre Deak 2015-11-27 22:23 ` Rafael J. Wysocki 2015-11-28 8:34 ` [PATCH v4] " Imre Deak 2015-11-30 2:07 ` Rafael J. Wysocki 2015-11-30 18:07 ` Bjorn Helgaas 2015-11-30 19:02 ` [PATCH v5] PCI / PM: Tune down retryable runtime suspend error messages Imre Deak 2015-12-02 1:54 ` Rafael J. Wysocki 2015-12-02 4:43 ` Bjorn Helgaas [this message] 2015-12-02 4:43 ` Bjorn Helgaas
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20151202044348.GA11971@localhost \ --to=helgaas@kernel.org \ --cc=bhelgaas@google.com \ --cc=daniel.vetter@ffwll.ch \ --cc=imre.deak@intel.com \ --cc=intel-gfx@lists.freedesktop.org \ --cc=jani.nikula@intel.com \ --cc=linux-pci@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=rjw@rjwysocki.net \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.