On Mon, 2008-05-19 at 14:01 +0530, Subrata Modak wrote:
> On Thu, 2008-05-15 at 17:59 +0530, Subrata Modak wrote:
> > On Thu, 2008-05-15 at 15:58 +0400, Pavel Emelyanov wrote:
> > > Pavel Emelyanov wrote:
> > > > Subrata Modak wrote:
> > > >> Hi Pavel,
> > > >>
> > > >> We have PID, SYSVIPC and UTSNAMESPACE test cases written recently in
> > > >> LTP
> > > >> (Linux Test Project) and used by people for their namespace testing.
> > > >> Would
> > > >> you like to use them and review them for us, telling about itÅ›
> > > >> effectiveness
> > > >> and other stuff:
> > >
> > > Well, current tests look sane, but there's something that caught my
> > > eyes immediately.
> > >
> >
> > Thanks Pavel for this immediate review.
> >
> > Rishi,
> >
> > Can you please look in to this when you have time for this ?
>
> Rishi,
>
> Would you like to look at it once.
>
> Regards--
> Subrata
>
> >
> > Regards--
> > Subrata
> >
> > >
> > > 1) The very important thing that has to be checked, but is missed is
> > > that parent from init_pid_ns can waitpid (not just wait) a child
> > > from a sub namespace by its pid returned from clone. It will be *not*
> > > 1, but some number above it.
> > >
> > > I.e. test like this:
> > >
> > > int child(void *x)
> > > {
> > > return 0;
> > > }
> > >
> > > int main(void)
> > > {
> > > int pid;
> > >
> > > pid = clone(child, CLONE_NEWPID, stack, NULL);
> > >
> > > if (waitpid(pid) == -1 && errno == ESRCH)
> > > /* FAIL - pid must exist */
> > > else
> > > /* Either we're OK, or some other bad
> > > thing (but not pid-ns-related) happened */
> > > }
> > >
> > >
> > > 2) The other thing to check (also very important) is that child from
> > > newly created namespace cannot see its parent from the init one by this
> > > parent's pid.
> > >
> > > I.e. smth like this
> > >
> > > int child(void *x)
> > > {
> > > int ppid = (int)x;
> > >
> > > if (kill(ppid, 0) == -1 && errno == ESRCH)
> > > /* PASS - parent is invisible */
> > > else
> > > /* FAIL - parent is visible (but maybe
> > > just not enough perms to kill one */
> > > return 0;
> > > }
> > >
> > > int main(void)
> > > {
> > > int pid;
> > >
> > > pid = clone(child, CLONE_NEWPID, stack, (void *)getpid());
> > >
> > > wait();
> > > }
> > >
> > >
> > > I will try to produce more feedback in the future :)
Hi Pavel,
Did you get a chance to generate more detailed feedback on the entire
set of namespace test cases in LTP.
Regards--
Subrata
> > >
> > > >> http://ltp.cvs.sourceforge.net/ltp/ltp/testcases/kernel/containers/
> > > >
> > > > I will try to take a look early.
> > > >
> > > > Thanks,
> > > > Pavel
> > > >
> > >
> > >
> > > -------------------------------------------------------------------------
> > > This SF.net email is sponsored by: Microsoft
> > > Defy all challenges. Microsoft(R) Visual Studio 2008.
> > > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> > > _______________________________________________
> > > Ltp-list mailing list
> > > [email protected]
> > > https://lists.sourceforge.net/lists/listinfo/ltp-list
> >
> >
> > -------------------------------------------------------------------------
> > This SF.net email is sponsored by: Microsoft
> > Defy all challenges. Microsoft(R) Visual Studio 2008.
> > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> > _______________________________________________
> > Ltp-list mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/ltp-list
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> Ltp-list mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ltp-list
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list