summaryrefslogtreecommitdiffstats
path: root/tools/dmsetup.c
diff options
context:
space:
mode:
authorAlasdair Kergon <agk@redhat.com>2009-08-03 18:01:45 +0000
committerAlasdair Kergon <agk@redhat.com>2009-08-03 18:01:45 +0000
commita74be32bf646dacb3a2930621cf53987cd3d8371 (patch)
tree008fa78748337bb6220bdeab815c87fba5767571 /tools/dmsetup.c
parent5e824754901ad4bef7568c70c8434e1cf5a14bf2 (diff)
downloadlvm2-a74be32bf646dacb3a2930621cf53987cd3d8371.tar.gz
lvm2-a74be32bf646dacb3a2930621cf53987cd3d8371.tar.xz
lvm2-a74be32bf646dacb3a2930621cf53987cd3d8371.zip
Manage without dm_udev_cleanup?
Diffstat (limited to 'tools/dmsetup.c')
-rw-r--r--tools/dmsetup.c28
1 files changed, 8 insertions, 20 deletions
diff --git a/tools/dmsetup.c b/tools/dmsetup.c
index d0da1906..bc0d15e1 100644
--- a/tools/dmsetup.c
+++ b/tools/dmsetup.c
@@ -588,12 +588,8 @@ static int _create(int argc, char **argv, void *data __attribute((unused)))
dm_udev_set_sync_support(0);
if (!dm_task_set_cookie(dmt, &cookie) ||
- !dm_task_run(dmt)) {
- (void) dm_udev_cleanup(cookie);
+ !dm_task_run(dmt))
goto out;
- }
-
- (void) dm_udev_wait(cookie);
r = 1;
@@ -601,6 +597,7 @@ static int _create(int argc, char **argv, void *data __attribute((unused)))
r = _display_info(dmt);
out:
+ (void) dm_udev_wait(cookie);
dm_task_destroy(dmt);
return r;
@@ -626,16 +623,13 @@ static int _rename(int argc, char **argv, void *data __attribute((unused)))
goto out;
if (!dm_task_set_cookie(dmt, &cookie) ||
- !dm_task_run(dmt)) {
- (void) dm_udev_cleanup(cookie);
+ !dm_task_run(dmt))
goto out;
- }
-
- (void) dm_udev_wait(cookie);
r = 1;
out:
+ (void) dm_udev_wait(cookie);
dm_task_destroy(dmt);
return r;
@@ -816,24 +810,18 @@ static int _simple(int task, const char *name, uint32_t event_nr, int display)
_read_ahead_flags))
goto out;
- if (udev_wait_flag && !dm_task_set_cookie(dmt, &cookie)) {
- (void) dm_udev_cleanup(cookie);
+ if (udev_wait_flag && !dm_task_set_cookie(dmt, &cookie))
goto out;
- }
r = dm_task_run(dmt);
- if (udev_wait_flag) {
- if (r)
- (void) dm_udev_wait(cookie);
- else
- (void) dm_udev_cleanup(cookie);
- }
-
if (r && display && _switches[VERBOSE_ARG])
r = _display_info(dmt);
out:
+ if (udev_wait_flag)
+ (void) dm_udev_wait(cookie);
+
dm_task_destroy(dmt);
return r;
}