Skip to content

runtime: short-term time.Sleep sleepers will be caught under long-term sleepers? #5439

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

Closed
mikioh opened this issue May 10, 2013 · 3 comments
Closed

Comments

@mikioh
Copy link
Contributor

mikioh commented May 10, 2013

What steps will reproduce the problem?
1. run attached timefor with flags -l and -s
Usage of ./timefor:
  -l=false: run long-term sleeper
  -s=false: use time.Sleep instead of time.After

What is the expected output?
all time.Sleep sleeper goroutines work well

What do you see instead?
all time.Sleep sleeper gorutines stalled

Attachments:

  1. timefor.go (908 bytes)
@mikioh
Copy link
Contributor Author

mikioh commented May 10, 2013

Comment 1:

I don't have much time to investigate it for now, but a little bit.
- looks like this happens on time.Sleep only, time.After works well (yay)
- so this issue is not a blocker for go 1.1 release (hooray)

@mikioh
Copy link
Contributor Author

mikioh commented May 10, 2013

Comment 2:

It's possible to repro this issue on both go1 and go1.1.

@ality
Copy link
Member

ality commented May 10, 2013

Comment 3:

This has to with overflow in runtime·tsleep.

Status changed to Duplicate.

Merged into issue #5321.

@golang golang locked and limited conversation to collaborators Jun 24, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants