Stream: git-wasmtime

Topic: wasmtime / PR #3117 cranelift: Move most debug-level logs...


view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:02):

fitzgen opened PR #3117 from log-levels to main:

Cranelift crates have historically been much more verbose with debug-level
logging than most other crates in the Rust ecosystem. We log things like how
many parameters a basic block has, the color of virtual registers during
regalloc, etc. Even for Cranelift hackers, these things are largely only useful
when hacking specifically on Cranelift and looking at a particular test case,
not even when using some Cranelift embedding (such as Wasmtime).

Most of the time, when people want logging for their Rust programs, they do
something like:

RUST_LOG=debug cargo run

This means that they get all that mostly not useful debug logging out of
Cranelift. So they might want to disable logging for Cranelift, or change it to
a higher log level:

RUST_LOG=debug,cranelift=info cargo run

The problem is that this is already more annoying to type that RUST_LOG=debug,
and that Cranelift isn't one single crate, so you actually have to play
whack-a-mole with naming all the Cranelift crates off the top of your head,
something more like this:

RUST_LOG=debug,cranelift=info,cranelift_codegen=info,cranelift_wasm=info,...

Therefore, we're changing most of the debug! logs into trace! logs: anything
that is very Cranelift-internal, unlikely to be useful/meaningful to the
"average" Cranelift embedder, or prints a message for each instruction visited
during a pass. On the other hand, things that just report a one line statistic
for a whole pass, for example, are left as debug!. The more verbose the log
messages are, the higher the bar they must clear to be debug! rather than
trace!.

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:02):

fitzgen requested cfallin for a review on PR #3117.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:13):

bjorn3 created PR review comment:

I think this should remain debug!.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:13):

bjorn3 submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:13):

bjorn3 created PR review comment:

And this one.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:13):

bjorn3 submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:14):

bjorn3 edited PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:39):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:39):

cfallin created PR review comment:

I think I actually err the other way on this one: the VCode dump is potentially thousands of lines for large functions; including all of the compiled IR in the debug log is potentially much more verbose than the other sorts of entries that people typically generate.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:52):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jul 26 2021 at 19:52):

cfallin merged PR #3117.


Last updated: Dec 23 2024 at 12:05 UTC