Re: IRQ off latency of printk is very high

Tim Bird
2007-10-25T23:42:18Z

Matt Mackall wrote:
> On Thu, Oct 25, 2007 at 03:52:28PM -0700, Tim Bird wrote:
>> Mathieu Desnoyers wrote:
>>> It might help to read this thread I posted on LKML in January 2006
>>> explaining the problem, which led to some discussion about the issue.
>>>
>>> http://lkml.org/lkml/2006/6/3/48
>> This is very helpful. Jon Smirl's answer seems to give the
>> rationale for supporting printk output in interrupt context.
>> I'm not sure, however, if extending the interrupt off period
>> to cover the console output is required. It didn't until
>> Ingo changed it in 2.6.17.
>
> Hmm, I see this at the beginning of the post-BK era (2.6.12-rc2):
>
> spin_lock_irqsave(&logbuf_lock, flags);
> ...
> spin_unlock(&logbuf_lock);
> call_console_drivers(_con_start, _log_end);
> local_irq_restore(flags);
>

Well, I need to do some more research. This must be in
release_console_sem(). I was looking at vprintk, through
the ages. At 2.6.16, it looked like this:

spin_lock_irqsave(&logbuf_lock, flags);
...
spin_unlock_irqrestore(&logbuf_lock, flags);
console_may_schedule = 0;
release_console_sem();

but the irq restore has been moving around to different places
in that function over the last few years. I suspect that in the
common case the irqsave in vprintk is the one that disables
ints.

It appears that formerly interrupts were enabled in vprintk but
re-disabled immediately upon entering release_console_sem().
As it is now, they're held during formatting, buffering,
and output, which seems excessive.

It seems draconian to drain the entire buffer with ints disabled.
Is it possible to break this up and send out smaller chunks
at a time? Maybe by putting a chunk loop in release_console_sem()?

Just an idea. I can see that there are a lot of requirements
for this code, and the concurrency control has gotten
complex over the years.
-- Tim

=============================
Tim Bird
Architecture Group Chair, CE Linux Forum
Senior Staff Engineer, Sony Corporation of America
=============================