]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Check element type before setting LEDs.
authormav <mav@FreeBSD.org>
Mon, 4 Feb 2019 01:24:10 +0000 (01:24 +0000)
committermav <mav@FreeBSD.org>
Mon, 4 Feb 2019 01:24:10 +0000 (01:24 +0000)
commit05fa3fd2c4977dba78932eb4e7556b3e0a57c579
tree49b2f6f8f091fc65e7b6f4dd527a5af8529a0160
parent6abbf0c797109751ae23ac78e9f83eaba4847c99
Check element type before setting LEDs.

With r319610, sesutil started twiddling the bits of every SES device.
Not everything is a disk slot, there are also fan controllers, temperature
sensors, even power supplies, among other things controlled by SES.

Add a type check to make sure we are only operating on device slot and array
device slot elements.  Other type elements will be skipped, but it would be
simple to add additional cases for controlling the ident LEDs of other
element types (which are not necessarily the same bits).

Rather than doing raw bit manipulation of an unstructured byte array using
unnamed numeric constants, leverage existing code abstractions.

Submitted by: Ryan Moeller <ryan@freqlabs.com>
MFC after: 1 week
Sponsored by: iXsystems, Inc.
Differential Revision: https://reviews.freebsd.org/D19052
usr.sbin/sesutil/sesutil.c