Just when I thought it was all working... Unfortunately cpu.h defines

clockframe -> intrframe, but that is included too late, because this
file includes systm.h and it is in the path of including systm.h. Fix
it by not including <systm.h>; it was only needed for the panic() calls
which I have disabled, since they look more like debugging calls to me.
Also add forward struct declaration for trapframe.

sys/arch/xen/include/hypervisor.h

@@ -1,4 +1,4 @@
-/*	$NetBSD: hypervisor.h,v 1.8 2004/06/14 13:55:52 cl Exp $	*/
+/*	$NetBSD: hypervisor.h,v 1.9 2004/12/10 18:51:15 christos Exp $	*/
 
 /*
  * 
@@ -47,8 +47,6 @@ struct xen_npx_attach_args {
 #define	u32 uint32_t
 #define	u64 uint64_t
 
-/* include the hypervisor interface */
-#include <sys/systm.h>
 #include <machine/hypervisor-ifs/block.h>
 #include <machine/hypervisor-ifs/hypervisor-if.h>
 #include <machine/hypervisor-ifs/dom0_ops.h>
@@ -73,6 +71,7 @@ extern union start_info_union start_info_union;
 
 
 /* hypervisor.c */
+struct trapframe;
 void do_hypervisor_callback(struct trapframe *regs);
 void hypervisor_enable_event(unsigned int ev);
 void hypervisor_disable_event(unsigned int ev);
@@ -101,8 +100,10 @@ static inline int HYPERVISOR_mmu_update(mmu_update_t *req, int count)
         : "=a" (ret) : "0" (__HYPERVISOR_mmu_update), 
         "b" (req), "c" (count) : "memory" );
 
+#ifdef notdef
     if (__predict_false(ret < 0))
         panic("Failed mmu update: %p, %d", req, count);
+#endif
 
     return ret;
 }
@@ -343,9 +344,11 @@ static inline int HYPERVISOR_update_va_mapping(
         : "=a" (ret) : "0" (__HYPERVISOR_update_va_mapping), 
         "b" (page_nr), "c" (new_val), "d" (flags) : "memory" );
 
+#ifdef notdef
     if (__predict_false(ret < 0))
         panic("Failed update VA mapping: %08lx, %08lx, %08lx",
               page_nr, new_val, flags);
+#endif
 
     return ret;
 }