@@ -1185,6 +1138,7 @@ void kbd_put_keysym_console(QemuConsole *s,
int keysym)
uint8_t buf[16], *q;
CharBackend *be;
int c;
+ uint32_t free;
Better call it num_free, to avoid symbol clash (even if we don't use
free() directly), it helps reading and can prevent mistakes.
Hi,
OK, I'll send a version 2 patch.
if (!s || (s->console_type == GRAPHIC_CONSOLE))
return;
@@ -2233,8 +2188,7 @@ static void text_console_do_init(Chardev
*chr, DisplayState *ds)
int g_width = 80 * FONT_WIDTH;
int g_height = 24 * FONT_HEIGHT;
- s->out_fifo.buf = s->out_fifo_buf;
- s->out_fifo.buf_size = sizeof(s->out_fifo_buf);
+ fifo8_create(&s->out_fifo, 16);
Missing a fif8_destroy() somewhere
An opened text console stays open until QEMU exits. There's no
text_console_close() function. Just like there's a ChardevClass open
call but no close call. I think this is one of the many cases in QEMU
where resources get allocated for the lifetime of QEMU.