sched/walt/core_ctl: use printk_deferred() in case of errors

It is possible that the Pause and Resume operations fail and
using pr_warn() could lead to core ctl task running for long
period when it has to push the logbuf data to console which
eventually cause RT throttling. Avoid this scenario by using
printk_deferred() instead of pr_warn().

Change-Id: I2717205216caf994a8995d6f11178499a7015201
Signed-off-by: Satya Durga Srinivasu Prabhala <satyap@codeaurora.org>
This commit is contained in:
Satya Durga Srinivasu Prabhala 2021-08-01 21:53:25 -07:00 committed by Rishabh Bhatnagar
parent 55bfad9a22
commit 2dd3a6b665

View File

@ -1155,7 +1155,7 @@ static void core_ctl_pause_cpus(struct cpumask *cpus_to_pause)
if (cpumask_any(cpus_to_pause) < nr_cpu_ids) {
if (walt_pause_cpus(cpus_to_pause) < 0)
pr_warn("core_ctl pause operation failed cpus=%*pbl paused_by_us=%*pbl\n",
printk_deferred("core_ctl pause operation failed cpus=%*pbl paused_by_us=%*pbl\n",
cpumask_pr_args(cpus_to_pause),
cpumask_pr_args(&cpus_paused_by_us));
else
@ -1184,7 +1184,7 @@ static void core_ctl_resume_cpus(struct cpumask *cpus_to_unpause)
if (cpumask_any(cpus_to_unpause) < nr_cpu_ids) {
if (walt_resume_cpus(cpus_to_unpause) < 0)
pr_warn("core_ctl resume operation failed cpus=%*pbl paused_by_us=%*pbl\n",
printk_deferred("core_ctl resume operation failed cpus=%*pbl paused_by_us=%*pbl\n",
cpumask_pr_args(cpus_to_unpause),
cpumask_pr_args(&cpus_paused_by_us));
else