<div dir="ltr"><div>Acked-by: Yuri Benditovich <<a href="mailto:yuri.benditovich@daynix.com">yuri.benditovich@daynix.com</a>> </div><div> </div></div><div class="gmail_extra"> <div class="gmail_quote">On Mon, Apr 10, 2017 at 6:30 PM, Frediano Ziglio <<a href="mailto:fziglio@redhat.com" target="_blank">fziglio@redhat.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">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(-) -- 2.9.3 _______________________________________________ Spice-devel mailing list <a href="mailto:Spice-devel@lists.freedesktop.org">Spice-devel@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/spice-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/spice-devel</a> </blockquote></div> </div>