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 test/lib/aux.sh Date: Thu, 30 Jun 2011 09:15:00 -0000 [thread overview] Message-ID: <20110630091553.19618.qmail@sourceware.org> (raw) CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: agk@sourceware.org 2011-06-30 09:15:53 Modified files: . : WHATS_NEW test/lib : aux.sh Log message: teardown stray filesystems too Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.2031&r2=1.2032 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/test/lib/aux.sh.diff?cvsroot=lvm2&r1=1.23&r2=1.24 --- LVM2/WHATS_NEW 2011/06/29 17:05:53 1.2031 +++ LVM2/WHATS_NEW 2011/06/30 09:15:53 1.2032 @@ -1,5 +1,6 @@ Version 2.02.86 - ================================= + Teardown any stray devices with $COMMON_PREFIX during test runs. Reinstate correct permissions when creating mirrors. [2.02.85] Append 'm' attribute to pv_attr for missing PVs. Annotate CLVMD_CMD_SYNC_NAMES in decode_cmd. --- LVM2/test/lib/aux.sh 2011/06/30 09:08:22 1.23 +++ LVM2/test/lib/aux.sh 2011/06/30 09:15:53 1.24 @@ -90,34 +90,47 @@ rm -f DEVICES # devs is set in prepare_devs() rm -f LOOP - # Try to remove test devices - # resume any linears to be sure we do not deadlock - STRAY_DEVS=$(dmsetup table | grep linear | sed 's/:.*//' | grep $COMMON_PREFIX | cut -d' ' -f 1) - for dm in $STRAY_DEVS ; do - # FIXME: only those really suspended - dmsetup resume $dm 2>/dev/null - done - - # Now try to remove devices - finish=0 - while [ $finish -eq 0 ] ; do - finish=1 + # Attempt to remove any loop devices that failed to get torn down if earlier tests aborted + test -n "$COMMON_PREFIX" && { + # Resume any linears to be sure we do not deadlock STRAY_DEVS=$(dmsetup table | sed 's/:.*//' | grep $COMMON_PREFIX | cut -d' ' -f 1) for dm in $STRAY_DEVS ; do - echo "Trying to remove stalled $dm" - dmsetup remove $dm 2>/dev/null - # Sucessful remove means repeat the loop once more - [ $? -eq 0 ] && finish=0 + # FIXME: only those really suspended + echo dmsetup resume $dm + dmsetup resume $dm || true done - done - # Remove any loop devices that failed to get torn down if earlier tests aborted - STRAY_LOOPS=`losetup -a | grep $COMMON_PREFIX | cut -d: -f1` - if test -n "$STRAY_LOOPS"; then - echo "Removing stray loop devices containing $COMMON_PREFIX:" - losetup -a | grep $COMMON_PREFIX - losetup -d $STRAY_LOOPS 2>/dev/null || true - fi + STRAY_MOUNTS=`mount | grep $COMMON_PREFIX | cut -d\ -f1` + if test -n "$STRAY_MOUNTS"; then + echo "Removing stray mounted devices containing $COMMON_PREFIX:" + mount | grep $COMMON_PREFIX + umount -fl $STRAY_MOUNTS || true + sleep 2 + fi + + init_udev_transaction + NUM_REMAINING_DEVS=999 + while NUM_DEVS=`dmsetup table | grep ^$COMMON_PREFIX | wc -l` && \ + test $NUM_DEVS -lt $NUM_REMAINING_DEVS -a $NUM_DEVS -ne 0; do + echo "Removing $NUM_DEVS stray mapped devices with names beginning with $COMMON_PREFIX:" + STRAY_DEVS=$(dmsetup table | sed 's/:.*//' | grep $COMMON_PREFIX | cut -d' ' -f 1) + dmsetup info -c | grep ^$COMMON_PREFIX + for dm in $STRAY_DEVS ; do + echo dmsetup remove $dm + dmsetup remove $dm || true + done + NUM_REMAINING_DEVS=$NUM_DEVS + done + finish_udev_transaction + udev_wait + + STRAY_LOOPS=`losetup -a | grep $COMMON_PREFIX | cut -d: -f1` + if test -n "$STRAY_LOOPS"; then + echo "Removing stray loop devices containing $COMMON_PREFIX:" + losetup -a | grep $COMMON_PREFIX + losetup -d $STRAY_LOOPS || true + fi + } } teardown() {
reply other threads:[~2011-06-30 9:15 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=20110630091553.19618.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).