Skip to content

std: Be resilient to failure in pthread_getattr_np #27073

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.

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 1 commit into from
Jul 21, 2015

Conversation

alexcrichton
Copy link
Member

This can fail on linux for various reasons, such as the /proc filesystem not
being mounted. There are already many cases where we can't set up stack guards,
so just don't worry about this case and communicate that no guard was enabled.

I've confirmed that this allows the compiler to run in a chroot without /proc
mounted.

Closes #22642

@rust-highfive
Copy link
Contributor

r? @aturon

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

@alexcrichton
Copy link
Member Author

cc @dwrensha

@dwrensha
Copy link
Contributor

\o/

@nagisa
Copy link
Member

nagisa commented Jul 16, 2015

This LGTM as a hack until we finally get LLVM to implement stack probes (please, we’ve been waiting for quite a while now!).

@alexcrichton
Copy link
Member Author

r? @brson

@rust-highfive rust-highfive assigned brson and unassigned aturon Jul 20, 2015
@brson
Copy link
Contributor

brson commented Jul 21, 2015

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

📌 Commit ba19824 has been approved by brson

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

⌛ Testing commit ba19824 with merge 85f9fcb...

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

💔 Test failed - auto-mac-64-opt

@alexcrichton
Copy link
Member Author

@bors: r=brson cfe464b

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

⌛ Testing commit cfe464b with merge 8420563...

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

💔 Test failed - auto-win-gnu-32-nopt-t

@alexcrichton
Copy link
Member Author

@bors: r=brson

On Mon, Jul 20, 2015 at 8:19 PM, bors [email protected] wrote:

[image: 💔] Test failed - auto-win-gnu-32-nopt-t
http://buildbot.rust-lang.org/builders/auto-win-gnu-32-nopt-t/builds/747


Reply to this email directly or view it on GitHub
#27073 (comment).

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

📌 Commit e966eed has been approved by brson

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

⌛ Testing commit e966eed with merge 4ff7780...

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

💔 Test failed - auto-linux-64-x-android-t

This can fail on linux for various reasons, such as the /proc filesystem not
being mounted. There are already many cases where we can't set up stack guards,
so just don't worry about this case and communicate that no guard was enabled.

I've confirmed that this allows the compiler to run in a chroot without /proc
mounted.

Closes rust-lang#22642
@alexcrichton
Copy link
Member Author

@bors: r=brson

On Tue, Jul 21, 2015 at 1:26 AM, bors [email protected] wrote:

[image: 💔] Test failed - auto-linux-64-x-android-t
http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/5740


Reply to this email directly or view it on GitHub
#27073 (comment).

@bors
Copy link
Collaborator

bors commented Jul 21, 2015

📌 Commit d68b152 has been approved by brson

bors added a commit that referenced this pull request Jul 21, 2015
This can fail on linux for various reasons, such as the /proc filesystem not
being mounted. There are already many cases where we can't set up stack guards,
so just don't worry about this case and communicate that no guard was enabled.

I've confirmed that this allows the compiler to run in a chroot without /proc
mounted.

Closes #22642
@bors
Copy link
Collaborator

bors commented Jul 21, 2015

⌛ Testing commit d68b152 with merge ee2d3bc...

@bors bors merged commit d68b152 into rust-lang:master Jul 21, 2015
@alexcrichton alexcrichton deleted the less-proc-fs branch July 21, 2015 22:38
@dwrensha
Copy link
Contributor

Thank you!

dwrensha pushed a commit to dwrensha/acronymy that referenced this pull request Jul 22, 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.

7 participants