Skip to content

prctl instead of pthread on linux for name setup - #23640

Merged
bors merged 1 commit into
rust-lang:masterfrom
nagisa:thread-less-weak
Mar 24, 2015
Merged

prctl instead of pthread on linux for name setup#23640
bors merged 1 commit into
rust-lang:masterfrom
nagisa:thread-less-weak

Conversation

@nagisa

@nagisa nagisa commented Mar 23, 2015

Copy link
Copy Markdown
Member

This is more portable as far as linux is concerned.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@nagisa

nagisa commented Mar 23, 2015

Copy link
Copy Markdown
Member Author

r? @alexcrichton

@rust-highfive rust-highfive assigned alexcrichton and unassigned brson Mar 23, 2015
Comment thread src/libstd/sys/unix/thread.rs Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't this call return a failure on kernels which don't suppot PR_SET_NAME? I'd also be fine just ignoring the return value, if this fails it's not really the end of the world!

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’m ambivalent about this. Updated.

This is more portable as far as linux is concerned.
@nagisa
nagisa force-pushed the thread-less-weak branch from dcef9d2 to d29d554 Compare March 23, 2015 18:08
@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ d29d554

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Mar 23, 2015
This is more portable as far as linux is concerned.
@bors
bors merged commit d29d554 into rust-lang:master Mar 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants