Merge branch 'fix_filename_overrun' of https://github.com/sitsofe/fio
authorJens Axboe <axboe@kernel.dk>
Sat, 16 Jan 2021 20:36:27 +0000 (13:36 -0700)
committerJens Axboe <axboe@kernel.dk>
Sat, 16 Jan 2021 20:36:27 +0000 (13:36 -0700)
* 'fix_filename_overrun' of https://github.com/sitsofe/fio:
  options: fix buffer overrun

backend.c
fio.h

index 2e6a377cfb4cb4317294eb854cd61712f402caf7..e20a2e07766eabf4ea0f86dfde6db40e9ed621f7 100644 (file)
--- a/backend.c
+++ b/backend.c
@@ -439,7 +439,7 @@ static int wait_for_completions(struct thread_data *td, struct timespec *time)
        if ((full && !min_evts) || !td->o.iodepth_batch_complete_min)
                min_evts = 1;
 
-       if (time && __should_check_rate(td))
+       if (time && should_check_rate(td))
                fio_gettime(time, NULL);
 
        do {
@@ -494,7 +494,7 @@ int io_queue_event(struct thread_data *td, struct io_u *io_u, int *ret,
                        requeue_io_u(td, &io_u);
                } else {
 sync_done:
-                       if (comp_time && __should_check_rate(td))
+                       if (comp_time && should_check_rate(td))
                                fio_gettime(comp_time, NULL);
 
                        *ret = io_u_sync_complete(td, io_u);
diff --git a/fio.h b/fio.h
index 4d439d988f088fba797ca858a5f0b8b0b1104910..ee582a7241f1710bcd3b82681932c764f8cd14df 100644 (file)
--- a/fio.h
+++ b/fio.h
@@ -757,17 +757,9 @@ static inline bool option_check_rate(struct thread_data *td, enum fio_ddir ddir)
        return false;
 }
 
-static inline bool __should_check_rate(struct thread_data *td)
-{
-       return (td->flags & TD_F_CHECK_RATE) != 0;
-}
-
 static inline bool should_check_rate(struct thread_data *td)
 {
-       if (!__should_check_rate(td))
-               return false;
-
-       return ddir_rw_sum(td->bytes_done) != 0;
+       return (td->flags & TD_F_CHECK_RATE) != 0;
 }
 
 static inline unsigned long long td_max_bs(struct thread_data *td)