Skip to content

feat: Thread support #118

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Sep 23, 2025
Merged

feat: Thread support #118

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
d66cfd5
feat(threads): install an internal thread event hook
nickysn Aug 22, 2025
77194b6
feat(threads): update runtime_tracing dependency to version 0.15.0
nickysn Sep 16, 2025
596dc41
feat(threads): update Cargo.lock
nickysn Sep 16, 2025
1ae5f6c
feat(threads): install the internal thread event hook only once
nickysn Sep 16, 2025
749bbc9
feat(threads): protect Recorder.tracer with a mutex
nickysn Sep 16, 2025
7fb04c0
chore: cargo fmt
nickysn Sep 16, 2025
41bbdbb
feat(threads): record ThreadStart() events
nickysn Sep 16, 2025
3475943
feat(threads): introduced flag recorder.data.in_event_hook to protect…
nickysn Sep 16, 2025
7f83876
feat(threads): fix deadlock when record_event_api() is called, during…
nickysn Sep 16, 2025
2b0bde6
chore: remove debug printlns
nickysn Sep 16, 2025
1948119
chore: cargo fmt
nickysn Sep 16, 2025
faad820
feat(threads): record ThreadExit() events
nickysn Sep 16, 2025
71ac828
feat(threads): fixed deadlock when logging a ThreadExit() event, duri…
nickysn Sep 17, 2025
5d825b7
feat(threads): add ThreadSwitch() events
nickysn Sep 17, 2025
c387478
chore: cargo fmt
nickysn Sep 17, 2025
860af99
test(threads): fix threads, by removing the ThreadSwitch record event…
nickysn Sep 18, 2025
5da521e
fix!: upgrade ruby to `3.4`
reo101 Sep 19, 2025
af4c09c
fix: fix tests after Ruby version upgrade
nickysn Sep 19, 2025
8e23b67
fix!: upgrade Ruby to 3.4 in CI/benchmark as well
nickysn Sep 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
- name: Setup Ruby
uses: ruby/setup-ruby@v1
with:
ruby-version: '3.2'
ruby-version: '3.4'
- name: Setup Rust
uses: dtolnay/rust-toolchain@stable
with:
Expand Down Expand Up @@ -75,7 +75,7 @@ jobs:
- name: Setup Ruby
uses: ruby/setup-ruby@v1
with:
ruby-version: '3.2'
ruby-version: '3.4'
- name: Setup Rust
uses: dtolnay/rust-toolchain@stable
- name: Install Ruby dependencies
Expand Down
4 changes: 3 additions & 1 deletion flake.nix
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,9 @@
in {
default = pkgs.mkShell {
packages = with pkgs; [
ruby
# WARNING: `3.4` needed in `./gems/codetracer-ruby-recorder/ext/native_tracer/src/lib.rs`
# for the `thread` field of `rb_internal_thread_event_data_t`
ruby_3_4

# The native extension is implemented in Rust
(rust-toolchain-for system)
Expand Down
66 changes: 62 additions & 4 deletions gems/codetracer-ruby-recorder/ext/native_tracer/Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion gems/codetracer-ruby-recorder/ext/native_tracer/Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ crate-type = ["cdylib"]

[dependencies]
rb-sys = "0.9"
runtime_tracing = "0.14.0"
runtime_tracing = "0.15.0"

[build-dependencies]
rb-sys-env = "0.2"
Expand Down
Loading
Loading