svn commit: r300640 - head/sys/cam/ata
Kenneth D. Merry
ken at FreeBSD.org
Wed May 25 01:37:40 UTC 2016
Author: ken
Date: Wed May 25 01:37:39 2016
New Revision: 300640
URL: https://svnweb.freebsd.org/changeset/base/300640
Log:
Fix a few ada(4) driver issues:
o Some Samsung drives do not support the ATA READ LOG EXT or READ
LOG DMA EXT commands, despite indicating that they do in their
IDENTIFY data. So, fix this in two ways:
1. Only start the log directory probe (ADA_STATE_LOGDIR) if
the drive claims to be an SMR drive in the first place.
We don't need to do the extra probing for other devices.
This will also serve to prevent problems with other
drives that have the same issue.
2. Add quirks for the two Samsung drives that have been
reported so far (thanks to Oleg Nauman and Alex Petrov).
If there is a reason to do a Read Log later on, we will
know that it doesn't work on these drives.
o Add a quirk entry to mark Seagate Lamarr Drive Managed drives as
drive managed. They don't report this in their Identify data.
sys/cam/ata/ata_da.c:
Add two new quirks:
1. ADA_Q_LOG_BROKEN, for drives that claim to support Read
Log but don't really.
2. ADA_Q_SMR_DM, for drives that are Drive Managed SMR, but
don't report it. This can matter for software that
wants to know when it should make an extra effort to
write sequentially.
Record two Samsung drives that don't support Read Log, and
one Seagate drive that doesn't report that it is a SMR drive.
The Seagate drive is already recorded in the da(4) driver.
We may have to come up with a similar solution in the da(4)
driver for SATA drives that don't properly support Read Log.
In adasetflags(), Dont' set the ADA_FLAG_CAN_LOG bit if the
device has the LOG_BROKEN quirk set. Also, look at the
SMR_DM quirk and set the device type accordingly if it is
actually a drive managed drive.
When deciding whether to go into the LOGDIR probe state,
look to see whether the device claims to be an SMR device.
If not, don't bother with the LOGDIR probe state.
Sponsored by: Spectra Logic
Modified:
head/sys/cam/ata/ata_da.c
Modified: head/sys/cam/ata/ata_da.c
==============================================================================
--- head/sys/cam/ata/ata_da.c Wed May 25 01:37:25 2016 (r300639)
+++ head/sys/cam/ata/ata_da.c Wed May 25 01:37:39 2016 (r300640)
@@ -115,12 +115,16 @@ typedef enum {
ADA_Q_NONE = 0x00,
ADA_Q_4K = 0x01,
ADA_Q_NCQ_TRIM_BROKEN = 0x02,
+ ADA_Q_LOG_BROKEN = 0x04,
+ ADA_Q_SMR_DM = 0x08
} ada_quirks;
#define ADA_Q_BIT_STRING \
"\020" \
"\0014K" \
- "\002NCQ_TRIM_BROKEN"
+ "\002NCQ_TRIM_BROKEN" \
+ "\003LOG_BROKEN" \
+ "\004SMR_DM"
typedef enum {
ADA_CCB_RAHEAD = 0x01,
@@ -680,6 +684,35 @@ static struct ada_quirk_entry ada_quirk_
/*quirks*/ADA_Q_4K
},
{
+ /*
+ * Samsung drive that doesn't support READ LOG EXT or
+ * READ LOG DMA EXT, despite reporting that it does in
+ * ATA identify data:
+ * SAMSUNG HD200HJ KF100-06
+ */
+ { T_DIRECT, SIP_MEDIA_FIXED, "*", "SAMSUNG HD200*", "*" },
+ /*quirks*/ADA_Q_LOG_BROKEN
+ },
+ {
+ /*
+ * Samsung drive that doesn't support READ LOG EXT or
+ * READ LOG DMA EXT, despite reporting that it does in
+ * ATA identify data:
+ * SAMSUNG HD501LJ CR100-10
+ */
+ { T_DIRECT, SIP_MEDIA_FIXED, "*", "SAMSUNG HD501*", "*" },
+ /*quirks*/ADA_Q_LOG_BROKEN
+ },
+ {
+ /*
+ * Seagate Lamarr 8TB Shingled Magnetic Recording (SMR)
+ * Drive Managed SATA hard drive. This drive doesn't report
+ * in firmware that it is a drive managed SMR drive.
+ */
+ { T_DIRECT, SIP_MEDIA_FIXED, "*", "ST8000AS0002*", "*" },
+ /*quirks*/ADA_Q_SMR_DM
+ },
+ {
/* Default */
{
T_ANY, SIP_MEDIA_REMOVABLE|SIP_MEDIA_FIXED,
@@ -1258,7 +1291,8 @@ adaasync(void *callback_arg, u_int32_t c
softc->state = ADA_STATE_RAHEAD;
else if (ADA_WC >= 0 && softc->flags & ADA_FLAG_CAN_RAHEAD)
softc->state = ADA_STATE_WCACHE;
- else if (softc->flags & ADA_FLAG_CAN_LOG)
+ else if ((softc->flags & ADA_FLAG_CAN_LOG)
+ && (softc->zone_mode != ADA_ZONE_NONE))
softc->state = ADA_STATE_LOGDIR;
else
break;
@@ -1581,7 +1615,8 @@ adasetflags(struct ada_softc *softc, str
*/
adasetdeletemethod(softc);
- if (cgd->ident_data.support.extension & ATA_SUPPORT_GENLOG)
+ if ((cgd->ident_data.support.extension & ATA_SUPPORT_GENLOG)
+ && ((softc->quirks & ADA_Q_LOG_BROKEN) == 0))
softc->flags |= ADA_FLAG_CAN_LOG;
else
softc->flags &= ~ADA_FLAG_CAN_LOG;
@@ -1589,8 +1624,9 @@ adasetflags(struct ada_softc *softc, str
if ((cgd->ident_data.support3 & ATA_SUPPORT_ZONE_MASK) ==
ATA_SUPPORT_ZONE_HOST_AWARE)
softc->zone_mode = ADA_ZONE_HOST_AWARE;
- else if ((cgd->ident_data.support3 & ATA_SUPPORT_ZONE_MASK) ==
- ATA_SUPPORT_ZONE_DEV_MANAGED)
+ else if (((cgd->ident_data.support3 & ATA_SUPPORT_ZONE_MASK) ==
+ ATA_SUPPORT_ZONE_DEV_MANAGED)
+ || (softc->quirks & ADA_Q_SMR_DM))
softc->zone_mode = ADA_ZONE_DRIVE_MANAGED;
else
softc->zone_mode = ADA_ZONE_NONE;
@@ -1818,7 +1854,8 @@ adaregister(struct cam_periph *periph, v
softc->state = ADA_STATE_RAHEAD;
} else if (ADA_WC >= 0 && softc->flags & ADA_FLAG_CAN_WCACHE) {
softc->state = ADA_STATE_WCACHE;
- } else if (softc->flags & ADA_FLAG_CAN_LOG) {
+ } else if ((softc->flags & ADA_FLAG_CAN_LOG)
+ && (softc->zone_mode != ADA_ZONE_NONE)) {
softc->state = ADA_STATE_LOGDIR;
} else {
/*
@@ -2863,7 +2900,8 @@ adadone(struct cam_periph *periph, union
/* Drop freeze taken due to CAM_DEV_QFREEZE */
cam_release_devq(path, 0, 0, 0, FALSE);
- if (softc->flags & ADA_FLAG_CAN_LOG) {
+ if ((softc->flags & ADA_FLAG_CAN_LOG)
+ && (softc->zone_mode != ADA_ZONE_NONE)) {
xpt_release_ccb(done_ccb);
softc->state = ADA_STATE_LOGDIR;
xpt_schedule(periph, priority);
More information about the svn-src-all
mailing list