mirror of https://github.com/xemu-project/xemu.git
vga: optimize computation of dirty memory region
The depth == 0 and depth == 15 have to be special cased because width * depth / 8 does not provide the correct scanline length. However, thanks to the recent reorganization of vga_draw_graphic() the correct value of VRAM bits per pixel is available in "bits". Use it (via the same "bwidth" computation that is used later in the function), thus restricting the slow path to the wraparound case. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
748e62dbf5
commit
ab75ecb79b
|
@ -1574,22 +1574,16 @@ static void vga_draw_graphic(VGACommonState *s, int full_update)
|
|||
|
||||
/* Horizontal pel panning bit 3 is only used in text mode. */
|
||||
hpel = bits <= 8 ? s->params.hpel & 7 : 0;
|
||||
bwidth = DIV_ROUND_UP(width * bits, 8); /* scanline length */
|
||||
if (hpel) {
|
||||
bwidth += 4;
|
||||
}
|
||||
|
||||
region_start = (s->params.start_addr * 4);
|
||||
region_end = region_start + (ram_addr_t)s->params.line_offset * height;
|
||||
region_end += width * depth / 8; /* scanline length */
|
||||
region_end -= s->params.line_offset;
|
||||
if (hpel) {
|
||||
region_end += 4;
|
||||
}
|
||||
if (region_end > s->vbe_size || depth == 0 || depth == 15) {
|
||||
region_end = region_start + (ram_addr_t)s->params.line_offset * (height - 1) + bwidth;
|
||||
if (region_end > s->vbe_size) {
|
||||
/*
|
||||
* We land here on:
|
||||
* - wraps around (can happen with cirrus vbe modes)
|
||||
* - depth == 0 (256 color palette video mode)
|
||||
* - depth == 15
|
||||
*
|
||||
* Take the safe and slow route:
|
||||
* On wrap around take the safe and slow route:
|
||||
* - create a dirty bitmap snapshot for all vga memory.
|
||||
* - force shadowing (so all vga memory access goes
|
||||
* through vga_read_*() helpers).
|
||||
|
@ -1667,10 +1661,6 @@ static void vga_draw_graphic(VGACommonState *s, int full_update)
|
|||
s->params.line_compare, sr(s, VGA_SEQ_CLOCK_MODE));
|
||||
#endif
|
||||
addr1 = (s->params.start_addr * 4);
|
||||
bwidth = DIV_ROUND_UP(width * bits, 8);
|
||||
if (hpel) {
|
||||
bwidth += 4;
|
||||
}
|
||||
y_start = -1;
|
||||
d = surface_data(surface);
|
||||
linesize = surface_stride(surface);
|
||||
|
|
Loading…
Reference in New Issue