Messages in this thread | | | From | Jan Kara <> | Subject | [PATCH 0/4 v6] Avoid softlockups in console_unlock() | Date | Thu, 7 Nov 2013 22:48:35 +0100 |
| |
Hello,
This is the next iteration of my printk patchset. Since v5 I've made the limit for printing configurable via kernel parameter and let it default to 0. So unless user sets printk.offload_chars on kernel command line, there will be no difference to current printk behavior.
Summary:
These patches avoid softlockups when a CPU gets caught in console_unlock() for a long time during heavy printing from other CPU. As is discussed in patch 3/4 it isn't enough to just silence the watchdog because if CPU spends too long in console_unlock() also RCU will complain, other CPUs can be blocked waiting for printing CPU to process IPI, and even disk can be offlined because interrupts would be disabled for too long.
This patch series solves the problem by stopping printing in console_unlock() after X (tunable) characters and the printing is postponed to irq work. To avoid hogging a single CPU (irq work gets processed on the same CPU where it was queued so it doesn't really help to reduce the printing load on that CPU) we introduce a new type of lazy irq work - IRQ_WORK_UNBOUND - which can be processed by any CPU.
The patch series has survived my testing without any softlockup reports. I've tested running sysrq-t (lots of printk output) while inserting modules (to generate IPIs and also some printk traffic) and also running two delayed works printing 10 KB of text each. All this was with simulated 9600 baud serial console.
Honza
| |