[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 10/11] trace: document how to specify multiple --trace pattern
From: |
Daniel P . Berrangé |
Subject: |
Re: [PULL 10/11] trace: document how to specify multiple --trace patterns |
Date: |
Mon, 1 Feb 2021 17:46:22 +0000 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Mon, Feb 01, 2021 at 06:25:33PM +0100, Paolo Bonzini wrote:
> On 01/02/21 17:54, Kevin Wolf wrote:
> > > How does this option parsing work? Would then multiple patterns separated
> > > by
> > > comma as in -trace pattern1,pattern2 also work?
> > This would be interpreted as an implied "enable" option with a value of
> > "pattern1,pattern2". I don't think anything splits that string at the
> > comma, so it would look for a trace event matching that string.
>
> Even worse, it would be interpreted as "-trace enable=pattern1,pattern2=on"
> (and raise a warning since recently).
Maybe we're trying to solve the problem at the wrong level.
The pattern is currently matched using the GLib glob matching APIs.
If we switched to use the GLib regex matching APIs, then we don't need
to repeat the args at all. We could just use regex syntax:
-trace 'enable=(kvm|virtio)*'
It is a little tedious to have to quote the args to avoid shell
expansion, but as a tradeoff you get much stronger ability todo
complex matches to filter out irrelevant cruft.
If we want to maintain back compat for glob syntax, then we should
support both in parallel and accept a different parameter name
for the regex style.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, (continued)
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, Kevin Wolf, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, Philippe Mathieu-Daudé, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, Kevin Wolf, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, Paolo Bonzini, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, Markus Armbruster, 2021/02/02
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/02/02
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, Paolo Bonzini, 2021/02/02
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns,
Daniel P . Berrangé <=
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, BALATON Zoltan, 2021/02/01
- Re: [PULL 10/11] trace: document how to specify multiple --trace patterns, Stefan Hajnoczi, 2021/02/02
[PULL 09/11] simpletrace: build() missing 2 required positional arguments, Stefan Hajnoczi, 2021/02/01
[PULL 05/11] tracetool: also strip %l and %ll from systemtap format strings, Stefan Hajnoczi, 2021/02/01
Re: [PULL 00/11] Tracing patches, Peter Maydell, 2021/02/02