mirrors/Bochs
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Implemented new text update API in the X gui and some related fixes.

Browse Source 
- The X gui behaviour should be unchanged.
- RFB / vncsrv: cursor shape fixes.
- Changed format of the font parameters to fix "double width" characters.
- Restore guest charmap after leaving gui console.
This commit is contained in:
Volker Ruppert 2020-11-16 15:39:35 +00:00
parent b94fff0501
commit 45507705bf
9 changed files with 137 additions and 275 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
bochs/gui/gui.cc
View File

@ -139,7 +139,7 @@ bx_gui_c::bx_gui_c(void): disp_mode(DISP_MODE_SIM)
led_timer_index = BX_NULL_TIMER_HANDLE;
framebuffer = NULL;
guest_textmode = 1;
guest_fsize = (16 << 4) | 8;
guest_fsize = (16 << 8) | 8;
guest_xres = 640;
guest_yres = 480;
guest_bpp = 8;
@ -1146,7 +1146,7 @@ void bx_gui_c::draw_char_common(Bit8u ch, Bit8u fc, Bit8u bc, Bit16u xc,
bx_bool dwidth;
buf = BX_GUI_THIS snapshot_buffer + yc * BX_GUI_THIS guest_xres + xc;
fwidth = BX_GUI_THIS guest_fsize & 0x0f;
fwidth = BX_GUI_THIS guest_fsize & 0x1f;
dwidth = (fwidth > 9);
font_ptr = &vga_charmap[(ch << 5) + fy];
do {
@ -1190,8 +1190,8 @@ void bx_gui_c::text_update_common(Bit8u *old_text, Bit8u *new_text,
bx_bool forceUpdate = 0, blink_mode = 0, blink_state = 0;
if (BX_GUI_THIS snapshot_mode || BX_GUI_THIS new_text_api) {
fheight = BX_GUI_THIS guest_fsize >> 4;
fwidth = BX_GUI_THIS guest_fsize & 0x0f;
fheight = BX_GUI_THIS guest_fsize >> 8;
fwidth = BX_GUI_THIS guest_fsize & 0x1f;
cursor_visible = ((tm_info->cs_start <= tm_info->cs_end) && (tm_info->cs_start < fheight));
if (BX_GUI_THIS snapshot_mode && (BX_GUI_THIS snapshot_buffer != NULL)) {
forceUpdate = 1;
@ -1204,7 +1204,7 @@ void bx_gui_c::text_update_common(Bit8u *old_text, Bit8u *new_text,
if (!blink_state) cursor_visible = 0;
}
if (BX_GUI_THIS charmap_updated) {
BX_GUI_THIS set_font();
BX_GUI_THIS set_font(tm_info->line_graphics);
BX_GUI_THIS charmap_updated = 0;
forceUpdate = 1;
}
@ -1413,6 +1413,7 @@ void bx_gui_c::console_init(void)
console.saved_yres = guest_yres;
console.saved_bpp = guest_bpp;
console.saved_fsize = guest_fsize;
memcpy(console.saved_charmap, BX_GUI_THIS vga_charmap, 0x2000);
for (i = 0; i < 256; i++) {
memcpy(&BX_GUI_THIS vga_charmap[0]+i*32, &sdl_font8x16[i], 16);
BX_GUI_THIS char_changed[i] = 1;
@ -1443,8 +1444,9 @@ void bx_gui_c::console_cleanup(void)
console.saved_palette[0]);
palette_change_common(0x07, console.saved_palette[30], console.saved_palette[29],
console.saved_palette[28]);
unsigned fheight = (console.saved_fsize >> 4);
unsigned fwidth = (console.saved_fsize & 0x0f);
unsigned fheight = (console.saved_fsize >> 8);
unsigned fwidth = (console.saved_fsize & 0x1f);
set_text_charmap(console.saved_charmap);
dimension_update(console.saved_xres, console.saved_yres, fheight, fwidth,
console.saved_bpp);
console.running = 0;

21
bochs/gui/gui.h
View File

@ -129,7 +129,7 @@ public:
virtual Bit8u *graphics_tile_get(unsigned x, unsigned y, unsigned *w, unsigned *h);
virtual void graphics_tile_update_in_place(unsigned x, unsigned y, unsigned w, unsigned h);
// new text update API
virtual void set_font(void) {}
virtual void set_font(bx_bool lg) {}
virtual void draw_char(Bit8u ch, Bit8u fc, Bit8u bc, Bit16u xc, Bit16u yc,
Bit8u fw, Bit8u fh, Bit8u fx, Bit8u fy,
bx_bool gfxcharw9, Bit8u cs, Bit8u ce, bx_bool curs) {}
@ -274,7 +274,7 @@ protected:
void (*f)(void);
} bx_headerbar_entry[BX_MAX_HEADERBAR_ENTRIES];
// text charmap
unsigned char vga_charmap[0x2000];
Bit8u vga_charmap[0x2000];
bx_bool charmap_updated;
bx_bool char_changed[256];
// status bar items
@ -308,10 +308,10 @@ protected:
unsigned y_tilesize;
// current guest display settings
bx_bool guest_textmode;
unsigned guest_fsize;
unsigned guest_xres;
unsigned guest_yres;
unsigned guest_bpp;
Bit16u guest_fsize;
Bit16u guest_xres;
Bit16u guest_yres;
Bit8u guest_bpp;
// graphics snapshot
bx_bool snapshot_mode;
Bit8u *snapshot_buffer;
@ -339,11 +339,12 @@ protected:
bx_bool running;
Bit8u *screen;
Bit8u *oldscreen;
unsigned saved_fsize;
unsigned saved_xres;
unsigned saved_yres;
unsigned saved_bpp;
Bit16u saved_fsize;
Bit16u saved_xres;
Bit16u saved_yres;
Bit8u saved_bpp;
Bit8u saved_palette[32];
Bit8u saved_charmap[0x2000];
unsigned cursor_x;
unsigned cursor_y;
Bit16u cursor_addr;

15
bochs/gui/rfb.cc
View File

@ -367,9 +367,16 @@ void bx_rfb_gui_c::draw_char(Bit8u ch, Bit8u fc, Bit8u bc, Bit16u xc, Bit16u yc,
DrawChar(xc, yc, fw, fh, fx, fy, (char *)&vga_charmap[ch << 5], fgcol, bgcol,
gfxcharw9);
rfbAddUpdateRegion(xc, yc, fw, fh);
if (curs) {
DrawChar(xc, yc + cs, fw, ce - cs + 1, fx, cs, (char *)&vga_charmap[ch << 5],
bgcol, fgcol, gfxcharw9);
if (curs && (ce >= fy) && (cs < (fh + fy))) {
if (cs > fy) {
yc += (cs - fy);
fh -= (cs - fy);
}
if ((ce - cs + 1) < fh) {
fh = ce - cs + 1;
}
DrawChar(xc, yc, fw, fh, fx, cs, (char *)&vga_charmap[ch << 5], bgcol,
fgcol, gfxcharw9);
}
}
@ -433,7 +440,7 @@ void bx_rfb_gui_c::dimension_update(unsigned x, unsigned y, unsigned fheight, un
BX_PANIC(("%d bpp graphics mode not supported yet", bpp));
}
guest_textmode = (fheight > 0);
guest_fsize = (fheight << 4) | fwidth;
guest_fsize = (fheight << 8) | fwidth;
guest_xres = x;
guest_yres = y;
if (guest_textmode) {

2
bochs/gui/sdl.cc
View File

@ -1203,7 +1203,7 @@ void bx_sdl_gui_c::dimension_update(unsigned x, unsigned y,
BX_PANIC(("%d bpp graphics mode not supported", bpp));
}
guest_textmode = (fheight > 0);
guest_fsize = (fheight << 4) | fwidth;
guest_fsize = (fheight << 8) | fwidth;
guest_xres = x;
guest_yres = y;
if (guest_textmode) {

2
bochs/gui/sdl2.cc
View File

@ -1118,7 +1118,7 @@ void bx_sdl2_gui_c::dimension_update(unsigned x, unsigned y,
BX_PANIC(("%d bpp graphics mode not supported", bpp));
}
guest_textmode = (fheight > 0);
guest_fsize = (fheight << 4) | fwidth;
guest_fsize = (fheight << 8) | fwidth;
guest_xres = x;
guest_yres = y;
if (guest_textmode) {

15
bochs/gui/vncsrv.cc
View File

@ -376,9 +376,16 @@ void bx_vncsrv_gui_c::draw_char(Bit8u ch, Bit8u fc, Bit8u bc, Bit16u xc,
DrawChar(xc, yc, fw, fh, fx, fy, (char *)&vga_charmap[ch << 5], fgcol, bgcol,
gfxcharw9);
SendUpdate(xc, yc, fw, fh);
if (curs) {
DrawChar(xc, yc + cs, fw, ce - cs + 1, fx, cs, (char *)&vga_charmap[ch << 5],
bgcol, fgcol, gfxcharw9);
if (curs && (ce >= fy) && (cs < (fh + fy))) {
if (cs > fy) {
yc += (cs - fy);
fh -= (cs - fy);
}
if ((ce - cs + 1) < fh) {
fh = ce - cs + 1;
}
DrawChar(xc, yc, fw, fh, fx, cs, (char *)&vga_charmap[ch << 5], bgcol,
fgcol, gfxcharw9);
}
}
@ -450,7 +457,7 @@ void bx_vncsrv_gui_c::dimension_update(unsigned x, unsigned y, unsigned fheight,
BX_PANIC(("%d bpp graphics mode not supported", bpp));
}
guest_textmode = (fheight > 0);
guest_fsize = (fheight << 4) | fwidth;
guest_fsize = (fheight << 8) | fwidth;
guest_xres = x;
guest_yres = y;
if (guest_textmode) {

2
bochs/gui/win32.cc
View File

@ -2009,7 +2009,7 @@ void bx_win32_gui_c::graphics_tile_update(Bit8u *tile, unsigned x0, unsigned y0)
void bx_win32_gui_c::dimension_update(unsigned x, unsigned y, unsigned fheight, unsigned fwidth, unsigned bpp)
{
guest_textmode = (fheight > 0);
guest_fsize = (fheight << 4) | fwidth;
guest_fsize = (fheight << 8) | fwidth;
guest_xres = x;
guest_yres = y;
if (guest_textmode) {

2
bochs/gui/wx.cc
View File

@ -1524,7 +1524,7 @@ void bx_wx_gui_c::dimension_update(unsigned x, unsigned y, unsigned fheight, uns
BX_PANIC(("%d bpp graphics mode not supported", bpp));
}
guest_textmode = (fheight > 0);
guest_fsize = (fheight << 4) | fwidth;
guest_fsize = (fheight << 8) | fwidth;
guest_xres = x;
guest_yres = y;
if (guest_textmode) {

229
bochs/gui/x.cc
View File

@ -62,6 +62,10 @@ public:
bx_x_gui_c(void);
DECLARE_GUI_VIRTUAL_METHODS()
DECLARE_GUI_NEW_VIRTUAL_METHODS()
virtual void set_font(bx_bool lg);
virtual void draw_char(Bit8u ch, Bit8u fc, Bit8u bc, Bit16u xc, Bit16u yc,
Bit8u fw, Bit8u fh, Bit8u fx, Bit8u fy,
bx_bool gfxcharw9, Bit8u cs, Bit8u ce, bx_bool curs);
virtual void beep_on(float frequency);
virtual void beep_off();
#if BX_HAVE_XRANDR_H
@ -105,10 +109,6 @@ static char *progname; /* name this program was invoked by */
// text display
static unsigned int text_rows=25, text_cols=80;
static unsigned font_width, font_height;
static Bit8u h_panning = 0, v_panning = 0;
static Bit16u line_compare = 1023;
static unsigned prev_cursor_x=0;
static unsigned prev_cursor_y=0;
// graphics display
static Window win;
@ -915,6 +915,7 @@ void bx_x_gui_c::specific_init(int argc, char **argv, unsigned headerbar_y)
#endif
new_gfx_api = 1;
new_text_api = 1;
dialog_caps |= (BX_GUI_DLG_USER | BX_GUI_DLG_SNAPSHOT | BX_GUI_DLG_CDROM);
console.present = 1;
}
@ -1137,34 +1138,19 @@ void bx_x_gui_c::handle_events(void)
#endif
}
void bx_x_gui_c::text_update(Bit8u *old_text, Bit8u *new_text,
unsigned long cursor_x, unsigned long cursor_y,
bx_vga_tminfo_t *tm_info)
void bx_x_gui_c::set_font(bx_bool lg)
{
Bit8u *old_line, *new_line, *text_base;
Bit8u cChar, fbits, fmask, frow;
unsigned int curs, hchars, i, j, k, offset, rows, x, y, xc, yc, yc2, cs_y;
unsigned new_foreground, new_background;
Bit8u cfwidth, cfheight, cfheight2, font_col, font_row, font_row2;
Bit8u split_textrow, split_fontrows;
bx_bool forceUpdate = 0, split_screen;
bx_bool blink_state, blink_mode;
unsigned i, j, k;
Bit8u fbits, fmask, frow;
unsigned char cell[96];
unsigned long text_palette[16];
bx_bool gfxchar, dwidth;
blink_mode = (tm_info->blink_flags & BX_TEXT_BLINK_MODE) > 0;
blink_state = (tm_info->blink_flags & BX_TEXT_BLINK_STATE) > 0;
if (blink_mode) {
if (tm_info->blink_flags & BX_TEXT_BLINK_TOGGLE)
forceUpdate = 1;
}
if (charmap_updated) {
BX_INFO(("charmap update. Font is %d x %d", font_width, font_height));
for (unsigned c = 0; c<256; c++) {
if (char_changed[c]) {
XFreePixmap(bx_x_display, vgafont[c]);
bx_bool gfxchar = tm_info->line_graphics && ((c & 0xE0) == 0xC0);
bx_bool dwidth = font_width > 9;
gfxchar = lg && ((c & 0xE0) == 0xC0);
dwidth = font_width > 9;
i = 0;
j = 0;
memset(cell, 0, sizeof(cell));
@ -1212,176 +1198,35 @@ void bx_x_gui_c::text_update(Bit8u *old_text, Bit8u *new_text,
char_changed[c] = 0;
}
}
forceUpdate = 1;
charmap_updated = 0;
}
for (i=0; i<16; i++) {
text_palette[i] = col_vals[tm_info->actl_palette[i]];
}
}
if((tm_info->h_panning != h_panning) || (tm_info->v_panning != v_panning)) {
forceUpdate = 1;
h_panning = tm_info->h_panning;
v_panning = tm_info->v_panning;
void bx_x_gui_c::draw_char(Bit8u ch, Bit8u fc, Bit8u bc, Bit16u xc, Bit16u yc,
Bit8u fw, Bit8u fh, Bit8u fx, Bit8u fy,
bx_bool gfxcharw9, Bit8u cs, Bit8u ce, bx_bool curs)
{
yc += bx_headerbar_y;
XSetForeground(bx_x_display, gc, col_vals[fc]);
XSetBackground(bx_x_display, gc, col_vals[bc]);
XCopyPlane(bx_x_display, vgafont[ch], win, gc, fx, fy, fw, fh, xc, yc, 1);
if (curs && (ce >= fy) && (cs < (fh + fy))) {
if (cs > fy) {
yc += (cs - fy);
fh -= (cs - fy);
}
if(tm_info->line_compare != line_compare) {
forceUpdate = 1;
line_compare = tm_info->line_compare;
if ((ce - cs + 1) < fh) {
fh = ce - cs + 1;
}
XSetForeground(bx_x_display, gc, col_vals[bc]);
XSetBackground(bx_x_display, gc, col_vals[fc]);
XCopyPlane(bx_x_display, vgafont[ch], win, gc, fx, cs, fw, fh, xc, yc, 1);
}
}
// first invalidate character at previous and new cursor location
if ((prev_cursor_y < text_rows) && (prev_cursor_x < text_cols)) {
curs = prev_cursor_y * tm_info->line_offset + prev_cursor_x * 2;
old_text[curs] = ~new_text[curs];
}
if((tm_info->cs_start <= tm_info->cs_end) && (tm_info->cs_start < font_height) &&
(cursor_y < text_rows) && (cursor_x < text_cols)) {
curs = cursor_y * tm_info->line_offset + cursor_x * 2;
old_text[curs] = ~new_text[curs];
} else {
curs = 0xffff;
}
rows = text_rows;
if (v_panning) rows++;
y = 0;
cs_y = 0;
offset = 0;
text_base = new_text - tm_info->start_address;
if (line_compare < dimension_y) {
split_textrow = (line_compare + v_panning) / font_height;
split_fontrows = ((line_compare + v_panning) % font_height) + 1;
} else {
split_textrow = rows + 1;
split_fontrows = 0;
}
split_screen = 0;
do {
hchars = text_cols;
if (h_panning) hchars++;
if (split_screen) {
yc = bx_headerbar_y + line_compare + cs_y * font_height + 1;
font_row = 0;
if (rows == 1) {
cfheight = (dimension_y - line_compare - 1) % font_height;
if (cfheight == 0) cfheight = font_height;
} else {
cfheight = font_height;
}
} else if (v_panning) {
if (y == 0) {
yc = bx_headerbar_y;
font_row = v_panning;
cfheight = font_height - v_panning;
} else {
yc = y * font_height + bx_headerbar_y - v_panning;
font_row = 0;
if (rows == 1) {
cfheight = v_panning;
} else {
cfheight = font_height;
}
}
} else {
yc = y * font_height + bx_headerbar_y;
font_row = 0;
cfheight = font_height;
}
if (y == split_textrow) {
cfheight = split_fontrows - font_row;
}
new_line = new_text;
old_line = old_text;
x = 0;
do {
if (h_panning) {
if (hchars > text_cols) {
xc = 0;
font_col = h_panning;
cfwidth = font_width - h_panning;
} else {
xc = x * font_width - h_panning;
font_col = 0;
if (hchars == 1) {
cfwidth = h_panning;
} else {
cfwidth = font_width;
}
}
} else {
xc = x * font_width;
font_col = 0;
cfwidth = font_width;
}
if (forceUpdate || (old_text[0] != new_text[0])
|| (old_text[1] != new_text[1])) {
cChar = new_text[0];
new_foreground = new_text[1] & 0x0f;
if (blink_mode) {
new_background = (new_text[1] & 0x70) >> 4;
if (!blink_state && (new_text[1] & 0x80))
new_foreground = new_background;
} else {
new_background = (new_text[1] & 0xf0) >> 4;
}
XSetForeground(bx_x_display, gc, text_palette[new_foreground]);
XSetBackground(bx_x_display, gc, text_palette[new_background]);
XCopyPlane(bx_x_display, vgafont[cChar], win, gc, font_col, font_row, cfwidth, cfheight,
xc, yc, 1);
if (offset == curs) {
XSetForeground(bx_x_display, gc, text_palette[new_background]);
XSetBackground(bx_x_display, gc, text_palette[new_foreground]);
if (font_row == 0) {
yc2 = yc + tm_info->cs_start;
font_row2 = tm_info->cs_start;
cfheight2 = tm_info->cs_end - tm_info->cs_start + 1;
if ((yc2 + cfheight2) > (dimension_y + bx_headerbar_y)) {
cfheight2 = dimension_y + bx_headerbar_y - yc2;
}
} else {
if (v_panning > tm_info->cs_start) {
yc2 = yc;
font_row2 = font_row;
cfheight2 = tm_info->cs_end - v_panning + 1;
} else {
yc2 = yc + tm_info->cs_start - v_panning;
font_row2 = tm_info->cs_start;
cfheight2 = tm_info->cs_end - tm_info->cs_start + 1;
}
}
if (yc2 < (dimension_y + bx_headerbar_y)) {
XCopyPlane(bx_x_display, vgafont[cChar], win, gc, font_col, font_row2, cfwidth,
cfheight2, xc, yc2, 1);
}
}
}
x++;
new_text+=2;
old_text+=2;
offset+=2;
} while (--hchars);
if (y == split_textrow) {
new_text = text_base;
forceUpdate = 1;
cs_y = 0;
curs += tm_info->start_address;
if (tm_info->split_hpanning) h_panning = 0;
rows = ((dimension_y - line_compare + font_height - 2) / font_height) + 1;
split_screen = 1;
} else {
cs_y++;
new_text = new_line + tm_info->line_offset;
old_text = old_line + tm_info->line_offset;
}
y++;
offset = cs_y * tm_info->line_offset;
} while (--rows);
h_panning = tm_info->h_panning;
prev_cursor_x = cursor_x;
prev_cursor_y = cursor_y;
void bx_x_gui_c::text_update(Bit8u *old_text, Bit8u *new_text,
unsigned long cursor_x, unsigned long cursor_y,
bx_vga_tminfo_t *tm_info)
{
// present for compatibility
}
void bx_x_gui_c::graphics_tile_update(Bit8u *tile, unsigned x0, unsigned y0)
@ -1512,7 +1357,7 @@ void bx_x_gui_c::dimension_update(unsigned x, unsigned y, unsigned fheight, unsi
BX_PANIC(("%d bpp graphics mode not supported", bpp));
}
guest_textmode = (fheight > 0);
guest_fsize = (fheight << 4) | fwidth;
guest_fsize = (fheight << 8) | fwidth;
guest_xres = x;
guest_yres = y;
if (guest_textmode) {