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

2011-01-17 Thread Arun R Bharadwaj
* Stefan Hajnoczi [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 but doesn't act

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 i

[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