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

From: Benjamin LaHaise
Date: Mon Jan 11 2016 - 20:30:23 EST


On Tue, Jan 12, 2016 at 12:11:28PM +1100, Dave Chinner wrote:
> On Mon, Jan 11, 2016 at 05:07:23PM -0500, Benjamin LaHaise wrote:
> > Enable a fully asynchronous fsync and fdatasync operations in aio using
> > the aio thread queuing mechanism.
> >
> > Signed-off-by: Benjamin LaHaise <ben.lahaise@xxxxxxxxxxxxxxxxx>
> > Signed-off-by: Benjamin LaHaise <bcrl@xxxxxxxxx>
>
> Insufficient. Needs the range to be passed through and call
> vfs_fsync_range(), as I implemented here:

Noted.

> https://lkml.org/lkml/2015/10/28/878

Please at least Cc the aio list in the future on aio patches, as I do
not have the time to read linux-kernel these days unless prodded to do
so...

-ben

> Cheers,
>
> Dave.
> --
> Dave Chinner
> david@xxxxxxxxxxxxx

--
"Thought is the essence of where you are now."