Re: [PATCH qxl-wddm-dod 0/8] Make easier to change Qxl memory layout

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]