Re: [PATCH REPOST 0/8] Privatize _virStoragePoolObj and _virStorageVolDefList (Batch #3)

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

 



On Fri, Oct 06, 2017 at 10:42:54AM -0400, John Ferlan wrote:
> Since the original series (19 patches):
>
> https://www.redhat.com/archives/libvir-list/2017-September/msg00594.html
>
> didn't garner any attention, I'm going with smaller patch piles to make
> forward progress.
>
> This is the last half of the storage backends plus one missed merge
> (because I broke things up <sigh>)
>
> John Ferlan (8):
>   storage: Use virStoragePoolObjGetDef accessor for iSCSI backend
>   storage: Use virStoragePoolObjGetDef accessor for MPATH backend
>   storage: Use virStoragePoolObjGetDef accessor for RBD backend
>   storage: Use virStoragePoolObjGetDef accessor for SCSI backend
>   storage: Use virStoragePoolObjGetDef accessor for VSTORAGE backend
>   storage: Use virStoragePoolObjGetDef accessor for ZFS backend
>   storage: Use virStoragePoolObjGetDef accessor for new driver events
>   storage: Privatize virStoragePoolObj and virStorageVolDefList
>
>  src/conf/storage_conf.h                |  4 ---
>  src/conf/virstorageobj.c               | 20 +++++++++++
>  src/conf/virstorageobj.h               | 15 --------
>  src/storage/storage_backend_iscsi.c    | 41 ++++++++++++----------
>  src/storage/storage_backend_mpath.c    |  8 +++--
>  src/storage/storage_backend_rbd.c      | 64 ++++++++++++++++++----------------
>  src/storage/storage_backend_scsi.c     | 30 +++++++++-------
>  src/storage/storage_backend_vstorage.c | 31 ++++++++--------
>  src/storage/storage_backend_zfs.c      | 39 ++++++++++++---------
>  src/storage/storage_driver.c           |  8 ++---
>  10 files changed, 144 insertions(+), 116 deletions(-)

Reviewed-by: Erik Skultety <eskultet@xxxxxxxxxx>

--
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]
  Powered by Linux