PR/52769 CVS commit: [netbsd-8] src/sys/dev/ic

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR kern/52769; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/52769 CVS commit: [netbsd-8] src/sys/dev/ic
Date: Sat, 17 Mar 2018 08:11:18 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Sat Mar 17 08:11:18 UTC 2018
 
 Modified Files:
 	src/sys/dev/ic [netbsd-8]: nvme.c nvmevar.h
 
 Log Message:
 Pull up following revision(s) (requested by jdolecek in ticket #636):
 	sys/dev/ic/nvme.c: revision 1.33
 	sys/dev/ic/nvmevar.h: revision 1.14
 stop using q_nccbs_avail for deciding whether there are available ccbs;
 no need to maintain a counter _and_ q_ccb_list
 this fixes deadlock when all ccbs happen to be taken before completion
 interrupt - nvme_q_complete() increased q_nccbs_avail only after
 processing all the completed commands, by then there was nothing
 left to actually kick the disk queue again into action
 this also fixes ccb leak on command errors e.g. with bus_dmamem_alloc()
 or bus_dmamel_load() - q_nccbs_avail was never decreased on the error path
 fixes PR kern/52769 by Martin Husemann, thanks to Paul Goyette
 for testing
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.30 -r1.30.2.1 src/sys/dev/ic/nvme.c
 cvs rdiff -u -r1.13 -r1.13.6.1 src/sys/dev/ic/nvmevar.h
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.