[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API |
Date: |
Mon, 19 Jan 2009 10:33:12 -0600 |
User-agent: |
Thunderbird 2.0.0.19 (X11/20090105) |
Paul Brook wrote:
It looks like what you're actually doing is pushing the bounce buffer
allocation into the individual packet consumers.
Maybe a solution to this is a 'do IO on IOVEC' actor, with an additional flag
that says whether it is acceptable to split the allocation. That way both
block and packet interfaces use the same API, and avoids proliferation of
manual bounce buffers in packet devices.
I think there may be utility in having packet devices provide the bounce
buffers, in which case, you could probably unique both into a single
function with a flag. But why not just have two separate functions?
Those two functions can live in exec.c too. The nice thing about using
map() is that it's easily overriden and chained. So what I'm proposing.
cpu_physical_memory_map()
cpu_physical_memory_unmap()
do_streaming_IO(map, unmap, ioworker, opaque);
do_packet_IO(map, unmap, buffer, size, ioworker, opaque);
Additional helpers that pass c_p_m_map/unmap would also be acceptable.
Regards,
Anthony Liguori
Paul
- [Qemu-devel] [PATCH 4/5] I/O vector helpers, (continued)
- [Qemu-devel] [PATCH 4/5] I/O vector helpers, Avi Kivity, 2009/01/18
- [Qemu-devel] [PATCH 5/5] Convert IDE to directly access guest memory, Avi Kivity, 2009/01/18
- [Qemu-devel] [PATCH 3/5] Vectored block device API, Avi Kivity, 2009/01/18
- [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Avi Kivity, 2009/01/18
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Ian Jackson, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Avi Kivity, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Anthony Liguori, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Paul Brook, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API,
Anthony Liguori <=
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Avi Kivity, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Anthony Liguori, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Avi Kivity, 2009/01/20
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Ian Jackson, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Anthony Liguori, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Avi Kivity, 2009/01/20
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Ian Jackson, 2009/01/20
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Ian Jackson, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Avi Kivity, 2009/01/19
- Re: [Qemu-devel] [PATCH 1/5] Add target memory mapping API, Ian Jackson, 2009/01/19