Add --unlink-aux-files option #142
Conversation
|
This looks good, will merge once you've added Linux support and addressed the failures. |
samply/src/main.rs
Outdated
|
|
||
| /// If a process produces jitdump or marker files, unlink them after | ||
| /// opening. This ensures that the files will not be left in /tmp, | ||
| /// but it will also be impossible to look at JIT disassembly. |
There was a problem hiding this comment.
[...], and line number information will be missing in the profiles.
|
Actually, I just had an idea -- I'll make this flag unlink jitdump files after the samply process exits. This would mean that while the server is up, the jitdump files are still around to provide all the info, but get cleaned up at exit. And if you're doing --save-only, well, don't pass this flag. |
|
Well, that'll work until I make the planned change to daemonize the server, so that you don't have to keep pressing Ctrl+C after every profiling run (and so that you can serve multiple profiles at the same time without using up excessive numbers of ports). I'd say don't worry about it too much at the moment - do whatever's easy and solves your use case, and we can sort out a long term solution later, once I've put the cards on the table by filing issues with all the things I have in mind. |
Adds an option to
recordto unlink jitdump/marker files after they've been read. Fixes #64... though looking over this, I only added it to mac, will add to linux.