sched_ext: Make handle_lockup() propagate scx_verror() result

handle_lockup() currently calls scx_verror() but ignores its return value,
always returning true when the scheduler is enabled. Make it capture and return
the result from scx_verror(). This prepares for hardlockup handling.

Cc: Dan Schatzberg <schatzberg.dan@gmail.com>
Cc: Emil Tsalapatis <etsal@meta.com>
Reviewed-by: Andrea Righi <arighi@nvidia.com>
Signed-off-by: Tejun Heo <tj@kernel.org>

diff --git a/kernel/sched/ext.c b/kernel/sched/ext.c
index ad5d3902e99d7041c564f477b3220ad0653e3233..85bb052459ec4f1d9f0eb47bbff64486bd71684c 100644 (file)
--- a/kernel/sched/ext.c
+++ b/kernel/sched/ext.c
@@ -3659,6 +3659,7 @@ static __printf(1, 2) bool handle_lockup(const char *fmt, ...)
 {
        struct scx_sched *sch;
        va_list args;
+       bool ret;
 
        guard(rcu)();
 
@@ -3670,9 +3671,9 @@ static __printf(1, 2) bool handle_lockup(const char *fmt, ...)
        case SCX_ENABLING:
        case SCX_ENABLED:
                va_start(args, fmt);
-               scx_verror(sch, fmt, args);
+               ret = scx_verror(sch, fmt, args);
                va_end(args);
-               return true;
+               return ret;
        default:
                return false;
        }