[libvirt] [PATCH 0/4] Complete storage pool object privatization and use of RWLockable

John Ferlan posted 4 patches 6 years, 3 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20171205014355.30524-1-jferlan@redhat.com
src/conf/virstorageobj.c     | 649 ++++++++++++++++++++++++++++++-------------
src/conf/virstorageobj.h     |  12 +-
src/libvirt_private.syms     |   2 +-
src/storage/storage_driver.c | 180 +++++-------
src/test/test_driver.c       |  51 ++--
5 files changed, 545 insertions(+), 349 deletions(-)
[libvirt] [PATCH 0/4] Complete storage pool object privatization and use of RWLockable
Posted by John Ferlan 6 years, 3 months ago
After this series the storage pool will be private to virstorageobj
and will be a self locking list of hash tables similar to other
drivers.  All that remains will be to make the storage volume lists
be hash tables, but that's only once this is accepted.

First patch is related to review comment from last series.

John Ferlan (4):
  storage: Fix path check in storagePoolLookupByTargetPath
  storage: Privatize virStoragePoolObjListPtr
  storage: Convert virStoragePoolObjList to use virObjectRWLockable
  storage: Reduce need for using storageDriverLock

 src/conf/virstorageobj.c     | 649 ++++++++++++++++++++++++++++++-------------
 src/conf/virstorageobj.h     |  12 +-
 src/libvirt_private.syms     |   2 +-
 src/storage/storage_driver.c | 180 +++++-------
 src/test/test_driver.c       |  51 ++--
 5 files changed, 545 insertions(+), 349 deletions(-)

-- 
2.13.6

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/4] Complete storage pool object privatization and use of RWLockable
Posted by John Ferlan 6 years, 3 months ago
ping?

Tks -

John

On 12/04/2017 08:43 PM, John Ferlan wrote:
> After this series the storage pool will be private to virstorageobj
> and will be a self locking list of hash tables similar to other
> drivers.  All that remains will be to make the storage volume lists
> be hash tables, but that's only once this is accepted.
> 
> First patch is related to review comment from last series.
> 
> John Ferlan (4):
>   storage: Fix path check in storagePoolLookupByTargetPath
>   storage: Privatize virStoragePoolObjListPtr
>   storage: Convert virStoragePoolObjList to use virObjectRWLockable
>   storage: Reduce need for using storageDriverLock
> 
>  src/conf/virstorageobj.c     | 649 ++++++++++++++++++++++++++++++-------------
>  src/conf/virstorageobj.h     |  12 +-
>  src/libvirt_private.syms     |   2 +-
>  src/storage/storage_driver.c | 180 +++++-------
>  src/test/test_driver.c       |  51 ++--
>  5 files changed, 545 insertions(+), 349 deletions(-)
> 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/4] Complete storage pool object privatization and use of RWLockable
Posted by Michal Privoznik 6 years, 3 months ago
On 12/05/2017 02:43 AM, John Ferlan wrote:
> After this series the storage pool will be private to virstorageobj
> and will be a self locking list of hash tables similar to other
> drivers.  All that remains will be to make the storage volume lists
> be hash tables, but that's only once this is accepted.
> 
> First patch is related to review comment from last series.
> 
> John Ferlan (4):
>   storage: Fix path check in storagePoolLookupByTargetPath
>   storage: Privatize virStoragePoolObjListPtr
>   storage: Convert virStoragePoolObjList to use virObjectRWLockable
>   storage: Reduce need for using storageDriverLock
> 
>  src/conf/virstorageobj.c     | 649 ++++++++++++++++++++++++++++++-------------
>  src/conf/virstorageobj.h     |  12 +-
>  src/libvirt_private.syms     |   2 +-
>  src/storage/storage_driver.c | 180 +++++-------
>  src/test/test_driver.c       |  51 ++--
>  5 files changed, 545 insertions(+), 349 deletions(-)
> 

ACK series.

Michal

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list