From 99614958d743d0fdea9b1875d3a41a5a51e9769d Mon Sep 17 00:00:00 2001 From: Jens Axboe Date: Wed, 1 Aug 2012 20:25:53 +0200 Subject: [PATCH] Revert "diskutil: ensure that we lock around disk_list access" This reverts commit feb418556a236aa041a625b083d6b99e55d23d74. It causes hangs, needs further testing. --- backend.c | 20 ++------------------ diskutil.c | 46 ++++++---------------------------------------- diskutil.h | 11 ++--------- iolog.h | 1 + 4 files changed, 11 insertions(+), 67 deletions(-) diff --git a/backend.c b/backend.c index 9cc8dbc3..e1dc0ac8 100644 --- a/backend.c +++ b/backend.c @@ -50,7 +50,6 @@ #include "server.h" static pthread_t disk_util_thread; -static struct fio_mutex *disk_thread_mutex; static struct fio_mutex *startup_mutex; static struct fio_mutex *writeout_mutex; static struct flist_head *cgroup_list; @@ -1589,28 +1588,20 @@ static void run_threads(void) fio_unpin_memory(); } -void wait_for_disk_thread_exit(void) -{ - fio_mutex_down(disk_thread_mutex); -} - static void *disk_thread_main(void *data) { - int ret = 0; - fio_mutex_up(startup_mutex); - while (threads && !ret) { + while (threads) { usleep(DISK_UTIL_MSEC * 1000); if (!threads) break; - ret = update_io_ticks(); + update_io_ticks(); if (!is_backend) print_thread_status(); } - fio_mutex_up(disk_thread_mutex); return NULL; } @@ -1618,20 +1609,14 @@ static int create_disk_util_thread(void) { int ret; - setup_disk_util(); - - disk_thread_mutex = fio_mutex_init(0); - ret = pthread_create(&disk_util_thread, NULL, disk_thread_main, NULL); if (ret) { - fio_mutex_remove(disk_thread_mutex); log_err("Can't create disk util thread: %s\n", strerror(ret)); return 1; } ret = pthread_detach(disk_util_thread); if (ret) { - fio_mutex_remove(disk_thread_mutex); log_err("Can't detatch disk util thread: %s\n", strerror(ret)); return 1; } @@ -1695,6 +1680,5 @@ int fio_backend(void) fio_mutex_remove(startup_mutex); fio_mutex_remove(writeout_mutex); - fio_mutex_remove(disk_thread_mutex); return exit_value; } diff --git a/diskutil.c b/diskutil.c index 32616b75..feb88526 100644 --- a/diskutil.c +++ b/diskutil.c @@ -14,9 +14,6 @@ static int last_majdev, last_mindev; static struct disk_util *last_du; -static struct fio_mutex *disk_util_mutex; -static int disk_util_exit; - FLIST_HEAD(disk_list); static struct disk_util *__init_per_file_disk_util(struct thread_data *td, @@ -105,26 +102,17 @@ static void update_io_tick_disk(struct disk_util *du) memcpy(ldus, &__dus, sizeof(__dus)); } -int update_io_ticks(void) +void update_io_ticks(void) { struct flist_head *entry; struct disk_util *du; - int ret = 0; dprint(FD_DISKUTIL, "update io ticks\n"); - fio_mutex_down(disk_util_mutex); - - if (!disk_util_exit) { - flist_for_each(entry, &disk_list) { - du = flist_entry(entry, struct disk_util, list); - update_io_tick_disk(du); - } - } else - ret = 1; - - fio_mutex_up(disk_util_mutex); - return ret; + flist_for_each(entry, &disk_list) { + du = flist_entry(entry, struct disk_util, list); + update_io_tick_disk(du); + } } static struct disk_util *disk_util_exists(int major, int minor) @@ -288,8 +276,6 @@ static struct disk_util *disk_util_add(struct thread_data *td, int majdev, du->lock = fio_mutex_init(1); du->users = 0; - fio_mutex_down(disk_util_mutex); - flist_for_each(entry, &disk_list) { __du = flist_entry(entry, struct disk_util, list); @@ -297,7 +283,6 @@ static struct disk_util *disk_util_add(struct thread_data *td, int majdev, if (!strcmp((char *) du->dus.name, (char *) __du->dus.name)) { disk_util_free(du); - fio_mutex_up(disk_util_mutex); return __du; } } @@ -309,7 +294,6 @@ static struct disk_util *disk_util_add(struct thread_data *td, int majdev, flist_add_tail(&du->list, &disk_list); find_add_disk_slaves(td, path, du); - fio_mutex_up(disk_util_mutex); return du; } @@ -537,11 +521,6 @@ void free_disk_util(void) { struct disk_util *du; - disk_util_exit = 1; - wait_for_disk_thread_exit(); - - fio_mutex_down(disk_util_mutex); - while (!flist_empty(&disk_list)) { du = flist_entry(disk_list.next, struct disk_util, list); flist_del(&du->list); @@ -549,8 +528,6 @@ void free_disk_util(void) } last_majdev = last_mindev = -1; - fio_mutex_up(disk_util_mutex); - fio_mutex_remove(disk_util_mutex); } void print_disk_util(struct disk_util_stat *dus, struct disk_util_agg *agg, @@ -596,12 +573,8 @@ void show_disk_util(int terse) struct flist_head *entry; struct disk_util *du; - fio_mutex_down(disk_util_mutex); - - if (flist_empty(&disk_list)) { - fio_mutex_up(disk_util_mutex); + if (flist_empty(&disk_list)) return; - } if (!terse) log_info("\nDisk stats (read/write):\n"); @@ -612,11 +585,4 @@ void show_disk_util(int terse) aggregate_slaves_stats(du); print_disk_util(&du->dus, &du->agg, terse); } - - fio_mutex_up(disk_util_mutex); -} - -void setup_disk_util(void) -{ - disk_util_mutex = fio_mutex_init(1); } diff --git a/diskutil.h b/diskutil.h index 88dde555..5a9b0796 100644 --- a/diskutil.h +++ b/diskutil.h @@ -94,8 +94,6 @@ static inline void disk_util_dec(struct disk_util *du) extern struct flist_head disk_list; -extern void wait_for_disk_thread_exit(void); - /* * disk util stuff */ @@ -104,18 +102,13 @@ extern void print_disk_util(struct disk_util_stat *, struct disk_util_agg *, int extern void show_disk_util(int terse); extern void free_disk_util(void); extern void init_disk_util(struct thread_data *); -extern int update_io_ticks(void); -extern void setup_disk_util(void); +extern void update_io_ticks(void); #else #define print_disk_util(dus, agg, terse) #define show_disk_util(terse) #define free_disk_util() #define init_disk_util(td) -#define setup_disk_util() -static inline int update_io_ticks(void) -{ - return 0; -} +#define update_io_ticks() #endif #endif diff --git a/iolog.h b/iolog.h index 1853846e..95617fcc 100644 --- a/iolog.h +++ b/iolog.h @@ -107,6 +107,7 @@ extern void add_bw_sample(struct thread_data *, enum fio_ddir, unsigned int, extern void add_iops_sample(struct thread_data *, enum fio_ddir, struct timeval *); extern void init_disk_util(struct thread_data *); extern void update_rusage_stat(struct thread_data *); +extern void update_io_ticks(void); extern void setup_log(struct io_log **, unsigned long); extern void finish_log(struct thread_data *, struct io_log *, const char *); extern void finish_log_named(struct thread_data *, struct io_log *, const char *, const char *); -- 2.25.1