Skip to content

Remove concurrency counters #1

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
Open
xaionaro opened this issue Mar 9, 2019 · 1 comment
Open

Remove concurrency counters #1

xaionaro opened this issue Mar 9, 2019 · 1 comment

Comments

@xaionaro
Copy link
Member

xaionaro commented Mar 9, 2019

Currently if we disable concurrency counters via SetForbidToGrow(true) then Get() performance rises up to 10 times. And we can remove this counters at all (they are not really required).

It's just required to use busySlots as detector of will we grow now or not for Get()'s
xaionaro added a commit that referenced this issue Mar 9, 2019
@xaionaro
Improved performance
af15df1 
See: #1
@xaionaro
Copy link
Member Author

I faced a strange problem with Golang (the most of the CPU is utilized on NOP instructions):

golang/go#30708

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xaionaro