Commit 0d5f0dc8 authored by Thomas Richter's avatar Thomas Richter Committed by Heiko Carstens

s390/cpum_cf: simplify PMC_INIT and PMC_RELEASE usage

Simplify the use of constants PMC_INIT and PMC_RELEASE.
Suggested-by: default avatarHeiko Carstens <hca@linux.ibm.com>
Signed-off-by: default avatarThomas Richter <tmricht@linux.ibm.com>
Signed-off-by: default avatarHeiko Carstens <hca@linux.ibm.com>
parent 1e99c242
...@@ -506,14 +506,14 @@ static void cpumf_pmu_disable(struct pmu *pmu) ...@@ -506,14 +506,14 @@ static void cpumf_pmu_disable(struct pmu *pmu)
cpuhw->flags &= ~PMU_F_ENABLED; cpuhw->flags &= ~PMU_F_ENABLED;
} }
#define PMC_INIT 0 #define PMC_INIT 0UL
#define PMC_RELEASE 1 #define PMC_RELEASE 1UL
static void cpum_cf_setup_cpu(void *flags) static void cpum_cf_setup_cpu(void *flags)
{ {
struct cpu_cf_events *cpuhw = this_cpu_ptr(&cpu_cf_events); struct cpu_cf_events *cpuhw = this_cpu_ptr(&cpu_cf_events);
switch (*((int *)flags)) { switch ((unsigned long)flags) {
case PMC_INIT: case PMC_INIT:
memset(&cpuhw->info, 0, sizeof(cpuhw->info)); memset(&cpuhw->info, 0, sizeof(cpuhw->info));
qctri(&cpuhw->info); qctri(&cpuhw->info);
...@@ -535,9 +535,7 @@ static void cpum_cf_setup_cpu(void *flags) ...@@ -535,9 +535,7 @@ static void cpum_cf_setup_cpu(void *flags)
/* Initialize the CPU-measurement counter facility */ /* Initialize the CPU-measurement counter facility */
static int __kernel_cpumcf_begin(void) static int __kernel_cpumcf_begin(void)
{ {
int flags = PMC_INIT; on_each_cpu(cpum_cf_setup_cpu, (void *)PMC_INIT, 1);
on_each_cpu(cpum_cf_setup_cpu, &flags, 1);
irq_subclass_register(IRQ_SUBCLASS_MEASUREMENT_ALERT); irq_subclass_register(IRQ_SUBCLASS_MEASUREMENT_ALERT);
return 0; return 0;
...@@ -546,9 +544,7 @@ static int __kernel_cpumcf_begin(void) ...@@ -546,9 +544,7 @@ static int __kernel_cpumcf_begin(void)
/* Release the CPU-measurement counter facility */ /* Release the CPU-measurement counter facility */
static void __kernel_cpumcf_end(void) static void __kernel_cpumcf_end(void)
{ {
int flags = PMC_RELEASE; on_each_cpu(cpum_cf_setup_cpu, (void *)PMC_RELEASE, 1);
on_each_cpu(cpum_cf_setup_cpu, &flags, 1);
irq_subclass_unregister(IRQ_SUBCLASS_MEASUREMENT_ALERT); irq_subclass_unregister(IRQ_SUBCLASS_MEASUREMENT_ALERT);
} }
...@@ -937,10 +933,10 @@ static struct pmu cpumf_pmu = { ...@@ -937,10 +933,10 @@ static struct pmu cpumf_pmu = {
.read = cpumf_pmu_read, .read = cpumf_pmu_read,
}; };
static int cpum_cf_setup(unsigned int cpu, int flags) static int cpum_cf_setup(unsigned int cpu, unsigned long flags)
{ {
local_irq_disable(); local_irq_disable();
cpum_cf_setup_cpu(&flags); cpum_cf_setup_cpu((void *)flags);
local_irq_enable(); local_irq_enable();
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment