Re: [RFC PATCH v1 00/25] printk: new implementation

From: John Ogness
Date: Wed Mar 13 2019 - 04:19:48 EST


On 2019-03-13, Sergey Senozhatsky <sergey.senozhatsky.work@xxxxxxxxx> wrote:
>>> I suggest the following way forward (separate patchsets):
>>>
>>> 1. Replace log buffer (least controversial thing)
>>
>> Yes. I will post a series that only implements the ringbuffer using
>> your simplified API. That will be enough to remove printk_safe and
>> actually does most of the work of updating devkmsg, kmsg_dump, and
>> syslog.
>
> This may _not_ be enough to remove printk_safe. One of the reasons
> printk_safe "condom" came into existence was console_sem (which
> is a bit too important to ignore it):
>
> printk()
> console_trylock()
> console_unlock()
> up()
> raw_spin_lock_irqsave(&sem->lock, flags)
> __up()
> wake_up_process()
> WARN/etc
> printk()
> console_trylock()
> down_trylock()
> raw_spin_lock_irqsave(&sem->lock, flags) << deadlock
>
> Back then we were looking at
>
> printk->console_sem->lock->printk->console_sem->lock
>
> deadlock report from LG, if I'm not mistaken.

The main drawback of printk_safe is the safe buffers, which, aside from
bogus timestamping, may never make it back to the printk log buffer.

With the new ring buffer the safe buffers are not needed, even in the
recursive situation. As you are pointing out, the notification/wake
component of printk_safe will still be needed. I will leave that (small)
part in printk_safe.c.

John Ogness