Re: [Qemu-devel] [RFC 0/5] nbd: Adapt for dataplane

2014-06-05 Thread Max Reitz
On 03.06.2014 16:38, Stefan Hajnoczi wrote: On Sat, May 31, 2014 at 08:43:07PM +0200, Max Reitz wrote: For the NBD server to work with dataplane, it needs to correctly access the exported BDS. It makes the most sense to run both in the same AioContext, therefore this series implements methods

Re: [Qemu-devel] [RFC 0/5] nbd: Adapt for dataplane

2014-06-04 Thread Stefan Hajnoczi
On Sat, May 31, 2014 at 08:43:07PM +0200, Max Reitz wrote: For the NBD server to work with dataplane, it needs to correctly access the exported BDS. It makes the most sense to run both in the same AioContext, therefore this series implements methods for tracking a BDS's AioContext and makes

Re: [Qemu-devel] [RFC 0/5] nbd: Adapt for dataplane

2014-06-04 Thread Stefan Hajnoczi
On Sat, May 31, 2014 at 10:24:47PM +0200, Max Reitz wrote: On 31.05.2014 20:43, Max Reitz wrote: [snip] However, if bs_aio_detach() is called from a different thread than the old AioContext is running in, we may still have coroutines running for which we should wait before returning from

Re: [Qemu-devel] [RFC 0/5] nbd: Adapt for dataplane

2014-06-03 Thread Stefan Hajnoczi
On Sat, May 31, 2014 at 08:43:07PM +0200, Max Reitz wrote: For the NBD server to work with dataplane, it needs to correctly access the exported BDS. It makes the most sense to run both in the same AioContext, therefore this series implements methods for tracking a BDS's AioContext and makes

[Qemu-devel] [RFC 0/5] nbd: Adapt for dataplane

2014-05-31 Thread Max Reitz
For the NBD server to work with dataplane, it needs to correctly access the exported BDS. It makes the most sense to run both in the same AioContext, therefore this series implements methods for tracking a BDS's AioContext and makes NBD make use of this for keeping the clients connected to that

Re: [Qemu-devel] [RFC 0/5] nbd: Adapt for dataplane

2014-05-31 Thread Max Reitz
On 31.05.2014 20:43, Max Reitz wrote: [snip] However, if bs_aio_detach() is called from a different thread than the old AioContext is running in, we may still have coroutines running for which we should wait before returning from bs_aio_detach(). After re-reading Stefan's RFC and the AIO