On Tue, Mar 10, 2020 at 11:14:17AM +0100, Miklos Szeredi wrote: > On Tue, Mar 3, 2020 at 10:15 PM Liu Bo <bo.liu@xxxxxxxxxxxxxxxxx> wrote: > > > > If this is a DSYNC write, make sure we push it to stable storage now > > that we've written data. > > If this is direct I/O then why do we need an fysnc() call? > > The only thing needed should be correct setting O_DSYNC in the flags > field of the WRITE request, and it appears to me that that is already > being done. Given direct IO itself doesn't guarantee FUA or FLUSH, I think we still need such a fsync() call to make sure a FUA/FLUSH is sent after direct IO. thanks, -liubo