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 ./WHATS_NEW lib/activate/activate.c Date: Fri, 30 Oct 2009 13:07:00 -0000 [thread overview] Message-ID: <20091030130750.365.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: agk@sourceware.org 2009-10-30 13:07:49 Modified files: . : WHATS_NEW lib/activate : activate.c Log message: Insert some missing stack macros into activation code. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1308&r2=1.1309 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/activate/activate.c.diff?cvsroot=lvm2&r1=1.157&r2=1.158 --- LVM2/WHATS_NEW 2009/10/27 17:00:44 1.1308 +++ LVM2/WHATS_NEW 2009/10/30 13:07:49 1.1309 @@ -1,5 +1,6 @@ Version 2.02.55 - =================================== + Insert some missing stack macros into activation code. Recognise DRBD device part and handle it similar to MD devices. Version 2.02.54 - 26th October 2009 --- LVM2/lib/activate/activate.c 2009/10/16 17:41:50 1.157 +++ LVM2/lib/activate/activate.c 2009/10/30 13:07:49 1.158 @@ -936,7 +936,7 @@ return 1; if (!(lv = lv_from_lvid(cmd, lvid_s, 0))) - goto out; + goto_out; if (test_mode()) { _skip("Resuming '%s'.", lv->name); @@ -949,11 +949,11 @@ if (!info.exists || !info.suspended) { r = error_if_not_active ? 0 : 1; - goto out; + goto_out; } if (!_lv_activate_lv(lv)) - goto out; + goto_out; memlock_dec(); fs_unlock(); @@ -1138,7 +1138,8 @@ lv->status |= ACTIVATE_EXCL; memlock_inc(); - r = _lv_activate_lv(lv); + if (!(r = _lv_activate_lv(lv))) + stack; memlock_dec(); fs_unlock(); @@ -1155,13 +1156,19 @@ /* Activate LV */ int lv_activate(struct cmd_context *cmd, const char *lvid_s, int exclusive) { - return _lv_activate(cmd, lvid_s, exclusive, 0); + if (!_lv_activate(cmd, lvid_s, exclusive, 0)) + return_0; + + return 1; } /* Activate LV only if it passes filter */ int lv_activate_with_filter(struct cmd_context *cmd, const char *lvid_s, int exclusive) { - return _lv_activate(cmd, lvid_s, exclusive, 1); + if (!_lv_activate(cmd, lvid_s, exclusive, 1)) + return_0; + + return 1; } int lv_mknodes(struct cmd_context *cmd, const struct logical_volume *lv)
next reply other threads:[~2009-10-30 13:07 UTC|newest] Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-10-30 13:07 agk [this message] -- strict thread matches above, loose matches on Subject: below -- 2012-01-25 22:16 zkabelac 2011-02-18 14:26 zkabelac 2010-08-16 23:29 agk 2010-07-12 11:37 prajnoha 2010-05-21 14:34 zkabelac 2010-01-27 13:24 mbroz 2009-12-03 19:23 mbroz 2009-12-01 19:10 mbroz 2009-09-29 18:50 agk 2009-09-29 15:17 mbroz 2007-08-01 20:29 jbrassow 2006-12-20 16:19 agk 2005-12-22 16:13 agk 2004-05-11 17:09 agk
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=20091030130750.365.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).