x86/decoder: Use stdout if insn decoder test is successful
authorPaul Bolle <pebolle@tiscali.nl>
Tue, 25 Oct 2016 20:56:04 +0000 (22:56 +0200)
committerIngo Molnar <mingo@kernel.org>
Wed, 26 Oct 2016 06:41:06 +0000 (08:41 +0200)
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 <pebolle@tiscali.nl>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1477428965-20548-2-git-send-email-pebolle@tiscali.nl
Signed-off-by: Ingo Molnar <mingo@kernel.org>
arch/x86/tools/test_get_len.c

index 56f04db0c9c0d8db9c459cc6199af85477fa0883..ecf31e0358c8140f5c0b468a3f3a98b5b877c728 100644 (file)
@@ -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;
 }