[PATCH] [PPC64] printf() consolidation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Remove the fprintf(stderr,...) in get_memory_ranges() that adds unnecessary
output in the normal kexec case that the user don't want to see.

Use dbgprintf() in get_base_ranges() instead of

    #ifdef DEBUG
        fprintf(stderr,...)
    #endif

to to make the code more readable.


Signed-off-by: Bernhard Walle <bwalle at suse.de>diff --git a/kexec/arch/ppc64/kexec-ppc64.c b/kexec/arch/ppc64/kexec-ppc64.c
index ad8a31c..8d4e42b 100644
Signed-off-by: Bernhard Walle <bwalle at suse.de>
---
 kexec/arch/ppc64/kexec-ppc64.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/kexec/arch/ppc64/kexec-ppc64.c b/kexec/arch/ppc64/kexec-ppc64.c
index ad8a31c..8d4e42b 100644
--- a/kexec/arch/ppc64/kexec-ppc64.c
+++ b/kexec/arch/ppc64/kexec-ppc64.c
@@ -263,9 +263,8 @@ static int get_base_ranges(void)
 	closedir(dir);
 	sort_base_ranges();
 	memory_max = base_memory_range[nr_memory_ranges - 1].end;
-#ifdef DEBUG
-	fprintf(stderr, "get base memory ranges:%d\n", nr_memory_ranges);
-#endif
+	dbgprintf("get base memory ranges:%d\n", nr_memory_ranges);
+
 	return 0;
 }
 
@@ -726,7 +725,7 @@ int get_memory_ranges(struct memory_range **range, int *ranges,
 		sizeof(struct memory_range) * nr_memory_ranges);
 
 	*ranges = nr_memory_ranges;
-	fprintf(stderr, "get memory ranges:%d\n", nr_memory_ranges);
+	dbgprintf("get memory ranges:%d\n", nr_memory_ranges);
 	return 0;
 }
 
-- 
1.6.0.2




[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux