public inbox for lvm2-cvs@sourceware.org help / color / mirror / Atom feed
From: agk@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2 tools/dmsetup.c libdm/libdm-common.c lib/ ... Date: Mon, 03 Aug 2009 11:20:00 -0000 [thread overview] Message-ID: <20090803112017.17487.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: agk@sourceware.org 2009-08-03 11:20:16 Modified files: tools : dmsetup.c libdm : libdm-common.c lib/activate : dev_manager.c Log message: cleanup some ignored return values & 'stack's Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/tools/dmsetup.c.diff?cvsroot=lvm2&r1=1.118&r2=1.119 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/libdm/libdm-common.c.diff?cvsroot=lvm2&r1=1.70&r2=1.71 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/activate/dev_manager.c.diff?cvsroot=lvm2&r1=1.154&r2=1.155 --- LVM2/tools/dmsetup.c 2009/07/31 17:51:46 1.118 +++ LVM2/tools/dmsetup.c 2009/08/03 11:20:15 1.119 @@ -593,7 +593,7 @@ goto out; } - dm_udev_wait(cookie); + (void) dm_udev_wait(cookie); r = 1; @@ -631,7 +631,7 @@ goto out; } - dm_udev_wait(cookie); + (void) dm_udev_wait(cookie); r = 1; --- LVM2/libdm/libdm-common.c 2009/08/03 11:01:26 1.70 +++ LVM2/libdm/libdm-common.c 2009/08/03 11:20:16 1.71 @@ -1065,7 +1065,7 @@ return 1; if (!_get_cookie_sem(cookie, &semid)) - return 0; + return_0; return _udev_notify_sem_destroy(semid, cookie); } --- LVM2/lib/activate/dev_manager.c 2009/07/31 18:30:31 1.154 +++ LVM2/lib/activate/dev_manager.c 2009/08/03 11:20:16 1.155 @@ -1133,10 +1133,11 @@ dm_tree_set_cookie(root, 0); if (!dm_tree_deactivate_children(root, uuid, strlen(uuid))) { - dm_udev_cleanup(dm_tree_get_cookie(root)); + (void) dm_udev_cleanup(dm_tree_get_cookie(root)); return_0; } - dm_udev_wait(dm_tree_get_cookie(root)); + if (!dm_udev_wait(dm_tree_get_cookie(root))) + stack; } return 1; @@ -1171,10 +1172,11 @@ /* Deactivate LV and all devices it references that nothing else has open. */ dm_tree_set_cookie(root, 0); if (!dm_tree_deactivate_children(root, dlid, ID_LEN + sizeof(UUID_PREFIX) - 1)) { - dm_udev_cleanup(dm_tree_get_cookie(root)); + (void) dm_udev_cleanup(dm_tree_get_cookie(root)); goto_out; } - dm_udev_wait(dm_tree_get_cookie(root)); + if (!dm_udev_wait(dm_tree_get_cookie(root))) + stack; if (!_remove_lv_symlinks(dm, root)) log_error("Failed to remove all device symlinks associated with %s.", lv->name); break; @@ -1195,10 +1197,11 @@ /* Preload any devices required before any suspensions */ dm_tree_set_cookie(root, 0); if (!dm_tree_preload_children(root, dlid, ID_LEN + sizeof(UUID_PREFIX) - 1)) { - dm_udev_cleanup(dm_tree_get_cookie(root)); + (void) dm_udev_cleanup(dm_tree_get_cookie(root)); goto_out; } - dm_udev_wait(dm_tree_get_cookie(root)); + if (!dm_udev_wait(dm_tree_get_cookie(root))) + stack; if (dm_tree_node_size_changed(root)) dm->flush_required = 1; @@ -1206,10 +1209,11 @@ if (action == ACTIVATE) { dm_tree_set_cookie(root, 0); if (!dm_tree_activate_children(root, dlid, ID_LEN + sizeof(UUID_PREFIX) - 1)) { - dm_udev_cleanup(dm_tree_get_cookie(root)); + (void) dm_udev_cleanup(dm_tree_get_cookie(root)); goto_out; } - dm_udev_wait(dm_tree_get_cookie(root)); + if (!dm_udev_wait(dm_tree_get_cookie(root))) + stack; } if (!_create_lv_symlinks(dm, root)) {
reply other threads:[~2009-08-03 11:20 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20090803112017.17487.qmail@sourceware.org \ --to=agk@sourceware.org \ --cc=lvm-devel@redhat.com \ --cc=lvm2-cvs@sourceware.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).