[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/7] memory: Flush coalesced MMIO on selected
From: |
Avi Kivity |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/7] memory: Flush coalesced MMIO on selected region access |
Date: |
Thu, 28 Jun 2012 19:07:40 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
On 06/27/2012 07:27 PM, Jan Kiszka wrote:
> Instead of flushing pending coalesced MMIO requests on every vmexit,
> this provides a mechanism to selectively flush when memory regions
> related to the coalesced one are accessed. This first of all includes
> the coalesced region itself but can also applied to other regions, e.g.
> of the same device, by calling memory_region_set_flush_coalesced.
>
> +
> +void memory_region_clear_flush_coalesced(MemoryRegion *mr)
> +{
> + qemu_flush_coalesced_mmio_buffer();
> + mr->flush_coalesced_mmio = false;
> +}
This will clear the implicit flag set by set_coalesced, but also any
explicit flag set by set_flush_coalesced. In some convoluted setup that
will never happen, this can cause breakage. This can be addresses
either using two flags (or perhaps using the coalesced list for the
implicit flag) or by a comment in clear_coalesced's documentation.
--
error compiling committee.c: too many arguments to function
- [Qemu-devel] [PATCH v2 0/7] kvm: Get coalesced MMIO flushing out of the hot-path, Jan Kiszka, 2012/06/27
- [Qemu-devel] [PATCH v2 1/7] i82378: Remove bogus MMIO coalescing, Jan Kiszka, 2012/06/27
- [Qemu-devel] [PATCH v2 2/7] memory: Flush coalesced MMIO on selected region access, Jan Kiszka, 2012/06/27
- Re: [Qemu-devel] [PATCH v2 2/7] memory: Flush coalesced MMIO on selected region access,
Avi Kivity <=
- [Qemu-devel] [PATCH v2 6/7] VGA: Flush coalesced MMIO on related MMIO/PIO accesses, Jan Kiszka, 2012/06/27
- [Qemu-devel] [PATCH v2 5/7] memory: Flush coalesced MMIO on mapping and state changes, Jan Kiszka, 2012/06/27
- [Qemu-devel] [PATCH v2 4/7] memory: Fold memory_region_update_topology into memory_region_transaction_commit, Jan Kiszka, 2012/06/27
- [Qemu-devel] [PATCH v2 3/7] memory: Use transaction_begin/commit also for single-step operations, Jan Kiszka, 2012/06/27
- [Qemu-devel] [PATCH v2 7/7] kvm: Stop flushing coalesced MMIO on vmexit, Jan Kiszka, 2012/06/27
- Re: [Qemu-devel] [PATCH v2 0/7] kvm: Get coalesced MMIO flushing out of the hot-path, Avi Kivity, 2012/06/28