"Dr. David Alan Gilbert" <dgilb...@redhat.com> wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> Creation of the threads, nothing inside yet.
>>
>> Signed-off-by: Juan Quintela <quint...@redhat.com>
>> +void migrate_multifd_send_threads_join(void)
>> +{
>> + int i, thread_count;
>> +
>> + if (!migrate_multifd()){
>
> You've missed the space prior to the { (and then copied
> it everywhere in this patch).
Fixed. As guessed, copy & paste O:-)
>> + thread_count = migrate_multifd_threads();
>> + multifd_send = g_new0(MultiFDSendParams, thread_count);
>> + for (i = 0; i < thread_count; i++) {
>> + qemu_mutex_init(&multifd_send[i].mutex);
>> + qemu_cond_init(&multifd_send[i].cond);
>> + multifd_send[i].quit = false;
>> + qemu_thread_create(&multifd_send[i].thread, "multifd_send",
>
> You could make the name of the thread include the thread number,
> that way you could easily see in top if any one of the threads
> was getting particularly busy (although be careful with the length
> I think linux will ignore the name if it's over 14 characters).
Until 99 thread will be show correctly :p
