Re: [PATCH 07/13] aio: enabled thread based async fsync

From: Dave Chinner
Date: Wed Jan 20 2016 - 15:00:12 EST


On Tue, Jan 19, 2016 at 07:59:35PM -0800, Linus Torvalds wrote:
> On Fri, Jan 15, 2016 at 12:21 PM, Benjamin LaHaise <bcrl@xxxxxxxxx> wrote:
> >>
> >> I'll have to think about this some more.
> >
> > Any further thoughts on this after a few days worth of pondering?
>
> Sorry about the delay, with the merge window and me being sick for a
> couple of days I didn't get around to this.
>
> After thinking it over some more, I guess I'm ok with your approach.
> The table-driven patch makes me a bit happier, and I guess not very
> many people end up ever wanting to do async system calls anyway.
>
> Are there other users outside of Solace? It would be good to get comments..

I know of quite a few storage/db products that use AIO. The most
recent high profile project that have been reporting issues with AIO
on XFS is http://www.scylladb.com/. That project is architected
around non-blocking AIO for scalability reasons...

Cheers,

Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx