[RFC PATCH v5 0/3] printk: new ringbuffer implementation
From: John Ogness
Date: Wed Nov 27 2019 - 20:53:14 EST
Hello,
This is a follow-up RFC on the work to re-implement much of the
core of printk. The threads for the previous RFC versions are
here[0][1][2][3].
This RFC includes only the ringbuffer and a test module. This is
a rewrite of the proposed ringbuffer, now based on the proof of
concept[4] from Petr Mladek as agreed at the meeting[5] during
LPC2019 in Lisbon.
The internal structure has been reworked such that the printk
strings are in their own array, each separated by a 32-bit
integer.
A 2nd array contains the dictionary strings (also with each
separated by a 32-bit integer).
A 3rd array is made up of descriptors that contain all the
meta-data for each printk record (sequence number, timestamp,
loglevel, caller, etc.) as well as pointers into the other data
arrays for the text and dictionary data.
The writer interface is somewhat similar to v4, but the reader
interface has changed significantly. Rather than using an
iterator object, readers just specify the sequence number they
want to read. In effect, the sequence number acts as the
iterator.
I have been communicating with Petr the last couple months to
make sure this implementation fits his expectations. This RFC is
mainly to get some feedback from anyone else that may see
something that Petr and I have missed.
This series also includes my test module. On a 16-core ARM64
test machine, the module runs without any errors. I am seeing
the 15 writing cores each writing about 34500 records per
second, while the 1 reading core misses only about 15% of the
total records.
John Ogness
[0] https://lkml.kernel.org/r/20190212143003.48446-1-john.ogness@xxxxxxxxxxxxx
[1] https://lkml.kernel.org/r/20190607162349.18199-1-john.ogness@xxxxxxxxxxxxx
[2] https://lkml.kernel.org/r/20190727013333.11260-1-john.ogness@xxxxxxxxxxxxx
[3] https://lkml.kernel.org/r/20190807222634.1723-1-john.ogness@xxxxxxxxxxxxx
[4] https://lkml.kernel.org/r/20190704103321.10022-1-pmladek@xxxxxxxx
[5] https://lkml.kernel.org/r/87k1acz5rx.fsf@xxxxxxxxxxxxx
John Ogness (3):
printk-rb: new printk ringbuffer implementation (writer)
printk-rb: new printk ringbuffer implementation (reader)
printk-rb: add test module
kernel/printk/Makefile | 3 +
kernel/printk/printk_ringbuffer.c | 910 ++++++++++++++++++++++++++++++
kernel/printk/printk_ringbuffer.h | 249 ++++++++
kernel/printk/test_prb.c | 347 ++++++++++++
4 files changed, 1509 insertions(+)
create mode 100644 kernel/printk/printk_ringbuffer.c
create mode 100644 kernel/printk/printk_ringbuffer.h
create mode 100644 kernel/printk/test_prb.c
--
2.20.1