qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 0/3] Name threads


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH 0/3] Name threads
Date: Tue, 28 Jan 2014 16:41:31 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0

Il 28/01/2014 16:20, Dr. David Alan Gilbert (git) ha scritto:
From: "Dr. David Alan Gilbert" <address@hidden>

This series uses pthread_setname_np (when available) to set the names on
threads that QEMU creates to make life easier when debugging.

It's turned off by default (because there were worries that it might break
tools that relied on process names) but is enabled by adding

   namethreads=on

to the --name option.

Note that the initial thread still has the default name (or the value passed
as the process= parameter to --name).

I plead guilty to bikeshedding and propose s/namethreads/threads/.

Also, how did you test the merging of options? Perhaps parse_name could itself call qemu_find_opts("name"), and could be called just once at the end of option parsing. But that's just how *I* would have written the code, and what you have is fine if it works with multiple occurrences of -name.

Thanks for submitting this patch!

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]