qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: cota@braap.org, famz@redhat.com, berrange@redhat.com,
	f4bug@amsat.org, richard.henderson@linaro.org, balrogg@gmail.com,
	aurelien@aurel32.net, agraf@suse.de, pbonzini@redhat.com,
	stefanha@redhat.com, stefanb@linux.vnet.ibm.com,
	marcandre.lureau@redhat.com
Cc: qemu-devel@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>
Subject: [Qemu-devel] [PATCH v6 12/49] tests/tcg/multiarch: move most output to stdout
Date: Fri,  8 Jun 2018 13:32:30 +0100	[thread overview]
Message-ID: <20180608123307.24773-13-alex.bennee@linaro.org> (raw)
In-Reply-To: <20180608123307.24773-1-alex.bennee@linaro.org>

The default test run outputs to stdout so it can be re-directed.
Errors are still reported to stderr.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

---
v4
  - remove space in fprintf () to keep checkpatch happy
---
 tests/tcg/multiarch/test-mmap.c | 40 ++++++++++++++++-----------------
 1 file changed, 20 insertions(+), 20 deletions(-)

diff --git a/tests/tcg/multiarch/test-mmap.c b/tests/tcg/multiarch/test-mmap.c
index cdefadfa4c..5c0afe6e49 100644
--- a/tests/tcg/multiarch/test-mmap.c
+++ b/tests/tcg/multiarch/test-mmap.c
@@ -36,7 +36,7 @@
 do                                                             \
 {                                                              \
   if (!(x)) {                                                  \
-    fprintf (stderr, "FAILED at %s:%d\n", __FILE__, __LINE__); \
+    fprintf(stderr, "FAILED at %s:%d\n", __FILE__, __LINE__); \
     exit (EXIT_FAILURE);                                       \
   }                                                            \
 } while (0)
@@ -57,7 +57,7 @@ void check_aligned_anonymous_unfixed_mmaps(void)
 	uintptr_t p;
 	int i;
 
-	fprintf (stderr, "%s", __func__);
+	fprintf(stdout, "%s", __func__);
 	for (i = 0; i < 0x1fff; i++)
 	{
 		size_t len;
@@ -106,7 +106,7 @@ void check_aligned_anonymous_unfixed_mmaps(void)
 		munmap (p4, len);
 		munmap (p5, len);
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_large_anonymous_unfixed_mmap(void)
@@ -115,7 +115,7 @@ void check_large_anonymous_unfixed_mmap(void)
 	uintptr_t p;
 	size_t len;
 
-	fprintf (stderr, "%s", __func__);
+	fprintf(stdout, "%s", __func__);
 
 	len = 0x02000000;
 	p1 = mmap(NULL, len, PROT_READ, 
@@ -130,7 +130,7 @@ void check_large_anonymous_unfixed_mmap(void)
 	/* Make sure we can read from the entire area.  */
 	memcpy (dummybuf, p1, pagesize);
 	munmap (p1, len);
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_aligned_anonymous_unfixed_colliding_mmaps(void)
@@ -141,7 +141,7 @@ void check_aligned_anonymous_unfixed_colliding_mmaps(void)
 	uintptr_t p;
 	int i;
 
-	fprintf (stderr, "%s", __func__);
+	fprintf(stdout, "%s", __func__);
 	for (i = 0; i < 0x2fff; i++)
 	{
 		int nlen;
@@ -180,7 +180,7 @@ void check_aligned_anonymous_unfixed_colliding_mmaps(void)
 		munmap (p2, pagesize);
 		munmap (p3, nlen);
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_aligned_anonymous_fixed_mmaps(void)
@@ -194,7 +194,7 @@ void check_aligned_anonymous_fixed_mmaps(void)
 	addr = mmap(NULL, pagesize * 40, PROT_READ | PROT_WRITE, 
 		    MAP_PRIVATE | MAP_ANONYMOUS,
 		    -1, 0);
-	fprintf (stderr, "%s addr=%p", __func__, addr);
+	fprintf(stdout, "%s addr=%p", __func__, addr);
 	fail_unless (addr != MAP_FAILED);
 
 	for (i = 0; i < 40; i++)
@@ -212,7 +212,7 @@ void check_aligned_anonymous_fixed_mmaps(void)
 		munmap (p1, pagesize);
 		addr += pagesize;
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_aligned_anonymous_fixed_mmaps_collide_with_host(void)
@@ -225,8 +225,8 @@ void check_aligned_anonymous_fixed_mmaps_collide_with_host(void)
 	/* Find a suitable address to start with.  Right were the x86 hosts
 	 stack is.  */
 	addr = ((void *)0x80000000);
-	fprintf (stderr, "%s addr=%p", __func__, addr);
-	fprintf (stderr, "FIXME: QEMU fails to track pages used by the host.");
+	fprintf(stdout, "%s addr=%p", __func__, addr);
+	fprintf(stdout, "FIXME: QEMU fails to track pages used by the host.");
 
 	for (i = 0; i < 20; i++)
 	{
@@ -243,7 +243,7 @@ void check_aligned_anonymous_fixed_mmaps_collide_with_host(void)
 		munmap (p1, pagesize);
 		addr += pagesize;
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_file_unfixed_mmaps(void)
@@ -252,7 +252,7 @@ void check_file_unfixed_mmaps(void)
 	uintptr_t p;
 	int i;
 
-	fprintf (stderr, "%s", __func__);
+	fprintf(stdout, "%s", __func__);
 	for (i = 0; i < 0x10; i++)
 	{
 		size_t len;
@@ -294,7 +294,7 @@ void check_file_unfixed_mmaps(void)
 		munmap (p2, len);
 		munmap (p3, len);
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_file_unfixed_eof_mmaps(void)
@@ -304,7 +304,7 @@ void check_file_unfixed_eof_mmaps(void)
 	uintptr_t p;
 	int i;
 
-	fprintf (stderr, "%s", __func__);
+	fprintf(stdout, "%s", __func__);
 	for (i = 0; i < 0x10; i++)
 	{
 		p1 = mmap(NULL, pagesize, PROT_READ, 
@@ -327,7 +327,7 @@ void check_file_unfixed_eof_mmaps(void)
 		fail_unless (cp[pagesize - 4] == 0);
 		munmap (p1, pagesize);
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_file_fixed_eof_mmaps(void)
@@ -343,7 +343,7 @@ void check_file_fixed_eof_mmaps(void)
 		    MAP_PRIVATE | MAP_ANONYMOUS,
 		    -1, 0);
 
-	fprintf (stderr, "%s addr=%p", __func__, (void *)addr);
+	fprintf(stdout, "%s addr=%p", __func__, (void *)addr);
 	fail_unless (addr != MAP_FAILED);
 
 	for (i = 0; i < 0x10; i++)
@@ -371,7 +371,7 @@ void check_file_fixed_eof_mmaps(void)
 		munmap (p1, pagesize);
 		addr += pagesize;
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void check_file_fixed_mmaps(void)
@@ -384,7 +384,7 @@ void check_file_fixed_mmaps(void)
 	addr = mmap(NULL, pagesize * 40 * 4, PROT_READ, 
 		    MAP_PRIVATE | MAP_ANONYMOUS,
 		    -1, 0);
-	fprintf (stderr, "%s addr=%p", __func__, (void *)addr);
+	fprintf(stdout, "%s addr=%p", __func__, (void *)addr);
 	fail_unless (addr != MAP_FAILED);
 
 	for (i = 0; i < 40; i++)
@@ -426,7 +426,7 @@ void check_file_fixed_mmaps(void)
 		munmap (p4, pagesize);
 		addr += pagesize * 4;
 	}
-	fprintf (stderr, " passed\n");
+	fprintf(stdout, " passed\n");
 }
 
 void checked_write(int fd, const void *buf, size_t count)
-- 
2.17.1

  parent reply	other threads:[~2018-06-08 12:49 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-08 12:32 [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 01/49] configure: add support for --cross-cc-FOO Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 02/49] configure: move i386_cc to cross_cc_i386 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 03/49] configure: allow user to specify --cross-cc-cflags-foo= Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 04/49] configure: set cross_cc_FOO for host compiler Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 05/49] docker: Add "cc" subcommand Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 06/49] docker: extend "cc" command to accept compiler Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 07/49] docker: allow "cc" command to run in user context Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 08/49] docker: Makefile.include introduce DOCKER_SCRIPT Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 09/49] tests/tcg: move architecture independent tests into subdir Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 10/49] tests/tcg/multiarch: Build fix for linux-test Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 11/49] tests/tcg/multiarch: enable additional linux-test tests Alex Bennée
2018-06-08 12:32 ` Alex Bennée [this message]
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 13/49] tests/tcg: move i386 specific tests into subdir Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 14/49] tests/tcg: enable building for i386 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 15/49] tests/tcg/i386: Build fix for hello-i386 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 16/49] tests/tcg/i386: fix test-i386 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 17/49] tests/tcg/i386: add runner for test-i386-fprem Alex Bennée
2018-06-11  2:22   ` Philippe Mathieu-Daudé
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 18/49] tests/tcg/x86_64: add Makefile.target Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 19/49] tests/tcg/i386/test-i386: use modern vector_size attributes Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 20/49] tests/tcg/i386/test-i386: fix printf format Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 21/49] tests/tcg: move ARM specific tests into subdir Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 22/49] tests/tcg: enable building for ARM Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 23/49] tests/tcg/arm: fix up test-arm-iwmmxt test Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 24/49] tests/tcg: enable building for AArch64 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 25/49] tests/tcg/arm: add fcvt test cases for AArch32/64 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 26/49] tests/tcg: move MIPS specific tests into subdir Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 27/49] tests/tcg: enable building for MIPS Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 28/49] tests/tcg/mips: include common mips hello-mips Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 29/49] tests/tcg: enable building for s390x Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 30/49] tests/tcg: enable building for ppc64 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 31/49] tests/tcg: enable building for Alpha Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 32/49] tests/tcg/alpha: add Alpha specific tests Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 33/49] tests/tcg: enable building for HPPA Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 34/49] tests/tcg: enable building for m68k Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 35/49] tests/tcg: enable building for sh4 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 36/49] tests/tcg: enable building for sparc64 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 37/49] tests/tcg: enable building for mips64 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 38/49] tests/tcg: enable building for RISCV64 Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 39/49] docker: move debian-powerpc-cross to sid based build Alex Bennée
2018-06-11  1:30   ` Philippe Mathieu-Daudé
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 40/49] tests/tcg: enable building for PowerPC Alex Bennée
2018-06-11  1:52   ` Philippe Mathieu-Daudé
2018-06-11  8:18     ` Alex Bennée
2018-06-08 12:32 ` [Qemu-devel] [PATCH v6 41/49] tests/tcg/Makefile: update to be called from Makefile.target Alex Bennée
2018-06-11  1:50   ` Philippe Mathieu-Daudé
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 42/49] Makefile.target: add (clean-/build-)guest-tests targets Alex Bennée
2018-06-11  2:15   ` Philippe Mathieu-Daudé
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 43/49] tests/Makefile.include: add [build|clean|check]-tcg targets Alex Bennée
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 44/49] tests/tcg: add run, diff, and skip helper macros Alex Bennée
2018-06-11  2:16   ` Philippe Mathieu-Daudé
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 45/49] tests/tcg: override runners for broken tests Alex Bennée
2018-06-11  1:58   ` Philippe Mathieu-Daudé
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 46/49] target/sh4: Fix translator.c assertion failure for gUSA Alex Bennée
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 47/49] tests: add top-level make dependency for docker builds Alex Bennée
2018-06-11  0:00   ` Philippe Mathieu-Daudé
2018-06-11  7:17     ` Alex Bennée
2018-06-11  1:34   ` Philippe Mathieu-Daudé
2018-06-11  7:18     ` Alex Bennée
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 48/49] tests/docker: prevent sub-makes re-building debian-sid Alex Bennée
2018-06-08 12:33 ` [Qemu-devel] [PATCH v6 49/49] .travis.yml: add check-tcg test Alex Bennée
2018-06-11  2:09   ` Philippe Mathieu-Daudé
2018-06-11  8:21     ` Alex Bennée
2018-06-08 14:23 ` [Qemu-devel] [PATCH v6 00/49] fix building of tests/tcg no-reply
2018-06-11  2:02 ` Philippe Mathieu-Daudé
2018-06-11  8:19   ` Alex Bennée
2018-06-11 10:47     ` Philippe Mathieu-Daudé
2018-06-11  2:27 ` Philippe Mathieu-Daudé

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180608123307.24773-13-alex.bennee@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=agraf@suse.de \
    --cc=aurelien@aurel32.net \
    --cc=balrogg@gmail.com \
    --cc=berrange@redhat.com \
    --cc=cota@braap.org \
    --cc=f4bug@amsat.org \
    --cc=famz@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=stefanb@linux.vnet.ibm.com \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).