Philippe Mathieu-Daudé <f4...@amsat.org> writes:
> On 04/15/2018 09:07 PM, Philippe Mathieu-Daudé wrote:
>> Hi Fam, Alex,
>>
>> On 04/10/2018 04:39 PM, Alex Bennée wrote:
>>> From: Fam Zheng <f...@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 <f...@redhat.com>
>>> [AJB: drop cutils/osdep dependencies]
>>> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
>>> ---
>>> 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
>>
>> Why keep this now dead code?
>
> You can use: #ifdef LINUX_TEST_CLONE
>
> But this my be even better to move this chunk as a stand-alone test in a
> new file, linux-clone-test.c and add a make rule to only include it in
> the TEST_SRCS var when the host is able to run it.
More of it gets un-#if 0'd in the follow-up patch. There is certainly
more clean-up to be done but for now I've just left it disabled. I think
splitting out a specific test-clone patch is a good follow-up candidate
though.
>
>>
>>> #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;
>>>
--
Alex Bennée
