On Wed, 2007-11-14 at 08:25 -0600, Kumar Gala wrote:
> I'm running through the testcases that are part of syscall suite at  
> this point. I think we should work on cleaning up the others.  Having  
> control over where temp files are created seems like a good thing.

You are welcome to do all the necessary cleanups you feel is necessary.

> 
> The issue with splice01 & tee01 is that you can't do certain form of  
> splice on a NFS root fs.  We could add a check to see if the test  
> files are going to be generated on a NFS fs and if so warn/not run  
> the test.
> 
> - k
> 
> On Nov 14, 2007, at 6:55 AM, Subrata Modak wrote:
> 
> > This Seems Fine with me. Applied too. But what about other hundreds of
> > instances in other testcases ??
> >
> > --Subrata--
> >
> > On Wed, 2007-11-14 at 00:34 -0600, Kumar Gala wrote:
> >> Create testfile in tmpdir to be consistent with other tests.
> >>
> >> The makes the behavior consistent with other testcases if /tmp is a
> >> different fs than where the testcase is run from.
> >>
> >> --- ltp-full-20071031.orig/testcases/kernel/syscalls/splice/ 
> >> splice01.c 2007-11-02 03:34:57.000000000 -0500
> >> +++ ltp-full-20071031/testcases/kernel/syscalls/splice/splice01.c   
> >> 2007-11-14 00:09:39.000000000 -0600
> >> @@ -150,6 +150,9 @@
> >>    int i, len;
> >>    int fd_in, fd_out;
> >>
> >> +  /* Make a temp directory and cd to it */
> >> +  tst_tmpdir();
> >> +
> >>    for (i = 0; i < SPLICE_TEST_BLOCK_SIZE; i++) {
> >>            buffer[i] = i & 0xff;
> >>    }
> >> @@ -260,6 +263,9 @@
> >>     */
> >>    TEST_CLEANUP;
> >>
> >> +  /* Remove tmp dir and all files in it */
> >> +  tst_rmdir();
> >> +
> >>    /* exit with return code appropriate for results */
> >>    tst_exit();
> >>  }                         /* End cleanup() */
> >> --- ltp-full-20071031.orig/testcases/kernel/syscalls/tee/tee01.c    
> >> 2007-11-02 03:34:57.000000000 -0500
> >> +++ ltp-full-20071031/testcases/kernel/syscalls/tee/tee01.c         
> >> 2007-11-14 00:10:49.000000000 -0600
> >> @@ -270,6 +270,9 @@
> >>   */
> >>  void setup()
> >>  {
> >> +  /* Make a temp directory and cd to it */
> >> +  tst_tmpdir();
> >> +
> >>    /* Initialize test file names */
> >>    sprintf(testfile1, "teetest%d_1.txt", getpid());
> >>    sprintf(testfile2, "teetest%d_2.txt", getpid());
> >> @@ -297,6 +300,9 @@
> >>     */
> >>    TEST_CLEANUP;
> >>
> >> +  /* Remove tmp dir and all files in it */
> >> +  tst_rmdir();
> >> +
> >>    /* exit with return code appropriate for results */
> >>    tst_exit();
> >>  }                         /* End cleanup() */
> >>
> >> --------------------------------------------------------------------- 
> >> ----
> >> This SF.net email is sponsored by: Splunk Inc.
> >> Still grepping through log files to find problems?  Stop.
> >> Now Search log events and configuration files using AJAX and a  
> >> browser.
> >> Download your FREE copy of Splunk now >> http://get.splunk.com/
> >> _______________________________________________
> >> Ltp-list mailing list
> >> [email protected]
> >> https://lists.sourceforge.net/lists/listinfo/ltp-list
> 


-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to