Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-12-03 Thread Dave Kleikamp
On 12/03/2012 08:52 PM, Dave Chinner wrote: > On Mon, Dec 03, 2012 at 10:59:39AM -0600, Dave Kleikamp wrote: >> On 11/22/2012 05:06 PM, Dave Chinner wrote: >>> And this extra fsync is now not done in the aio path. I.e. the AIO >>> completion path needs to issue the fsync to maintain correct

Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-12-03 Thread Dave Chinner
On Mon, Dec 03, 2012 at 10:59:39AM -0600, Dave Kleikamp wrote: > On 11/22/2012 05:06 PM, Dave Chinner wrote: > > On Wed, Nov 21, 2012 at 04:40:56PM -0600, Dave Kleikamp wrote: > >> From: Zach Brown > >> > >> This uses the new kernel aio interface to process loopback IO by > >> submitting

Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-12-03 Thread Dave Kleikamp
On 11/22/2012 05:06 PM, Dave Chinner wrote: > On Wed, Nov 21, 2012 at 04:40:56PM -0600, Dave Kleikamp wrote: >> From: Zach Brown >> >> This uses the new kernel aio interface to process loopback IO by >> submitting concurrent direct aio. Previously loop's IO was serialized >> by synchronous

Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-12-03 Thread Dave Kleikamp
On 11/22/2012 05:06 PM, Dave Chinner wrote: On Wed, Nov 21, 2012 at 04:40:56PM -0600, Dave Kleikamp wrote: From: Zach Brown z...@zabbo.net This uses the new kernel aio interface to process loopback IO by submitting concurrent direct aio. Previously loop's IO was serialized by synchronous

Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-12-03 Thread Dave Chinner
On Mon, Dec 03, 2012 at 10:59:39AM -0600, Dave Kleikamp wrote: On 11/22/2012 05:06 PM, Dave Chinner wrote: On Wed, Nov 21, 2012 at 04:40:56PM -0600, Dave Kleikamp wrote: From: Zach Brown z...@zabbo.net This uses the new kernel aio interface to process loopback IO by submitting

Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-12-03 Thread Dave Kleikamp
On 12/03/2012 08:52 PM, Dave Chinner wrote: On Mon, Dec 03, 2012 at 10:59:39AM -0600, Dave Kleikamp wrote: On 11/22/2012 05:06 PM, Dave Chinner wrote: And this extra fsync is now not done in the aio path. I.e. the AIO completion path needs to issue the fsync to maintain correct REQ_FUA

Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-11-22 Thread Dave Chinner
On Wed, Nov 21, 2012 at 04:40:56PM -0600, Dave Kleikamp wrote: > From: Zach Brown > > This uses the new kernel aio interface to process loopback IO by > submitting concurrent direct aio. Previously loop's IO was serialized > by synchronous processing in a thread. > > The aio operations specify

Re: [PATCH v4 16/31] loop: use aio to perform io on the underlying file

2012-11-22 Thread Dave Chinner
On Wed, Nov 21, 2012 at 04:40:56PM -0600, Dave Kleikamp wrote: From: Zach Brown z...@zabbo.net This uses the new kernel aio interface to process loopback IO by submitting concurrent direct aio. Previously loop's IO was serialized by synchronous processing in a thread. The aio operations