git: DragonFly_RELEASE_4_6 nvme - Fix BUF_KERNPROC() SMP race

Matthew Dillon dillon at crater.dragonflybsd.org
Fri Jul 22 11:43:50 PDT 2016


commit 6c1feaf50952b51987811bfc4ec6466cfdaab389
Author: Matthew Dillon <dillon at apollo.backplane.com>
Date:   Wed Jul 20 19:29:06 2016 -0700

    nvme - Fix BUF_KERNPROC() SMP race
    
    * BUF_KERNPROC() must be issued before we submit the request.  The subq
      lock is not sufficient to interlock request completion (which only needs
      the comq lock).
    
    * Only occurs under extreme loads, probably due to an IPI or Xinvltlb
      causing enough of a pause that the completion can run.  NVMe is so fast,
      probably no other controller would hit this particular race condition.
    
    * Also fix a bio queueing race which can leave a bio hanging.  If no
      requests are available (which can only happen under very heavy I/O
      loads), the signaling to the admin thread on the next I/O completion
      can race the queueing of the bio.  Fix the race by making sure the
      admin thread is signalled *after* queueing the bio.

Summary of changes:
 sys/dev/disk/nvme/nvme.c      |  7 +++----
 sys/dev/disk/nvme/nvme_disk.c | 18 ++++++++++++++++--
 2 files changed, 19 insertions(+), 6 deletions(-)

http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/6c1feaf50952b51987811bfc4ec6466cfdaab389


-- 
DragonFly BSD source repository



More information about the Commits mailing list