Skip to content
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

Jump to bottom

Assign names to application threads #103

Merged
merged 1 commit into from
Nov 15, 2024
Merged

Assign names to application threads #103

merged 1 commit into from
Nov 15, 2024

Conversation

maxsharabayko
Copy link
Owner

@maxsharabayko maxsharabayko commented Nov 15, 2024 •
edited
Loading

Thread names:

  • XTR:Gen
  • XTR:Metrics
  • XTR:Stats
  • XTR:Rcv
  • XTR:Route

Example top output:

  PID USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND
21620 admin     20   0  316m 7096 5336 S   10  0.5   0:01.81 SRT:RcvQ:w1
21626 admin     20   0  316m 7096 5336 S    8  0.5   0:01.30 XTR:Rcv
21627 admin     20   0  316m 7096 5336 S    8  0.5   0:01.31 SRT:TsbPd 
21617 admin     20   0  316m 7096 5336 S    0  0.5   0:00.01 srt-xtransmit 
21618 admin     20   0  316m 7096 5336 S    0  0.5   0:00.00 SRT:GC 
21619 admin     20   0  316m 7096 5336 S    0  0.5   0:00.00 SRT:SndQ:w1 
21625 admin     20   0  316m 7096 5336 S    0  0.5   0:00.00 XTR:Stats

github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 15, 2024
View reviewed changes
xtransmit/misc.cpp
@@ -177,6 +177,7 @@
void common_run(const vector<string>& urls, const stats_config& cfg_stats, const conn_config& cfg_conn,
const atomic_bool& break_token, processing_fn_t& processing_fn)
{
//XTR_THREADNAME(std::string("XTR:ConnMngmt"));

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
@maxsharabayko maxsharabayko merged commit 45eca3e into master Nov 15, 2024
6 checks passed
@maxsharabayko maxsharabayko deleted the develop/thread-name branch November 15, 2024 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@maxsharabayko