handle race to mkdir at startup
authorJeff Moyer <jmoyer@redhat.com>
Fri, 17 Apr 2009 06:50:22 +0000 (08:50 +0200)
committerJens Axboe <jens.axboe@oracle.com>
Fri, 17 Apr 2009 06:50:22 +0000 (08:50 +0200)
commit60886290e2ba6ddae1213f3a13929193180c8c55
tree61ee51a7ab1b177f0c97e7fb39ff079f57b4412a
parentbb4a66074cecd59afc37f0053fdf42661503f51c
handle race to mkdir at startup

I ran into a problem when specifying -D dirname-that-doesnt-yet-exist.
Blktrace would fail, spewing the following messages:

[root@megadeth blktrace]# ./blktrace -d /dev/cciss/c0d1 -D ./2.6.30-rc2-cfq-local
Destination dir ./2.6.30-rc2-cfq-local/ can't be made: 17/File exists
Destination dir ./2.6.30-rc2-cfq-local/ can't be made: 17/File exists
Destination dir ./2.6.30-rc2-cfq-local/ can't be made: 17/File exists
Destination dir ./2.6.30-rc2-cfq-local/ can't be made: 17/File exists
Destination dir ./2.6.30-rc2-cfq-local/ can't be made: 17/File exists
FAILED to start thread on CPU 0: 1/Operation not permitted
FAILED to start thread on CPU 4: 1/Operation not permitted
FAILED to start thread on CPU 5: 1/Operation not permitted
FAILED to start thread on CPU 6: 1/Operation not permitted
FAILED to start thread on CPU 7: 1/Operation not permitted

I tracked it down to the fact that there is no synchronization between
threads when trying to create the output directory.  The fix is simple,
just allow the race to happen and detect it.  It's not really worth
putting in any extra synchronization.  It looks like no place else in
that startup path needs synchronization either.

This patch fixes the issue for me.  I tested it by running the very
command that caused me headaches 100% of the time before.  I also did a
chattr +i on the directory and verified that it would really fail in the
case where it couldn't create the directory.

Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
blktrace.c