On 8.8.2017 10:35, Hannes Reinecke wrote:
> Older Controller might not support volume status, so assume
> the volume is online here.
>
> Signed-off-by: Hannes Reinecke <hare@xxxxxxxx>
> ---
> drivers/scsi/hpsa.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
> index 7ca6078..4ebf5d4 100644
> --- a/drivers/scsi/hpsa.c
> +++ b/drivers/scsi/hpsa.c
> @@ -3832,6 +3832,17 @@ static int hpsa_update_device_info(struct ctlr_info *h,
> if (h->fw_support & MISC_FW_RAID_OFFLOAD_BASIC)
> hpsa_get_ioaccel_status(h, scsi3addr, this_device);
> volume_offline = hpsa_volume_offline(h, scsi3addr);
> + if (volume_offline == HPSA_VPD_LV_STATUS_UNSUPPORTED &&
> + !h->supported) {
> + /*
> + * Older / unsupported controllers might not support
> + * volume status
> + */
> + dev_info(&h->pdev->dev,
> + "C0:T%d:L%d Volume status not available, assuming online.\n",
> + this_device->target, this_device->lun);
> + volume_offline = 0;
Hi,
could we have here
volume_offline = HPSA_LV_OK;
instead ?
tomash
> + }
> this_device->volume_offline = volume_offline;
> if (volume_offline == HPSA_LV_FAILED) {
> rc = HPSA_LV_FAILED;
