At 2016-09-22 05:32:10, "John Ferlan" <jferlan(a)redhat.com> wrote:
On 09/19/2016 07:06 AM, Chen Hanxiao wrote:
> From: Chen Hanxiao <chenhanxiao(a)gmail.com>
>
> This series will pass the path of compress programe
> directly to functions such as doCoreDump, qemuDomainSaveInternal, etc.
> this can avoid one extra virFindFileInPath call in during virExec.
>
> Also reduce code duplication by macros.
>
> Chen Hanxiao (3):
> qemu_driver: pass path of compress prog directly
> qemu_driver: simplify compress prog check by macros
> qemu_driver: pass path of compress prog directly to doCoreDump
>
> src/qemu/qemu_driver.c | 242 ++++++++++++++++++++++++++-----------------------
> 1 file changed, 128 insertions(+), 114 deletions(-)
>
Not really what I had in mind.
I've posted a different approach instead. See
http://www.redhat.com/archives/libvir-list/2016-September/msg00971.html
I'll review your series as soon as possible.
BTW: Patch 2 fails syntax-check
I make syntax-check again and passed on a centos7 machine.
Maybe it's a syntax-check bug?
Regards,
- Chen