]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/libvirt.git/commit
scsi: Adjust return values from processLU
authorJohn Ferlan <jferlan@redhat.com>
Mon, 30 Mar 2015 22:55:16 +0000 (18:55 -0400)
committerJohn Ferlan <jferlan@redhat.com>
Tue, 21 Apr 2015 14:20:17 +0000 (10:20 -0400)
commit9832205263d5e3adefc49fe05422db6732409d81
treeac919ec54164af840c15c188aaf8d4c81b869603
parent1e13eff4354af1d6f9c774d90995151aa94e78d0
scsi: Adjust return values from processLU

https://bugzilla.redhat.com/show_bug.cgi?id=1171933

Adjust the processLU error returns to be a bit more logical. Currently,
the calling code cannot determine the difference between a non disk/lun
volume and a processed/found disk/lun. It can also not differentiate
between perhaps real/fatal error and one that won't necessarily stop
the code from finding other volumes.

After this patch virStorageBackendSCSIFindLUsInternal will stop processing
as soon as a "fatal" message occurs rather than continuting processing
for no apparent reason. It will also only set the *found value when
at least one of the processLU's was successful.

With the failed return, if the reason for the stop was that the pool
target path did not exist, was /dev, was /dev/, or did not start with
/dev, then iSCSI pool startup and refresh will fail.
src/storage/storage_backend_scsi.c