Messages in this thread | | | Date | Mon, 8 Aug 2022 09:37:50 -0400 | From | Steven Rostedt <> | Subject | Re: [PATCH v5 31/32] tracing: Convert to printbuf |
| |
On Sun, 7 Aug 2022 23:32:01 -0400 Kent Overstreet <kent.overstreet@gmail.com> wrote:
> > Please remove the two tracing patches and the deletion of the seq_buf from > > the series. > > Well, that's not really an option, as Christoph already (rightly) > pointed out.
These are the last patches of the series. There's no dependency on them. You should be able to simply drop them. If others find that these patches are worth their while then by all means, let them have them.
> > If you've got actual engineering concerns that you'd care to articulate > I'd (still) like to try to work with you - otherwise, I don't think this > is something I can accommodate you on.
Here's my technical reason. These are non trivial changes that are replacing code that has been stable for 8 years that the tracing infrastructure highly depends on. I do not have the time to sit down and review this code as it is not a priority. My time is extremely limited (as my wife keeps complaining to me about, as I'm not spending enough time with the family).
This change is likely to cause subtle regressions for no benefit to the tracing subsystem. Hence, when it comes to risk vs reward, I see none.
I'm not NACKing the series, just the changes to the tracing subsystem.
When this code is mature enough, I may reconsider my stance on this.
-- Steve
| |