87 lines
2.5 KiB
Diff
87 lines
2.5 KiB
Diff
From e4e671838ea60af90e00b14e9877e9cc6d2a9476 Mon Sep 17 00:00:00 2001
|
|
Message-Id: <e4e671838ea60af90e00b14e9877e9cc6d2a9476.1601675152.git.zanussi@kernel.org>
|
|
In-Reply-To: <5b5a156f9808b1acf1205606e03da117214549ea.1601675151.git.zanussi@kernel.org>
|
|
References: <5b5a156f9808b1acf1205606e03da117214549ea.1601675151.git.zanussi@kernel.org>
|
|
From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
|
|
Date: Thu, 21 Mar 2013 19:01:05 +0100
|
|
Subject: [PATCH 208/333] printk: Drop the logbuf_lock more often
|
|
Origin: https://www.kernel.org/pub/linux/kernel/projects/rt/4.19/older/patches-4.19.148-rt64.tar.xz
|
|
|
|
The lock is hold with irgs off. The latency drops 500us+ on my arm bugs
|
|
with a "full" buffer after executing "dmesg" on the shell.
|
|
|
|
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
|
|
---
|
|
kernel/printk/printk.c | 28 ++++++++++++++++++++++++++++
|
|
1 file changed, 28 insertions(+)
|
|
|
|
diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c
|
|
index c5fbc51e61ed..4ac542763a09 100644
|
|
--- a/kernel/printk/printk.c
|
|
+++ b/kernel/printk/printk.c
|
|
@@ -1455,12 +1455,23 @@ static int syslog_print_all(char __user *buf, int size, bool clear)
|
|
u64 next_seq;
|
|
u64 seq;
|
|
u32 idx;
|
|
+ int attempts = 0;
|
|
+ int num_msg;
|
|
|
|
text = kmalloc(LOG_LINE_MAX + PREFIX_MAX, GFP_KERNEL);
|
|
if (!text)
|
|
return -ENOMEM;
|
|
|
|
logbuf_lock_irq();
|
|
+
|
|
+try_again:
|
|
+ attempts++;
|
|
+ if (attempts > 10) {
|
|
+ len = -EBUSY;
|
|
+ goto out;
|
|
+ }
|
|
+ num_msg = 0;
|
|
+
|
|
/*
|
|
* Find first record that fits, including all following records,
|
|
* into the user-provided buffer for this dump.
|
|
@@ -1473,6 +1484,14 @@ static int syslog_print_all(char __user *buf, int size, bool clear)
|
|
len += msg_print_text(msg, true, NULL, 0);
|
|
idx = log_next(idx);
|
|
seq++;
|
|
+ num_msg++;
|
|
+ if (num_msg > 5) {
|
|
+ num_msg = 0;
|
|
+ logbuf_unlock_irq();
|
|
+ logbuf_lock_irq();
|
|
+ if (clear_seq < log_first_seq)
|
|
+ goto try_again;
|
|
+ }
|
|
}
|
|
|
|
/* move first record forward until length fits into the buffer */
|
|
@@ -1484,6 +1503,14 @@ static int syslog_print_all(char __user *buf, int size, bool clear)
|
|
len -= msg_print_text(msg, true, NULL, 0);
|
|
idx = log_next(idx);
|
|
seq++;
|
|
+ num_msg++;
|
|
+ if (num_msg > 5) {
|
|
+ num_msg = 0;
|
|
+ logbuf_unlock_irq();
|
|
+ logbuf_lock_irq();
|
|
+ if (clear_seq < log_first_seq)
|
|
+ goto try_again;
|
|
+ }
|
|
}
|
|
|
|
/* last message fitting into this dump */
|
|
@@ -1521,6 +1548,7 @@ static int syslog_print_all(char __user *buf, int size, bool clear)
|
|
clear_seq = log_next_seq;
|
|
clear_idx = log_next_idx;
|
|
}
|
|
+out:
|
|
logbuf_unlock_irq();
|
|
|
|
kfree(text);
|
|
--
|
|
2.17.1
|
|
|