diff mbox

Fix compilation for non-POSIX system (w64)

Message ID 1308856793-29724-1-git-send-email-weil@mail.berlios.de
State Superseded
Headers show

Commit Message

Stefan Weil June 23, 2011, 7:19 p.m. UTC 
compatfd.h is only needed with CONFIG_POSIX.
Compilation fails in compatfd.h for at least one non-POSIX systems (w64).

Signed-off-by: Stefan Weil <weil@mail.berlios.de>
---
 cpus.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Comments

Stefan Hajnoczi June 24, 2011, 10:24 a.m. UTC | #1 
On Thu, Jun 23, 2011 at 09:19:53PM +0200, Stefan Weil wrote:
> compatfd.h is only needed with CONFIG_POSIX.
> Compilation fails in compatfd.h for at least one non-POSIX systems (w64).
> 
> Signed-off-by: Stefan Weil <weil@mail.berlios.de>
> ---
>  cpus.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)

Thanks!  Jan's patch which is similar but uses #ifndef _WIN32 is already
queued.  The cpus.c qemu_signalfd() usage is #ifndef _WIN32, so I think
there's a logic to keeping that patch.

Stefan
diff mbox

Patch

diff --git a/cpus.c b/cpus.c
index 0699f37..8501254 100644
--- a/cpus.c
+++ b/cpus.c
@@ -34,7 +34,9 @@ 
 
 #include "qemu-thread.h"
 #include "cpus.h"
+#ifdef CONFIG_POSIX
 #include "compatfd.h"
+#endif
 
 #ifdef SIGRTMIN
 #define SIG_IPI (SIGRTMIN+4)