All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] Reg: sendfile01_server
@ 2014-05-06 23:17 C Nandi
  2014-05-29 14:40 ` chrubis
  0 siblings, 1 reply; 2+ messages in thread
From: C Nandi @ 2014-05-06 23:17 UTC (permalink / raw)
  To: ltp-list

[-- Attachment #1: Type: text/plain, Size: 1115 bytes --]

From "testcases/network/tcp_cmds/sendfile/sendfile01"
    53      CLIENT="testsf_c${EXEC_SUFFIX}"
    54      SERVER="testsf_s${EXEC_SUFFIX}"

SERVER can be "testsf_s" or "testfs_s6"

But, in testcases/network/tcp_cmds/sendfile/sendfile01_server, 

     1  #!/bin/sh
     2  #set -x
     3  cd $3
     4  exec 1< /dev/null       # open descriptor 1
     5  exec 2< /dev/null       # open descriptor 1
     6  nohup ./testsf_s $1 $2 &


SERVER will always execute as "testfs_s"

I guess, below changes would be good.

diff --git a/testcases/network/tcp_cmds/sendfile/sendfile01_server b/testcases/network/tcp_cmds/sendfile/sendfile01_server
index 2383ce4..9933db4 100755
--- a/testcases/network/tcp_cmds/sendfile/sendfile01_server
+++ b/testcases/network/tcp_cmds/sendfile/sendfile01_server
@@ -3,4 +3,4 @@
 cd $3
 exec 1< /dev/null       # open descriptor 1
 exec 2< /dev/null       # open descriptor 1
-nohup ./testsf_s $1 $2 &
+nohup ./$4 $1 $2 &




If you feel this is correct, please find the patch attached with this e-mail.


 
Thanks and Regards, 
CHANDRASHEKHAR NANDI

[-- Attachment #2: 0001-sendfile01_server-Correction-to-run-both-testsf_s-an.patch --]
[-- Type: application/octet-stream, Size: 939 bytes --]

From 470a3b680a9764b82a9ba5345b604f02d1eaa277 Mon Sep 17 00:00:00 2001
From: Chandrashekhar Nandi <Chandrashekhar.Nandi@mphasis.com>
Date: Tue, 6 May 2014 17:11:07 -0600
Subject: [PATCH] sendfile01_server: Correction to run both 'testsf_s' and
 'testsf_s6' Thanks to "Roberts, Lee" for pointing this out

Signed-off-by: Chandrashekhar Nandi <Chandrashekhar.Nandi@mphasis.com>
---
 testcases/network/tcp_cmds/sendfile/sendfile01_server |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testcases/network/tcp_cmds/sendfile/sendfile01_server b/testcases/network/tcp_cmds/sendfile/sendfile01_server
index 2383ce4..9933db4 100755
--- a/testcases/network/tcp_cmds/sendfile/sendfile01_server
+++ b/testcases/network/tcp_cmds/sendfile/sendfile01_server
@@ -3,4 +3,4 @@
 cd $3
 exec 1< /dev/null       # open descriptor 1
 exec 2< /dev/null       # open descriptor 1
-nohup ./testsf_s $1 $2 &
+nohup ./$4 $1 $2 &
-- 
1.7.10.4


[-- Attachment #3: Type: text/plain, Size: 346 bytes --]

------------------------------------------------------------------------------
Is your legacy SCM system holding you back? Join Perforce May 7 to find out:
&#149; 3 signs your SCM is hindering your productivity
&#149; Requirements for releasing software faster
&#149; Expert tips and advice for migrating your SCM now
http://p.sf.net/sfu/perforce

[-- Attachment #4: Type: text/plain, Size: 155 bytes --]

_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [LTP] Reg: sendfile01_server
  2014-05-06 23:17 [LTP] Reg: sendfile01_server C Nandi
@ 2014-05-29 14:40 ` chrubis
  0 siblings, 0 replies; 2+ messages in thread
From: chrubis @ 2014-05-29 14:40 UTC (permalink / raw)
  To: C Nandi; +Cc: ltp-list

Hi!
Pushed, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Time is money. Stop wasting it! Get your web API in 5 minutes.
www.restlet.com/download
http://p.sf.net/sfu/restlet
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-05-29 14:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-06 23:17 [LTP] Reg: sendfile01_server C Nandi
2014-05-29 14:40 ` chrubis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.