[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/5] vga optimization.
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH 5/5] vga optimization. |
Date: |
Tue, 11 Nov 2008 15:50:40 +0000 |
User-agent: |
Thunderbird 2.0.0.14 (X11/20080505) |
Glauber Costa wrote:
>> @@ -2102,6 +2122,11 @@ static void vga_map(PCIDevice *pci_dev, int
>> region_num,
>> } else {
>> cpu_register_physical_memory(addr, s->vram_size, s->vram_offset);
>> }
>> +
>> + s->map_addr = addr;
>> + s->map_end = addr + VGA_RAM_SIZE;
>> +
>> + vga_dirty_log_start(s);
>> }
>>
>> I think you should set lfb_addr and lfb_end here instead of map_addr and
>> map_end.
>>
> It is possible for the lfb to be set, but not mapped.
> So map_addr is what really indicates we're interested in tracking this region.
> Or I am totally wrong?
>
Yes, you are right.
However these changes are in vga_map that is the pci_register_io_region
function, much like cirrus_pci_lfb_map for cirrus.
They are supposed to set the lfb_addr before we remap it.
Besides I just noticed that you should set lfb_addr only in case that
region_num != PCI_ROM_SLOT.
- [Qemu-devel] [PATCH 0/5] VGA Optimization Episode IV - A new hope, Glauber Costa, 2008/11/10
- [Qemu-devel] [PATCH 1/5] better type checking for vga, Glauber Costa, 2008/11/10
- [Qemu-devel] [PATCH 2/5] move vga_io_address to VGA State, Glauber Costa, 2008/11/10
- [Qemu-devel] [PATCH 3/5] de-register mem region for MMIO., Glauber Costa, 2008/11/10
- [Qemu-devel] [PATCH 4/5] Introduce kvm logging interface., Glauber Costa, 2008/11/10
- [Qemu-devel] [PATCH 5/5] vga optimization., Glauber Costa, 2008/11/10
- Re: [Qemu-devel] [PATCH 5/5] vga optimization., Anthony Liguori, 2008/11/11
- Re: [Qemu-devel] [PATCH 5/5] vga optimization., Stefano Stabellini, 2008/11/11
- Re: [Qemu-devel] [PATCH 5/5] vga optimization., Glauber Costa, 2008/11/11
- Re: [Qemu-devel] [PATCH 5/5] vga optimization.,
Stefano Stabellini <=