[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/9] fbdev: add linux framebuffer display driver
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH 4/9] fbdev: add linux framebuffer display driver. |
Date: |
Mon, 24 Sep 2012 12:06:55 +0100 |
User-agent: |
Alpine 2.02 (DEB 1266 2009-07-14) |
On Fri, 21 Sep 2012, Gerd Hoffmann wrote:
> Hi,
>
> >> + case FB_ACQ_REQ:
> >> + fbdev_switch_acquire();
> >> + redraw_screen++;
> >
> > Rather than introducing redraw_screen, I would just call vga_hw_invalidate
> > instead here. It is always a win if we can avoid to introduce one more
> > state machine.
>
> Wouldn't that make the vga emulation do additional work which is simply
> not needed?
It causes full_update to be set and that causes vga_draw_line to be
called for every line on the screen.
However in the important cases (32bpp and 16bpp) the buffer is shared
and nothing happens. So no more work.
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, (continued)
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, Gerd Hoffmann, 2012/09/20
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, Stefano Stabellini, 2012/09/20
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, Gerd Hoffmann, 2012/09/20
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, Stefano Stabellini, 2012/09/20
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, Stefano Stabellini, 2012/09/20
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, Gerd Hoffmann, 2012/09/21
- Re: [Qemu-devel] [PATCH 7/9] fbdev: move to pixman, Stefano Stabellini, 2012/09/21
[Qemu-devel] [PATCH 4/9] fbdev: add linux framebuffer display driver., Gerd Hoffmann, 2012/09/19