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.

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

Multiple races when starting daemon #3992

Closed
magik6k opened this issue Jun 19, 2017 · 2 comments
Closed

Multiple races when starting daemon #3992

magik6k opened this issue Jun 19, 2017 · 2 comments
Labels
help wanted Seeking public contribution on this issue

Comments

@magik6k
Copy link
Member

magik6k commented Jun 19, 2017

Version information:

Current master

Severity:

Medium/Low

Description:

When compiling with go race detector (GOFLAGS='-gcflags "-N -l" -race' make build ), when daemon is starting, there are multiple races reported:

https://gist.github.com/magik6k/0d5474ca940dc257abfc448c7e4e9abf

I haven't looked at them in detail, it's just what i have noticed when debugging different issue.

@magik6k magik6k marked this as a duplicate of #4105 Jul 29, 2017
@whyrusleeping whyrusleeping added the help wanted Seeking public contribution on this issue label Aug 31, 2017
@whyrusleeping whyrusleeping added this to the Ipfs 0.4.12 milestone Aug 31, 2017
@whyrusleeping
Copy link
Member

In general, It would be great to be able to run the ipfs daemon with race detection enabled. It currently fails because we open too many goroutines (too many connections + too many goroutines per connection in bitswap mostly). I remember @Kubuxu was working on addressing some of this

@Kubuxu Kubuxu modified the milestones: Ipfs 0.4.12, go-ipfs 0.4.13 Nov 6, 2017
@Stebalien
Copy link
Member

As far as I can tell, the only thing left is the mDNS race: libp2p/go-libp2p#257

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Seeking public contribution on this issue
Projects
None yet
Development

No branches or pull requests

4 participants