https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-10178 Lustre <p>While processing CLIO fsync IO an OSC sends separate SYNC request when BRW is finished. The optimization idea is to avoid this just by making last BRW synchronous instead. Refer to the osc_io_fsync_start(), osc_fsync_ost() functions and mdc_io_fsync_start() as well.</p> LU-10178

DoM: Use synced last BRW instead of extra SYNC RPC for CLIO fsync

Technical task LU-10176 Minor Open Unresolved WC Triage Mikhail Pershin DoM2 medium Wed, 1 Nov 2017 15:14:20 +0000 Mon, 1 May 2023 07:53:59 +0000 0 3 <p>I thought this was implemented or was in a patch somewhere already?</p> <p>Mike,</p> <p>Is there still work to do here?</p> <p>Thanks.</p> <p>Joe</p> <p>Joseph, yes, there are work to do.</p> <p>Patrick, is this something that would be easily done?</p> <p>Hmm, I'm only about 80% sure I understand.  It sounds like we're saying when there's data to flush on an fsync, we should just make the RPC sync, so it forces the OST to do a sync, rather than send a separate sync request to the OST.</p> <p>It wouldn't be the hardest thing in the world; we'd modify how FSYNC works a little so it would force data written out to be synchronous.  The harder part would be making sure we tracked if sync actually sent anything, but it wouldn't be that hard.</p> <p>Is fsync expected to force a journal commit?  It is, right?  So we for sure have to send a sync command to the OST, whether or not we have any dirty data to sync.  It's just that if we have dirty data we are sync'ing, we could skip the separate sync request.</p> <p>May I ask why this is of interest?  It seems a pretty small optimization for a case that I don't normally think of as really performance sensitive?  (I'm happy to learn here, that's why I'm asking)</p> <p>The goal here would be to send the <b>last</b> RPC without <tt>OBD_BRW_ASYNC</tt> if there is data to be flushed, so that it forces a sync and returns with an updated transno. This would be most useful for flushing small files, where it would cut the RPC count in half.</p> <p>Nothing urgent, just an open ticket that I found that seems like it would be easily fixed. </p> Related LU-3285 Development Rank 1|hzzmvz: Rank (Obsolete) 9223372036854775807