From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759457AbcJYU4X (ORCPT ); Tue, 25 Oct 2016 16:56:23 -0400 Received: from lb1-smtp-cloud3.xs4all.net ([194.109.24.22]:43343 "EHLO lb1-smtp-cloud3.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754104AbcJYU4S (ORCPT ); Tue, 25 Oct 2016 16:56:18 -0400 From: Paul Bolle To: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" Cc: Andrew Morton , x86@kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 1/2] x86: use stdout if insn decoder test is successful Date: Tue, 25 Oct 2016 22:56:04 +0200 Message-Id: <1477428965-20548-2-git-send-email-pebolle@tiscali.nl> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1477428965-20548-1-git-send-email-pebolle@tiscali.nl> References: <1477428965-20548-1-git-send-email-pebolle@tiscali.nl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If the instruction decoder test ran successful it prints a message like this to stderr: Succeed: decoded and checked 1767380 instructions But, as described in "console mode programming user interface guidelines version 101" which doesn't exist, programs should use stderr for errors or warnings. We're told about a successful run here, so the instruction decoder test should use stdout. Let's fix the typo too, while we're at it. Signed-off-by: Paul Bolle --- The joke about the guidelines is shamelessly copied from commit 55a6e622e66a ("arch/x86/tools/insn_sanity.c: Identify source of messages"). Let's hope Andrew doesn't mind! arch/x86/tools/test_get_len.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/tools/test_get_len.c b/arch/x86/tools/test_get_len.c index 56f04db0c9c0..ecf31e0358c8 100644 --- a/arch/x86/tools/test_get_len.c +++ b/arch/x86/tools/test_get_len.c @@ -167,7 +167,7 @@ int main(int argc, char **argv) fprintf(stderr, "Warning: decoded and checked %d" " instructions with %d warnings\n", insns, warnings); else - fprintf(stderr, "Succeed: decoded and checked %d" + fprintf(stdout, "Success: decoded and checked %d" " instructions\n", insns); return 0; } -- 2.7.4