Project

General

Profile

Feature #9455

Expose drive speed and temperature on disk topo node

Added by Rob Johnston over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
2018-04-11
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:

Description

As part of the general effort to flesh out the hc-scheme topology, it would be nice if the drive speed and temperature were exposed on the disk topo nodes. Both speed and temp are available via libdiskmgt. Drive speed would be exposed as a uint32 topo prop directly on the disk node for non-SSDs. Drive temp would be exposed via a child sensor facility node.

Note this change has already been integrated into illumos-joyent via the commit linked below:

https://github.com/joyent/illumos-joyent/commit/0d2eb7a53abcabd58cac94fda80cb021bae84390

Details on how the test was changed can be found here:

https://smartos.org/bugview/OS-6607


Related issues

Related to illumos gate - Support #9849: Add support for pull temperature and speed off of SATA drivesNew2018-09-18

Actions

History

#1

Updated by Rob Johnston about 1 year ago

Just a clarification that the temperature sensor requires support for the DKIOCGTEMPERATURE ioctl in the underlying driver. As such, this will work with SCSI-based devices, but not ATA/SATA, USB storage or NMVE. On those devices, the "facility/reading" property will fail to read.

Similar story for the speed-in-rpm property. It relies on underlying DKIO support in the driver.

#2

Updated by Electric Monk about 1 year ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

git commit 00d7a6fb939233ed021b823016c33fcae1dac964

commit  00d7a6fb939233ed021b823016c33fcae1dac964
Author: Rob Johnston <rob.johnston@joyent.com>
Date:   2018-09-18T16:57:51.000Z

    9455 Expose drive speed and temperature on disk topo node
    Reviewed by: Yuri Pankov <yuripv@yuripv.net>
    Reviewed by: Igor Kozhukhov <igor@dilos.org>
    Approved by: Dan McDonald <danmcd@joyent.com>

#3

Updated by Rob Johnston about 1 year ago

  • Related to Support #9849: Add support for pull temperature and speed off of SATA drives added

Also available in: Atom PDF