[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 00/13] [RfC] fix tracing for modules
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH v4 00/13] [RfC] fix tracing for modules |
Date: |
Thu, 10 Jun 2021 08:58:50 +0100 |
On Thu, Jun 10, 2021 at 08:32:05AM +0200, Gerd Hoffmann wrote:
> On Wed, Jun 09, 2021 at 04:14:12PM +0100, Stefan Hajnoczi wrote:
> > On Tue, Jun 01, 2021 at 03:24:01PM +0200, Gerd Hoffmann wrote:
> > > First version that actually works. Only qxl covered for this RfC,
> > > other modules will follow once the basics are hashed out.
> > >
> > > v4:
> > > - rebase to latest master.
> > > - fix systemtap tracing.
> > > - also cover virtio-gpu modules.
> > > - pick up some review tags.
> > > - misc minor tweaks.
> > >
> > > v3:
> > > - handle initialization of modular tracepoints.
> > >
> > > TODO:
> > > Enabling modular tracepoints via -trace cmd line doesn't work yet.
> > > Guess we need to store the list somewhere for later re-processing.
> > > Error handling is tricky, specifically the "tracepoint doesn't exist"
> > > error. Suggestions / ideas are welcome.
> >
> > Excellent! I think we discussed these TODOs in the past. Can we merge
> > this series in close to its current form or does it degrade the tracing
> > experience (e.g. does the -trace command-line no longer work for
> > trace-events files that were split off into modules)?
>
> Main focus for v4 was to make stap work.
> The cmd line issue is still there.
>
> Patches 1-5 can be cherry-picked without breaking something.
Thanks, I have applied Patches 1-5 to my tracing tree:
https://gitlab.com/stefanha/qemu/-/tree/tracing
Stefan
signature.asc
Description: PGP signature
- [PATCH v4 08/13] meson: add module_trace & module_trace_src, (continued)
- [PATCH v4 08/13] meson: add module_trace & module_trace_src, Gerd Hoffmann, 2021/06/01
- [PATCH v4 10/13] meson: move qxl trace events to separate file, Gerd Hoffmann, 2021/06/01
- [PATCH v4 11/13] virtio-gpu: split trace points, Gerd Hoffmann, 2021/06/01
- [PATCH v4 12/13] meson: move virtio trace events to separate file, Gerd Hoffmann, 2021/06/01
- [PATCH v4 13/13] meson: move virtio-gl trace events to separate file, Gerd Hoffmann, 2021/06/01
- Re: [PATCH v4 00/13] [RfC] fix tracing for modules, Stefan Hajnoczi, 2021/06/09