[libvirt] [PATCH v4 4/4] scsi: Adjust return values from processLU

[Ján Tomko]

On Sun, Apr 19, 2015 at 08:38:36PM -0400, John Ferlan wrote:
> 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.
> 
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
>  src/storage/storage_backend_scsi.c | 38 +++++++++++++++++++++++++-------------
>  1 file changed, 25 insertions(+), 13 deletions(-)
> 

ACK

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150421/a7d6cfff/attachment-0001.sig>