Signed-off-by: Kir Kolyshkin <kir@openvz.org>

(1) mkostemp with O_DIRECT in flags can fail, leaving a temporary file:

Apparently, O_DIRECT is not working on top of LVM. The problem is
in this case mkostemp() creates the temporary file but still returns
an error, and the temp file is not removed:

 $ ls
 ioping  ioping.1  ioping.c  ioping.o  ioping.spec  Makefile
 $ strace ./ioping -D .
 ....
 open("./ioping.UFuMHz", O_RDWR|O_CREAT|O_EXCL|O_DIRECT, 0600) = -1 EINVAL (Invalid argument)
 ....
 $ ls
 ioping  ioping.1  ioping.c  ioping.o  ioping.UFuMHz  ioping.spec  Makefile

Solution -- do not use flags, apply those later using fcntl()

(2) While trying to rebuild on centos5:

 cc -c -o ioping.o ioping.c -std=c99 -g -Wall -Wextra -pedantic -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64
 ioping.c: In function 'main':
 ioping.c:369: warning: implicit declaration of function 'mkostemp'
Signed-off-by: Kir Kolyshkin <kir@openvz.org>

'ioping -w 7 -i 2' was taking 8 seconds to run, not 7 or less.
Fix it by moving the check to before usleep(), and taking
the interval (ie usleep argument) into account.
Signed-off-by: Kir Kolyshkin <kir@openvz.org>

This commit addresses two similar (but distinct) bugs.

(1) 'ioping -c' was needlessly sleeping after the last iteration
(and before showing final stats). Fix it by moving the check
to before the usleep(), not after.

(2) 'ioping -w' was going over the time limit specified. For example,
'ioping -w 7 -i 2' was taking 8 seconds to run, not 7 or less.
Fix it by moving the check to before usleep(), and taking
the interval (ie usleep argument) into account.
Signed-off-by: Kir Kolyshkin <kir@openvz.org>

err(1, NULL) prints program name and errno string, which is
just what we need in this case.
Signed-off-by: Kir Kolyshkin <kir@openvz.org>

* free temp path string

getopt() already emits warning about unknown option, so if we do that, too,
the result will look like this:

	$ ./ioping -z
	./ioping: invalid option -- 'z'
	ioping: unknown option: -z

Don't warn about unknown option, instead show usage().
Signed-off-by: Kir Kolyshkin <kir@openvz.org>

Needed mostly for distro vendors (.spec writers etc.)

* Makefile: do not strip when installing
Otherwise distros could not produce debuginfo packages

* Makefile: add BINDIR
Signed-off-by: Kir Kolyshkin <kir@openvz.org>

Signed-off-by: Kir Kolyshkin <kir@openvz.org>