<html><body><div style="font-family: times new roman, new york, times, serif; font-size: 12pt; color: #000000"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div><br></div><div dir="ltr"><div>Acked-by: Yuri Benditovich <<a href="mailto:yuri.benditovich@daynix.com" target="_blank">yuri.benditovich@daynix.com</a>> <br></div></div></blockquote><div>Thanks for the review.<br></div><div>I'll wait for your series to be merged, is almost ready.<br></div><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 10, 2017 at 6:30 PM, Frediano Ziglio <span dir="ltr"><<a href="mailto:fziglio@redhat.com" target="_blank">fziglio@redhat.com</a>></span> wrote:<br><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<br>
by different memory allocations.<br>
These set of patches remove the needs to pass the Bar to<br>
some functions (FreeMem, VA and PA) making easier to<br>
change memory layout in the future.<br>
These patches also change memory allocation moving commands<br>
to Bar0 as Qemu has some issues with migrations if commands<br>
are on Bar1.<br>
These patches make also easier to move allocations to Bar0<br>
or implementing some sort of fallback is one Bar is full.<br><br>
NOTE: currently Qemu requires that<br>
- primary surface;<br>
- monitor configurations;<br>
- commands;<br>
all reside in Bar0.<br><br>
Frediano Ziglio (8):<br>
Compute automatically memory space in FreeMem<br>
Remove m_NumMemSlots member<br>
Allocate m_MemSlots statically inside QxlDevice<br>
Use normal pointer for VA addresses<br>
Compute dynamically slot_id in PA and VA functions<br>
Use inline attribute for VA and PA<br>
Allocate commands on Bar0<br>
Remove unused parameters from VA and PA functions<br><br>
qxldod/QxlDod.cpp | 137 ++++++++++++++++++++++++++----------------------------<br>
qxldod/QxlDod.h | 18 ++++---<br>
2 files changed, 73 insertions(+), 82 deletions(-)<br></blockquote></div></div></blockquote><div>Frediano</div><div><br></div></div></body></html>