Acked-by: Yuri Benditovich <yuri.benditovich@xxxxxxxxxx>
Thanks for the review.
I'll wait for your series to be merged, is almost ready.
On Mon, Apr 10, 2017 at 6:30 PM, Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:Current code require different function to know the Bar used
by different memory allocations.
These set of patches remove the needs to pass the Bar to
some functions (FreeMem, VA and PA) making easier to
change memory layout in the future.
These patches also change memory allocation moving commands
to Bar0 as Qemu has some issues with migrations if commands
are on Bar1.
These patches make also easier to move allocations to Bar0
or implementing some sort of fallback is one Bar is full.
NOTE: currently Qemu requires that
- primary surface;
- monitor configurations;
- commands;
all reside in Bar0.
Frediano Ziglio (8):
Compute automatically memory space in FreeMem
Remove m_NumMemSlots member
Allocate m_MemSlots statically inside QxlDevice
Use normal pointer for VA addresses
Compute dynamically slot_id in PA and VA functions
Use inline attribute for VA and PA
Allocate commands on Bar0
Remove unused parameters from VA and PA functions
qxldod/QxlDod.cpp | 137 ++++++++++++++++++++++++++----------------------------
qxldod/QxlDod.h | 18 ++++---
2 files changed, 73 insertions(+), 82 deletions(-)
Frediano
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel