From: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
To: Mike Frysinger <vapier@gentoo.org>, ltp-list@lists.sourceforge.net
Cc: vasily.isaenko@oracle.com
Subject: Re: [LTP] [PATCH 2/3] isofs.sh: be more quiet
Date: Fri, 21 Mar 2014 17:43:29 +0400 [thread overview]
Message-ID: <532C4201.3030605@oracle.com> (raw)
In-Reply-To: <532C1813.9060609@oracle.com>
On 03/21/2014 02:44 PM, Stanislav Kholmanskikh wrote:
>
> I think that according to the patch the described behaviour is
> absolutely expected. Therefore It would be better to leave "ls -lR" in
> the test or insert a 'sleep' instead. In this case we would use no more
> than 2 loop devices per iteration.
>
I suppose it would be a better solution (and without any waitings):
loop_dev=$(losetup --show -f isofs.iso)
# do mount and other stuff
# do umount
while /bin/true; do
out=$(losetup -d "$loop_dev" 2>&1)
if echo "$out" | grep -q -i "No such device"; then
break
fi
done
This way we doesn't depend on asynchrony in the loop-driver code.
> Guys, what do you think?
>
------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their
applications. Written by three acclaimed leaders in the field,
this first edition is now available. Download your free book today!
http://p.sf.net/sfu/13534_NeoTech
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next prev parent reply other threads:[~2014-03-21 13:43 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-13 12:38 [LTP] Let's use isofs.sh as an ordinary test case Stanislav Kholmanskikh
2014-03-13 12:38 ` [LTP] [PATCH 1/3] isofs.sh: use $TMPDIR for temporary directory Stanislav Kholmanskikh
2014-03-13 20:55 ` Mike Frysinger
2014-03-17 14:39 ` Stanislav Kholmanskikh
2014-03-13 12:38 ` [LTP] [PATCH 2/3] isofs.sh: be more quiet Stanislav Kholmanskikh
2014-03-13 20:56 ` Mike Frysinger
2014-03-17 14:29 ` Stanislav Kholmanskikh
2014-03-21 10:44 ` Stanislav Kholmanskikh
2014-03-21 13:43 ` Stanislav Kholmanskikh [this message]
2014-03-24 10:04 ` [LTP] [PATCH V2 1/4] testscripts/isofs.sh: cleanup Stanislav Kholmanskikh
2014-03-24 10:04 ` [LTP] [PATCH V2 2/4] testscripts/isofs.sh: use the new shell interface Stanislav Kholmanskikh
2014-03-24 10:04 ` [LTP] [PATCH V2 3/4] testscripts/isofs.sh: removed some "unnecessary" steps Stanislav Kholmanskikh
2014-03-24 10:04 ` [LTP] [PATCH V2 4/4] Declare an iso9660 fs testcase based on isofs.sh Stanislav Kholmanskikh
2014-03-27 6:15 ` [LTP] [PATCH 2/3] isofs.sh: be more quiet Mike Frysinger
2014-03-27 14:25 ` chrubis
2014-03-13 12:38 ` [LTP] [RFC PATCH 3/3] Declare iso9660 fs testcase based on isofs.sh Stanislav Kholmanskikh
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=532C4201.3030605@oracle.com \
--to=stanislav.kholmanskikh@oracle.com \
--cc=ltp-list@lists.sourceforge.net \
--cc=vapier@gentoo.org \
--cc=vasily.isaenko@oracle.com \
/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: link
Be 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