diff options
| author | Hans Rosenfeld <hans.rosenfeld@joyent.com> | 2018-10-25 18:01:09 +0200 |
|---|---|---|
| committer | Hans Rosenfeld <hans.rosenfeld@joyent.com> | 2018-11-02 12:02:57 +0100 |
| commit | 42de8792aded98091c221467e867ae64295e8f24 (patch) | |
| tree | 988c74a1738cb88b36e72bebf2ddcc1c5face84b | |
| parent | 76683f77aeee2619b447d8ea5843399835fbc586 (diff) | |
| download | illumos-joyent-OS-7320.tar.gz | |
OS-7313 Samsung PM1635a breaks when using power conditionsOS-7320
| -rw-r--r-- | usr/src/uts/common/io/scsi/targets/sd.c | 29 |
1 files changed, 14 insertions, 15 deletions
diff --git a/usr/src/uts/common/io/scsi/targets/sd.c b/usr/src/uts/common/io/scsi/targets/sd.c index 550d1afe23..95f0b3b82a 100644 --- a/usr/src/uts/common/io/scsi/targets/sd.c +++ b/usr/src/uts/common/io/scsi/targets/sd.c @@ -7109,7 +7109,20 @@ sdpower(dev_info_t *devi, int component, int level) sd_ssc_assessment(ssc, SD_FMT_IGNORE); } - if (un->un_f_power_condition_supported) { + /* + * We've encountered certain classes of drives that pass a TUR, but fail + * the START STOP UNIT when using power conditions, or worse leave the + * drive in an unusable state despite passing SSU. Strictly speaking, + * for SPC-4 or greater, no additional actions are required to make the + * drive operational when a TUR passes. If we have something that + * matches this condition, we continue on and presume the drive is + * successfully powered on. + */ + if (un->un_f_power_condition_supported && + SD_SCSI_VERS_IS_GE_SPC_4(un) && SD_PM_IS_IO_CAPABLE(un, level) && + level == SD_SPINDLE_ACTIVE && tursval == 0) { + sval = 0; + } else if (un->un_f_power_condition_supported) { char *pm_condition_name[] = {"STOPPED", "STANDBY", "IDLE", "ACTIVE"}; SD_TRACE(SD_LOG_IO_PM, un, @@ -7132,20 +7145,6 @@ sdpower(dev_info_t *devi, int component, int level) } - /* - * We've encountered certain classes of drives that pass a TUR, but fail - * the START STOP UNIT when using power conditions. Strictly speaking, - * for SPC-4 or greater, no additional actions are required to make the - * drive operational when a TUR passes. If we have something that - * matches this condition, we continue on and presume the drive is - * successfully powered on. - */ - if (un->un_f_power_condition_supported && sval == ENOTSUP && - SD_SCSI_VERS_IS_GE_SPC_4(un) && SD_PM_IS_IO_CAPABLE(un, level) && - level == SD_SPINDLE_ACTIVE && tursval == 0) { - sval = 0; - } - /* Command failed, check for media present. */ if ((sval == ENXIO) && un->un_f_has_removable_media) { medium_present = FALSE; |