Re: [PATCH] tracing: Account bottom half disabled sections.

From: Sebastian Andrzej Siewior
Date: Mon Dec 13 2021 - 04:28:18 EST


On 2021-12-10 20:32:40 [-0500], Steven Rostedt wrote:
> > @@ -4226,7 +4228,7 @@ static void print_func_help_header_irq(struct array_buffer *buf, struct seq_file
> >
> > seq_printf(m, "# %.*s _-----=> irqs-off\n", prec, space);
> > seq_printf(m, "# %.*s / _----=> need-resched\n", prec, space);
> > - seq_printf(m, "# %.*s| / _---=> hardirq/softirq\n", prec, space);
> > + seq_printf(m, "# %.*s| / _---=> hardirq/softirq/BH-disabled\n", prec, space);
>
> So I went to update the documentation on this, and realized that this is
> wrong. Really, we want this in the irqs-off section probably.
>
> Note, the above is to show we are running in a hardirq or softirq context.
> But BH-disabled does not match that. Should this be with irqs-off being:
>
> d - irqs are disabled
> b - BH is disabled?
> D - irqs and BH is disabled?

We are not in hardirq/softirq/nmi but have simply BH disabled.
Makes sense.

> -- Steve

Sebastian