From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: "Stefan Hajnoczi" <stefanha@redhat.com>,
qemu-devel@nongnu.org, qemu-ppc@nongnu.org,
"Markus Armbruster" <armbru@redhat.com>,
"Eric Blake" <eblake@redhat.com>,
"Pierrick Bouvier" <pierrick.bouvier@oss.qualcomm.com>,
"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Helge Deller" <deller@gmx.de>,
xen-devel@lists.xenproject.org,
"Anthony PERARD" <anthony@xenproject.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Stefano Stabellini" <sstabellini@kernel.org>,
"Gerd Hoffmann" <kraxel@redhat.com>,
"Alexey Kardashevskiy" <aik@ozlabs.ru>
Subject: Re: [PULL 00/25] Tracing patches
Date: Wed, 4 Nov 2026 10:30:00 +0000 [thread overview]
Message-ID: <ausJKNfBbqXtPDe9@redhat.com> (raw)
In-Reply-To: <102e313c-1193-44b1-9908-a03c01a85e8d@linaro.org>
On Tue, Nov 03, 2026 at 10:59:43AM -0800, Richard Henderson wrote:
> On 11/2/26 10:58, Stefan Hajnoczi wrote:
> > The following changes since commit 56afd82bd4db9688f1c1c9d0e2ea163b8cde3283:
> >
> > Merge tag 'qemu-sparc-20261101' ofhttps://github.com/mcayland/qemu into staging (2026-11-01 08:37:48 -0800)
> >
> > are available in the Git repository at:
> >
> > https://gitlab.com/stefanha/qemu.git tags/tracing-pull-request
> >
> > for you to fetch changes up to 3b68720390b98f21e26ffcff8874851b7b49b7b7:
> >
> > trace: remove 'trace_event_get_state' wrapper (2026-11-02 13:52:14 -0500)
> >
> > ----------------------------------------------------------------
> > Pull request
> >
> > Daniel Berrangé's 'disable' property removal and associated cleanups.
>
> Function signature change in here somewhere that didn't get propagated to
> rust?
>
> https://gitlab.com/qemu-project/qemu/-/jobs/17710633655#L4275
>
> > error[E0308]: mismatched types
> > --> /builds/qemu-project/qemu/build/trace/trace-hw_char.rs:236:14
> > |
> > 236 | unsafe { (bindings::qemu_loglevel & bindings::LOG_TRACE) != 0 &&
> > | ______________^
> > 237 | | trace_event_state_is_enabled(_TRACE_PARALLEL_IOPORT_READ_DSTATE) } ||
> > | |_____________________________________________________________________________^ expected `()`, found `bool`
> > |
>
> etc.
Bug in the "trace: include qemu_loglevel_mask(LOG_TRACE) in guard" patch
Guess I must have not had --enable-rust set after I changed that code.
With regards,
Daniel
--
|: https://berrange.com ~~ https://hachyderm.io/@berrange :|
|: https://libvirt.org ~~ https://entangle-photo.org :|
|: https://pixelfed.art/berrange ~~ https://fstop138.berrange.com :|
prev parent reply other threads:[~2026-11-04 10:30 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-02 18:58 [PULL 00/25] Tracing patches Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 01/25] osdep: add 'ignore_value(x)' macro Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 02/25] trace: use 'ignore_value()' instead of a dummy variable Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 03/25] trace: fix handling of char** in UST backend Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 04/25] hw/xen: replace "long unsigned int" with "unsigned long " for dtrace Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 05/25] trace: block use of "long int" " Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 06/25] trace: drop unused trace_event_set_state_dynamic_init() Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 07/25] trace: remove obsolete references to vcpu property Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 08/25] trace: drop 'id' from TraceEvent struct when not required Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 09/25] trace: change "DSTATE" variables from uint16_t to uint8_t Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 10/25] trace: add testing of 'nop' trace backend Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 11/25] trace: don't emit TraceEvent structs when 'nop' backend is used Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 12/25] trace: add test of generation with all backends enabled Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 13/25] trace: remove redundant headers from simple trace backend code Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 14/25] trace: include probe name in Rust format strings Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 15/25] trace: add missing newline in ftrace Rust format string Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 16/25] trace: eliminate dupe trace_event_get_state_dynamic_by_id() calls Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 17/25] audio: remove mistaken use of trace_event_get_state() Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 18/25] hw/ppc: " Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 19/25] monitor: " Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 20/25] trace: include qemu_loglevel_mask(LOG_TRACE) in guard Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 21/25] target/hppa: use PRIx64 in trace-events Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 22/25] trace: remove "disable" property from trace files Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 23/25] trace: stop tracking static disabled state Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 24/25] trace: remove parsing of "disable" event property Stefan Hajnoczi
2026-11-02 18:58 ` [PULL 25/25] trace: remove 'trace_event_get_state' wrapper Stefan Hajnoczi
2026-11-03 18:59 ` [PULL 00/25] Tracing patches Richard Henderson
2026-11-04 10:30 ` Daniel P. Berrangé [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ausJKNfBbqXtPDe9@redhat.com \
--to=berrange@redhat.com \
--cc=aik@ozlabs.ru \
--cc=anthony@xenproject.org \
--cc=armbru@redhat.com \
--cc=deller@gmx.de \
--cc=eblake@redhat.com \
--cc=edgar.iglesias@gmail.com \
--cc=kraxel@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=pbonzini@redhat.com \
--cc=pierrick.bouvier@oss.qualcomm.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=sstabellini@kernel.org \
--cc=stefanha@redhat.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox