Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#1045 closed patch (fixed)

add support for SAS host managed drives

Reported by: chris Owned by: Christian Franke
Priority: minor Milestone: Release 7.0
Component: smartctl Version: 6.6
Keywords: scsi Cc:

Description

The industry is working on SAS host managed drives supporting the ZBC specification. It would be helpful if smartctl could read some basic SAS parameters on these drives, such as power on hours and grown defect list entries.

The current issue is that these SAS HM drives are a different peripheral type (0x14) from a standard direct access drive (0x0), as per SPC-5, so in scsiprint.cpp they are not being recognized in the is_disk variable in the scsiPrintMain function.

I have made local changes to support Seagate SAS HM drives. All that needs changing is an additional SCSI Peripheral Type in scsicmds.h (host managed device, 14h, see link below), and a check for that in scsiprint.cpp (in scsiPrintMain, in the is_disk check).

https://en.wikipedia.org/wiki/SCSI_Peripheral_Device_Type

Attachments (1)

SAS_HM_support.patch (1.2 KB ) - added by chris 6 years ago.

Download all attachments as: .zip

Change History (6)

in reply to:  description comment:1 by Christian Franke, 6 years ago

Keywords: scsi added; SAS HM ZBC removed
Milestone: unscheduled

I have made local changes to support Seagate SAS HM drives. All that needs changing is an additional SCSI Peripheral Type in scsicmds.h (host managed device, 14h, see link below), and a check for that in scsiprint.cpp (in scsiPrintMain, in the is_disk check).

Please provide these changes as a patch file.

by chris, 6 years ago

Attachment: SAS_HM_support.patch added

comment:2 by chris, 6 years ago

I have submitted a patch file. Thanks.

comment:3 by Christian Franke, 6 years ago

Milestone: unscheduledRelease 6.7
Owner: set to Christian Franke
Status: newaccepted
Type: enhancementpatch

comment:4 by Christian Franke, 6 years ago

Resolution: fixed
Status: acceptedclosed

Patch applied in r4744, thanks.

comment:5 by Christian Franke, 5 years ago

Milestone: Release 6.7Release 7.0

Milestone renamed

Note: See TracTickets for help on using tickets.