Commit 03e9e42f authored by Thomas Richter's avatar Thomas Richter Committed by Vasily Gorbik

s390/cpum_sf: Fix line length and format string

Rewrite some lines to match line length and replace
format string 0x%x to %#x.  Add and remove blank line.
Signed-off-by: default avatarThomas Richter <tmricht@linux.ibm.com>
Reviewed-by: default avatarHendrik Brueckner <brueckner@linux.ibm.com>
Signed-off-by: default avatarVasily Gorbik <gor@linux.ibm.com>
parent cf2c4a3f
...@@ -514,7 +514,6 @@ static void extend_sampling_buffer(struct sf_buffer *sfb, ...@@ -514,7 +514,6 @@ static void extend_sampling_buffer(struct sf_buffer *sfb,
sfb_pending_allocs(sfb, hwc)); sfb_pending_allocs(sfb, hwc));
} }
/* Number of perf events counting hardware events */ /* Number of perf events counting hardware events */
static atomic_t num_events; static atomic_t num_events;
/* Used to avoid races in calling reserve/release_cpumf_hardware */ /* Used to avoid races in calling reserve/release_cpumf_hardware */
...@@ -923,9 +922,10 @@ static void cpumsf_pmu_enable(struct pmu *pmu) ...@@ -923,9 +922,10 @@ static void cpumsf_pmu_enable(struct pmu *pmu)
lpp(&S390_lowcore.lpp); lpp(&S390_lowcore.lpp);
debug_sprintf_event(sfdbg, 6, "pmu_enable: es=%i cs=%i ed=%i cd=%i " debug_sprintf_event(sfdbg, 6, "pmu_enable: es=%i cs=%i ed=%i cd=%i "
"tear=%p dear=%p\n", cpuhw->lsctl.es, cpuhw->lsctl.cs, "tear=%p dear=%p\n", cpuhw->lsctl.es,
cpuhw->lsctl.ed, cpuhw->lsctl.cd, cpuhw->lsctl.cs, cpuhw->lsctl.ed, cpuhw->lsctl.cd,
(void *) cpuhw->lsctl.tear, (void *) cpuhw->lsctl.dear); (void *) cpuhw->lsctl.tear,
(void *) cpuhw->lsctl.dear);
} }
static void cpumsf_pmu_disable(struct pmu *pmu) static void cpumsf_pmu_disable(struct pmu *pmu)
...@@ -1083,7 +1083,8 @@ static void debug_sample_entry(struct hws_basic_entry *sample, ...@@ -1083,7 +1083,8 @@ static void debug_sample_entry(struct hws_basic_entry *sample,
struct hws_trailer_entry *te) struct hws_trailer_entry *te)
{ {
debug_sprintf_event(sfdbg, 4, "hw_collect_samples: Found unknown " debug_sprintf_event(sfdbg, 4, "hw_collect_samples: Found unknown "
"sampling data entry: te->f=%i basic.def=%04x (%p)\n", "sampling data entry: te->f=%i basic.def=%04x "
"(%p)\n",
te->f, sample->def, sample); te->f, sample->def, sample);
} }
...@@ -1216,7 +1217,7 @@ static void hw_perf_event_update(struct perf_event *event, int flush_all) ...@@ -1216,7 +1217,7 @@ static void hw_perf_event_update(struct perf_event *event, int flush_all)
/* Timestamps are valid for full sample-data-blocks only */ /* Timestamps are valid for full sample-data-blocks only */
debug_sprintf_event(sfdbg, 6, "hw_perf_event_update: sdbt=%p " debug_sprintf_event(sfdbg, 6, "hw_perf_event_update: sdbt=%p "
"overflow=%llu timestamp=0x%llx\n", "overflow=%llu timestamp=%#llx\n",
sdbt, te->overflow, sdbt, te->overflow,
(te->f) ? trailer_timestamp(te) : 0ULL); (te->f) ? trailer_timestamp(te) : 0ULL);
...@@ -1879,10 +1880,12 @@ static struct attribute_group cpumsf_pmu_events_group = { ...@@ -1879,10 +1880,12 @@ static struct attribute_group cpumsf_pmu_events_group = {
.name = "events", .name = "events",
.attrs = cpumsf_pmu_events_attr, .attrs = cpumsf_pmu_events_attr,
}; };
static struct attribute_group cpumsf_pmu_format_group = { static struct attribute_group cpumsf_pmu_format_group = {
.name = "format", .name = "format",
.attrs = cpumsf_pmu_format_attr, .attrs = cpumsf_pmu_format_attr,
}; };
static const struct attribute_group *cpumsf_pmu_attr_groups[] = { static const struct attribute_group *cpumsf_pmu_attr_groups[] = {
&cpumsf_pmu_events_group, &cpumsf_pmu_events_group,
&cpumsf_pmu_format_group, &cpumsf_pmu_format_group,
...@@ -1938,7 +1941,8 @@ static void cpumf_measurement_alert(struct ext_code ext_code, ...@@ -1938,7 +1941,8 @@ static void cpumf_measurement_alert(struct ext_code ext_code,
/* Report measurement alerts only for non-PRA codes */ /* Report measurement alerts only for non-PRA codes */
if (alert != CPU_MF_INT_SF_PRA) if (alert != CPU_MF_INT_SF_PRA)
debug_sprintf_event(sfdbg, 6, "measurement alert: 0x%x\n", alert); debug_sprintf_event(sfdbg, 6, "measurement alert: %#x\n",
alert);
/* Sampling authorization change request */ /* Sampling authorization change request */
if (alert & CPU_MF_INT_SF_SACA) if (alert & CPU_MF_INT_SF_SACA)
...@@ -1959,6 +1963,7 @@ static void cpumf_measurement_alert(struct ext_code ext_code, ...@@ -1959,6 +1963,7 @@ static void cpumf_measurement_alert(struct ext_code ext_code,
sf_disable(); sf_disable();
} }
} }
static int cpusf_pmu_setup(unsigned int cpu, int flags) static int cpusf_pmu_setup(unsigned int cpu, int flags)
{ {
/* Ignore the notification if no events are scheduled on the PMU. /* Ignore the notification if no events are scheduled on the PMU.
...@@ -2096,5 +2101,6 @@ static int __init init_cpum_sampling_pmu(void) ...@@ -2096,5 +2101,6 @@ static int __init init_cpum_sampling_pmu(void)
out: out:
return err; return err;
} }
arch_initcall(init_cpum_sampling_pmu); arch_initcall(init_cpum_sampling_pmu);
core_param(cpum_sfb_size, CPUM_SF_MAX_SDB, sfb_size, 0640); core_param(cpum_sfb_size, CPUM_SF_MAX_SDB, sfb_size, 0640);
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