<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 11, 2020 at 3:36 AM Gerd Hoffmann <<a href="mailto:kraxel@redhat.com">kraxel@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">  Hi,<br>
<br>
> I should've been more clear -- this is an internal cleanup/preparation and<br>
> the per-context changes are invisible to host userspace.<br>
<br>
Ok, it wasn't clear that you don't flip the switch yet.  In general the<br>
commit messages could be a bit more verbose ...<br>
<br>
I'm wondering though why we need the new fence_id in the first place.<br>
Isn't it enough to have per-context (instead of global) last_seq?<br></blockquote><div><br></div><div>Heh, that was to leave open the possibility of multiple timelines per context.  Roughly speaking,</div><div><br></div><div>V2 -- multiple processes</div><div>V3 -- multiple processes and multiple threads (due to VK multi-threaded command buffers)</div><div><br></div><div>I think we all agree on V2.  It seems we still have to discuss V3 (multi-queue, thread pools, a fence context associated with each thread) a bit more before we start landing pieces. </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Multi-queue sounds very interesting indeed, especially with VK<br>
> multi-threaded command submission.  That to me is V3 rather than V2.. let's<br>
> start easy!<br>
<br>
Having v2 if we plan to obsolete it with v3 soon doesn't look like a<br>
good plan to me.  It'll make backward compatibility more complex for<br>
no good reason ...<br>
<br>
Also: Does virglrenderer render different contexts in parallel today?<br>
Only in case it does we'll actually get benefits from per-context<br>
fences.  But I think it doesn't, so there is no need to rush.<br>
<br>
I think we should better have a rough plan for parallel rendering first,<br>
then go start implementing the pieces needed.<br>
<br>
cheers,<br>
  Gerd<br>
<br>
</blockquote></div></div>