Commit 420f42ec authored by Heiko Carstens's avatar Heiko Carstens Committed by Martin Schwidefsky
Browse files

s390/irq: remove split irq fields from /proc/stat 



Now that irq sum accounting for /proc/stat's "intr" line works again we
have the oddity that the sum field (first field) contains only the sum
of the second (external irqs) and third field (I/O interrupts).
The reason for that is that these two fields are already sums of all other
fields. So if we would sum up everything we would count every interrupt
twice.
This is broken since the split interrupt accounting was merged two years
ago: 052ff461 "[S390] irq: have detailed
statistics for interrupt types".
To fix this remove the split interrupt fields from /proc/stat's "intr"
line again and only have them in /proc/interrupts.

This restores the old behaviour, seems to be the only sane fix and mimics
a behaviour from other architectures where /proc/interrupts also contains
more than /proc/stat's "intr" line does.

Signed-off-by: default avatarHeiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: default avatarMartin Schwidefsky <schwidefsky@de.ibm.com>
parent add9bde2

arch/s390/include/asm/irq.h

+47 −30
Original line number Original line Diff line number Diff line
@@ -2,43 +2,60 @@ 
#define _ASM_IRQ_H

#define _ASM_IRQ_H





#include <linux/hardirq.h>

#include <linux/hardirq.h>

#include <linux/percpu.h>

#include <linux/cache.h>

#include <linux/types.h>

#include <linux/types.h>





enum interruption_class {

enum interruption_main_class {

	EXTERNAL_INTERRUPT,

	EXTERNAL_INTERRUPT,

	IO_INTERRUPT,

	IO_INTERRUPT,

	EXTINT_CLK,

	NR_IRQS

	EXTINT_EXC,

};

	EXTINT_EMS,



	EXTINT_TMR,

enum interruption_class {

	EXTINT_TLA,

	IRQEXT_CLK,

	EXTINT_PFL,

	IRQEXT_EXC,

	EXTINT_DSD,

	IRQEXT_EMS,

	EXTINT_VRT,

	IRQEXT_TMR,

	EXTINT_SCP,

	IRQEXT_TLA,

	EXTINT_IUC,

	IRQEXT_PFL,

	EXTINT_CMS,

	IRQEXT_DSD,

	EXTINT_CMC,

	IRQEXT_VRT,

	EXTINT_CMR,

	IRQEXT_SCP,

	IOINT_CIO,

	IRQEXT_IUC,

	IOINT_QAI,

	IRQEXT_CMS,

	IOINT_DAS,

	IRQEXT_CMC,

	IOINT_C15,

	IRQEXT_CMR,

	IOINT_C70,

	IRQIO_CIO,

	IOINT_TAP,

	IRQIO_QAI,

	IOINT_VMR,

	IRQIO_DAS,

	IOINT_LCS,

	IRQIO_C15,

	IOINT_CLW,

	IRQIO_C70,

	IOINT_CTC,

	IRQIO_TAP,

	IOINT_APB,

	IRQIO_VMR,

	IOINT_ADM,

	IRQIO_LCS,

	IOINT_CSC,

	IRQIO_CLW,

	IOINT_PCI,

	IRQIO_CTC,

	IOINT_MSI,

	IRQIO_APB,

	IRQIO_ADM,

	IRQIO_CSC,

	IRQIO_PCI,

	IRQIO_MSI,

	NMI_NMI,

	NMI_NMI,

	NR_IRQS,

	NR_ARCH_IRQS

};

};





struct irq_stat {

	unsigned int irqs[NR_ARCH_IRQS];

};



DECLARE_PER_CPU_SHARED_ALIGNED(struct irq_stat, irq_stat);



static __always_inline void inc_irq_stat(enum interruption_class irq)

{

	__get_cpu_var(irq_stat).irqs[irq]++;

}



struct ext_code {

struct ext_code {

	unsigned short subcode;

	unsigned short subcode;

	unsigned short code;

	unsigned short code;

arch/s390/kernel/irq.c

+73 −48
Original line number Original line Diff line number Diff line
@@ -24,42 +24,63 @@ 
#include <asm/irq.h>

#include <asm/irq.h>

#include "entry.h"

#include "entry.h"





DEFINE_PER_CPU_SHARED_ALIGNED(struct irq_stat, irq_stat);

EXPORT_PER_CPU_SYMBOL_GPL(irq_stat);



struct irq_class {

struct irq_class {

	char *name;

	char *name;

	char *desc;

	char *desc;

};

};





static const struct irq_class intrclass_names[] = {

/*

 * The list of "main" irq classes on s390. This is the list of interrrupts

 * that appear both in /proc/stat ("intr" line) and /proc/interrupts.

 * Historically only external and I/O interrupts have been part of /proc/stat.

 * We can't add the split external and I/O sub classes since the first field

 * in the "intr" line in /proc/stat is supposed to be the sum of all other

 * fields.

 * Since the external and I/O interrupt fields are already sums we would end

 * up with having a sum which accounts each interrupt twice.

 */

static const struct irq_class irqclass_main_desc[NR_IRQS] = {

	[EXTERNAL_INTERRUPT] = {.name = "EXT"},

	[EXTERNAL_INTERRUPT] = {.name = "EXT"},

	[IO_INTERRUPT]	     = {.name = "I/O"},

	[IO_INTERRUPT]	     = {.name = "I/O"}

	[EXTINT_CLK] = {.name = "CLK", .desc = "[EXT] Clock Comparator"},

};

	[EXTINT_EXC] = {.name = "EXC", .desc = "[EXT] External Call"},



	[EXTINT_EMS] = {.name = "EMS", .desc = "[EXT] Emergency Signal"},

/*

	[EXTINT_TMR] = {.name = "TMR", .desc = "[EXT] CPU Timer"},

 * The list of split external and I/O interrupts that appear only in

	[EXTINT_TLA] = {.name = "TAL", .desc = "[EXT] Timing Alert"},

 * /proc/interrupts.

	[EXTINT_PFL] = {.name = "PFL", .desc = "[EXT] Pseudo Page Fault"},

 * In addition this list contains non external / I/O events like NMIs.

	[EXTINT_DSD] = {.name = "DSD", .desc = "[EXT] DASD Diag"},

 */

	[EXTINT_VRT] = {.name = "VRT", .desc = "[EXT] Virtio"},

static const struct irq_class irqclass_sub_desc[NR_ARCH_IRQS] = {

	[EXTINT_SCP] = {.name = "SCP", .desc = "[EXT] Service Call"},

	[IRQEXT_CLK] = {.name = "CLK", .desc = "[EXT] Clock Comparator"},

	[EXTINT_IUC] = {.name = "IUC", .desc = "[EXT] IUCV"},

	[IRQEXT_EXC] = {.name = "EXC", .desc = "[EXT] External Call"},

	[EXTINT_CMS] = {.name = "CMS", .desc = "[EXT] CPU-Measurement: Sampling"},

	[IRQEXT_EMS] = {.name = "EMS", .desc = "[EXT] Emergency Signal"},

	[EXTINT_CMC] = {.name = "CMC", .desc = "[EXT] CPU-Measurement: Counter"},

	[IRQEXT_TMR] = {.name = "TMR", .desc = "[EXT] CPU Timer"},

	[EXTINT_CMR] = {.name = "CMR", .desc = "[EXT] CPU-Measurement: RI"},

	[IRQEXT_TLA] = {.name = "TAL", .desc = "[EXT] Timing Alert"},

	[IOINT_CIO]  = {.name = "CIO", .desc = "[I/O] Common I/O Layer Interrupt"},

	[IRQEXT_PFL] = {.name = "PFL", .desc = "[EXT] Pseudo Page Fault"},

	[IOINT_QAI]  = {.name = "QAI", .desc = "[I/O] QDIO Adapter Interrupt"},

	[IRQEXT_DSD] = {.name = "DSD", .desc = "[EXT] DASD Diag"},

	[IOINT_DAS]  = {.name = "DAS", .desc = "[I/O] DASD"},

	[IRQEXT_VRT] = {.name = "VRT", .desc = "[EXT] Virtio"},

	[IOINT_C15]  = {.name = "C15", .desc = "[I/O] 3215"},

	[IRQEXT_SCP] = {.name = "SCP", .desc = "[EXT] Service Call"},

	[IOINT_C70]  = {.name = "C70", .desc = "[I/O] 3270"},

	[IRQEXT_IUC] = {.name = "IUC", .desc = "[EXT] IUCV"},

	[IOINT_TAP]  = {.name = "TAP", .desc = "[I/O] Tape"},

	[IRQEXT_CMS] = {.name = "CMS", .desc = "[EXT] CPU-Measurement: Sampling"},

	[IOINT_VMR]  = {.name = "VMR", .desc = "[I/O] Unit Record Devices"},

	[IRQEXT_CMC] = {.name = "CMC", .desc = "[EXT] CPU-Measurement: Counter"},

	[IOINT_LCS]  = {.name = "LCS", .desc = "[I/O] LCS"},

	[IRQEXT_CMR] = {.name = "CMR", .desc = "[EXT] CPU-Measurement: RI"},

	[IOINT_CLW]  = {.name = "CLW", .desc = "[I/O] CLAW"},

	[IRQIO_CIO]  = {.name = "CIO", .desc = "[I/O] Common I/O Layer Interrupt"},

	[IOINT_CTC]  = {.name = "CTC", .desc = "[I/O] CTC"},

	[IRQIO_QAI]  = {.name = "QAI", .desc = "[I/O] QDIO Adapter Interrupt"},

	[IOINT_APB]  = {.name = "APB", .desc = "[I/O] AP Bus"},

	[IRQIO_DAS]  = {.name = "DAS", .desc = "[I/O] DASD"},

	[IOINT_ADM]  = {.name = "ADM", .desc = "[I/O] EADM Subchannel"},

	[IRQIO_C15]  = {.name = "C15", .desc = "[I/O] 3215"},

	[IOINT_CSC]  = {.name = "CSC", .desc = "[I/O] CHSC Subchannel"},

	[IRQIO_C70]  = {.name = "C70", .desc = "[I/O] 3270"},

	[IOINT_PCI]  = {.name = "PCI", .desc = "[I/O] PCI Interrupt" },

	[IRQIO_TAP]  = {.name = "TAP", .desc = "[I/O] Tape"},

	[IOINT_MSI] =  {.name = "MSI", .desc = "[I/O] MSI Interrupt" },

	[IRQIO_VMR]  = {.name = "VMR", .desc = "[I/O] Unit Record Devices"},

	[IRQIO_LCS]  = {.name = "LCS", .desc = "[I/O] LCS"},

	[IRQIO_CLW]  = {.name = "CLW", .desc = "[I/O] CLAW"},

	[IRQIO_CTC]  = {.name = "CTC", .desc = "[I/O] CTC"},

	[IRQIO_APB]  = {.name = "APB", .desc = "[I/O] AP Bus"},

	[IRQIO_ADM]  = {.name = "ADM", .desc = "[I/O] EADM Subchannel"},

	[IRQIO_CSC]  = {.name = "CSC", .desc = "[I/O] CHSC Subchannel"},

	[IRQIO_PCI]  = {.name = "PCI", .desc = "[I/O] PCI Interrupt" },

	[IRQIO_MSI]  = {.name = "MSI", .desc = "[I/O] MSI Interrupt" },

	[NMI_NMI]    = {.name = "NMI", .desc = "[NMI] Machine Check"},

	[NMI_NMI]    = {.name = "NMI", .desc = "[NMI] Machine Check"},

};

};
@@ -68,28 +89,32 @@ static const struct irq_class intrclass_names[] = { 
 */

 */

int show_interrupts(struct seq_file *p, void *v)

int show_interrupts(struct seq_file *p, void *v)

{

{

	int i = *(loff_t *) v, j;

	int irq = *(loff_t *) v;

	int cpu;





	get_online_cpus();

	get_online_cpus();

	if (i == 0) {

	if (irq == 0) {

		seq_puts(p, "           ");

		seq_puts(p, "           ");

		for_each_online_cpu(j)

		for_each_online_cpu(cpu)

			seq_printf(p, "CPU%d       ",j);

			seq_printf(p, "CPU%d       ", cpu);

		seq_putc(p, '\n');

		seq_putc(p, '\n');

	}

	}



	if (irq < NR_IRQS) {

	if (i < NR_IRQS) {

		seq_printf(p, "%s: ", irqclass_main_desc[irq].name);

		seq_printf(p, "%s: ", intrclass_names[i].name);

		for_each_online_cpu(cpu)

#ifndef CONFIG_SMP

			seq_printf(p, "%10u ", kstat_cpu(cpu).irqs[irq]);

		seq_printf(p, "%10u ", kstat_irqs(i));

		seq_putc(p, '\n');

#else

		goto skip_arch_irqs;

		for_each_online_cpu(j)

	}

			seq_printf(p, "%10u ", kstat_cpu(j).irqs[i]);

	for (irq = 0; irq < NR_ARCH_IRQS; irq++) {

#endif

		seq_printf(p, "%s: ", irqclass_sub_desc[irq].name);

		if (intrclass_names[i].desc)

		for_each_online_cpu(cpu)

			seq_printf(p, "  %s", intrclass_names[i].desc);

			seq_printf(p, "%10u ", per_cpu(irq_stat, cpu).irqs[irq]);

		if (irqclass_sub_desc[irq].desc)

			seq_printf(p, "  %s", irqclass_sub_desc[irq].desc);

		seq_putc(p, '\n');

		seq_putc(p, '\n');

	}

	}

skip_arch_irqs:

	put_online_cpus();

	put_online_cpus();

	return 0;

	return 0;

}

}

arch/s390/kernel/nmi.c

+1 −1
Original line number Original line Diff line number Diff line
@@ -254,7 +254,7 @@ void notrace s390_do_machine_check(struct pt_regs *regs) 
	int umode;

	int umode;





	nmi_enter();

	nmi_enter();

	kstat_cpu(smp_processor_id()).irqs[NMI_NMI]++;

	inc_irq_stat(NMI_NMI);

	mci = (struct mci *) &S390_lowcore.mcck_interruption_code;

	mci = (struct mci *) &S390_lowcore.mcck_interruption_code;

	mcck = &__get_cpu_var(cpu_mcck);

	mcck = &__get_cpu_var(cpu_mcck);

	umode = user_mode(regs);

	umode = user_mode(regs);

arch/s390/kernel/perf_cpum_cf.c

+1 −1
Original line number Original line Diff line number Diff line
@@ -229,7 +229,7 @@ static void cpumf_measurement_alert(struct ext_code ext_code, 
	if (!(alert & CPU_MF_INT_CF_MASK))

	if (!(alert & CPU_MF_INT_CF_MASK))

		return;

		return;





	kstat_cpu(smp_processor_id()).irqs[EXTINT_CMC]++;

	inc_irq_stat(IRQEXT_CMC);

	cpuhw = &__get_cpu_var(cpu_hw_events);

	cpuhw = &__get_cpu_var(cpu_hw_events);





	/* Measurement alerts are shared and might happen when the PMU

	/* Measurement alerts are shared and might happen when the PMU

arch/s390/kernel/runtime_instr.c

+1 −1
Original line number Original line Diff line number Diff line
@@ -71,7 +71,7 @@ static void runtime_instr_int_handler(struct ext_code ext_code, 
	if (!(param32 & CPU_MF_INT_RI_MASK))

	if (!(param32 & CPU_MF_INT_RI_MASK))

		return;

		return;





	kstat_cpu(smp_processor_id()).irqs[EXTINT_CMR]++;

	inc_irq_stat(IRQEXT_CMR);





	if (!current->thread.ri_cb)

	if (!current->thread.ri_cb)

		return;

		return;

CRA Git | Maintained and supported by SUSTech CRA and CCSE