Re: [PATCH]lxc: improve readability of lxcContainer[Send|Waitfor]Continue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Michal Privoznik [mailto:mprivozn@xxxxxxxxxx]
> Sent: Monday, October 21, 2013 9:29 PM
> To: Chen Hanxiao
> Cc: libvir-list@xxxxxxxxxx
> Subject: Re:  [PATCH]lxc: improve readability of
> lxcContainer[Send|Waitfor]Continue
> 
> On 16.10.2013 08:27, Chen Hanxiao wrote:
> > From: Chen Hanxiao <chenhanxiao@xxxxxxxxxxxxxx>
> 
> I think this can be achieved even without this hack. We already can
> produce a stack trace on VIR_DEBUG() [1]. And if you don't want to keep
> only some debug messages, apply filters.
> 

If we only config debug option for output values, we got:
604+0000: 12010: debug : lxcContainerWaitForContinue:392 : Wait continue on
fd 21

If we config filter option as "1:+lxc", we got:
2013-10-22 02:24:30.365+0000: 13579: debug : lxcContainerWaitForContinue:392
: Wait continue on fd 12
/usr/libexec/libvirt_lxc(virLogMessage+0x97)[0x7f2e1bdfa387]
/usr/libexec/libvirt_lxc(lxcContainerWaitForContinue+0x4b)[0x7f2e1bd7346b]
/usr/libexec/libvirt_lxc(+0x3197b)[0x7f2e1bd7b97b]
/usr/libexec/libvirt_lxc(main+0xcdf)[0x7f2e1bd70e4f]
/lib64/libc.so.6(__libc_start_main+0xf5)[0x7f2e19761b75]
/usr/libexec/libvirt_lxc(+0x27065)[0x7f2e1bd71065]

We still can't see who is the caller, and we got too many logs.
Additionally, filters with long name like ' lxcContainerWaitForContinue'
will not take effect.

With this patch, we got:
17931: debug : lxcContainerWaitForContinue:394 : virLXCControllerRun wait
continue on fd 12

I think we still need this patch.

Thanks!

> Michal
> 
> 1: http://libvirt.org/logging.html#log_syntax


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]