qemu coredumps on RELENG_7

[Juergen Lock]

On Sat, Mar 22, 2008 at 04:13:29PM +0100, Oliver Fromme wrote:
> 
> Juergen Lock wrote:
>  > Oliver Fromme wrote:
>  > > D Hill <d.hill at yournetplus.com> wrote:
>  > > > Rick C. Petty wrote:
>  > > > > Oliver Fromme wrote:
>  > > > > > Kostik Belousov wrote:
>  > > > > > > 
>  > > > > > > Definitely,
>  > > > > > > kldload aio
>  > > > > > > before running qemu.
>  > > > > > 
>  > > > > > Thank you, that seems to solve the problem indeed.
>  > > > > > Is that documented somewhere?  It's not in the manpage.
>  > > > > > I think it should be in the manpage ...  would have
>  > > > > > saved me quite some time.
>  > > > > 
>  > Actually it is also documented in the pkg-message of the port(s), but
>  > apparently nobody reads that...
> 
> How are you supposed to read it when you install a bunch
> of ports in batch mode, i.e. unattended (for example,
> over night or during lunch break)?
> 
> Therefore I think that pkg-message is _not_ the correct
> place for critical ionformation like that.
> 
>  > (OK I could patch a pointer to that
>  > into the manpage, do other ports do that?)
> 
> There are several ports that patch port-specific things
> into the manpage.  Prominent examples are gzip, jpeg,
> bash, cdrtools and openssh-portable.
> 
Alright, so how about the following:

Index: qemu/qemu-doc.texi
@@ -985,7 +985,8 @@
 
 @c man begin SEEALSO
 The HTML documentation of QEMU for more precise information and Linux
-user mode emulator invocation.
+user mode emulator invocation, as well as the FreeBSD host notes in
+ at file{pkg-message} in the relevant qemu port directory.
 @c man end
 
 @c man begin AUTHOR

>  >  Anyway, this was easier than I thought: (it didn't occur to me that
>  > modfind(2) also finds modules that are statically linked into the kernel...)
>  > 
>  > Index: qemu/vl.c
>  > @@ -8423,6 +8423,12 @@
>  >      nb_nics = 0;
>  >      /* default mac address of the first network interface */
>  >  
>  > +#ifdef __FreeBSD__
>  > +    loadmodules(0, "aio", NULL);
>  > +    if (modfind("aio") == -1)
>  > +        fprintf(stderr, "warning: aio not (kld)loaded, may cause `Invalid system call' traps on disk IO\n");
>  > +#endif
> 
> I prefer it didn't try to load the module itself.  That's
> a decision I want to make myself.

 Yeah I removed the load attempt.

>  But the message is
> perfectly fine.  It would have saved me quite some time.
> 
> Thanks!
> 
> Best regards
>    Oliver

 You're welcome,
	Juergen