[PATCH v2 0/6] storage: handle scsi/iscsi error paths better

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



v1 here:
http://www.redhat.com/archives/libvir-list/2015-March/msg01562.html

changes:

1/6: Removed the virGetLastError() == NULL check
2/6: New - reading more closely showed an error path without a goto
3/6: New - Adjust return for virStorageBackendSCSINewLun to be able
           to differentiate real error vs. deciding to return because
           a stable path could not be determined
4/6: New - Found an unused variable in processLU
5/6: Adjust the logic to return -1 for "real" errors and -2 for errors
     that are "recoverable". Removed the need for goto cleanup.
6/6: Add check for non-existent stable path at startup (and fail). Add
     check during refresh for a found non standard stable path that returns
     no volumes in the pool
     
John Ferlan (6):
  iscsi: Use error message from virStorageBackendSCSIFindLUs
  iscsi: Fix exit path for virStorageBackendISCSIFindLUs failure
  scsi: Adjust return value for virStorageBackendSCSINewLun
  scsi: Remove unused 'type_path' in processLU
  scsi: Adjust return values from processLU
  iscsi: Add checks for non standard stable target.path

 src/storage/storage_backend_iscsi.c | 33 ++++++++++++++++-----
 src/storage/storage_backend_scsi.c  | 59 ++++++++++++++++++++++++-------------
 2 files changed, 64 insertions(+), 28 deletions(-)

-- 
2.1.0

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]