syscalls/{getcontext, profil}: Move test.h header and TCID variable to the top

Hi, please review the attached patch

markos

From 12fe59ab99a573210f38ca6c40b41aa68504ad37 Mon Sep 17 00:00:00 2001
From: Markos Chandras <[email protected]>
Date: Fri, 18 Nov 2011 16:08:21 +0000
Subject: [PATCH 12/21] syscalls/{getcontext,profil}: Move test.h header and TCID variable to the top

The tst_brkm() function is used independently of UCLIBC macro so they need to be defined outside of #ifdef statement

---
 .../kernel/syscalls/getcontext/getcontext01.c      |    8 +++++---
 testcases/kernel/syscalls/profil/profil01.c        |    8 +++++---
 2 files changed, 10 insertions(+), 6 deletions(-)

index 7cadc67..aea0b1d 100644
@@ -34,6 +34,10 @@
  **********************************************************/
 
 #include <features.h>
+#include "test.h"
+
+char *TCID = "getcontext01";	/* Test program identifier.    */
+
 #if !defined(__UCLIBC__)
 
 #include <stdio.h>

@@ -41,13 +45,11 @@
 #include <errno.h>
 #include <ucontext.h>
 
-#include "test.h"
 #include "usctest.h"
 
 void setup();
 void cleanup();
 
-char *TCID = "getcontext01";	/* Test program identifier.    */
 int TST_TOTAL = 1;		/* Total number of test cases. */
 
 int exp_enos[] = { 0 };		/* must be a 0 terminated list */

@@ -102,4 +104,4 @@ int main()
 {
 	tst_brkm(TCONF, NULL, "system doesn't have getcontext support");
 }
-#endif
\ No newline at end of file
+#endif

index 63bed83..1af7e5d 100644
@@ -41,6 +41,10 @@
 #endif
 
 #include <features.h>
+#include "test.h"
+
+char *TCID = "profil01";
+
 #ifndef __UCLIBC__
 
 #ifdef __arm__

@@ -58,7 +62,6 @@
 #include <errno.h>
 #include <sys/types.h>
 #include <sys/wait.h>
-#include "test.h"
 #include "usctest.h"
 #define FAILED 0
 #define PASSED 1

@@ -73,7 +76,6 @@ volatile int t_flag;
 //char progname[]= "profil1()";
 
 /*****	LTP Port	*****/
-char *TCID = "profil01";
 int local_flag = PASSED;
 int block_number;
 FILE *temp;

@@ -310,4 +312,4 @@ int main(void)
 	/* uClibc does not have profiling support */
 	tst_exit();
 }
-#endif
\ No newline at end of file
+#endif

1.7.1

------------------------------------------------------------------------------
Write once. Port to many.
Get the SDK and tools to simplify cross-platform app development. Create new or port existing apps to sell to consumers worldwide. Explore the Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join
http://p.sf.net/sfu/intel-appdev

_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

------------------------------------------------------------------------------
Write once. Port to many.
Get the SDK and tools to simplify cross-platform app development. Create new or port existing apps to sell to consumers worldwide. Explore the Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join
http://p.sf.net/sfu/intel-appdev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

the test.h header should be after all system includes. moving it before them is not a good idea. so NAK this patch.
-mike

------------------------------------------------------------------------------
Write once. Port to many.
Get the SDK and tools to simplify cross-platform app development. Create new or port existing apps to sell to consumers worldwide. Explore the Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join
http://p.sf.net/sfu/intel-appdev

_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Good point. Rescinding my Reviewed-by...
Thanks,
-Garrett

------------------------------------------------------------------------------
Write once. Port to many.
Get the SDK and tools to simplify cross-platform app development. Create new or port existing apps to sell to consumers worldwide. Explore the Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join
http://p.sf.net/sfu/intel-appdev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list