Opened 9 months ago

Closed 8 months ago

Last modified 3 months 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 9 months ago.

Download all attachments as: .zip

Change History (6)

comment:1 in reply to:  description Changed 9 months ago by Christian Franke

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.

Changed 9 months ago by chris

Attachment: SAS_HM_support.patch added

comment:2 Changed 9 months ago by chris

I have submitted a patch file. Thanks.

comment:3 Changed 9 months ago by Christian Franke

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

comment:4 Changed 8 months ago by Christian Franke

Resolution: fixed
Status: acceptedclosed

Patch applied in r4744, thanks.

comment:5 Changed 3 months ago by Christian Franke

Milestone: Release 6.7Release 7.0

Milestone renamed

Note: See TracTickets for help on using tickets.