On 10.11.20 г. 15:14 ч., John Ogness wrote:
> On 2020-11-10, Nikolay Borisov <nbori...@suse.com> wrote:
>> data_alloc's 2nd argument is always rb::text_data_ring and that functino
>> always takes a struct printk_ringbuffer. Instead of passing the data
>> ring buffer as an argument simply make it a local variable.
>
> This is a relic of when we had a second data ring (for
> dictionaries). The patch is a nice cleanup, but there are actually
> several functions that could use this exact same cleanup:
>
> - data_make_reusable()
> - data_push_tail()
> - data_alloc()
> - data_realloc()
>
> Perhaps we should fix them all in a single patch?
I observed that right after sending this patch, so I have authored the
necessary changes I can squash them and send them.
>
> John Ogness
>
