On 2019-03-12, Mikulas Patocka <mpatocka@xxxxxxxxxx> wrote: >>> But it doesn't turn SMP system into UP. >> >> In this example it turned it into a brick. >> >> The problems I see are: >> >> 1. The current loglevels used in the kernel are not sufficient to >> distinguish between emergency and informational messages. Addressing >> this issue may require things like using a new printk flag and >> manually marking the printks that we(?) decide are critical. I was > > It depends on what the user considers important. The kernel doesn't > know what's critical and can't know it. > > For example, if the user plugs an USB stick and the USB stick reports > an I/O error, it is not critical condition. > > If the user runs the whole system from the USB stick and the USB stick > reports an I/O error, it is critical condition. I would argue that it is _not_ a critical condition. I/O errors are something that happen. This is normal operation. _My_ definition of critical is when something with the _kernel itself_ has gone wrong. If the kernel is in error paths that should never happen (for example, when a WARN_ON is hit) or if a BUG/panic is hit. This is critical information. > The filesystems and the block device drivers can't know if the data > stored on them are important. Perhaps it would be nice if an administrator could additionally "mark" what _they_ consider critical, i.e. they are aware of and accept the consequences of a synchronous printk for certain messages. When I look at things like dynamic_printk and ftrace, I see mechanisms that provide excellent runtime tuning without much complexity. We could extend dynamic_printk to include all printk messages and also support the emergency classification. We could have something like ftrace-events, where certain _events_ (i.e a group of printk messages) could be classified as an emergency (for example, a WARN_ON event, a watchdog event, a general stacktrace event, etc.). These are just ideas that I'm bouncing around my brain. But as you said, only the administrator can know what is important. So perhaps we need to provide the administrator an interface to specify that. John Ogness
