s390/cpum_sf: Rework call to sf_disable()

Setup_pmc_cpu() function body consists of one single switch
statement with two cases PMC_INIT and PMC_RELEASE.
In both of these cases sf_disable() is invoked to turn off the
CPU Measurement sampling facility.
Move sf_disable() out of the switch statement.
No functional change.

Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
Reviewed-by: Hendrik Brueckner <brueckner@linux.ibm.com>
Reviewed-by: Sumanth Korikkar <sumanthk@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>

diff --git a/arch/s390/kernel/perf_cpum_sf.c b/arch/s390/kernel/perf_cpum_sf.c
index 713999681b608a16940db79ccf2c639039f9775d..2c1d4188e9874a90c901a67f38718baf6809b82a 100644 (file)
--- a/arch/s390/kernel/perf_cpum_sf.c
+++ b/arch/s390/kernel/perf_cpum_sf.c
@@ -559,16 +559,15 @@ static void setup_pmc_cpu(void *flags)
 {
        struct cpu_hw_sf *cpuhw = this_cpu_ptr(&cpu_hw_sf);
 
+       sf_disable();
        switch (*((int *)flags)) {
        case PMC_INIT:
                memset(cpuhw, 0, sizeof(*cpuhw));
                qsi(&cpuhw->qsi);
                cpuhw->flags |= PMU_F_RESERVED;
-               sf_disable();
                break;
        case PMC_RELEASE:
                cpuhw->flags &= ~PMU_F_RESERVED;
-               sf_disable();
                deallocate_buffers(cpuhw);
                break;
        }