From: Ram Pai <linuxram@us.ibm.com>
To: Tetsuo Handa <from-linux-fsdevel@I-love.SAKURA.ne.jp>
Cc: Avantika Mathur <mathur@linux.ibm.com>, linux-fsdevel@vger.kernel.org
Subject: Re: Can I download testing programs for "shared subtree"?
Date: Fri, 17 Mar 2006 12:38:38 -0800 [thread overview]
Message-ID: <1142627918.4809.141.camel@localhost> (raw)
In-Reply-To: <200603172019.BBI87068.WFtOVStNMGMPFETGYJ@I-love.SAKURA.ne.jp>
On Fri, 2006-03-17 at 20:19 +0900, Tetsuo Handa wrote:
> Hello.
>
> > I noticed that there are a couple of mount requests that are rejected by my patch.
> > May be the requested directories didn't exist.
> > I will report the cause if I could find it.
If you are seeing some warning and missing directories with your patch,
which you did not see without the patch, you will have to analyse the
behavior carefully.
That said the testcases needs fixing too. It leaves too many mounts
around which needs fixing.
rbind/test08 has issues for sure which needs to be analysed. and there
is one more move/test14 test case which needs to be analysed.
Avantika developed these testcases. Avantika, can you help us with this?
Thanks,
RP
>
> I made my kernel to report mount requests on nonexistent directory.
> The following is the change I made to the testing programs.
> Are these missing directories intended?
>
> ----- Start of patch -----
> diff -ur TESTSUITE.org/testcases/move/test05 TESTSUITE/testcases/move/test05
> --- TESTSUITE.org/testcases/move/test05 2005-09-27 07:10:32.000000000 +0900
> +++ TESTSUITE/testcases/move/test05 2006-03-17 11:40:28.000000000 +0900
> @@ -56,6 +56,7 @@
> check -n dir/grandchild parent2/child2/grandchild
> check -n share1/grandchild parent2/child2/grandchild
>
> +mkdir -p share2/grandchild/a
> mount --bind $disk2 share2/grandchild/a
>
> check parent2/child2/grandchild/a share2/child2/grandchild/a
> diff -ur TESTSUITE.org/testcases/move/test09 TESTSUITE/testcases/move/test09
> --- TESTSUITE.org/testcases/move/test09 2005-09-27 07:10:32.000000000 +0900
> +++ TESTSUITE/testcases/move/test09 2006-03-17 11:36:36.000000000 +0900
> @@ -57,6 +57,7 @@
> check -n dir/grandchild parent2/child2/grandchild
> check -n share1/grandchild parent2/child2/grandchild
>
> +mkdir -p share2/grandchild/a
> mount --bind $disk2 share2/grandchild/a
>
> check parent2/child2/grandchild/a share2/child2/grandchild/a
> diff -ur TESTSUITE.org/testcases/move/test21 TESTSUITE/testcases/move/test21
> --- TESTSUITE.org/testcases/move/test21 2005-09-27 07:10:32.000000000 +0900
> +++ TESTSUITE/testcases/move/test21 2006-03-17 11:37:18.000000000 +0900
> @@ -55,6 +55,7 @@
>
> check parent2/b parent2/a/b parent2/a/a/b
>
> +mkdir -p parent1/a/c
> mount --bind $disk3 parent1/a/c
>
> check parent2/c parent2/a/c parent2/a/a/c
> diff -ur TESTSUITE.org/testcases/rbind/test08 TESTSUITE/testcases/rbind/test08
> --- TESTSUITE.org/testcases/rbind/test08 2005-09-27 07:10:32.000000000 +0900
> +++ TESTSUITE/testcases/rbind/test08 2006-03-17 11:48:28.000000000 +0900
> @@ -51,16 +51,18 @@
>
> check -n parent1/child1 share1/child1
>
> -mkdir parent2/child2
> +mkdir -p parent2/child2
>
> mount --rbind parent1/child1 parent2/child2
>
> +mkdir -p parent2/child2
> check parent1/child1 parent2/child2
>
> mount --rbind $disk2 parent1/child1/a
>
> check -n parent1/child1/a parent2/child2/a
>
> +mkdir -p parent2/child2/b
> mount --rbind $disk3 parent2/child2/b
>
> check -n parent1/child1/b parent2/child2/b
> ----- End of patch -----
>
> Although I made directories as above, I couldn't suppress the following two cases.
>
> /root/TESTSUITE/testcases/rbind/test08 /bin/mount requested mount on nonexistent parent2/child2
> /root/TESTSUITE/testcases/rbind/test08 /bin/mount requested mount on nonexistent parent2/child2/b
>
> Also, I had to run the following command for several times whenever I ran the testing programs.
>
> grep TESTSUITE /proc/mounts | awk ' { print $2 } ' | xargs umount --
>
> Thank you.
prev parent reply other threads:[~2006-03-17 20:38 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-13 12:56 Can I download testing programs for "shared subtree"? Tetsuo Handa
2006-03-13 22:12 ` Ram Pai
2006-03-14 11:15 ` Tetsuo Handa
2006-03-17 11:19 ` Tetsuo Handa
2006-03-17 20:38 ` Ram Pai [this message]
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=1142627918.4809.141.camel@localhost \
--to=linuxram@us.ibm.com \
--cc=from-linux-fsdevel@I-love.SAKURA.ne.jp \
--cc=linux-fsdevel@vger.kernel.org \
--cc=mathur@linux.ibm.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;
as well as URLs for NNTP newsgroup(s).