[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/13] build-sys: fix qemu-ga -pthread linkin
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/13] build-sys: fix qemu-ga -pthread linking |
Date: |
Tue, 19 Dec 2017 16:43:06 +0100 |
Hi
On Fri, Dec 15, 2017 at 7:21 PM, Philippe Mathieu-Daudé <address@hidden> wrote:
> Hi Marc-André,
>
> On 12/15/2017 12:06 PM, Marc-André Lureau wrote:
>> When linking qemu-ga under some configuration (when gthread-2.0.pc
>> doesn't have -pthread, as happening atm with meson build), you may
>> have this linking issue:
>>
>> /usr/bin/ld: libqemuutil.a(qemu-thread-posix.o): undefined reference to
>> symbol 'pthread_setname_np@@GLIBC_2.12'
>> /usr/lib64/libpthread.so.0: error adding symbols: DSO missing from command
>> line
>>
>> Make sure qemu-ga links with the pthread library, by adding correct
>> flags to libs_qga.
>>
>> Signed-off-by: Marc-André Lureau <address@hidden>
>> ---
>> configure | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/configure b/configure
>> index 0c6e7572db..2b8c71f522 100755
>> --- a/configure
>> +++ b/configure
>> @@ -3436,6 +3436,7 @@ else
>> done
>> if test "$found" = "no"; then
>> LIBS="$pthread_lib $LIBS"
>> + libs_qga="$pthread_lib $libs_qga"
>> fi
>> PTHREAD_LIB="$pthread_lib"
>> break
>
> Hmm why not add it later, around line 4270:
This is check place for librt usage, better to keep it around pthread checks.
>
> if compile_prog "" "" ; then
> :
> # we need pthread for static linking. use previous pthread test result
> elif compile_prog "" "$pthread_lib -lrt" ; then
> LIBS="$LIBS -lrt"
> libs_qga="$libs_qga -lrt" # <-- here
> fi
>
--
Marc-André Lureau