Re: [Qemu-devel] [PATCH 00/12] Threadlets Infrastructure.

2011-01-17 Thread Stefan Hajnoczi
This series needs a new pair of eyes for review. I'm probably missing things here after seeing it many times. posix-aio-compat.c:handle_work() doesn't need to take aiocb_mutex for container_of(), which just calculates an address but doesn't actually access the aiocb. dequeue_work_on_queue() is

Re: [Qemu-devel] [PATCH 00/12] Threadlets Infrastructure.

2011-01-17 Thread Arun R Bharadwaj
* Stefan Hajnoczi stefa...@gmail.com [2011-01-17 10:32:24]: This series needs a new pair of eyes for review. I'm probably missing things here after seeing it many times. posix-aio-compat.c:handle_work() doesn't need to take aiocb_mutex for container_of(), which just calculates an address

[Qemu-devel] [PATCH 00/12] Threadlets Infrastructure.

2011-01-13 Thread Arun R Bharadwaj
Hi, This series implements threadlets infrastructure: Changelog: * Changed the name aio_thread to handle_work. * Corrected the usage of aiocb-ret which has to be accessed under aiocb_mutex. * Merged the patche which add dequeue_work with the patch which removes the unused active field in