[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory regio
From: |
Avi Kivity |
Subject: |
Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API |
Date: |
Fri, 20 May 2011 12:20:35 +0300 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc14 Thunderbird/3.1.10 |
On 05/19/2011 10:27 PM, Jan Kiszka wrote:
On 2011-05-19 16:12, Avi Kivity wrote:
> +/* Sets an offset to be added to MemoryRegionOps callbacks. */
> +void memory_region_set_offset(MemoryRegion *mr, target_phys_addr_t offset);
Please mark this as a legacy helper, ideally to be removed after the
complete conversion to this API. During that phase we should try to
identify those devices which still depend on offset=0 and maybe directly
fix them.
Okay.
> +/* Turn loggging on or off for specified client (display, migration) */
> +void memory_region_set_log(MemoryRegion *mr, bool log, unsigned client);
> +/* Enable memory coalescing for the region. MMIO ->write callbacks may be
> + * delayed until a non-coalesced MMIO is issued.
> + */
> +void memory_region_set_coalescing(MemoryRegion *mr);
> +/* Enable memory coalescing for a sub-range of the region. MMIO ->write
> + * callbacks may be delayed until a non-coalesced MMIO is issued.
> + */
> +void memory_region_add_coalescing(MemoryRegion *mr,
> + target_phys_addr_t offset,
> + target_phys_addr_t size);
Will this be such a common use case that requesting the user to split up
the region and then use set_coalescing will generate too much boiler
plate code?
Look at e1000, coalescing ranges have byte granularity.
> +/* Disable MMIO coalescing for the region. */
> +void memory_region_clear_coalescing(MemoryRegion *mr);
And what about clearing coalescing for sub-ranges?
Clear them all and rebuild.
Maybe skip
add_coalescing for the first run and see how far we get.
We get as far as e.
--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.
- [Qemu-devel] [RFC v1] Memory API, Avi Kivity, 2011/05/19
- [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Avi Kivity, 2011/05/19
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Alex Williamson, 2011/05/19
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Jan Kiszka, 2011/05/19
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API,
Avi Kivity <=
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Anthony Liguori, 2011/05/19
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Avi Kivity, 2011/05/20
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Richard Henderson, 2011/05/20
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Anthony Liguori, 2011/05/20
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Richard Henderson, 2011/05/20
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Anthony Liguori, 2011/05/20
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Blue Swirl, 2011/05/20
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Avi Kivity, 2011/05/22
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Avi Kivity, 2011/05/22
- Re: [Qemu-devel] [RFC v1] Add declarations for hierarchical memory region API, Anthony Liguori, 2011/05/22