qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: peter.maydell@linaro.org
Cc: qemu-devel@nongnu.org, 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,
	"Alex Bennée" <alex.bennee@linaro.org>
Subject: [Qemu-devel] [PULL 13/56] tests/tcg/multiarch: move most output to stdout
Date: Tue, 19 Jun 2018 16:43:52 +0100	[thread overview]
Message-ID: <20180619154435.18898-14-alex.bennee@linaro.org> (raw)
In-Reply-To: <20180619154435.18898-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>
Reviewed-by: Thomas Huth <thuth@redhat.com>
---
v4
  - remove space in fprintf () to keep checkpatch happy

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-19 15:44 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-19 15:43 [Qemu-devel] [PULL 00/56] add check-tcg and associated machinery Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 01/56] configure: add support for --cross-cc-FOO Alex Bennée
2018-06-20 15:22   ` Peter Maydell
2018-06-19 15:43 ` [Qemu-devel] [PULL 02/56] configure: move i386_cc to cross_cc_i386 Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 03/56] configure: allow user to specify --cross-cc-cflags-foo= Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 04/56] configure: set cross_cc_FOO for host compiler Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 05/56] docker: Add "cc" subcommand Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 06/56] docker: extend "cc" command to accept compiler Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 07/56] docker: allow "cc" command to run in user context Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 08/56] docker: Makefile.include introduce DOCKER_SCRIPT Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 09/56] tests/tcg: move architecture independent tests into subdir Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 10/56] tests/tcg/multiarch: Build fix for linux-test Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 11/56] tests/tcg/multiarch: enable additional linux-test tests Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 12/56] tests/tcg/multiarch: don't hard code paths/ports for linux-test Alex Bennée
2018-06-19 15:43 ` Alex Bennée [this message]
2018-06-19 15:43 ` [Qemu-devel] [PULL 14/56] tests/tcg: move i386 specific tests into subdir Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 15/56] tests/tcg: enable building for i386 Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 16/56] tests/tcg/i386: Build fix for hello-i386 Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 17/56] tests/tcg/i386: fix test-i386 Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 18/56] tests/tcg/i386: add runner for test-i386-fprem Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 19/56] tests/tcg/x86_64: add Makefile.target Alex Bennée
2018-06-19 15:43 ` [Qemu-devel] [PULL 20/56] tests/tcg/i386/test-i386: use modern vector_size attributes Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 21/56] tests/tcg/i386/test-i386: fix printf format Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 22/56] tests/tcg: move ARM specific tests into subdir Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 23/56] tests/tcg: enable building for ARM Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 24/56] tests/tcg/arm: fix up test-arm-iwmmxt test Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 25/56] tests/tcg: enable building for AArch64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 26/56] tests/tcg/arm: add fcvt test cases for AArch32/64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 27/56] tests/tcg: move MIPS specific tests into subdir Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 28/56] tests/tcg: enable building for MIPS Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 29/56] tests/docker/Makefile.include: fix mipsel-cross dependancy Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 30/56] tests/tcg/mips: include common mips hello-mips Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 31/56] tests/tcg: enable building for s390x Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 32/56] tests/tcg: enable building for ppc64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 33/56] tests/tcg: enable building for Alpha Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 34/56] tests/tcg/alpha: add Alpha specific tests Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 35/56] tests/tcg: enable building for HPPA Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 36/56] tests/tcg: enable building for m68k Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 37/56] tests/tcg: enable building for sh4 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 38/56] tests/tcg: enable building for sparc64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 39/56] tests/tcg: enable building for mips64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 40/56] tests/tcg: enable building for RISCV64 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 41/56] docker: move debian-powerpc-cross to sid based build Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 42/56] tests/tcg: enable building for PowerPC Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 43/56] tests/tcg/Makefile: update to be called from Makefile.target Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 44/56] Makefile.target: add (clean-/build-)guest-tests targets Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 45/56] tests/Makefile.include: add [build|clean|check]-tcg targets Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 46/56] tests/tcg: add run, diff, and skip helper macros Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 47/56] tests/tcg: override runners for broken tests Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 48/56] tests/tcg/i386: extend timeout for runcom test Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 49/56] tests: add top-level make dependency for docker builds Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 50/56] docker: docker.py use "version" to probe usage Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 51/56] docker: docker.py don't conflate checksums for extra_files Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 52/56] docker: docker.py add check sub-command Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 53/56] tests/Makefile: call sub-makes with SKIP_DOCKER_BUILD=1 Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 54/56] docker: docker.py adding age check command Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 55/56] tests/docker/Makefile.include: only force SID to NOCACHE if old Alex Bennée
2018-06-19 15:44 ` [Qemu-devel] [PULL 56/56] .travis.yml: add check-tcg test Alex Bennée
2018-06-20 10:43 ` [Qemu-devel] [PULL 00/56] add check-tcg and associated machinery Peter Maydell
2018-06-20 14:09   ` Alex Bennée
2018-06-20 14:13   ` Daniel P. Berrangé
2018-06-20 14:21 ` Peter Maydell
2018-06-20 14:34   ` Alex Bennée

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=20180619154435.18898-14-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=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /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).