Re: [PATCH v3 0/6] Add read error ignore to storage backend vol info APIs

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

 



On Fri, Dec 04, 2015 at 05:23:47PM -0500, John Ferlan wrote:
> v2 here:
> http://www.redhat.com/archives/libvir-list/2015-November/thread.html
> 
> Based on review and well just the amount of change I figured I'd
> generate a v3.  The patches were mostly ACK'd, but patch 3 required
> removal of some flags which resulted in patch 1 needing updates... 
> Then I found that what I'd done in patch 3 regarding returning errors
> needed to generate a patch 2.5... So I just bit the bullet and am
> posting the patches again.  Patches 2, 4, and 5 of this series are
> unchanged. While patch 1 removes 'readflags' from an API, patch 3
> is new, and patch 6 uses the new name.
> 
> John Ferlan (6):
>   storage: Add readflags for backend error processing
>   storage: Add comments for backend APIs
>   storage: Set ret = -1 on failures in
>     virStorageBackendUpdateVolTargetInfo
>   storage: Handle readflags errors
>   storage: Add debug message
>   storage: Ignore block devices that fail format detection
> 
>  src/storage/storage_backend.c         | 83 ++++++++++++++++++++++++++++++-----
>  src/storage/storage_backend.h         | 15 ++++++-
>  src/storage/storage_backend_disk.c    |  5 ++-
>  src/storage/storage_backend_fs.c      |  4 +-
>  src/storage/storage_backend_logical.c |  2 +-
>  src/storage/storage_backend_mpath.c   |  2 +-
>  src/storage/storage_backend_scsi.c    |  6 ++-
>  7 files changed, 96 insertions(+), 21 deletions(-)
> 

ACK series

Jan

Attachment: signature.asc
Description: Digital signature

--
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]