diff options
| author | Sheng Yong <shengyong1@xiaomi.com> | 2025-07-23 22:24:56 +0800 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2025-07-24 20:16:19 +0000 |
| commit | 95d7c508b21235144f6cef611ec5686bbdeeec25 (patch) | |
| tree | 543b145e3f28cc36334c8359f9446cbee3b09be1 /fs/f2fs | |
| parent | f2fs: merge the two conditions to avoid code duplication (diff) | |
| download | linux-95d7c508b21235144f6cef611ec5686bbdeeec25.tar.gz linux-95d7c508b21235144f6cef611ec5686bbdeeec25.zip | |
f2fs: remove unnecessary tracepoint enabled check
There is no extra work before trace_f2fs_[dataread|datawrite]_end(),
so there is no need to check trace_<tracepoint>_enabled().
Signed-off-by: Sheng Yong <shengyong1@xiaomi.com>
Reviewed-by: Chao Yu <chao@kernel.org>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/file.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 4039ccb5022c..09ba8bef2f63 100644 --- a/fs/f2fs/file.c +++ b/fs/f2fs/file.c @@ -4855,8 +4855,7 @@ static ssize_t f2fs_file_read_iter(struct kiocb *iocb, struct iov_iter *to) f2fs_update_iostat(F2FS_I_SB(inode), inode, APP_BUFFERED_READ_IO, ret); } - if (trace_f2fs_dataread_end_enabled()) - trace_f2fs_dataread_end(inode, pos, ret); + trace_f2fs_dataread_end(inode, pos, ret); return ret; } @@ -4879,8 +4878,7 @@ static ssize_t f2fs_file_splice_read(struct file *in, loff_t *ppos, f2fs_update_iostat(F2FS_I_SB(inode), inode, APP_BUFFERED_READ_IO, ret); - if (trace_f2fs_dataread_end_enabled()) - trace_f2fs_dataread_end(inode, pos, ret); + trace_f2fs_dataread_end(inode, pos, ret); return ret; } @@ -5225,8 +5223,7 @@ static ssize_t f2fs_file_write_iter(struct kiocb *iocb, struct iov_iter *from) f2fs_dio_write_iter(iocb, from, &may_need_sync) : f2fs_buffered_write_iter(iocb, from); - if (trace_f2fs_datawrite_end_enabled()) - trace_f2fs_datawrite_end(inode, orig_pos, ret); + trace_f2fs_datawrite_end(inode, orig_pos, ret); } /* Don't leave any preallocated blocks around past i_size. */ |
