|
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
[Prev in Thread] | Current Thread | [Next in Thread] |