Commit dd49ded8 authored by Vishakha Channapattan's avatar Vishakha Channapattan Committed by Martin K. Petersen
Browse files

scsi: pm80xx: Add sysfs attribute to track RAAE count 

A new sysfs variable 'ctl_raae_count' is being introduced that tells if the
controller is alive by indicating controller ticks. If on subsequent run we
see the ticks changing in RAAE count that indicates that controller is not
dead.

Using the 'ctl_raae_count' sysfs variable we can see ticks incrementing:

    linux-9saw:~# cat /sys/class/scsi_host/host*/ctl_raae_count
    0x00002245
    0x00002253
    0x0000225e

Link: https://lore.kernel.org/r/20210415103352.3580-4-Viswas.G@microchip.com


Acked-by: default avatarJack Wang <jinpu.wang@ionos.com>
Signed-off-by: default avatarVishakha Channapattan <vishakhavc@google.com>
Signed-off-by: default avatarViswas G <Viswas.G@microchip.com>
Signed-off-by: default avatarRuksar Devadi <Ruksar.devadi@microchip.com>
Signed-off-by: default avatarAshokkumar N <Ashokkumar.N@microchip.com>
Signed-off-by: default avatarRadha Ramachandran <radha@google.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent a4c55e16

drivers/scsi/pm8001/pm8001_ctl.c

+22 −0
Original line number Diff line number Diff line
@@ -932,6 +932,27 @@ static ssize_t ctl_hmi_error_show(struct device *cdev, 
}

static DEVICE_ATTR_RO(ctl_hmi_error);



/**

 * ctl_raae_count_show - controller raae count check

 * @cdev: pointer to embedded class device

 * @buf: the buffer returned

 *

 * A sysfs 'read-only' shost attribute.

 */



static ssize_t ctl_raae_count_show(struct device *cdev,

		struct device_attribute *attr, char *buf)

{

	struct Scsi_Host *shost = class_to_shost(cdev);

	struct sas_ha_struct *sha = SHOST_TO_SAS_HA(shost);

	struct pm8001_hba_info *pm8001_ha = sha->lldd_ha;

	unsigned int raaecnt;



	raaecnt = pm8001_mr32(pm8001_ha->general_stat_tbl_addr, 12);

	return sysfs_emit(buf, "0x%08x\n", raaecnt);

}

static DEVICE_ATTR_RO(ctl_raae_count);



struct device_attribute *pm8001_host_attrs[] = {

	&dev_attr_interface_rev,

	&dev_attr_controller_fatal_error,
@@ -957,6 +978,7 @@ struct device_attribute *pm8001_host_attrs[] = { 
	&dev_attr_inc_fw_ver,

	&dev_attr_ctl_mpi_state,

	&dev_attr_ctl_hmi_error,

	&dev_attr_ctl_raae_count,

	NULL,

};