[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] fix tests/migration/stress build
From: |
Alexey Perevalov |
Subject: |
[Qemu-devel] [PATCH] fix tests/migration/stress build |
Date: |
Fri, 29 Sep 2017 13:11:13 +0300 |
This small commit fixes build of tests/migration/stress on Ubuntu16.04
(on RHEL7.4 the same situation, after typical configure with following options
--target-list=x86_64-linux-user,i386-linux-user,x86_64-softmmu,i386-softmmu \
--enable-numa --enable-kvm --enable-vhost-net --enable-tpm --enable-debug \
--disable-gtk )
PTHREAD_LIB is empty after successfully compiled pthread sample in configure.
tests/migration/stress relay on it, but -pthread is not there
and linker fails trying to resolve pthread symbols.
I still have a questions:
1. Is there a better way to obtain -thread option from QEMU_CFLAGS
2. Is linker options really necessary for stress application, because only -O3
is used, so w/o linker options in Makefile there is no need in PTHREAD_LIB
at all, I mean in whole project.
3. LINKPROG will be expanded into c++ firstly, and -static in
tests/Makefile.include will require static libstdc++, but configure doesn't
check it and in case of missing libstdc++-static.x86_64 (RHEL package name),
build will fail with the following message: "ld: cannot find -lstdc++",
I think here reasonably to use just $(CC) instead of LINKPROG, stress.c is on C.
Also ./tests/migration/guestperf.py can not be started w/o
super user privileges, and reports in this case something useless like this:
Error: [Errno 104] Connection reset by peer
This patch is based on commit 4a0d8b34e8ccf5ababdd11b29ce954aa9a5da365
Alexey Perevalov (1):
configure: correctly define PTHREAD_LIB
configure | 1 +
1 file changed, 1 insertion(+)
--
2.7.4
- [Qemu-devel] [PATCH] fix tests/migration/stress build,
Alexey Perevalov <=
- Message not available
- [Qemu-devel] [PATCH] configure: correctly define PTHREAD_LIB, Alexey Perevalov, 2017/09/29
- Re: [Qemu-devel] [PATCH] configure: correctly define PTHREAD_LIB, Daniel P. Berrange, 2017/09/29
- Re: [Qemu-devel] [PATCH] configure: correctly define PTHREAD_LIB, Alexey Perevalov, 2017/09/29
- Re: [Qemu-devel] [PATCH] configure: correctly define PTHREAD_LIB, Daniel P. Berrange, 2017/09/29
- Re: [Qemu-devel] [PATCH] configure: correctly define PTHREAD_LIB, Alexey Perevalov, 2017/09/29
- Re: [Qemu-devel] [PATCH] configure: correctly define PTHREAD_LIB, Daniel P. Berrange, 2017/09/29
Re: [Qemu-devel] [PATCH] fix tests/migration/stress build, Daniel P. Berrange, 2017/09/29