On 2014-04-02 02:33, Erwan Velu wrote:
Hey Jens,
The following code is pretty confusing for me :
http://git.kernel.dk/?p=fio.git;a=blob;f=init.c;h=c3996a76c12840e3ab3cf6514e5d5dce611a1077;hb=HEAD#l1962
The "if (nr_clients && fio_clients_connect()) {" is using the do_exit
concept while being after the code that manage it and does a return -1
while exit code is set to 1.
I don't get how it should be fixed.
1- we shall move the if nr_clients before the "if (do_exit", remove the
return -1 and adjust the exit_code ?
2- we shall get rid of the do_exit & exit code semantic of the if
nr_clients.
What was the intent while reading the code ? ;o)
It should probably just be turned into an exit(1) or similar exit code.
The current code will work to the extent that it will properly exit,
however it wont signal the error correctly in the return value.
--
Jens Axboe
--
To unsubscribe from this list: send the line "unsubscribe fio" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
