Re: [PATCH 00/10] Convert secret driver to use hashed object

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

 




On 03/02/2016 01:54 PM, John Ferlan wrote:
> This series replaces the usage of linked list in secret driver with a
> hashed object list as was suggested in the review in the last patch of
> the previous secret driver changes series:
> 
> http://www.redhat.com/archives/libvir-list/2016-February/msg01274.html
> 
> Patch 1 - starts the ball rolling
> Patch 2 - is yet another code optimization (as found in bridge_driver)
> Patches 3-8 - Add the new API's "slowly" (for review purposes)
> Patch 9 - replaces the usage of linked lists with the hashed object
> Patch 10 - moves the secretLoadAllConfigs to secret_conf
> 
> The changes are loosely modeled after both virdomainobj and network_conf
> functionality.  The real meat and potato(e)s is found in patches 5 and 9.
> Other functions should be relatively straightforward.
> 
> I've done testing of various virsh secret-* commands (both valid and invalid
> options) as well as performing driver start, stop, and reload testing.
> Each patch was built using 'check' and 'syntax-check'.
> 
> The end result is much more code in secret_conf and much less in secret_driver.
> 
> John Ferlan (10):
>   secret: Move virSecretObj to secret_conf.h
>   Add secretObjFromSecret
>   secret: Add hashed virSecretObj and virSecretObjList
>   secret: Introduce virSecretObjListFindBy{UUID|Usage} support
>   secret: Introduce virSecretObjListAdd* and virSecretObjListRemove
>   secret: Introduce virSecretObjListNumOfSecrets
>   secret: Introduce virSecretObjListExport
>   secret: Introduce virSecretObjListGetUUIDs
>   secret: Use the hashed virSecretObjList
>   secret: Move and rename secretLoadAllConfigs
> 
>  src/conf/secret_conf.c     | 819 ++++++++++++++++++++++++++++++++++++++++++++-
>  src/conf/secret_conf.h     |  76 ++++-
>  src/libvirt_private.syms   |  11 +
>  src/secret/secret_driver.c | 647 +++--------------------------------
>  4 files changed, 957 insertions(+), 596 deletions(-)
> 

Ping - any chance these can be finished up?  Would it be better to
repost the entire series. It should apply cleanly as there hasn't been
changes in this space.

Tks -

John

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