Skip to content

Commit

Permalink
Do not enable fibers in tracing-tracy
Browse files Browse the repository at this point in the history
Fibers introduce a significant overhead to each invocation of Tracy
instrumentation calls, which in turn makes all tracing instrumentation
points (with callstack collection disabled) 2x slower than they could
be.

I anticipate that `fibers` still may become unconditional requirement in
the future due to #35, but there's no reason to impose that overhead on
users today.
  • Loading branch information
nagisa committed Jun 11, 2022
1 parent d5d2476 commit de4325c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions README.mkd
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,6 @@ The following table lists the version correspondence between the libraries.
| v0.7.8 | 0.16.0 | 0.12.* | 0.6.* |
| v0.7.8 | 0.16.0 | 0.12.* | 0.7.* |
| v0.7.8 | 0.16.0 | 0.12.* | 0.8.* |
| v0.8.1 | 0.17.0 | 0.13.* | 0.9.* |
| v0.8.1 | 0.17.* | 0.14.* | ... |
| v0.8.1 | 0.17.* | 0.13.* | 0.9.* |
| v0.8.1 | 0.17.* | 0.14.* | 0.10.* |
<!-- AUTO-UPDATE -->
4 changes: 2 additions & 2 deletions tracing-tracy/Cargo.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[package]
name = "tracing-tracy"
version = "0.9.0"
version = "0.10.0"
authors = ["Simonas Kazlauskas <[email protected]>"]
license = "MIT/Apache-2.0"
edition = "2018"
Expand All @@ -19,7 +19,7 @@ bench = true
[dependencies]
tracing-core = { version = "0.1", default-features = false, features = ["std"] }
tracing-subscriber = { version = "0.3", default-features = false, features = ["fmt", "registry"] }
client = { package = "tracy-client", path = "../tracy-client", version = "0.13.0", default-features = false, features = ["fibers"] }
client = { package = "tracy-client", path = "../tracy-client", version = "0.14.0", default-features = false }

[dev-dependencies]
tracing = { version = "0.1", default-features = false, features = ["std"] }
Expand Down

0 comments on commit de4325c

Please sign in to comment.