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
This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig1F04BE72E9400407B5E44918 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Bez=FCglich 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 > =20 > In cam_periph_runccb, cam_periph_ccbwait was using the value of the c= cb > pinfo.index and status fields to determine whether the ccb was done, > but these fields are updated without a contending lock and could glit= ch > 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. > =20 > Submitted by: Ryan Libby <rlibby@gmail.com> > Reviewed by: mav > MFC after: 3 weeks > Sponsored by: Dell EMC Isilon=20 Were there problems causing MFC delay or is it just slipped through? Thanks, -harry --------------enig1F04BE72E9400407B5E44918 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (FreeBSD) iEYEARECAAYFAlgXvJQACgkQLDqVQ9VXb8iDFACfWlIL88SMViyYCjrsqBnR/lU3 jtwAn1YDyHO5QMDxwyGVsClcHjDdzq5O =I/Id -----END PGP SIGNATURE----- --------------enig1F04BE72E9400407B5E44918--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5817BC94.40204>