[v3] Introducing TPM Proxy device support for PPC64

[PATCH v3 4/9] conf, domain: register AUTOPTR_CLEANUP_FUNC for virDomainTPMDef

Posted by Daniel Henrique Barboza 5 years, 9 months ago

Next patch will make use of g_autoptr() with virDomainTPMDefPtr.

Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
---
 src/conf/domain_conf.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index 8f178ade34..60dbba3b19 100644
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -3028,6 +3028,7 @@ virDomainDeviceInfoPtr virDomainDeviceGetInfo(virDomainDeviceDefPtr device);
 void virDomainDeviceSetData(virDomainDeviceDefPtr device,
                             void *devicedata);
 void virDomainTPMDefFree(virDomainTPMDefPtr def);
+G_DEFINE_AUTOPTR_CLEANUP_FUNC(virDomainTPMDef, virDomainTPMDefFree);
 
 typedef int (*virDomainDeviceInfoCallback)(virDomainDefPtr def,
                                            virDomainDeviceDefPtr dev,
-- 
2.26.2

Re: [PATCH v3 4/9] conf, domain: register AUTOPTR_CLEANUP_FUNC for virDomainTPMDef

Posted by Stefan Berger 5 years, 8 months ago

On 5/13/20 3:30 PM, Daniel Henrique Barboza wrote:
> Next patch will make use of g_autoptr() with virDomainTPMDefPtr.
>
> Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com>
> ---
>   src/conf/domain_conf.h | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
> index 8f178ade34..60dbba3b19 100644
> --- a/src/conf/domain_conf.h
> +++ b/src/conf/domain_conf.h
> @@ -3028,6 +3028,7 @@ virDomainDeviceInfoPtr virDomainDeviceGetInfo(virDomainDeviceDefPtr device);
>   void virDomainDeviceSetData(virDomainDeviceDefPtr device,
>                               void *devicedata);
>   void virDomainTPMDefFree(virDomainTPMDefPtr def);
> +G_DEFINE_AUTOPTR_CLEANUP_FUNC(virDomainTPMDef, virDomainTPMDefFree);
>   
>   typedef int (*virDomainDeviceInfoCallback)(virDomainDefPtr def,
>                                              virDomainDeviceDefPtr dev,

It seems there are several other ones of those now already in this file. 
So it should be good.


Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>