X-Git-Url: https://git.kernel.dk/?p=fio.git;a=blobdiff_plain;f=diskutil.c;h=feb88526bbb90af4d67fbcec8b038c2305328ec2;hp=4ddaa33355bdd7a4093ec2aaa056f94617c228ed;hb=89adce7343fd523e4560d00e797d43b4510416fc;hpb=6914f59e5e3560325dc8892c9f77689df7b9aa8b diff --git a/diskutil.c b/diskutil.c index 4ddaa333..feb88526 100644 --- a/diskutil.c +++ b/diskutil.c @@ -31,7 +31,7 @@ static void disk_util_free(struct disk_util *du) flist_del(&slave->slavelist); slave->users--; } - + fio_mutex_remove(du->lock); sfree(du); } @@ -215,7 +215,7 @@ static void find_add_disk_slaves(struct thread_data *td, char *path, !strcmp(dirent->d_name, "..")) continue; - sprintf(temppath, "%s/%s", slavesdir, dirent->d_name); + sprintf(temppath, "%s%s%s", slavesdir, FIO_OS_PATH_SEPARATOR, dirent->d_name); /* Can we always assume that the slaves device entries * are links to the real directories for the slave * devices? @@ -240,7 +240,7 @@ static void find_add_disk_slaves(struct thread_data *td, char *path, if (slavedu) continue; - sprintf(temppath, "%s/%s", slavesdir, slavepath); + sprintf(temppath, "%s%s%s", slavesdir, FIO_OS_PATH_SEPARATOR, slavepath); __init_per_file_disk_util(td, majdev, mindev, temppath); slavedu = disk_util_exists(majdev, mindev); @@ -327,7 +327,7 @@ static int find_block_dir(int majdev, int mindev, char *path, int link_ok) if (!strcmp(dir->d_name, ".") || !strcmp(dir->d_name, "..")) continue; - sprintf(full_path, "%s/%s", path, dir->d_name); + sprintf(full_path, "%s%s%s", path, FIO_OS_PATH_SEPARATOR, dir->d_name); if (!strcmp(dir->d_name, "dev")) { if (!check_dev_match(majdev, mindev, full_path)) { @@ -470,7 +470,7 @@ static void show_agg_stats(struct disk_util_agg *agg, int terse) agg->time_in_queue / agg->slavecount, agg->max_util.u.f); } else { - log_info("slaves;%u;%u;%u;%u;%u;%u;%u;%3.2f%%", + log_info(";slaves;%u;%u;%u;%u;%u;%u;%u;%3.2f%%", agg->ios[0] / agg->slavecount, agg->ios[1] / agg->slavecount, agg->merges[0] / agg->slavecount, @@ -540,10 +540,10 @@ void print_disk_util(struct disk_util_stat *dus, struct disk_util_agg *agg, if (util > 100.0) util = 100.0; - if (agg->slavecount) - log_info(" "); - if (!terse) { + if (agg->slavecount) + log_info(" "); + log_info(" %s: ios=%u/%u, merge=%u/%u, ticks=%u/%u, " "in_queue=%u, util=%3.2f%%", dus->name, dus->ios[0], dus->ios[1], @@ -562,8 +562,7 @@ void print_disk_util(struct disk_util_stat *dus, struct disk_util_agg *agg, * If the device has slaves, aggregate the stats for * those slave devices also. */ - if (agg->slavecount) - show_agg_stats(agg, terse); + show_agg_stats(agg, terse); if (!terse) log_info("\n");