[Top][All Lists]
[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