[libvirt] [Xen-devel] [PATCH] libxl: include a XLU_Config in _libxlDriverConfig

Jim Fehlig jfehlig at suse.com
Wed May 6 15:12:05 UTC 2015


Wei Liu wrote:
> On Wed, May 06, 2015 at 10:58:11AM +0200, Olaf Hering wrote:
>   
>> On Wed, May 06, Olaf Hering wrote:
>>
>>     
>>> Since libvirtd runs forever there is very little code to undo most
>>> things. But I will see if there is any unload function, did not actively
>>> look for such thing.
>>>       
>> Looking through libxlStateDriver it seems that libxl and libxlu remains
>> as is. Not sure if thats supposed that way or if perhaps libxl should be
>> fully reinitialized during ->stateReload, and if logfiles should be
>> closed in ->stateCleanup.
>>
>>     
>
> FWIW libxl ctx is destroy in libxlDriverConfigDispose. You can probably
> just call xlu_cfg_destroy there.
>   

Right. Any additions to the libxlDriverConfig object that needs to be
cleaned up should be done in libxlDriverConfigDispose.

Regards,
Jim




More information about the libvir-list mailing list