Re: [PATCH v4 00/34] Printbufs - new data structure for building strings

From: Steven Rostedt
Date: Tue Jul 19 2022 - 19:15:31 EST


On Sun, 19 Jun 2022 20:41:59 -0400
Kent Overstreet <kent.overstreet@xxxxxxxxx> wrote:

> Core idea: Wouldn't it be nice if we had a common data structure and calling
> convention for outputting strings?

Because seq_buf gives us this already, the cover letter really just needs
to state exactly what the benefit is to replace seq_buf with printbuf (and
why seq_buf can not be simply extended to do some extra features).

I just applied your series and ran the tracing selftests and several of
them failed.

# cd tools/testing/selftests/ftrace/
# ./ftracetest

This means that this is not a simple replacement and that there's going to
be regressions with this change. The question is, is the added benefits of
doing the change greater than the fallout of the regressions?

-- Steve