[libvirt] [PATCH] storage: make debug log more useful
Eric Blake
eblake at redhat.com
Mon Feb 21 22:23:26 UTC 2011
On 02/21/2011 12:08 AM, Osier Yang wrote:
> "__func__" should be replaced with "__FUNCTION__" instead for
> better portability,
Actually, __FUNCTION__ is gcc-specific, while __func__ is the more
portable C99 construct (we #define __FUNCTION__ __func__ in internal.h,
and rely on the gnulib module for __func__, so either spelling works
with equal portability from within libvirt; meanwhile, we should
probably be gradually getting rid of __FUNCTION__).
> however, VIR_DEBUG already prints the function
> name, so it's useless, replace it with more useful log.
>
> * src/storage/storage_backend_mpath.c
> ---
> src/storage/storage_backend_mpath.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/storage/storage_backend_mpath.c b/src/storage/storage_backend_mpath.c
> index 2e85561..2d48a9d 100644
> --- a/src/storage/storage_backend_mpath.c
> +++ b/src/storage/storage_backend_mpath.c
> @@ -316,7 +316,7 @@ virStorageBackendMpathRefreshPool(virConnectPtr conn ATTRIBUTE_UNUSED,
> {
> int retval = 0;
>
> - VIR_DEBUG("in %s", __func__);
> + VIR_DEBUG("conn=%p, pool=%p", conn, pool);
ACK.
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110221/21c0c122/attachment-0001.sig>
More information about the libvir-list
mailing list