public inbox for lvm2-cvs@sourceware.org help / color / mirror / Atom feed
From: mbroz@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2 ./WHATS_NEW daemons/clvmd/lvm-functions.c ... Date: Mon, 23 Nov 2009 10:44:00 -0000 [thread overview] Message-ID: <20091123104452.1396.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: mbroz@sourceware.org 2009-11-23 10:44:51 Modified files: . : WHATS_NEW daemons/clvmd : lvm-functions.c lib/metadata : metadata-exported.h metadata.c Log message: Revert vg_read_internal change, clvmd cannot use vg_read now. (2.02.55) Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1320&r2=1.1321 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/daemons/clvmd/lvm-functions.c.diff?cvsroot=lvm2&r1=1.71&r2=1.72 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/metadata/metadata-exported.h.diff?cvsroot=lvm2&r1=1.122&r2=1.123 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/metadata/metadata.c.diff?cvsroot=lvm2&r1=1.297&r2=1.298 --- LVM2/WHATS_NEW 2009/11/19 19:53:58 1.1320 +++ LVM2/WHATS_NEW 2009/11/23 10:44:50 1.1321 @@ -1,5 +1,6 @@ Version 2.02.56 - ==================================== + Revert vg_read_internal change, clvmd cannot use vg_read now. (2.02.55) Version 2.02.55 - 19th November 2009 ==================================== --- LVM2/daemons/clvmd/lvm-functions.c 2009/11/19 12:13:37 1.71 +++ LVM2/daemons/clvmd/lvm-functions.c 2009/11/23 10:44:51 1.72 @@ -774,14 +774,15 @@ void lvm_do_backup(const char *vgname) { struct volume_group * vg; + int consistent = 0; DEBUGLOG("Triggering backup of VG metadata for %s. suspended=%d\n", vgname, suspended); pthread_mutex_lock(&lvm_lock); - vg = vg_read(cmd, vgname, NULL /*vgid*/, 0 /*flags*/); + vg = vg_read_internal(cmd, vgname, NULL /*vgid*/, &consistent); - if (!vg_read_error(vg)) + if (vg && consistent) check_current_backup(vg); else log_error("Error backing up metadata, can't find VG for group %s", vgname); --- LVM2/lib/metadata/metadata-exported.h 2009/11/19 12:13:37 1.122 +++ LVM2/lib/metadata/metadata-exported.h 2009/11/23 10:44:51 1.123 @@ -380,6 +380,8 @@ int vg_write(struct volume_group *vg); int vg_commit(struct volume_group *vg); int vg_revert(struct volume_group *vg); +struct volume_group *vg_read_internal(struct cmd_context *cmd, const char *vg_name, + const char *vgid, int *consistent); struct physical_volume *pv_read(struct cmd_context *cmd, const char *pv_name, struct dm_list *mdas, uint64_t *label_sector, int warnings, int scan_label_only); @@ -470,7 +472,7 @@ force_t force); /* * vg_release() must be called on every struct volume_group allocated - * by vg_create() or vg_read() to free it when no longer required. + * by vg_create() or vg_read_internal() to free it when no longer required. */ void vg_release(struct volume_group *vg); --- LVM2/lib/metadata/metadata.c 2009/11/19 13:44:37 1.297 +++ LVM2/lib/metadata/metadata.c 2009/11/23 10:44:51 1.298 @@ -58,10 +58,6 @@ static uint32_t _vg_bad_status_bits(const struct volume_group *vg, uint32_t status); -static struct volume_group *_vg_read_internal(struct cmd_context *cmd, - const char *vgname, - const char *vgid, int *consistent); - const char _really_init[] = "Really INITIALIZE physical volume \"%s\" of volume group \"%s\" [y/n]? "; @@ -288,7 +284,7 @@ struct pv_list *pvl; int r = 0, consistent = 0; - if (!(vg = _vg_read_internal(fmt->cmd, vg_name, vgid, &consistent))) { + if (!(vg = vg_read_internal(fmt->cmd, vg_name, vgid, &consistent))) { log_error("get_pv_from_vg_by_id: vg_read_internal failed to read VG %s", vg_name); return 0; @@ -790,10 +786,10 @@ /* NOTE: let caller decide - this may be check for existence */ return _vg_make_handle(cmd, NULL, rc); - /* FIXME: Is this _vg_read_internal necessary? Move it inside + /* FIXME: Is this vg_read_internal necessary? Move it inside vg_lock_newname? */ /* is this vg name already in use ? */ - if ((vg = _vg_read_internal(cmd, vg_name, NULL, &consistent))) { + if ((vg = vg_read_internal(cmd, vg_name, NULL, &consistent))) { log_error("A volume group called '%s' already exists.", vg_name); unlock_and_release_vg(cmd, vg, vg_name); return _vg_make_handle(cmd, NULL, FAILED_EXIST); @@ -2494,7 +2490,7 @@ if (is_orphan_vg(vgname)) { if (use_precommitted) { - log_error("Internal error: _vg_read requires vgname " + log_error("Internal error: vg_read_internal requires vgname " "with pre-commit."); return NULL; } @@ -2783,9 +2779,8 @@ return correct_vg; } -static struct volume_group *_vg_read_internal(struct cmd_context *cmd, - const char *vgname, - const char *vgid, int *consistent) +struct volume_group *vg_read_internal(struct cmd_context *cmd, const char *vgname, + const char *vgid, int *consistent) { struct volume_group *vg; struct lv_list *lvl; @@ -3050,7 +3045,7 @@ stack; continue; } - if (!(vg = _vg_read_internal(cmd, vgname, vgid, &consistent))) { + if (!(vg = vg_read_internal(cmd, vgname, vgid, &consistent))) { stack; continue; } @@ -3284,7 +3279,7 @@ if (!lock_vol(cmd, lock_name, lock_flags)) return_NULL; - if (!(vg = _vg_read_internal(cmd, vg_name, vgid, &consistent))) + if (!(vg = vg_read_internal(cmd, vg_name, vgid, &consistent))) return_NULL; if (!consistent) { @@ -3341,7 +3336,7 @@ consistent_in = consistent; /* If consistent == 1, we get NULL here if correction fails. */ - if (!(vg = _vg_read_internal(cmd, vg_name, vgid, &consistent))) { + if (!(vg = vg_read_internal(cmd, vg_name, vgid, &consistent))) { if (consistent_in && !consistent) { log_error("Volume group \"%s\" inconsistent.", vg_name); failure |= FAILED_INCONSISTENT;
next reply other threads:[~2009-11-23 10:44 UTC|newest] Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-11-23 10:44 mbroz [this message] -- strict thread matches above, loose matches on Subject: below -- 2012-01-20 0:27 jbrassow 2011-12-08 21:24 agk 2011-09-27 22:43 agk 2011-08-10 20:25 zkabelac 2011-02-18 14:16 zkabelac 2011-02-18 0:36 jbrassow 2011-02-04 20:30 jbrassow 2011-02-03 16:03 zkabelac 2011-02-03 1:58 zkabelac 2010-12-08 20:51 agk 2010-11-23 1:56 agk 2010-03-26 15:40 snitzer 2010-03-23 22:30 snitzer 2010-01-19 13:25 mbroz 2010-01-05 16:09 mbroz 2010-01-05 16:06 mbroz 2010-01-05 16:03 mbroz 2009-07-24 18:15 agk 2009-07-16 0:37 agk 2009-07-15 23:57 agk 2009-07-13 19:49 agk 2009-06-12 8:30 mbroz 2009-02-22 21:14 agk 2009-01-26 19:01 agk 2008-09-19 6:42 agk 2007-08-07 9:06 meyering 2007-01-25 14:37 agk 2007-01-23 15:58 agk 2007-01-19 22:21 agk 2006-05-11 19:05 agk 2006-03-09 22:34 agk 2005-08-14 23:18 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=20091123104452.1396.qmail@sourceware.org \ --to=mbroz@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).