[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/13] migration: create multifd migration threa
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH 05/13] migration: create multifd migration threads |
Date: |
Mon, 23 Jan 2017 18:15:49 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
"Dr. David Alan Gilbert" <address@hidden> wrote:
> * Juan Quintela (address@hidden) wrote:
>> Creation of the threads, nothing inside yet.
>>
>> Signed-off-by: Juan Quintela <address@hidden>
>> +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
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 05/13] migration: create multifd migration threads,
Juan Quintela <=