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

2015-05-07 Thread Wei Liu
On Thu, May 07, 2015 at 10:20:23AM +0200, Olaf Hering wrote: > On Wed, May 06, 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

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

2015-05-07 Thread Olaf Hering
On Wed, May 06, 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 > > > lo

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

2015-05-06 Thread Jim Fehlig
Olaf Hering wrote: > Upcoming changes for vscsi will use libxlutil.so to prepare the > configuration for libxl. The helpers needs a xlu struct for logging. > Provide one and reuse the existing output as log target. > > Signed-off-by: Olaf Hering > Cc: Jim Fehlig > --- > src/libxl/libxl_conf.c |

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

2015-05-06 Thread Jim Fehlig
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 t

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

2015-05-06 Thread Jim Fehlig
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 a

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

2015-05-06 Thread Wei Liu
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 libx

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

2015-05-06 Thread Olaf Hering
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 that

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

2015-05-06 Thread Olaf Hering
On Tue, May 05, Wei Liu wrote: > On Tue, Apr 28, 2015 at 06:40:05AM +, Olaf Hering wrote: > > Upcoming changes for vscsi will use libxlutil.so to prepare the > > configuration for libxl. The helpers needs a xlu struct for logging. > > Provide one and reuse the existing output as log target. >

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

2015-05-05 Thread Wei Liu
On Tue, Apr 28, 2015 at 06:40:05AM +, Olaf Hering wrote: > Upcoming changes for vscsi will use libxlutil.so to prepare the > configuration for libxl. The helpers needs a xlu struct for logging. > Provide one and reuse the existing output as log target. > Do you not need to call xlu_cfg_destro

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

2015-04-27 Thread Olaf Hering
Upcoming changes for vscsi will use libxlutil.so to prepare the configuration for libxl. The helpers needs a xlu struct for logging. Provide one and reuse the existing output as log target. Signed-off-by: Olaf Hering Cc: Jim Fehlig --- src/libxl/libxl_conf.c | 6 ++ src/libxl/libxl_conf.h |

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

2015-04-17 Thread Jim Fehlig
On 04/17/2015 11:44 AM, Olaf Hering wrote: Upcoming changes for vscsi will use libxlutil.so to prepare the configuration for libxl. The helpers needs a xlu struct for logging. Provide one and reuse the existing output as log target. Signed-off-by: Olaf Hering Cc: Jim Fehlig --- src/libxl/lib

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

2015-04-17 Thread Olaf Hering
Upcoming changes for vscsi will use libxlutil.so to prepare the configuration for libxl. The helpers needs a xlu struct for logging. Provide one and reuse the existing output as log target. Signed-off-by: Olaf Hering Cc: Jim Fehlig --- src/libxl/libxl_conf.c | 6 ++ src/libxl/libxl_conf.h |