Messages in this thread | | | Date | Tue, 23 May 2023 10:43:47 -0400 | From | Steven Rostedt <> | Subject | Re: [PATCH v6] f2fs: add async reset zone command support |
| |
On Tue, 9 May 2023 11:53:35 -0700 Jaegeuk Kim <jaegeuk@kernel.org> wrote:
> On 05/08, Daejun Park wrote: > > v5 -> v6 > > Added trace_f2fs_iostat support for zone reset command. > > > > v4 -> v5 > > Added f2fs iostat for zone reset command. > > > > v3 -> v4 > > Fixed build error caused by unused function. > > > > v2 -> v3 > > Modified arguments to be correct for ftrace parameter. > > Changed __submit_zone_reset_cmd to void return. > > Refactored the f2fs_wait_discard_bio function. > > Fixed code that was previously incorrectly merged. > > > > v1 -> v2 > > Changed to apply the optional async reset write pointer by default. > > Don't add the history in the patch description.
Well, you can (and should) ...
> > > > > This patch enables submit reset zone command asynchornously. It helps > > decrease average latency of write IOs in high utilization scenario by > > faster checkpointing. > > > > Signed-off-by: Daejun Park <daejun7.park@samsung.com> > > ---
But it must go below the three dashes above. That will keep 'git am' from adding it to the change log.
-- Steve
> > fs/f2fs/f2fs.h | 1 + > > fs/f2fs/iostat.c | 1 + > > fs/f2fs/segment.c | 84 +++++++++++++++++++++++++++++++++++-- > > include/trace/events/f2fs.h | 24 +++++++++-- > > 4 files changed, 104 insertions(+), 6 deletions(-) > > > > diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h > > index d211ee89c158..51b68a629814 100644 > > --- a/fs/f2fs/f2fs.h > > +++ b/fs/f2fs/f2fs.h > > @@ -1175,6 +1175,7 @@ enum iostat_type { > > /* other */ > > FS_DISCARD_IO, /* discard */ > > FS_FLUSH_IO, /* flush */ > > + FS_ZONE_RESET_IO, /* zone reset */ > > NR_IO_TYPE, > > }; > >
| |