From: wei.guo.simon@gmail.com
To: linuxppc-dev@lists.ozlabs.org
Cc: Shuah Khan <shuah@kernel.org>,
Michael Ellerman <mpe@ellerman.id.au>,
Cyril Bur <cyrilbur@gmail.com>, Chris Smart <chris@distroguy.com>,
Suraj Jitindar Singh <sjitindarsingh@gmail.com>,
Michael Neuling <mikey@neuling.org>,
Anshuman Khandual <khandual@linux.vnet.ibm.com>,
Simon Guo <wei.guo.simon@gmail.com>,
Jack Miller <jack@codezen.org>,
Rashmica Gupta <rashmicy@gmail.com>,
linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org
Subject: [PATCH v15 15/15] selftests/powerpc: Fix a build issue
Date: Fri, 30 Sep 2016 10:33:04 +0800 [thread overview]
Message-ID: <1475202784-5653-16-git-send-email-wei.guo.simon@gmail.com> (raw)
In-Reply-To: <1475202784-5653-1-git-send-email-wei.guo.simon@gmail.com>
From: Anshuman Khandual <khandual@linux.vnet.ibm.com>
Fixes the following build failure -
cp_abort.c:90:3: error: ‘for’ loop initial declarations are only
allowed in C99 or C11 mode
for (int i = 0; i < NUM_LOOPS; i++) {
^
cp_abort.c:90:3: note: use option -std=c99, -std=gnu99, -std=c11 or
-std=gnu11 to compile your code
cp_abort.c:97:3: error: ‘for’ loop initial declarations are only
allowed in C99 or C11 mode
for (int i = 0; i < NUM_LOOPS; i++) {
Signed-off-by: Anshuman Khandual <khandual@linux.vnet.ibm.com>
Signed-off-by: Simon Guo <wei.guo.simon@gmail.com>
Reviewed-by: Cyril Bur <cyrilbur@gmail.com>
---
tools/testing/selftests/powerpc/context_switch/cp_abort.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tools/testing/selftests/powerpc/context_switch/cp_abort.c b/tools/testing/selftests/powerpc/context_switch/cp_abort.c
index 5a5b55a..1ce7dce 100644
--- a/tools/testing/selftests/powerpc/context_switch/cp_abort.c
+++ b/tools/testing/selftests/powerpc/context_switch/cp_abort.c
@@ -67,7 +67,7 @@ int test_cp_abort(void)
/* 128 bytes for a full cache line */
char buf[128] __cacheline_aligned;
cpu_set_t cpuset;
- int fd1[2], fd2[2], pid;
+ int fd1[2], fd2[2], pid, i;
char c;
/* only run this test on a P9 or later */
@@ -87,14 +87,14 @@ int test_cp_abort(void)
FAIL_IF(pid < 0);
if (!pid) {
- for (int i = 0; i < NUM_LOOPS; i++) {
+ for (i = 0; i < NUM_LOOPS; i++) {
FAIL_IF((write(fd1[WRITE_FD], &c, 1)) != 1);
FAIL_IF((read(fd2[READ_FD], &c, 1)) != 1);
/* A paste succeeds if CR0 EQ bit is set */
FAIL_IF(paste(buf) & 0x20000000);
}
} else {
- for (int i = 0; i < NUM_LOOPS; i++) {
+ for (i = 0; i < NUM_LOOPS; i++) {
FAIL_IF((read(fd1[READ_FD], &c, 1)) != 1);
copy(buf);
FAIL_IF((write(fd2[WRITE_FD], &c, 1) != 1));
--
1.8.3.1
prev parent reply other threads:[~2016-09-30 2:37 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-30 2:32 [PATCH v15 00/15] selftests/powerpc: Add ptrace tests for ppc registers wei.guo.simon
2016-09-30 2:32 ` [PATCH v15 01/15] selftests/powerpc: Add more SPR numbers, TM & VMX instructions to 'reg.h'/'instructions.h' wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:32 ` [PATCH v15 02/15] selftests/powerpc: move shared utility files into new utility/ dir wei.guo.simon
2016-09-30 2:32 ` [PATCH v15 03/15] selftests/powerpc: Add ptrace tests for EBB wei.guo.simon
2016-10-06 21:44 ` Michael Ellerman
2016-10-12 14:06 ` Simon Guo
2016-11-17 12:03 ` [v15,03/15] " Michael Ellerman
2016-09-30 2:32 ` [PATCH v15 04/15] selftests/powerpc: Add ptrace tests for GPR/FPR registers wei.guo.simon
2016-09-30 2:32 ` [PATCH v15 05/15] selftests/powerpc: Add ptrace tests for GPR/FPR registers in TM wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:32 ` [PATCH v15 06/15] selftests/powerpc: Add ptrace tests for GPR/FPR registers in suspended TM wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:32 ` [PATCH v15 07/15] selftests/powerpc: Add ptrace tests for TAR, PPR, DSCR registers wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:32 ` [PATCH v15 08/15] selftests/powerpc: Add ptrace tests for TAR, PPR, DSCR in TM wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:32 ` [PATCH v15 09/15] selftests/powerpc: Add ptrace tests for TAR, PPR, DSCR in suspended TM wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:32 ` [PATCH v15 10/15] selftests/powerpc: Add ptrace tests for VSX, VMX registers wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:33 ` [PATCH v15 11/15] selftests/powerpc: Add ptrace tests for VSX, VMX registers in TM wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:33 ` [PATCH v15 12/15] selftests/powerpc: Add ptrace tests for VSX, VMX registers in suspended TM wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:33 ` [PATCH v15 13/15] selftests/powerpc: Add ptrace tests for TM SPR registers wei.guo.simon
2016-11-17 12:03 ` [v15, " Michael Ellerman
2016-09-30 2:33 ` [PATCH v15 14/15] selftests/powerpc: Add .gitignore file for ptrace executables wei.guo.simon
2016-09-30 2:33 ` wei.guo.simon [this message]
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=1475202784-5653-16-git-send-email-wei.guo.simon@gmail.com \
--to=wei.guo.simon@gmail.com \
--cc=chris@distroguy.com \
--cc=cyrilbur@gmail.com \
--cc=jack@codezen.org \
--cc=khandual@linux.vnet.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mikey@neuling.org \
--cc=mpe@ellerman.id.au \
--cc=rashmicy@gmail.com \
--cc=shuah@kernel.org \
--cc=sjitindarsingh@gmail.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).