Actions
Bug #8156
closeddbuf_evict_notify() does not need dbuf_evict_lock
Start date:
2017-05-04
Due date:
% Done:
100%
Estimated time:
Difficulty:
Medium
Tags:
needs-triage
Gerrit CR:
Description
dbuf_evict_notify() holds the dbuf_evict_lock while checking if it should do the eviction itself
(because the evict thread is not able to keep up). This can result in massive lock contention.
It isn't necessary to hold the lock, because if we make the wrong choice occasionally, nothing
bad will happen.
Updated by Electric Monk almost 5 years ago
- Status changed from New to Closed
- % Done changed from 0 to 100
git commit dbfd9f930004c390a2ce2cf850c71b4f880eef9c
commit dbfd9f930004c390a2ce2cf850c71b4f880eef9c Author: Matthew Ahrens <mahrens@delphix.com> Date: 2017-06-08T01:56:23.000Z 8156 dbuf_evict_notify() does not need dbuf_evict_lock Reviewed by: Dan Kimmel <dan.kimmel@delphix.com> Reviewed by: Paul Dagnelie <pcd@delphix.com> Approved by: Robert Mustacchi <rm@joyent.com>
Actions