From fefefa002d044098f2d48e42c27e200d1f95a97b Mon Sep 17 00:00:00 2001 From: Shane Snyder Date: Tue, 14 Aug 2018 17:08:04 -0500 Subject: [PATCH] make sure to allow fsyncs in RADOS --- src/ior.c | 3 ++- src/parse_options.c | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/ior.c b/src/ior.c index db87b2e..1fed65e 100755 --- a/src/ior.c +++ b/src/ior.c @@ -1541,7 +1541,8 @@ static void ValidateTests(IOR_param_t * test) if (((strcasecmp(test->api, "POSIX") != 0) && (strcasecmp(test->api, "MPIIO") != 0) && (strcasecmp(test->api, "MMAP") != 0) - && (strcasecmp(test->api, "HDFS") != 0)) && test->fsync) + && (strcasecmp(test->api, "HDFS") != 0) + && (strcasecmp(test->api, "RADOS") != 0)) && test->fsync) WARN_RESET("fsync() not supported in selected backend", test, &defaults, fsync); if ((strcasecmp(test->api, "MPIIO") != 0) && test->preallocate) diff --git a/src/parse_options.c b/src/parse_options.c index d0dec17..0e46c27 100755 --- a/src/parse_options.c +++ b/src/parse_options.c @@ -449,7 +449,7 @@ IOR_test_t *ParseCommandLine(int argc, char **argv) {.help=" -O stoneWallingWearOut=1 -- once the stonewalling timout is over, all process finish to access the amount of data", .arg = OPTION_OPTIONAL_ARGUMENT}, {.help=" -O stoneWallingWearOutIterations=N -- stop after processing this number of iterations, needed for reading data back written with stoneWallingWearOut", .arg = OPTION_OPTIONAL_ARGUMENT}, {.help=" -O stoneWallingStatusFile=FILE -- this file keeps the number of iterations from stonewalling during write and allows to use them for read", .arg = OPTION_OPTIONAL_ARGUMENT}, - {'e', NULL, "fsync -- perform fsync/msync upon POSIX/MMAP write close", OPTION_FLAG, 'd', & initialTestParams.fsync}, + {'e', NULL, "fsync -- perform sync operation after each block write", OPTION_FLAG, 'd', & initialTestParams.fsync}, {'E', NULL, "useExistingTestFile -- do not remove test file before write access", OPTION_FLAG, 'd', & initialTestParams.useExistingTestFile}, {'f', NULL, "scriptFile -- test script name", OPTION_OPTIONAL_ARGUMENT, 's', & testscripts}, {'F', NULL, "filePerProc -- file-per-process", OPTION_FLAG, 'd', & initialTestParams.filePerProc}, @@ -491,7 +491,7 @@ IOR_test_t *ParseCommandLine(int argc, char **argv) {'W', NULL, "checkWrite -- check read after write", OPTION_FLAG, 'd', & initialTestParams.checkWrite}, {'x', NULL, "singleXferAttempt -- do not retry transfer if incomplete", OPTION_FLAG, 'd', & initialTestParams.singleXferAttempt}, {'X', NULL, "reorderTasksRandomSeed -- random seed for -Z option", OPTION_OPTIONAL_ARGUMENT, 'd', & initialTestParams.reorderTasksRandomSeed}, - {'Y', NULL, "fsyncPerWrite -- perform fsync/msync after each POSIX/MMAP write", OPTION_FLAG, 'd', & initialTestParams.fsyncPerWrite}, + {'Y', NULL, "fsyncPerWrite -- perform sync operation after every write operation", OPTION_FLAG, 'd', & initialTestParams.fsyncPerWrite}, {'z', NULL, "randomOffset -- access is to random, not sequential, offsets within a file", OPTION_FLAG, 'd', & initialTestParams.randomOffset}, {'Z', NULL, "reorderTasksRandom -- changes task ordering to random ordering for readback", OPTION_FLAG, 'd', & initialTestParams.reorderTasksRandom}, {.help=" -O summaryFile=FILE -- store result data into this file", .arg = OPTION_OPTIONAL_ARGUMENT},