Get rid of __ prefix for internal frand state

[fio.git] / backend.c

diff --git a/backend.c b/backend.c
index 14803c3835de6c0cda1e155f07e271bafce45c81..cae4e38de4b9fb3adc871863558dd1230bba32b2 100644 (file)
--- a/backend.c
+++ b/backend.c
@@ -55,9 +55,10 @@
 #include "err.h"
 #include "lib/tp.h"
 
-static pthread_t disk_util_thread;
-static pthread_cond_t du_cond;
-static pthread_mutex_t du_lock;
+static pthread_t helper_thread;
+static pthread_mutex_t helper_lock;
+pthread_cond_t helper_cond;
+int helper_do_stat = 0;
 
 static struct fio_mutex *startup_mutex;
 static struct flist_head *cgroup_list;
@@ -75,7 +76,7 @@ unsigned int stat_number = 0;
 int shm_id = 0;
 int temp_stall_ts;
 unsigned long done_secs = 0;
-volatile int disk_util_exit = 0;
+volatile int helper_exit = 0;
 
 #define PAGE_ALIGN(buf)        \
        (char *) (((uintptr_t) (buf) + page_mask) & ~page_mask)
@@ -752,9 +753,9 @@ static uint64_t do_io(struct thread_data *td)
                    ((io_u->flags & IO_U_F_VER_LIST) || !td_rw(td))) {
 
                        if (!td->o.verify_pattern_bytes) {
-                               io_u->rand_seed = __rand(&td->__verify_state);
+                               io_u->rand_seed = __rand(&td->verify_state);
                                if (sizeof(int) != sizeof(long *))
-                                       io_u->rand_seed *= __rand(&td->__verify_state);
+                                       io_u->rand_seed *= __rand(&td->verify_state);
                        }
 
                        if (td->o.verify_async)
@@ -1564,13 +1565,17 @@ err:
        if (o->write_iolog_file)
                write_iolog_close(td);
 
-       fio_mutex_remove(td->rusage_sem);
-       td->rusage_sem = NULL;
-
        fio_mutex_remove(td->mutex);
        td->mutex = NULL;
 
        td_set_runstate(td, TD_EXITED);
+
+       /*
+        * Do this last after setting our runstate to exited, so we
+        * know that the stat thread is signaled.
+        */
+       check_update_rusage(td);
+
        return (void *) (uintptr_t) td->error;
 }
 
@@ -1972,23 +1977,23 @@ static void run_threads(void)
        update_io_ticks();
 }
 
-static void wait_for_disk_thread_exit(void)
+static void wait_for_helper_thread_exit(void)
 {
        void *ret;
 
-       disk_util_start_exit();
-       pthread_cond_signal(&du_cond);
-       pthread_join(disk_util_thread, &ret);
+       helper_exit = 1;
+       pthread_cond_signal(&helper_cond);
+       pthread_join(helper_thread, &ret);
 }
 
 static void free_disk_util(void)
 {
        disk_util_prune_entries();
 
-       pthread_cond_destroy(&du_cond);
+       pthread_cond_destroy(&helper_cond);
 }
 
-static void *disk_thread_main(void *data)
+static void *helper_thread_main(void *data)
 {
        int ret = 0;
 
@@ -2008,12 +2013,15 @@ static void *disk_thread_main(void *data)
                        ts.tv_sec++;
                }
 
-               ret = pthread_cond_timedwait(&du_cond, &du_lock, &ts);
-               if (ret != ETIMEDOUT)
-                       break;
+               pthread_cond_timedwait(&helper_cond, &helper_lock, &ts);
 
                ret = update_io_ticks();
 
+               if (helper_do_stat) {
+                       helper_do_stat = 0;
+                       __show_running_run_stats();
+               }
+
                if (!is_backend)
                        print_thread_status();
        }
@@ -2021,18 +2029,18 @@ static void *disk_thread_main(void *data)
        return NULL;
 }
 
-static int create_disk_util_thread(void)
+static int create_helper_thread(void)
 {
        int ret;
 
        setup_disk_util();
 
-       pthread_cond_init(&du_cond, NULL);
-       pthread_mutex_init(&du_lock, NULL);
+       pthread_cond_init(&helper_cond, NULL);
+       pthread_mutex_init(&helper_lock, NULL);
 
-       ret = pthread_create(&disk_util_thread, NULL, disk_thread_main, NULL);
+       ret = pthread_create(&helper_thread, NULL, helper_thread_main, NULL);
        if (ret) {
-               log_err("Can't create disk util thread: %s\n", strerror(ret));
+               log_err("Can't create helper thread: %s\n", strerror(ret));
                return 1;
        }
 
@@ -2072,14 +2080,14 @@ int fio_backend(void)
 
        set_genesis_time();
        stat_init();
-       create_disk_util_thread();
+       create_helper_thread();
 
        cgroup_list = smalloc(sizeof(*cgroup_list));
        INIT_FLIST_HEAD(cgroup_list);
 
        run_threads();
 
-       wait_for_disk_thread_exit();
+       wait_for_helper_thread_exit();
 
        if (!fio_abort) {
                __show_run_stats();
@@ -2095,8 +2103,11 @@ int fio_backend(void)
                }
        }
 
-       for_each_td(td, i)
+       for_each_td(td, i) {
                fio_options_free(td);
+               fio_mutex_remove(td->rusage_sem);
+               td->rusage_sem = NULL;
+       }
 
        free_disk_util();
        cgroup_kill(cgroup_list);