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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.