[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vhost-user-test: revert changes to make 'make c
From: |
Gonglei |
Subject: |
Re: [Qemu-devel] [PATCH] vhost-user-test: revert changes to make 'make check' happy |
Date: |
Sun, 26 Oct 2014 16:04:20 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; rv:11.0) Gecko/20120327 Thunderbird/11.0.1 |
On 2014/10/26 15:55, Paolo Bonzini wrote:
>
>
> On 10/26/2014 08:46 AM, address@hidden wrote:
>> From: Gonglei <address@hidden>
>>
>> After commit 89b516d8, some logics is turbid.
>> First, vhost-usr-test.c rely on glib-compat.h because
>> of using G_TIME_SPAN_SECOND [glib < 2.26] and g_get_monotonic_time(),
>> but vhost-usr-test.c defined QEMU_GLIB_COMPAT_H, which make
>> glib-compat.h will not be included.
>> Second, if we remove QEMU_GLIB_COMPAT_H definability in
>> vhost-usr-test.c, then we will get warning as below:
>>
>> tests/vhost-user-test.c: In function 'read_guest_mem':
>> tests/vhost-user-test.c:190: warning: passing argument 1
>> of 'g_mutex_lock' from incompatible pointer type
>> tests/vhost-user-test.c:234: warning: passing argument 1
>> of 'g_mutex_unlock' from incompatible pointer type
>>
>> That's because glib-compat.h redefine the g_mutex_lock/unlock
>> function. Those functions' arguments is CompatGMutex/CompatGCond,
>> but vhost-user-test.c is using GMutex/GCond, which cause the type
>> is not consistent.
>>
>> We can rerealize those functions of vhost-user-test.c,
>> which need a lots of patches. Let's simply address it, and
>> leave this file alone.
>>
>> Signed-off-by: Gonglei <address@hidden>
>> ---
>> The errors and warnings I got:
>>
>> tests/vhost-user-test.c: In function '_cond_wait_until':
>> tests/vhost-user-test.c:154: error: 'G_TIME_SPAN_SECOND' undeclared (first
>> use in this function)
>> tests/vhost-user-test.c:154: error: (Each undeclared identifier is reported
>> only once
>> tests/vhost-user-test.c:154: error: for each function it appears in.)
>> tests/vhost-user-test.c: In function 'read_guest_mem':
>> tests/vhost-user-test.c:192: warning: implicit declaration of function
>> 'g_get_monotonic_time'
>> tests/vhost-user-test.c:192: warning: nested extern declaration of
>> 'g_get_monotonic_time'
>> tests/vhost-user-test.c:192: error: 'G_TIME_SPAN_SECOND' undeclared (first
>> use in this function)
>> make: *** [tests/vhost-user-test.o] Error 1
>>
>> After remove QEMU_GLIB_COMPAT_H definability in vhost-usr-test.c:
>>
>> tests/vhost-user-test.c: In function 'read_guest_mem':
>> tests/vhost-user-test.c:190: warning: passing argument 1 of 'g_mutex_lock'
>> from incompatible pointer type
>> tests/vhost-user-test.c:234: warning: passing argument 1 of 'g_mutex_unlock'
>> from incompatible pointer type
>> tests/vhost-user-test.c: In function 'chr_read':
>> tests/vhost-user-test.c:262: warning: passing argument 1 of 'g_mutex_lock'
>> from incompatible pointer type
>> tests/vhost-user-test.c:295: warning: passing argument 1 of 'g_cond_signal'
>> from incompatible pointer type
>> tests/vhost-user-test.c:312: warning: passing argument 1 of 'g_mutex_unlock'
>> from incompatible pointer type
>> ---
>> tests/vhost-user-test.c | 23 ++++++++++++++++++++++-
>> 1 file changed, 22 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/vhost-user-test.c b/tests/vhost-user-test.c
>> index fdf91e7..75fedf0 100644
>> --- a/tests/vhost-user-test.c
>> +++ b/tests/vhost-user-test.c
>> @@ -21,6 +21,15 @@
>> #include <sys/vfs.h>
>> #include <qemu/sockets.h>
>>
>> +/* GLIB version compatibility flags */
>> +#if !GLIB_CHECK_VERSION(2, 26, 0)
>> +#define G_TIME_SPAN_SECOND (G_GINT64_CONSTANT(1000000))
>> +#endif
>> +
>> +#if GLIB_CHECK_VERSION(2, 28, 0)
>> +#define HAVE_MONOTONIC_TIME
>> +#endif
>> +
>> #if GLIB_CHECK_VERSION(2, 32, 0)
>> #define HAVE_MUTEX_INIT
>> #define HAVE_COND_INIT
>> @@ -107,6 +116,18 @@ static VhostUserMemory memory;
>> static GMutex *data_mutex;
>> static GCond *data_cond;
>>
>> +static gint64 _get_time(void)
>> +{
>> +#ifdef HAVE_MONOTONIC_TIME
>> + return g_get_monotonic_time();
>> +#else
>> + GTimeVal time;
>> + g_get_current_time(&time);
>> +
>> + return time.tv_sec * G_TIME_SPAN_SECOND + time.tv_usec;
>> +#endif
>> +}
>> +
>> static GMutex *_mutex_new(void)
>> {
>> GMutex *mutex;
>> @@ -189,7 +210,7 @@ static void read_guest_mem(void)
>>
>> g_mutex_lock(data_mutex);
>>
>> - end_time = g_get_monotonic_time() + 5 * G_TIME_SPAN_SECOND;
>> + end_time = _get_time() + 5 * G_TIME_SPAN_SECOND;
>> while (!fds_num) {
>> if (!_cond_wait_until(data_cond, data_mutex, end_time)) {
>> /* timeout has passed */
>>
>
> Looks good; sooner or later we should convert vhost-user-test.c to use
> glib-compat.h.
>
Yes. First we should re-realize _cond_wait_until() in glib-compat.h, then
using CompatGMutex/CompatGCond instead of GMutex/GCond.
Best regards,
-Gonglei