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

[Frediano Ziglio]

> Acked-by: Yuri Benditovich < yuri.benditovich at daynix.com >

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 at redhat.com >
> 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 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20170411/dc6bdcd7/attachment.html>