ad9577
From 26c75797103da70f85a5fdd646625edb3eaa300c Mon Sep 17 00:00:00 2001
ad9577
From: Karel Zak <kzak@redhat.com>
ad9577
Date: Wed, 5 Mar 2014 11:23:16 +0100
ad9577
Subject: [PATCH 171/173] chcpu: cleanup stdout/stderr usage
ad9577
ad9577
Signed-off-by: Karel Zak <kzak@redhat.com>
ad9577
Upstream: http://github.com/karelzak/util-linux/commit/69e74525bb3212944feebc8c2848317e04d95ec0
ad9577
Addresses: http://bugzilla.redhat.com/show_bug.cgi?id=1579439
ad9577
---
ad9577
 sys-utils/chcpu.c | 17 +++++++----------
ad9577
 1 file changed, 7 insertions(+), 10 deletions(-)
ad9577
ad9577
diff --git a/sys-utils/chcpu.c b/sys-utils/chcpu.c
ad9577
index 304b80d7a..ada0eaacc 100644
ad9577
--- a/sys-utils/chcpu.c
ad9577
+++ b/sys-utils/chcpu.c
ad9577
@@ -84,12 +84,12 @@ static int cpu_enable(cpu_set_t *cpu_set, size_t setsize, int enable)
ad9577
 		if (!CPU_ISSET(cpu, cpu_set))
ad9577
 			continue;
ad9577
 		if (!path_exist(_PATH_SYS_CPU "/cpu%d", cpu)) {
ad9577
-			printf(_("CPU %d does not exist\n"), cpu);
ad9577
+			warnx(_("CPU %d does not exist"), cpu);
ad9577
 			fails++;
ad9577
 			continue;
ad9577
 		}
ad9577
 		if (!path_exist(_PATH_SYS_CPU "/cpu%d/online", cpu)) {
ad9577
-			printf(_("CPU %d is not hot pluggable\n"), cpu);
ad9577
+			warnx(_("CPU %d is not hot pluggable"), cpu);
ad9577
 			fails++;
ad9577
 			continue;
ad9577
 		}
ad9577
@@ -107,8 +107,7 @@ static int cpu_enable(cpu_set_t *cpu_set, size_t setsize, int enable)
ad9577
 		if (enable) {
ad9577
 			rc = path_write_str("1", _PATH_SYS_CPU "/cpu%d/online", cpu);
ad9577
 			if ((rc == -1) && (configured == 0)) {
ad9577
-				warnx(_("CPU %d enable failed "
ad9577
-					 "(CPU is deconfigured)"), cpu);
ad9577
+				warn(_("CPU %d enable failed (CPU is deconfigured)"), cpu);
ad9577
 				fails++;
ad9577
 			} else if (rc == -1) {
ad9577
 				warn(_("CPU %d enable failed"), cpu);
ad9577
@@ -117,8 +116,7 @@ static int cpu_enable(cpu_set_t *cpu_set, size_t setsize, int enable)
ad9577
 				printf(_("CPU %d enabled\n"), cpu);
ad9577
 		} else {
ad9577
 			if (onlinecpus && num_online_cpus() == 1) {
ad9577
-				printf(_("CPU %d disable failed "
ad9577
-					 "(last enabled CPU)\n"), cpu);
ad9577
+				warnx(_("CPU %d disable failed (last enabled CPU)"), cpu);
ad9577
 				fails++;
ad9577
 				continue;
ad9577
 			}
ad9577
@@ -178,12 +176,12 @@ static int cpu_configure(cpu_set_t *cpu_set, size_t setsize, int configure)
ad9577
 		if (!CPU_ISSET(cpu, cpu_set))
ad9577
 			continue;
ad9577
 		if (!path_exist(_PATH_SYS_CPU "/cpu%d", cpu)) {
ad9577
-			printf(_("CPU %d does not exist\n"), cpu);
ad9577
+			warnx(_("CPU %d does not exist"), cpu);
ad9577
 			fails++;
ad9577
 			continue;
ad9577
 		}
ad9577
 		if (!path_exist(_PATH_SYS_CPU "/cpu%d/configure", cpu)) {
ad9577
-			printf(_("CPU %d is not configurable\n"), cpu);
ad9577
+			warnx(_("CPU %d is not configurable"), cpu);
ad9577
 			fails++;
ad9577
 			continue;
ad9577
 		}
ad9577
@@ -198,8 +196,7 @@ static int cpu_configure(cpu_set_t *cpu_set, size_t setsize, int configure)
ad9577
 		}
ad9577
 		if ((current == 1) && (configure == 0) && onlinecpus &&
ad9577
 		    is_cpu_online(cpu)) {
ad9577
-			printf(_("CPU %d deconfigure failed "
ad9577
-				 "(CPU is enabled)\n"), cpu);
ad9577
+			warnx(_("CPU %d deconfigure failed (CPU is enabled)"), cpu);
ad9577
 			fails++;
ad9577
 			continue;
ad9577
 		}
ad9577
-- 
ad9577
2.14.4
ad9577