[Qemu-devel] [PATCH] BSD build fixes

[Qemu-devel] [PATCH] BSD build fixes

[Christoph Egger]

[-- Attachment #1: Type: text/plain, Size: 547 bytes --]


Hi!

Attached patch fixes some build issues for BSD.

Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>



-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy

[-- Attachment #2: qemu.diff --]
[-- Type: text/x-diff, Size: 2902 bytes --]

Index: vl.c
===================================================================
--- vl.c	(revision 5608)
+++ vl.c	(working copy)
@@ -54,26 +54,31 @@
 #include <termios.h>
 #include <sys/mman.h>
 #include <sys/ioctl.h>
+#include <sys/resource.h>
 #include <sys/socket.h>
 #include <netinet/in.h>
+#include <net/if.h>
+#if defined(__NetBSD__)
+#include <net/if_tap.h>
+#endif
+#ifdef __linux__
+#include <linux/if_tun.h>
+#endif
+#include <arpa/inet.h>
 #include <dirent.h>
 #include <netdb.h>
 #include <sys/select.h>
-#include <arpa/inet.h>
 #ifdef _BSD
 #include <sys/stat.h>
-#if !defined(__APPLE__) && !defined(__OpenBSD__)
+#ifdef __FreeBSD__
 #include <libutil.h>
+#else
+#include <util.h>
 #endif
-#ifdef __OpenBSD__
-#include <net/if.h>
-#endif
 #elif defined (__GLIBC__) && defined (__FreeBSD_kernel__)
 #include <freebsd/stdlib.h>
 #else
 #ifdef __linux__
-#include <linux/if.h>
-#include <linux/if_tun.h>
 #include <pty.h>
 #include <malloc.h>
 #include <linux/rtc.h>
Index: net.c
===================================================================
--- net.c	(revision 5608)
+++ net.c	(working copy)
@@ -54,26 +54,31 @@
 #include <termios.h>
 #include <sys/mman.h>
 #include <sys/ioctl.h>
+#include <sys/resource.h>
 #include <sys/socket.h>
 #include <netinet/in.h>
+#include <net/if.h>
+#ifdef __NetBSD__
+#include <net/if_tap.h>
+#endif
+#ifdef __linux__
+#include <linux/if_tun.h>
+#endif
+#include <arpa/inet.h>
 #include <dirent.h>
 #include <netdb.h>
 #include <sys/select.h>
-#include <arpa/inet.h>
 #ifdef _BSD
 #include <sys/stat.h>
-#if !defined(__APPLE__) && !defined(__OpenBSD__)
+#ifdef __FreeBSD__
 #include <libutil.h>
+#else
+#include <util.h>
 #endif
-#ifdef __OpenBSD__
-#include <net/if.h>
-#endif
 #elif defined (__GLIBC__) && defined (__FreeBSD_kernel__)
 #include <freebsd/stdlib.h>
 #else
 #ifdef __linux__
-#include <linux/if.h>
-#include <linux/if_tun.h>
 #include <pty.h>
 #include <malloc.h>
 #include <linux/rtc.h>
Index: qemu-char.c
===================================================================
--- qemu-char.c	(revision 5608)
+++ qemu-char.c	(working copy)
@@ -45,25 +45,31 @@
 #include <termios.h>
 #include <sys/mman.h>
 #include <sys/ioctl.h>
+#include <sys/resource.h>
 #include <sys/socket.h>
 #include <netinet/in.h>
+#include <net/if.h>
+#ifdef __NetBSD__
+#include <net/if_tap.h>
+#endif
+#ifdef __linux__
+#include <linux/if_tun.h>
+#endif
+#include <arpa/inet.h>
 #include <dirent.h>
 #include <netdb.h>
 #include <sys/select.h>
-#include <arpa/inet.h>
 #ifdef _BSD
 #include <sys/stat.h>
-#if !defined(__APPLE__) && !defined(__OpenBSD__)
+#ifdef __FreeBSD__
 #include <libutil.h>
+#else
+#include <util.h>
 #endif
-#ifdef __OpenBSD__
-#include <net/if.h>
-#endif
 #elif defined (__GLIBC__) && defined (__FreeBSD_kernel__)
 #include <freebsd/stdlib.h>
 #else
 #ifdef __linux__
-#include <linux/if.h>
 #include <pty.h>
 
 #include <linux/ppdev.h>

Re: [Qemu-devel] [PATCH] BSD build fixes

[Blue Swirl]

On 11/4/08, Christoph Egger <Christoph.Egger@amd.com> wrote:
>
>  Hi!
>
>  Attached patch fixes some build issues for BSD.

Qemu compiled fine on OpenBSD without the patch, but it's still OK
after the patch too. Did you test both FreeBSD and NetBSD?

[Qemu-devel] Re: [PATCH] BSD build fixes

[Christoph Egger]

> > Hi!
> >
> > Attached patch fixes some build issues for BSD.
> 
> Qemu compiled fine on OpenBSD without the patch, but it's still OK
> after the patch too. Did you test both FreeBSD and NetBSD?
>
> Blue Swirl

Yes. NetBSD in particular.

(I have more patches in the queue, but the complete patch as a whole was
ignored on this list, so I splitted it in smaller pieces.)

Christoph


-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy

Re: [Qemu-devel] Re: [PATCH] BSD build fixes

[Blue Swirl]

On 11/6/08, Christoph Egger <Christoph.Egger@amd.com> wrote:
>
>
>  > > Hi!
>  > >
>  > > Attached patch fixes some build issues for BSD.
>  >
>
> > Qemu compiled fine on OpenBSD without the patch, but it's still OK
>  > after the patch too. Did you test both FreeBSD and NetBSD?
>  >
>
> > Blue Swirl
>
>  Yes. NetBSD in particular.

Thanks, applied.