Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 8 Dec 2012 07:37:12 +0000 (UTC)
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r244020 - stable/9/sys/cam/ata
Message-ID:  <201212080737.qB87bCn5093446@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mav
Date: Sat Dec  8 07:37:12 2012
New Revision: 244020
URL: http://svnweb.freebsd.org/changeset/base/244020

Log:
  MFC r243571:
  Fix problem with the Samsung 840 PRO series SSD detection.
  The device reports support for SATA Asynchronous Notification in its
  IDENTIFY data, but returns error on attempt to enable that feature.
  Make SATA XPT of CAM only report these errors, but not fail the device.

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

Modified: stable/9/sys/cam/ata/ata_xpt.c
==============================================================================
--- stable/9/sys/cam/ata/ata_xpt.c	Sat Dec  8 07:34:57 2012	(r244019)
+++ stable/9/sys/cam/ata/ata_xpt.c	Sat Dec  8 07:37:12 2012	(r244020)
@@ -745,6 +745,14 @@ probedone(struct cam_periph *periph, uni
 			goto noerror;
 
 		/*
+		 * Some Samsung SSDs report supported Asynchronous Notification,
+		 * but return ABORT on attempt to enable it.
+		 */
+		} else if (softc->action == PROBE_SETAN &&
+		    status == CAM_ATA_STATUS_ERROR) {
+			goto noerror;
+
+		/*
 		 * SES and SAF-TE SEPs have different IDENTIFY commands,
 		 * but SATA specification doesn't tell how to identify them.
 		 * Until better way found, just try another if first fail.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201212080737.qB87bCn5093446>