Skip to contentSkip to conversation

process_stress/process: Fix dprt() function definition when DEBUG macro is used

PATCH In review Markos Chandras 3 participants 3 messages Jan 3, 2012 – Jan 4, 2012

lore
Share & tools
lore

Markos Chandras

author
#0 lore raw

PATCH 11/21 process_stress/process: Fix dprt() function definition when DEBUG macro is used

Hi, please review the attached patch

Signature · 1 line
markos

From ba7dc3499dc3e7389943345a3c042a2b6910b9eb Mon Sep 17 00:00:00 2001
From: Markos Chandras <[email protected]>
Date: Mon, 21 Nov 2011 10:24:12 +0000
Subject: [PATCH 11/21] process_stress/process: Fix dprt() function definition when DEBUG macro is used

Signed-off-by Markos Chandras <[email protected]>
---
 testcases/kernel/sched/process_stress/process.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
testcases/kernel/sched/process_stress/process.c +2 -2
index b7d68b8..a72c1bc 100644
@@ -68,7 +68,7 @@
 #if defined _LINUX && defined DEBUG
 #define prtln()	printf("At line number: %d\n", __LINE__); \
 		fflush(NULL)
-#define dprt(fmt, args...) printf(fmt, args...)
+#define dprt(fmt, args...) printf(fmt, ## args)
 #else
 #define prtln()
 #define dprt(fmt, args...)

@@ -1210,4 +1210,4 @@ dprt("value of nodesum is initiallized to: %d\n", nodesum);
 	prtln();
 	return 0;
 
-}
\ No newline at end of file
+}
Signature · 1 line
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

Reviewed-by on v1 11/21
#1 lore raw
Quoted 1 line · Markos Chandras

Hi, please review the attached patch

Reviewed-by Garrett Cooper <[email protected]>

------------------------------------------------------------------------------
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

#2 lore raw
In reply to Markos Chandras

Hi!

Quoted 1 line

Subject: [PATCH 11/21] process_stress/process: Fix dprt() function definition when DEBUG macro is used

Commited, thanks.

Signature · 13 lines
Cyril Hrubis
[email protected]

------------------------------------------------------------------------------
Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
infrastructure or vast IT resources to deliver seamless, secure access to
virtual desktops. With this all-in-one solution, easily deploy virtual 
desktops for less than the cost of PCs and save 60% on VDI infrastructure 
costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list