s390/cpum_sf: Assign error value to err variable
authorThomas Richter <tmricht@linux.ibm.com>
Thu, 31 Oct 2019 09:38:53 +0000 (10:38 +0100)
committerVasily Gorbik <gor@linux.ibm.com>
Tue, 12 Nov 2019 10:24:10 +0000 (11:24 +0100)
When starting the CPU Measurement sampling facility using
qsi() function, this function may return an error value.
This error value is referenced in the else part of the
if statement to dump its value in a debug statement.
Right now this value is always zero because it has not been
assigned a value.

Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
arch/s390/kernel/perf_cpum_sf.c

index e758734..69506fd 100644 (file)
@@ -1007,7 +1007,8 @@ static void cpumsf_pmu_disable(struct pmu *pmu)
        }
 
        /* Save state of TEAR and DEAR register contents */
-       if (!qsi(&si)) {
+       err = qsi(&si);
+       if (!err) {
                /* TEAR/DEAR values are valid only if the sampling facility is
                 * enabled.  Note that cpumsf_pmu_disable() might be called even
                 * for a disabled sampling facility because cpumsf_pmu_enable()