* Paolo Bonzini (pbonz...@redhat.com) wrote: > This makes it easier to find the desired thread > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > --- > iothread.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/iothread.c b/iothread.c > index da6ce7b..8a1d6f8 100644 > --- a/iothread.c > +++ b/iothread.c > @@ -72,6 +72,7 @@ static void iothread_complete(UserCreatable *obj, Error > **errp) > { > Error *local_error = NULL; > IOThread *iothread = IOTHREAD(obj); > + char *name, *thread_name; > > iothread->stopping = false; > iothread->thread_id = -1; > @@ -87,8 +88,12 @@ static void iothread_complete(UserCreatable *obj, Error > **errp) > /* This assumes we are called from a thread with useful CPU affinity for > us > * to inherit. > */ > - qemu_thread_create(&iothread->thread, "iothread", iothread_run, > + name = object_get_canonical_path_component(OBJECT(obj)); > + thread_name = g_strdup_printf("iothread %s", name);
Yes, that's a good idea; Can you shorten that to just "IO %s" please, for three reasons: 1) There is a ~14 character limit on the size of that string 2) We use CPU ... for the CPU threads. 3) It's a threadname, it doesn't need to say thread Dave > + qemu_thread_create(&iothread->thread, thread_name, iothread_run, > iothread, QEMU_THREAD_JOINABLE); > + g_free(thread_name); > + g_free(name); > > /* Wait for initialization to complete */ > qemu_mutex_lock(&iothread->init_done_lock); > -- > 1.8.3.1 > -- Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK