svn commit: r350545 - stable/11/sys/cam/ata

Brooks Davis brooks at FreeBSD.org
Fri Aug 2 22:33:44 UTC 2019


Author: brooks
Date: Fri Aug  2 22:33:43 2019
New Revision: 350545
URL: https://svnweb.freebsd.org/changeset/base/350545

Log:
  MFC r350228:
  
  ata_xpt: Use the correct union member when accessing valid.
  
  In principle this should not matter as it's a union and they point to
  the same memory location but based on the code above we should be
  accessing .sata and not .ata.
  
  Submitted by:	arichardson
  Reviewed by:	scottl, imp
  Obtained from:	CheriBSD
  Sponsored by:	DARPA, AFRL
  Differential Revision:	https://reviews.freebsd.org/D21002

Modified:
  stable/11/sys/cam/ata/ata_xpt.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/cam/ata/ata_xpt.c
==============================================================================
--- stable/11/sys/cam/ata/ata_xpt.c	Fri Aug  2 22:09:56 2019	(r350544)
+++ stable/11/sys/cam/ata/ata_xpt.c	Fri Aug  2 22:33:43 2019	(r350545)
@@ -440,7 +440,7 @@ negotiate:
 			if (cts.xport_specific.ata.valid & CTS_ATA_VALID_MODE)
 				mode = cts.xport_specific.ata.mode;
 		} else {
-			if (cts.xport_specific.ata.valid & CTS_SATA_VALID_MODE)
+			if (cts.xport_specific.sata.valid & CTS_SATA_VALID_MODE)
 				mode = cts.xport_specific.sata.mode;
 		}
 		/* If SIM disagree - renegotiate. */


More information about the svn-src-all mailing list