Re: [PATCH] printk: fix kmsg_dump_get_buffer length calulations
On Wed 2021-01-13 17:50:13, John Ogness wrote: > kmsg_dump_get_buffer() uses @syslog to determine if the syslog > prefix should be written to the buffer. However, when calculating > the maximum number of records that can fit into the buffer, it > always counts the bytes from the syslog prefix. > > Use @syslog when calculating the maximum number of records that can > fit into the buffer. > > Fixes: e2ae715d66bf ("kmsg - kmsg_dump() use iterator to receive log buffer > content") > Signed-off-by: John Ogness The patch is committed in printk/linux.git, branch printk-rework. I plan to send it for-5.11 the following week after it spends few days in linux-next. Best Regards, Petr
Re: [PATCH] printk: fix kmsg_dump_get_buffer length calulations
On (21/01/13 17:50), John Ogness wrote: > > kmsg_dump_get_buffer() uses @syslog to determine if the syslog > prefix should be written to the buffer. However, when calculating > the maximum number of records that can fit into the buffer, it > always counts the bytes from the syslog prefix. > > Use @syslog when calculating the maximum number of records that can > fit into the buffer. > > Fixes: e2ae715d66bf ("kmsg - kmsg_dump() use iterator to receive log buffer > content") > Signed-off-by: John Ogness Acked-by: Sergey Senozhatsky -ss
Re: [PATCH] printk: fix kmsg_dump_get_buffer length calulations
On Wed 2021-01-13 17:50:13, John Ogness wrote: > kmsg_dump_get_buffer() uses @syslog to determine if the syslog > prefix should be written to the buffer. However, when calculating > the maximum number of records that can fit into the buffer, it > always counts the bytes from the syslog prefix. > > Use @syslog when calculating the maximum number of records that can > fit into the buffer. Another great catch! > Fixes: e2ae715d66bf ("kmsg - kmsg_dump() use iterator to receive log buffer > content") > Signed-off-by: John Ogness Reviewed-by: Petr Mladek Best Regards, Petr
[PATCH] printk: fix kmsg_dump_get_buffer length calulations
kmsg_dump_get_buffer() uses @syslog to determine if the syslog prefix should be written to the buffer. However, when calculating the maximum number of records that can fit into the buffer, it always counts the bytes from the syslog prefix. Use @syslog when calculating the maximum number of records that can fit into the buffer. Fixes: e2ae715d66bf ("kmsg - kmsg_dump() use iterator to receive log buffer content") Signed-off-by: John Ogness --- kernel/printk/printk.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index ffdd0dc7ec6d..848b56efc9d7 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -3427,7 +3427,7 @@ bool kmsg_dump_get_buffer(struct kmsg_dumper *dumper, bool syslog, while (prb_read_valid_info(prb, seq, , _count)) { if (r.info->seq >= dumper->next_seq) break; - l += get_record_print_text_size(, line_count, true, time); + l += get_record_print_text_size(, line_count, syslog, time); seq = r.info->seq + 1; } @@ -3437,7 +3437,7 @@ bool kmsg_dump_get_buffer(struct kmsg_dumper *dumper, bool syslog, , _count)) { if (r.info->seq >= dumper->next_seq) break; - l -= get_record_print_text_size(, line_count, true, time); + l -= get_record_print_text_size(, line_count, syslog, time); seq = r.info->seq + 1; } -- 2.20.1