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

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

 



On 10/06/2017 01:23 PM, 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 series is the first half of the storage backend changes to use
> the virStoragePoolObjGetDef accessor.
> 
> The next/last series will be the next half, plus the change to move
> virStoragePoolObj and virStorageVolDefList into virstoragepoolobj.c.
> 
> After that adjustments to virstoragepoolobj similar to other drivers
> and perhaps even generating a virStorageVolObjPtr - e.g. a hash table
> of volume defs rather than a linked list of defs.
> 
> John Ferlan (6):
>   storage: Use virStoragePoolObjGetDef accessor for storage_util
>   storage: Use virStoragePoolObjGetDef accessor for Disk backend
>   storage: Use virStoragePoolObjGetDef accessor for Logical backend
>   storage: Use virStoragePoolObjGetDef accessor for Sheepdog backend
>   storage: Use virStoragePoolObjGetDef accessor for FS backend
>   storage: Use virStoragePoolObjGetDef accessor for Gluster backend
> 
>  src/storage/storage_backend_disk.c     |  98 ++++++++++++++++------------
>  src/storage/storage_backend_fs.c       |  90 +++++++++++++------------
>  src/storage/storage_backend_gluster.c  |  20 +++---
>  src/storage/storage_backend_logical.c  |  63 ++++++++++--------
>  src/storage/storage_backend_sheepdog.c |  23 ++++---
>  src/storage/storage_util.c             | 116 ++++++++++++++++++---------------
>  6 files changed, 230 insertions(+), 180 deletions(-)
> 

ACK

Michal

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