qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] Re: [PATCH 2/3] qemu-thread: add cleanup_push() and cl


From: Corentin Chary
Subject: Re: [Qemu-devel] Re: [PATCH 2/3] qemu-thread: add cleanup_push() and cleanup_pop()
Date: Thu, 3 Jun 2010 09:46:06 +0200

On Thu, Jun 3, 2010 at 9:27 AM, Paolo Bonzini <address@hidden> wrote:
> On 05/29/2010 09:38 AM, Corentin Chary wrote:
>>
>> Signed-off-by: Corentin Chary<address@hidden>
>> ---
>>  qemu-thread.h |    4 ++++
>>  1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/qemu-thread.h b/qemu-thread.h
>> index 19bb30c..e5006bb 100644
>> --- a/qemu-thread.h
>> +++ b/qemu-thread.h
>> @@ -41,4 +41,8 @@ void qemu_thread_self(QemuThread *thread);
>>  int qemu_thread_equal(QemuThread *thread1, QemuThread *thread2);
>>  void qemu_thread_exit(void *retval);
>>
>> +#define qemu_thread_cleanup_pop(execute) pthread_cleanup_pop(execute)
>> +#define qemu_thread_cleanup_push(routine, arg)  \
>> +    pthread_cleanup_push(routine, arg)
>
> I agree with Paul that this isn't necessary.  Also you're not using
> pthread_exit.  Probably stale from a previous version of the patch?

Right pthread_exit() is missing.

Anyway, I don't use thread cancellation in the threaded vnc server (I
send a signal to the condition with an empty queue to stop the
thread), so we can skip this patch.



-- 
Corentin Chary
http://xf.iksaif.net



reply via email to

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