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 man/lvconvert.8 tools/lvchange.c Date: Wed, 10 Jan 2007 19:56:00 -0000 [thread overview] Message-ID: <20070110195639.20093.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: agk@sourceware.org 2007-01-10 19:56:39 Modified files: . : WHATS_NEW man : lvconvert.8 tools : lvchange.c Log message: Prevent permission changes on active mirrors. Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.533&r2=1.534 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/man/lvconvert.8.diff?cvsroot=lvm2&r1=1.4&r2=1.5 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/tools/lvchange.c.diff?cvsroot=lvm2&r1=1.74&r2=1.75 --- LVM2/WHATS_NEW 2007/01/10 14:13:46 1.533 +++ LVM2/WHATS_NEW 2007/01/10 19:56:38 1.534 @@ -1,7 +1,8 @@ Version 2.02.18 - ==================================== - Print warning instead of error message if cannot zero volume - Update lvconvert man page (snapshot option) + Prevent permission changes on active mirrors. + Print warning instead of error message if lvconvert cannot zero volume. + Add snapshot options to lvconvert man page. dumpconfig accepts a list of configuration variables to display. Change dumpconfig to use --file to redirect output to a file. Avoid vgreduce error when mirror code removes the log LV. --- LVM2/man/lvconvert.8 2007/01/10 14:13:46 1.4 +++ LVM2/man/lvconvert.8 2007/01/10 19:56:38 1.5 @@ -58,7 +58,7 @@ .TP .I \-Z, \-\-zero y/n Controls zeroing of the first KB of data in the snapshot. -If the volume is read only snapshot will not be zeroed. +If the volume is read-only the snapshot will not be zeroed. .br .SH Examples "lvconvert -m1 vg00/lvol1" --- LVM2/tools/lvchange.c 2006/11/03 21:07:15 1.74 +++ LVM2/tools/lvchange.c 2007/01/10 19:56:39 1.75 @@ -19,6 +19,7 @@ struct logical_volume *lv) { uint32_t lv_access; + struct lvinfo info; lv_access = arg_uint_value(cmd, permission_ARG, 0); @@ -34,6 +35,13 @@ return 0; } + if ((lv->status & MIRRORED) && (lv->vg->status & CLUSTERED) && + lv_info(cmd, lv, &info, 0) && info.exists) { + log_error("Cannot change permissions of mirror \"%s\" " + "while active.", lv->name); + return 0; + } + if (lv_access & LVM_WRITE) { lv->status |= LVM_WRITE; log_verbose("Setting logical volume \"%s\" read/write",
reply other threads:[~2007-01-10 19:56 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=20070110195639.20093.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).