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

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

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

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

 Module Name:	src
 Committed By:	martin
 Date:		Sun Mar 18 11:05:27 UTC 2018
 
 Modified Files:
 	src/sys/dev/ic [netbsd-8]: ld_nvme.c nvme.c nvmevar.h
 
 Log Message:
 Pull up following revision(s) (requested by jdolecek in ticket #641):
 	sys/dev/ic/nvme.c: revision 1.34
 	sys/dev/ic/nvme.c: revision 1.35
 	sys/dev/ic/nvme.c: revision 1.36
 	sys/dev/ic/nvme.c: revision 1.37
 	sys/dev/ic/ld_nvme.c: revision 1.19
 	sys/dev/ic/nvmevar.h: revision 1.15
 
 refactor the locking code around DIOCGCACHE handling to be reusable
 for other infrequent commands,it uses single condvar for simplicity,
 and uses it both when waiting for ccb or command completion - this
 is fine, since usually there will be just one such command qeueued anyway
 use this to finally properly implement DIOCCACHESYNC - return only after
 the command is confirmed as completed by the controller.
 
 switch handling of passthrough commands to use queue, instead of polling
 should fix PR kern/53059 by Frank Kardel
 
 fix passthrough command usage also in nvme_get_number_of_queues(), fixes
 memory corruption and possible panic on boot
 
 also remove now duplicate nvme_ccb_put() call from
 nvme_get_number_of_queues()
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.16.2.1 -r1.16.2.2 src/sys/dev/ic/ld_nvme.c
 cvs rdiff -u -r1.30.2.1 -r1.30.2.2 src/sys/dev/ic/nvme.c
 cvs rdiff -u -r1.13.6.1 -r1.13.6.2 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.