scsi: smartpqi: cleanup list initialization
Better initialization of linked list heads. Reviewed-by: Scott Benesh <scott.benesh@microsemi.com> Signed-off-by: Kevin Barnett <kevin.barnett@microsemi.com> Signed-off-by: Don Brace <don.brace@microsemi.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
parent
a9f9339241
commit
8a994a04fc
@ -1591,11 +1591,8 @@ static void pqi_update_device_list(struct pqi_ctrl_info *ctrl_info,
|
|||||||
struct pqi_scsi_dev *device;
|
struct pqi_scsi_dev *device;
|
||||||
struct pqi_scsi_dev *next;
|
struct pqi_scsi_dev *next;
|
||||||
struct pqi_scsi_dev *matching_device;
|
struct pqi_scsi_dev *matching_device;
|
||||||
struct list_head add_list;
|
LIST_HEAD(add_list);
|
||||||
struct list_head delete_list;
|
LIST_HEAD(delete_list);
|
||||||
|
|
||||||
INIT_LIST_HEAD(&add_list);
|
|
||||||
INIT_LIST_HEAD(&delete_list);
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* The idea here is to do as little work as possible while holding the
|
* The idea here is to do as little work as possible while holding the
|
||||||
@ -1755,7 +1752,7 @@ static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info)
|
|||||||
{
|
{
|
||||||
int i;
|
int i;
|
||||||
int rc;
|
int rc;
|
||||||
struct list_head new_device_list_head;
|
LIST_HEAD(new_device_list_head);
|
||||||
struct report_phys_lun_extended *physdev_list = NULL;
|
struct report_phys_lun_extended *physdev_list = NULL;
|
||||||
struct report_log_lun_extended *logdev_list = NULL;
|
struct report_log_lun_extended *logdev_list = NULL;
|
||||||
struct report_phys_lun_extended_entry *phys_lun_ext_entry;
|
struct report_phys_lun_extended_entry *phys_lun_ext_entry;
|
||||||
@ -1773,8 +1770,6 @@ static int pqi_update_scsi_devices(struct pqi_ctrl_info *ctrl_info)
|
|||||||
static char *out_of_memory_msg =
|
static char *out_of_memory_msg =
|
||||||
"failed to allocate memory, device discovery stopped";
|
"failed to allocate memory, device discovery stopped";
|
||||||
|
|
||||||
INIT_LIST_HEAD(&new_device_list_head);
|
|
||||||
|
|
||||||
rc = pqi_get_device_lists(ctrl_info, &physdev_list, &logdev_list);
|
rc = pqi_get_device_lists(ctrl_info, &physdev_list, &logdev_list);
|
||||||
if (rc)
|
if (rc)
|
||||||
goto out;
|
goto out;
|
||||||
|
Loading…
Reference in New Issue
Block a user