Jan Kiszka <jan.kis...@siemens.com> writes:
> On 22.07.21 11:48, Philippe Gerum via Xenomai wrote: >> From: Philippe Gerum <r...@xenomai.org> >> >> Signed-off-by: Philippe Gerum <r...@xenomai.org> >> --- >> kernel/irq/proc.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/kernel/irq/proc.c b/kernel/irq/proc.c >> index 7c5cd42df3b93e7..1428d83d7da8658 100644 >> --- a/kernel/irq/proc.c >> +++ b/kernel/irq/proc.c >> @@ -518,6 +518,9 @@ int show_interrupts(struct seq_file *p, void *v) >> seq_printf(p, " %*s", prec, ""); >> #ifdef CONFIG_GENERIC_IRQ_SHOW_LEVEL >> seq_printf(p, " %-8s", irqd_is_level_type(&desc->irq_data) ? "Level" : >> "Edge"); >> +#endif >> +#ifdef CONFIG_IRQ_PIPELINE >> + seq_printf(p, " %-3s", irq_settings_is_oob(desc) ? "oob" : ""); >> #endif >> if (desc->name) >> seq_printf(p, "-%-8s", desc->name); >> > > Small but valuable - already received a question in this direction. > This was deeply buried in my todo list until Byron fortunately mentioned it. I'm not 100% happy with the format since this new field might affect some parsers fed by /proc/interrupts, but I could not find any better way which would be generic enough, not depending on the specific printout which may be done by the irqchip driver. -- Philippe.