Project

General

Profile

Actions

Bug #8156

closed

dbuf_evict_notify() does not need dbuf_evict_lock

Added by Serapheim Dimitropoulos over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Category:
zfs - Zettabyte File System
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.

Actions #1

Updated by Electric Monk over 4 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

Also available in: Atom PDF