Re: [PATCH] libxenlight: Clean up logging arrangements

[Vincent Hanquez <vincent.hanquez@eu.citrix.com>]

Ian Jackson wrote:
> libxenlight: Clean up logging arrangements
> 
> * Introduce new variants of the logging functions which include
>   errno values (converted using strerror) in the messages passed to the
>   application's logging callback.
> 
> * While we're at it, introduce xl_logv.
> 
> * Use the new errno-including logging functions everywhere where
>   appropriate.  In general, xc_... functions return errno values or 0;
>   xs_... functions return 0 or -1 (or some such) setting errno.
> 
> * When libxl_xs_get_dompath fails, do not treat it as an allocation
>   error.  It isn't: it usually means xenstored failed.
> 
> * Remove many spurious \n's from log messages.  (The applications log
>   callback is expected to add a \n if it wants to do that, so libxl's
>   logging functions should be passes strings without \n.)
> 
> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

both patches look good.

Acked-by: Vincent Hanquez <vincent.hanquez@eu.citrix.com>

Thanks,
-- 
Vincent