qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 20/39] virtio-pci: convert to memory API


From: Michael S. Tsirkin
Subject: Re: [Qemu-devel] [PATCH 20/39] virtio-pci: convert to memory API
Date: Tue, 2 Aug 2011 12:34:52 +0300
User-agent: Mutt/1.5.21 (2010-09-15)

On Tue, Aug 02, 2011 at 12:17:06PM +0300, Avi Kivity wrote:
> On 08/01/2011 10:53 PM, Michael S. Tsirkin wrote:
> >>
> >>
> >>  Just because a memory region becomes visible to the cpu is no reason
> >>  to have a callback.  From the device perspective, it can't tell that
> >>  it happened.
> >
> >BTW this is what qxl does, too, conceptually: on config writes, it peeks
> >at the bar to detect whether that got unmapped.
> 
> Can you elaborate?  why?  what does it do?


Disables vga mapping when it sees io has been disabled.
What it really should do is check the io enable bit I think ...

    vga_dirty_log_stop(vga);
    pci_default_write_config(d, address, val, len);
    if (vga->map_addr && qxl->pci.io_regions[0].addr == -1) {
        vga->map_addr = 0;
    }
    vga_dirty_log_start(vga);

> -- 
> error compiling committee.c: too many arguments to function



reply via email to

[Prev in Thread] Current Thread [Next in Thread]