[Qemu-devel] [5322] Fix build on FreeBSD

[Qemu-devel] [5322] Fix build on FreeBSD

[Anthony Liguori]

Revision: 5322
          http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=5322
Author:   aliguori
Date:     2008-09-26 15:52:17 +0000 (Fri, 26 Sep 2008)

Log Message:
-----------
Fix build on FreeBSD

__GLIBC_PREREQ is defined in such a way that the ! cannot be used in front of
it on FreeBSD.  Also, -lpthread is not implied by the build and we definitely
use it for compatfd support.

While at it, I added a default initialization for posix-aio that seems to
perform well in our testing.

Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>

Modified Paths:
--------------
    trunk/block-raw-posix.c
    trunk/configure

Modified: trunk/block-raw-posix.c
===================================================================
--- trunk/block-raw-posix.c	2008-09-25 20:24:19 UTC (rev 5321)
+++ trunk/block-raw-posix.c	2008-09-26 15:52:17 UTC (rev 5322)
@@ -525,6 +525,7 @@
 {
     sigset_t mask;
     PosixAioState *s;
+    struct aioinit ai;
   
     if (posix_aio_state)
         return 0;
@@ -545,18 +546,19 @@
 
     qemu_aio_set_fd_handler(s->fd, posix_aio_read, NULL, posix_aio_flush, s);
 
-#if defined(__linux__) && defined(__GLIBC_PREREQ) && !__GLIBC_PREREQ(2, 4)
-    {
-        /* XXX: aio thread exit seems to hang on RedHat 9 and this init
-           seems to fix the problem. */
-        struct aioinit ai;
-        memset(&ai, 0, sizeof(ai));
-        ai.aio_threads = 1;
-        ai.aio_num = 1;
-        ai.aio_idle_time = 365 * 100000;
-        aio_init(&ai);
+    memset(&ai, 0, sizeof(ai));
+#if !defined(__linux__) || (defined(__GLIBC_PREREQ) && __GLIBC_PREREQ(2, 4))
+    ai.aio_threads = 5;
+    ai.aio_num = 1;
+#else
+    /* XXX: aio thread exit seems to hang on RedHat 9 and this init
+       seems to fix the problem. */
+    ai.aio_threads = 1;
+    ai.aio_num = 1;
+    ai.aio_idle_time = 365 * 100000;
     }
 #endif
+    aio_init(&ai);
     posix_aio_state = s;
 
     return 0;

Modified: trunk/configure
===================================================================
--- trunk/configure	2008-09-25 20:24:19 UTC (rev 5321)
+++ trunk/configure	2008-09-26 15:52:17 UTC (rev 5322)
@@ -482,8 +482,10 @@
     linux_user="no"
 fi
 
-if [ "$bsd" = "yes" -o "$darwin" = "yes" -o "$mingw32" = "yes" ] ; then
+if [ "$darwin" = "yes" -o "$mingw32" = "yes" ] ; then
     AIOLIBS=
+elif [ "$bsd" = "yes" ]; then
+    AIOLIBS="-lpthread"
 else
     # Some Linux architectures (e.g. s390) don't imply -lpthread automatically.
     AIOLIBS="-lrt -lpthread"