From: "Alex Bennée" <alex.bennee@linaro.org>
To: peter.maydell@linaro.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
Cc: qemu-devel@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>
Subject: [Qemu-devel] [PATCH v3 13/46] tests/tcg/multiarch: Build fix for linux-test
Date: Tue, 24 Apr 2018 16:23:32 +0100 [thread overview]
Message-ID: <20180424152405.10304-14-alex.bennee@linaro.org> (raw)
In-Reply-To: <20180424152405.10304-1-alex.bennee@linaro.org>
From: Fam Zheng <famz@redhat.com>
To keep the compiler happy, and to fit in our buildsys flags:
- Make local functions "static"
- #ifdef out unused functions
- drop cutils/osdep dependencies
Signed-off-by: Fam Zheng <famz@redhat.com>
[AJB: drop cutils/osdep dependencies]
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
---
tests/tcg/multiarch/linux-test.c | 68 ++++++++++----------------------
1 file changed, 21 insertions(+), 47 deletions(-)
diff --git a/tests/tcg/multiarch/linux-test.c b/tests/tcg/multiarch/linux-test.c
index 5070d31446..4457bd04ba 100644
--- a/tests/tcg/multiarch/linux-test.c
+++ b/tests/tcg/multiarch/linux-test.c
@@ -16,7 +16,6 @@
* You should have received a copy of the GNU General Public License
* along with this program; if not, see <http://www.gnu.org/licenses/>.
*/
-#define _GNU_SOURCE
#include <stdarg.h>
#include <stdlib.h>
#include <stdio.h>
@@ -31,6 +30,7 @@
#include <utime.h>
#include <time.h>
#include <sys/time.h>
+#include <sys/resource.h>
#include <sys/uio.h>
#include <sys/socket.h>
#include <netinet/in.h>
@@ -39,13 +39,12 @@
#include <dirent.h>
#include <setjmp.h>
#include <sys/shm.h>
-#include "qemu/cutils.h"
#define TESTPATH "/tmp/linux-test.tmp"
#define TESTPORT 7654
#define STACK_SIZE 16384
-void error1(const char *filename, int line, const char *fmt, ...)
+static void error1(const char *filename, int line, const char *fmt, ...)
{
va_list ap;
va_start(ap, fmt);
@@ -56,7 +55,7 @@ void error1(const char *filename, int line, const char *fmt, ...)
exit(1);
}
-int __chk_error(const char *filename, int line, int ret)
+static int __chk_error(const char *filename, int line, int ret)
{
if (ret < 0) {
error1(filename, line, "%m (ret=%d, errno=%d)",
@@ -73,7 +72,7 @@ int __chk_error(const char *filename, int line, int ret)
#define FILE_BUF_SIZE 300
-void test_file(void)
+static void test_file(void)
{
int fd, i, len, ret;
uint8_t buf[FILE_BUF_SIZE];
@@ -210,7 +209,7 @@ void test_file(void)
chk_error(rmdir(TESTPATH));
}
-void test_fork(void)
+static void test_fork(void)
{
int pid, status;
@@ -224,7 +223,7 @@ void test_fork(void)
error("waitpid status=0x%x", status);
}
-void test_time(void)
+static void test_time(void)
{
struct timeval tv, tv2;
struct timespec ts, rem;
@@ -251,34 +250,7 @@ void test_time(void)
error("getrusage");
}
-void pstrcpy(char *buf, int buf_size, const char *str)
-{
- int c;
- char *q = buf;
-
- if (buf_size <= 0)
- return;
-
- for(;;) {
- c = *str++;
- if (c == 0 || q >= buf + buf_size - 1)
- break;
- *q++ = c;
- }
- *q = '\0';
-}
-
-/* strcat and truncate. */
-char *pstrcat(char *buf, int buf_size, const char *s)
-{
- int len;
- len = strlen(buf);
- if (len < buf_size)
- pstrcpy(buf + len, buf_size - len, s);
- return buf;
-}
-
-int server_socket(void)
+static int server_socket(void)
{
int val, fd;
struct sockaddr_in sockaddr;
@@ -298,7 +270,7 @@ int server_socket(void)
}
-int client_socket(void)
+static int client_socket(void)
{
int fd;
struct sockaddr_in sockaddr;
@@ -312,9 +284,9 @@ int client_socket(void)
return fd;
}
-const char socket_msg[] = "hello socket\n";
+static const char socket_msg[] = "hello socket\n";
-void test_socket(void)
+static void test_socket(void)
{
int server_fd, client_fd, fd, pid, ret, val;
struct sockaddr_in sockaddr;
@@ -348,9 +320,10 @@ void test_socket(void)
chk_error(close(server_fd));
}
+#if 0
#define WCOUNT_MAX 512
-void test_pipe(void)
+static void test_pipe(void)
{
fd_set rfds, wfds;
int fds[2], fd_max, ret;
@@ -391,10 +364,10 @@ void test_pipe(void)
chk_error(close(fds[1]));
}
-int thread1_res;
-int thread2_res;
+static int thread1_res;
+static int thread2_res;
-int thread1_func(void *arg)
+static int thread1_func(void *arg)
{
int i;
for(i=0;i<5;i++) {
@@ -404,7 +377,7 @@ int thread1_func(void *arg)
return 0;
}
-int thread2_func(void *arg)
+static int thread2_func(void *arg)
{
int i;
for(i=0;i<6;i++) {
@@ -435,27 +408,28 @@ void test_clone(void)
thread2_res != 6)
error("clone");
}
+#endif
/***********************************/
volatile int alarm_count;
jmp_buf jmp_env;
-void sig_alarm(int sig)
+static void sig_alarm(int sig)
{
if (sig != SIGALRM)
error("signal");
alarm_count++;
}
-void sig_segv(int sig, siginfo_t *info, void *puc)
+static void sig_segv(int sig, siginfo_t *info, void *puc)
{
if (sig != SIGSEGV)
error("signal");
longjmp(jmp_env, 1);
}
-void test_signal(void)
+static void test_signal(void)
{
struct sigaction act;
struct itimerval it, oit;
@@ -510,7 +484,7 @@ void test_signal(void)
#define SHM_SIZE 32768
-void test_shm(void)
+static void test_shm(void)
{
void *ptr;
int shmid;
--
2.17.0
next prev parent reply other threads:[~2018-04-24 15:43 UTC|newest]
Thread overview: 102+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-24 15:23 [Qemu-devel] [PATCH v3 00/46] fix building of tests/tcg Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 01/46] docker: add "probe" command for configure Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 02/46] configure: add test for docker availability Alex Bennée
2018-04-25 0:06 ` Richard Henderson
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 03/46] configure: add support for --cross-cc-FOO Alex Bennée
2018-04-25 0:10 ` Richard Henderson
2018-04-25 9:10 ` Alex Bennée
2018-04-25 16:27 ` Murilo Opsfelder Araujo
2018-05-16 15:18 ` Alex Bennée
2018-05-16 17:36 ` Murilo Opsfelder Araujo
2018-05-16 20:37 ` Alex Bennée
2018-04-25 17:03 ` Peter Maydell
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 04/46] configure: move i386_cc to cross_cc_i386 Alex Bennée
2018-04-25 0:11 ` Richard Henderson
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 05/46] configure: allow user to specify --cross-cc-cflags-foo= Alex Bennée
2018-04-25 0:12 ` Richard Henderson
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 06/46] configure: set cross_cc_FOO for host compiler Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 07/46] Makefile: Rename TARGET_DIRS to TARGET_LIST Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 08/46] docker: Add "cc" subcommand Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 09/46] docker: extend "cc" command to accept compiler Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 10/46] docker: allow "cc" command to run in user context Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 11/46] docker: Makefile.include introduce DOCKER_SCRIPT Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 12/46] tests/tcg: move architecture independent tests into subdir Alex Bennée
2018-04-24 15:23 ` Alex Bennée [this message]
2018-04-25 0:13 ` [Qemu-devel] [PATCH v3 13/46] tests/tcg/multiarch: Build fix for linux-test Richard Henderson
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 14/46] tests/tcg/multiarch: enable additional linux-test tests Alex Bennée
2018-04-25 0:14 ` Richard Henderson
2018-04-25 4:07 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 15/46] tests/tcg/multiarch: move most output to stdout Alex Bennée
2018-04-25 0:14 ` Richard Henderson
2018-04-25 4:06 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 16/46] tests/tcg: move i386 specific tests into subdir Alex Bennée
2018-04-25 0:15 ` Richard Henderson
2018-04-25 3:27 ` Philippe Mathieu-Daudé
2018-04-25 9:08 ` Alex Bennée
2018-04-25 13:42 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 17/46] docker: Add fedora-i386-cross image Alex Bennée
2018-04-24 23:04 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 18/46] tests/tcg: enable building for i386 Alex Bennée
2018-04-25 3:33 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 19/46] tests/tcg/i386: Build fix for hello-i386 Alex Bennée
2018-04-25 0:22 ` Richard Henderson
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 20/46] tests/tcg/i386: fix test-i386 Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 21/46] tests/tcg/i386: fix test-i386-fprem Alex Bennée
2018-04-25 0:23 ` Richard Henderson
2018-04-25 3:26 ` Thomas Huth
2018-04-25 8:46 ` Alex Bennée
2018-04-25 3:42 ` Philippe Mathieu-Daudé
2018-04-25 8:45 ` Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 22/46] tests/tcg/i386: disable i386 version of test-i386-ssse Alex Bennée
2018-04-25 21:28 ` Richard Henderson
2018-04-26 8:59 ` Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 23/46] tests/tcg/x86_64: add Makefile.target Alex Bennée
2018-04-25 4:03 ` Philippe Mathieu-Daudé
2018-04-25 8:43 ` Alex Bennée
2018-04-25 14:01 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 24/46] tests/tcg/i386/test-i386: use modern vector_size attributes Alex Bennée
2018-04-25 3:47 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 25/46] tests/tcg/i386/test-i386: fix printf format Alex Bennée
2018-04-25 14:06 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 26/46] tests/tcg: move ARM specific tests into subdir Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 27/46] tests/tcg/arm: fix up test-arm-iwmmxt test Alex Bennée
2018-04-25 3:48 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 28/46] tests/tcg/aarch64: add Makefile.target Alex Bennée
2018-04-25 3:57 ` Philippe Mathieu-Daudé
2018-04-25 8:43 ` Alex Bennée
2018-04-25 14:21 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 29/46] tests/tcg/aarch64: add fcvt test cases for AArch64 Alex Bennée
2018-04-25 14:22 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 30/46] tests/tcg: move MIPS specific tests into subdir Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 31/46] tests/tcg: enable building for MIPS Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 32/46] tests/tcg/mips: include common mips hello-mips Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 33/46] tests/tcg: enable building for s390x Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 34/46] tests/tcg: enable building for ppc64 Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 35/46] tests/tcg: enable building for Alpha Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 36/46] tests/tcg/alpha: add Alpha specific tests Alex Bennée
2018-04-24 18:28 ` Richard Henderson
2018-04-25 9:11 ` Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 37/46] tests/tcg/alpha: fix warnings about write() Alex Bennée
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 38/46] tests/tcg: enable building for HPPA Alex Bennée
2018-04-25 14:08 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 39/46] tests/tcg: enable building for m68k Alex Bennée
2018-04-25 14:28 ` Philippe Mathieu-Daudé
2018-04-24 15:23 ` [Qemu-devel] [PATCH v3 40/46] tests/tcg: enable building for sh4 Alex Bennée
2018-04-25 14:16 ` Philippe Mathieu-Daudé
2018-04-24 15:24 ` [Qemu-devel] [PATCH v3 41/46] tests/tcg: enable building for sparc64 Alex Bennée
2018-04-24 15:24 ` [Qemu-devel] [PATCH v3 42/46] tests/tcg: enable building for mips64 Alex Bennée
2018-04-24 15:24 ` [Qemu-devel] [PATCH v3 43/46] tests/tcg/Makefile: update to be called from Makefile.target Alex Bennée
2018-04-25 21:26 ` Richard Henderson
2018-04-26 9:30 ` Alex Bennée
2018-04-26 20:51 ` Richard Henderson
2018-04-24 15:24 ` [Qemu-devel] [PATCH v3 44/46] Makefile.target: add (clean-)guest-tests targets Alex Bennée
2018-04-24 15:24 ` [Qemu-devel] [PATCH v3 45/46] tests/Makefile.include: add [build|clean|check]-tcg targets Alex Bennée
2018-04-24 23:13 ` Philippe Mathieu-Daudé
2018-04-25 14:15 ` Philippe Mathieu-Daudé
2018-04-25 15:40 ` Alex Bennée
2018-04-25 3:18 ` Philippe Mathieu-Daudé
2018-04-24 15:24 ` [Qemu-devel] [PATCH v3 46/46] tests/tcg: override runners for broken tests (!UPSTREAM) Alex Bennée
2018-04-24 16:58 ` [Qemu-devel] [PATCH v3 00/46] fix building of tests/tcg no-reply
2018-04-25 1:25 ` Philippe Mathieu-Daudé
2018-04-25 6:57 ` Samuel Thibault
2018-04-25 8:39 ` 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=20180424152405.10304-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).