-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
dbuf_find() mutex contention #1291
Comments
@behlendorf I'm testing right now with:
it's a bit early to say it's helping but so far it doesn't seem to have hurt (performance is the same not worse, I don't see rcu stalls (yet)) I'll probably leave this change in my tree for a while and revisit this in a week or two. |
@behlendorf I just noticed this was still open. As reported on IRC this did indeed help. |
@cwedgwood It helped by getting rid of the 'stall on CPU' messages or it measurably helped performance in some way? |
Due to evidence of contention both the buf_hash_table and the dbuf_hash_table sizes have been increased from 256 to 8192. Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Issue openzfs#1291
Due to evidence of contention both the buf_hash_table and the dbuf_hash_table sizes have been increased from 256 to 8192. Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Issue openzfs#1291
Due to evidence of contention both the buf_hash_table and the dbuf_hash_table sizes have been increased from 256 to 8192. Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Issue openzfs#1291 Signed-off-by: Pavel Snajdr <snajpa@snajpa.net>
Due to evidence of contention both the buf_hash_table and the dbuf_hash_table sizes have been increased from 256 to 8192. Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Issue openzfs#1291 Signed-off-by: Pavel Snajdr <snajpa@snajpa.net>
Due to evidence of contention both the buf_hash_table and the dbuf_hash_table sizes have been increased from 256 to 8192. This increase in hash table size adds approximating 0.5M to our fixed memory footprint. This relatively small increase is not expected to cause problems even on low memory machines. This footprint will also become dynamic when the persistent L2ARC support is finalized. In the meanwhile, this small change significantly reduces contention for certain workloads. Signed-off-by: Chris Wedgwood <cw@f00f.org> Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: Pavel Snajdr <snajpa@snajpa.net> Closes openzfs#1291
kvm & arc fighting it out for memory
The text was updated successfully, but these errors were encountered: