search

mstange / samply

Command-line sampling profiler for macOS and Linux
Apache License 2.0
1.96k stars 47 forks source link

README updates? #249

Open djc opened 3 weeks ago

djc commented 3 weeks ago

Following up on https://phanpy.social/#/hachyderm.io/s/112571606961741408, the root README.md has:

samply works on macOS and Linux. Windows support is planned.

But the samply crate README.md has:

At the moment, the macOS implementation works best. The Linux implementation is extremely new and experimental. There is no Windows implementation yet.

Per the Mastodon response from @jrmuizel, both of these might not be up to date? I would suggest maybe merging the README's so you don't forget to make updates to both? (The crate-level one gets shown on crates.io.)

Might also want to keep https://nnethercote.github.io/perf-book/profiling.html up to date.

I think samply is great, have been it using this week on rustup, thanks for building it! (It is dampening my motivation for maintaining cargo-flamegraph, so will try to make sure to point it out to people.)

djc commented 3 weeks ago

Added this in the flamegraph README:

Screenshot 2024-06-07 at 12 01 44

PRs with more accurate/extensive information welcome!