public inbox for lvm2-cvs@sourceware.org help / color / mirror / Atom feed
From: mornfall@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2 lib/metadata/lv_manip.c test/t-lvcreate-s ... Date: Fri, 16 Sep 2011 09:59:00 -0000 [thread overview] Message-ID: <20110916095943.957.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: mornfall@sourceware.org 2011-09-16 09:59:43 Modified files: lib/metadata : lv_manip.c Added files: test : t-lvcreate-striped-mirror.sh Log message: Fix the divisibility check in the allocator for the mirror+stripe case (require divisibility by stripe count alone, not by (mirror*stripe)). Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/lib/metadata/lv_manip.c.diff?cvsroot=lvm2&r1=1.285&r2=1.286 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/test/t-lvcreate-striped-mirror.sh.diff?cvsroot=lvm2&r1=NONE&r2=1.1 --- LVM2/lib/metadata/lv_manip.c 2011/09/14 14:20:16 1.285 +++ LVM2/lib/metadata/lv_manip.c 2011/09/16 09:59:42 1.286 @@ -1917,10 +1917,11 @@ return 1; } - if (ah->area_multiple > 1 && - (ah->new_extents - alloc_state.allocated) % ah->area_count) { + if (ah->area_multiple > 1 && + (ah->new_extents - alloc_state.allocated) % ah->area_multiple) { log_error("Number of extents requested (%d) needs to be divisible by %d.", - ah->new_extents - alloc_state.allocated, ah->area_count); + ah->new_extents - alloc_state.allocated, + ah->area_multiple); return 0; } /cvs/lvm2/LVM2/test/t-lvcreate-striped-mirror.sh,v --> standard output revision 1.1 --- LVM2/test/t-lvcreate-striped-mirror.sh +++ - 2011-09-16 09:59:43.392905000 +0000 @@ -0,0 +1,65 @@ +#!/bin/sh +# Copyright (C) 2010 Red Hat, Inc. All rights reserved. +# +# This copyrighted material is made available to anyone wishing to use, +# modify, copy, or redistribute it subject to the terms and conditions +# of the GNU General Public License v.2. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software Foundation, +# Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + +. lib/test +aux prepare_vg 9 + +lvcreate -i2 -l2 -m1 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +check mirror_images_redundant $vg $lv1 +lvremove -ff $vg + +lvcreate -i2 -l4 -m1 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +check mirror_images_redundant $vg $lv1 +lvremove -ff $vg + +lvcreate -i3 -l3 -m1 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +check mirror_images_redundant $vg $lv1 +lvremove -ff $vg + +lvcreate -i4 -l4 -m1 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +check mirror_images_redundant $vg $lv1 +lvremove -ff $vg + + +lvcreate -i2 -l2 -m2 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +check mirror_images_redundant $vg $lv1 +lvremove -ff $vg + +lvcreate -i3 -l3 -m2 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +check mirror_images_redundant $vg $lv1 +lvremove -ff $vg + +lvcreate -i2 -l2 -m3 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +check mirror_images_redundant $vg $lv1 +lvremove -ff $vg + +lvcreate -i3 -l2 -m2 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +grep "Rounding size (2 extents) up to .* (3 extents)" log +lvremove -ff $vg + +lvcreate -i3 -l4 -m2 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +grep "Rounding size (4 extents) up to .* (6 extents)" log +lvremove -ff $vg + +lvcreate -i3 -l4 -m1 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +grep "Rounding size (4 extents) up to .* (6 extents)" log +lvremove -ff $vg + +lvcreate -i4 -l4 -m1 --mirrorlog core -n $lv1 $vg 2>&1 | tee log +not grep "Rounding" log +lvremove -ff $vg
next reply other threads:[~2011-09-16 9:59 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-09-16 9:59 mornfall [this message] -- strict thread matches above, loose matches on Subject: below -- 2010-10-13 13:52 mornfall
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=20110916095943.957.qmail@sourceware.org \ --to=mornfall@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).