qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [diff] BSD support (NetBSD, FreeBSD, Dragonfly)


From: Juergen Lock
Subject: Re: [Qemu-devel] [diff] BSD support (NetBSD, FreeBSD, Dragonfly)
Date: Mon, 12 Oct 2009 01:31:23 +0200
User-agent: Mutt/1.5.20 (2009-06-14)

On Sun, Oct 11, 2009 at 08:57:14PM +0400, Aleksej Saushev wrote:
>   Hello!
Hi!
> 
> qemu 0.11.0 doesn't build on NetBSD out of the box,
> it doesn't on FreeBSD and Dragonfly either.
> 
> --- exec.c.orig       2009-09-23 23:01:32.000000000 +0400
> +++ exec.c    2009-09-30 14:03:55.000000000 +0400
> @@ -464,7 +464,7 @@
>              exit(1);
>          }
>      }
> -#elif defined(__FreeBSD__) || defined(__DragonFly__)
> +#elif defined(__FreeBSD__) || defined(__DragonFly__) || defined(__NetBSD__)
>      {
>          int flags;
>          void *addr = NULL;
> --- osdep.c.orig      2009-09-23 23:01:32.000000000 +0400
> +++ osdep.c   2009-09-30 14:05:39.000000000 +0400
> @@ -110,7 +110,7 @@
>      void *ptr;
>  
>  /* no need (?) for a dummy file on OpenBSD/FreeBSD */
> -#if defined(__OpenBSD__) || defined(__FreeBSD__) || defined(__DragonFly__)
> +#if defined(__OpenBSD__) || defined(__FreeBSD__) || defined(__DragonFly__) 
> || defined(__NetBSD__)
>      int map_anon = MAP_ANON;
>  #else
>      int map_anon = 0;
> @@ -181,7 +181,7 @@
>      }
>      size = (size + 4095) & ~4095;
>      ftruncate(phys_ram_fd, phys_ram_size + size);
> -#endif /* !(__OpenBSD__ || __FreeBSD__ || __DragonFly__) */
> +#endif /* !(__OpenBSD__ || __FreeBSD__ || __DragonFly__ || __NetBSD__) */
>      ptr = mmap(NULL,
>                 size,
>                 PROT_WRITE | PROT_READ, map_anon | MAP_SHARED,

 While you are patching this code...  There's a bug here that I've
mentioned before:  size needs to be rounded up on the BSDs too,
at least if you want to use kqemu:  (which this oode is for.)

--- a/qemu/osdep.c
+++ b/qemu/osdep.c
@@ -110,7 +110,7 @@
     void *ptr;
 
 /* no need (?) for a dummy file on OpenBSD/FreeBSD */
-#if defined(__OpenBSD__) || defined(__FreeBSD__) || defined(__DragonFly__)
+#if defined(__OpenBSD__) || defined(__FreeBSD__) || defined(__DragonFly__) || 
defined(__NetBSD__)
     int map_anon = MAP_ANON;
 #else
     int map_anon = 0;
@@ -179,9 +179,11 @@
         }
         unlink(phys_ram_file);
     }
+#endif /* !(__OpenBSD__ || __FreeBSD__ || __DragonFly__ || __NetBSD__) */
     size = (size + 4095) & ~4095;
+#if !defined(__OpenBSD__) && !defined(__FreeBSD__) && !defined(__DragonFly__) 
|| defined(__NetBSD__)
     ftruncate(phys_ram_fd, phys_ram_size + size);
-#endif /* !(__OpenBSD__ || __FreeBSD__ || __DragonFly__) */
+#endif /* !(__OpenBSD__ || __FreeBSD__ || __DragonFly__ || __NetBSD__) */
     ptr = mmap(NULL,
                size,
                PROT_WRITE | PROT_READ, map_anon | MAP_SHARED,

 (Otoh kqemu has been removed in qemu git now so this can't be
committed anyway...)

> --- vl.c.orig 2009-09-23 23:01:32.000000000 +0400
> +++ vl.c      2009-09-30 14:10:21.000000000 +0400
> @@ -567,7 +567,7 @@
>  {
>      use_rt_clock = 0;
>  #if defined(__linux__) || (defined(__FreeBSD__) && __FreeBSD_version >= 
> 500000) \
> -    || defined(__DragonFly__)
> +    || defined(__DragonFly__) || defined(__NetBSD__)
>      {
>          struct timespec ts;
>          if (clock_gettime(CLOCK_MONOTONIC, &ts) == 0) {
> @@ -580,7 +580,7 @@
>  static int64_t get_clock(void)
>  {
>  #if defined(__linux__) || (defined(__FreeBSD__) && __FreeBSD_version >= 
> 500000) \
> -     || defined(__DragonFly__)
> +     || defined(__DragonFly__) || defined(__NetBSD__)
>      if (use_rt_clock) {
>          struct timespec ts;
>          clock_gettime(CLOCK_MONOTONIC, &ts);
> --- cpu-exec.c.orig   2009-09-23 23:01:32.000000000 +0400
> +++ cpu-exec.c        2009-09-30 14:18:16.000000000 +0400
> @@ -1247,6 +1247,20 @@
>  # define TRAP_sig(context)    ((context)->uc_mcontext->es.trapno)
>  # define ERROR_sig(context)   ((context)->uc_mcontext->es.err)
>  # define MASK_sig(context)    ((context)->uc_sigmask)
> +#elif defined (__NetBSD__)
> +# include <ucontext.h>
> +
> +# define EIP_sig(context)     ((context)->uc_mcontext.__gregs[_REG_EIP])
> +# define TRAP_sig(context)    ((context)->uc_mcontext.__gregs[_REG_TRAPNO])
> +# define ERROR_sig(context)   ((context)->uc_mcontext.__gregs[_REG_ERR])
> +# define MASK_sig(context)    ((context)->uc_sigmask)
> +#elif defined (__FreeBSD__) || defined(__DragonFly__)
> +# include <ucontext.h>
> +
> +# define EIP_sig(context)  (*((unsigned 
> long*)&(context)->uc_mcontext.mc_eip))
> +# define TRAP_sig(context)    ((context)->uc_mcontext.mc_trapno)
> +# define ERROR_sig(context)   ((context)->uc_mcontext.mc_err)
> +# define MASK_sig(context)    ((context)->uc_sigmask)
>  #elif defined(__OpenBSD__)
>  # define EIP_sig(context)     ((context)->sc_eip)
>  # define TRAP_sig(context)    ((context)->sc_trapno)
> @@ -1263,7 +1277,9 @@
>                         void *puc)
>  {
>      siginfo_t *info = pinfo;
> -#if defined(__OpenBSD__)
> +#if defined(__NetBSD__) || defined (__FreeBSD__) || defined(__DragonFly__)
> +    ucontext_t *uc = puc;
> +#elif defined(__OpenBSD__)
>      struct sigcontext *uc = puc;
>  #else
>      struct ucontext *uc = puc;
> 
> 

 And I had a simpler (FreeBSD-only) version of this diff in my bsd-user
patches [1], so I took yours instead and added amd64 code:

--- a/qemu/cpu-exec.c
+++ b/qemu/cpu-exec.c
@@ -805,6 +805,20 @@ static inline int handle_cpu_signal(unsi
 # define TRAP_sig(context)    ((context)->uc_mcontext->es.trapno)
 # define ERROR_sig(context)   ((context)->uc_mcontext->es.err)
 # define MASK_sig(context)    ((context)->uc_sigmask)
+#elif defined (__NetBSD__)
+# include <ucontext.h>
+
+# define EIP_sig(context)     ((context)->uc_mcontext.__gregs[_REG_EIP])
+# define TRAP_sig(context)    ((context)->uc_mcontext.__gregs[_REG_TRAPNO])
+# define ERROR_sig(context)   ((context)->uc_mcontext.__gregs[_REG_ERR])
+# define MASK_sig(context)    ((context)->uc_sigmask)
+#elif defined (__FreeBSD__) || defined(__DragonFly__)
+# include <ucontext.h>
+
+# define EIP_sig(context)  (*((unsigned long*)&(context)->uc_mcontext.mc_eip))
+# define TRAP_sig(context)    ((context)->uc_mcontext.mc_trapno)
+# define ERROR_sig(context)   ((context)->uc_mcontext.mc_err)
+# define MASK_sig(context)    ((context)->uc_sigmask)
 #elif defined(__OpenBSD__)
 # define EIP_sig(context)     ((context)->sc_eip)
 # define TRAP_sig(context)    ((context)->sc_trapno)
@@ -821,7 +835,9 @@ int cpu_signal_handler(int host_signum, 
                        void *puc)
 {
     siginfo_t *info = pinfo;
-#if defined(__OpenBSD__)
+#if defined(__NetBSD__) || defined (__FreeBSD__) || defined(__DragonFly__)
+    ucontext_t *uc = puc;
+#elif defined(__OpenBSD__)
     struct sigcontext *uc = puc;
 #else
     struct ucontext *uc = puc;
@@ -855,6 +871,13 @@ int cpu_signal_handler(int host_signum, 
 #define TRAP_sig(context)     ((context)->sc_trapno)
 #define ERROR_sig(context)    ((context)->sc_err)
 #define MASK_sig(context)     ((context)->sc_mask)
+#elif defined (__FreeBSD__) || defined(__DragonFly__)
+#include <ucontext.h>
+
+#define PC_sig(context)  (*((unsigned long*)&(context)->uc_mcontext.mc_rip))
+#define TRAP_sig(context)     ((context)->uc_mcontext.mc_trapno)
+#define ERROR_sig(context)    ((context)->uc_mcontext.mc_err)
+#define MASK_sig(context)     ((context)->uc_sigmask)
 #else
 #define PC_sig(context)       ((context)->uc_mcontext.gregs[REG_RIP])
 #define TRAP_sig(context)     ((context)->uc_mcontext.gregs[REG_TRAPNO])
@@ -867,7 +890,7 @@ int cpu_signal_handler(int host_signum, 
 {
     siginfo_t *info = pinfo;
     unsigned long pc;
-#ifdef __NetBSD__
+#if defined(__NetBSD__) || defined (__FreeBSD__) || defined(__DragonFly__)
     ucontext_t *uc = puc;
 #elif defined(__OpenBSD__)
     struct sigcontext *uc = puc;

 Signed-off-by: Juergen Lock <address@hidden>

 (Btw you need to post such a line too when submitting patches, I can't
find the relevant thread from the qemu list now but I found at least this:
        http://kerneltrap.org/node/3180
)

 Cheers,
        Juergen

[1] See this thread:
        
http://lists.freebsd.org/pipermail/freebsd-emulation/2009-October/006953.html




reply via email to

[Prev in Thread] Current Thread [Next in Thread]