Date: Mon, 31 Oct 2016 22:50:12 +0100 From: Harald Schmalzbauer <h.schmalzbauer@omnilan.de> To: Mark Johnston <markj@freebsd.org> Cc: svn-src-all@freebsd.org Subject: Re: svn commit: r306529 - head/sys/cam Message-ID: <5817BC94.40204@omnilan.de> In-Reply-To: <201609302100.u8UL0A3w034815@repo.freebsd.org> References: <201609302100.u8UL0A3w034815@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] Bezüglich Mark Johnston's Nachricht vom 30.09.2016 23:00 (localtime): > Author: markj > Date: Fri Sep 30 21:00:09 2016 > New Revision: 306529 > URL: https://svnweb.freebsd.org/changeset/base/306529 > > Log: > cam_periph_ccbwait could return while ccb in progress > > In cam_periph_runccb, cam_periph_ccbwait was using the value of the ccb > pinfo.index and status fields to determine whether the ccb was done, > but these fields are updated without a contending lock and could glitch > into states that would be erroneously interpreted as done. Instead, > have cam_periph_ccbwait look for the explicit result of the function > cam_periph_done. > > Submitted by: Ryan Libby <rlibby@gmail.com> > Reviewed by: mav > MFC after: 3 weeks > Sponsored by: Dell EMC Isilon Were there problems causing MFC delay or is it just slipped through? Thanks, -harry [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (FreeBSD) iEYEARECAAYFAlgXvJQACgkQLDqVQ9VXb8iDFACfWlIL88SMViyYCjrsqBnR/lU3 jtwAn1YDyHO5QMDxwyGVsClcHjDdzq5O =I/Id -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5817BC94.40204>