[Qemu-devel] sdl-less qemu - shouldnt -nographic be default?

[Juergen Lock]

After adding a WITHOUT_SDL option to the FreeBSD qemu port (see
        http://docs.freebsd.org/cgi/mid.cgi?20061028183618.GA46200
, and hi joshe! :) I got a crash that looks like monitor output
going to an uninitialized vty:

$ gdb work/qemu-snapshot-2006-10-26_05/i386-softmmu/qemu qemu.core
GNU gdb 6.1.1 [FreeBSD]
Copyright 2004 Free Software Foundation, Inc.
GDB is free software, covered by the GNU General Public License, and you are
welcome to change it and/or distribute copies of it under certain conditions.
Type "show copying" to see the conditions.
There is absolutely no warranty for GDB.  Type "show warranty" for details.
This GDB was configured as "i386-marcel-freebsd"...
Core was generated by `qemu'.
Program terminated with signal 11, Segmentation fault.
Reading symbols from /lib/libm.so.4...done.
Loaded symbols for /lib/libm.so.4
Reading symbols from /lib/libz.so.3...done.
Loaded symbols for /lib/libz.so.3
Reading symbols from /lib/libutil.so.5...done.
Loaded symbols for /lib/libutil.so.5
Reading symbols from /lib/libc.so.6...done.
Loaded symbols for /lib/libc.so.6
Reading symbols from /libexec/ld-elf.so.1...done.
Loaded symbols for /libexec/ld-elf.so.1
#0  console_puts (chr=0x0, 
    buf=0x8125f20 "QEMU 0.8.2 monitor - type 'help' for more information\r\n", 
    len=55)
    at 
/ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/console.c:790
790                 c->ch = ch;
(gdb) l
785                 s->state = TTY_STATE_ESC;
786                 break;
787             default:
788                 y1 = (s->y_base + s->y) % s->total_height;
789                 c = &s->cells[y1 * s->width + s->x];
790                 c->ch = ch;
791                 c->t_attrib = s->t_attrib;
792                 update_xy(s, s->x, s->y);
793                 s->x++;
794                 if (s->x >= s->width) {
(gdb) p s
$1 = (TextConsole *) 0x9b18100
(gdb) p *s
$2 = {text_console = 1, ds = 0x8123740, hw_update = 0, hw_invalidate = 0, 
  hw_screen_dump = 0, hw = 0x0, g_width = 0, g_height = 0, width = 0, 
  height = 0, total_height = 512, backscroll_height = 0, x = 0, y = 0, 
  y_displayed = 0, y_base = 0, t_attrib_default = {fgcol = 7 '\a', 
    bgcol = 0 '\0', bold = 0 '\0', uline = 0 '\0', blink = 0 '\0', 
    invers = 0 '\0', unvisible = 0 '\0'}, t_attrib = {fgcol = 7 '\a', 
    bgcol = 0 '\0', bold = 0 '\0', uline = 0 '\0', blink = 0 '\0', 
    invers = 0 '\0', unvisible = 0 '\0'}, cells = 0x800, 
  state = TTY_STATE_NORM, esc_params = {0, 0, 0}, nb_esc_params = 0, 
  fd_can_read = 0, fd_read = 0, fd_opaque = 0x0, out_fifo = {
    buf = 0x9b1817c "", buf_size = 16, count = 0, wptr = 0, rptr = 0}, 
  out_fifo_buf = '\0' <repeats 15 times>, kbd_timer = 0x9b10120}
(gdb) bt
#0  console_puts (chr=0x0, 
    buf=0x8125f20 "QEMU 0.8.2 monitor - type 'help' for more information\r\n", 
    len=55)
    at 
/ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/console.c:790
#1  0x08052d75 in term_flush ()
    at 
/ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/monitor.c:72
#2  0x08052dbb in term_puts (str=0xbfbfade6 "")
    at 
/ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/monitor.c:90
#3  0x08052e01 in term_vprintf (
    fmt=0x80ffa50 "QEMU %s monitor - type 'help' for more information\n", 
    ap=0xbfbfbde4 "UK\020\bXè¿¿\002")
    at 
/ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/monitor.c:98
#4  0x08052e1a in term_printf (
    fmt=0x80ffa50 "QEMU %s monitor - type 'help' for more information\n")
    at 
/ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/monitor.c:105
#5  0x08055602 in monitor_init (hd=0x9b10100, show_banner=1)
    at 
/ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/monitor.c:2382
---Type <return> to continue, or q <return> to quit---
#6  0x08051244 in main (argc=8, argv=0xbfbfe8a4)
    at /ad4/usr/ports/emulators/qemu/work/qemu-snapshot-2006-10-26_05/vl.c:7025
(gdb) q

 Is this supposed to work?  Is it a bug?  Should -nographic be
default in this case?

 Curious...
        Juergen