Hi Dave, As the attached, our Vulkan team is verifying it. Thanks, David Zhou On 2016å¹´12æ??01æ?¥ 13:44, Dave Airlie wrote: > > On 1 Dec. 2016 15:22, "zhoucm1" <david1.zhou at amd.com > <mailto:david1.zhou at amd.com>> wrote: > > > > Yes, the old implementation which is already in upstream libdrm is > out of data, there isn't other user, so we want to drop it when new > semaphore is verified OK. > > Could you post some patches for the new one? Otherwise I'll have to > write one for radv. > > Dave. > > > > Thanks, > > David Zhou > > > > > > On 2016å¹´12æ??01æ?¥ 10:36, Mao, David wrote: > >> > >> Hi Dave, > >> i believe your first attempt is correct. > >> The export/import semaphore needs refine of the semaphore > implementation. > >> We are working on that. > >> > >> Thanks. > >> Best Regards, > >> David > >>> > >>> On 1 Dec 2016, at 10:12 AM, Dave Airlie <airlied at gmail.com > <mailto:airlied at gmail.com>> wrote: > >>> > >>> Hey all, > >>> > >>> So I've started adding semaphore support to radv but I'm not really > >>> sure what the API to the semaphore code is. > >>> > >>> the Vulkan API is you get a command submission of a number of submit > >>> units which have a 0-n wait semaphore, 0-n command buffers and 0-n > >>> signal semaphores. > >>> > >>> Now I'm not sure how I should use the APIs with those. > >>> > >>> My first attempt is > >>> > >>> call amdgpu_cs_wait_semaphore on all the wait ones, call the cs submit > >>> API, then call the amdgpu_cs_signal_semaphore on all the signal ones? > >>> > >>> or should I be up front calling wait/signal then submitting the > command streams? > >>> > >>> Also upcoming work requires possibly sharing semaphores between > >>> processes, is there any indication how this might be made work with > >>> the libdrm_amdgpu semaphore implementation? > >>> > >>> Thanks, > >>> Dave. > >>> _______________________________________________ > >>> amd-gfx mailing list > >>> amd-gfx at lists.freedesktop.org <mailto:amd-gfx at lists.freedesktop.org> > >>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx > >> > >> _______________________________________________ > >> amd-gfx mailing list > >> amd-gfx at lists.freedesktop.org <mailto:amd-gfx at lists.freedesktop.org> > >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx > > > > > -------------- next part -------------- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161201/2162b3a9/attachment-0001.html> -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-amdgpu-add-new-semaphore-support.patch Type: text/x-patch Size: 7651 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161201/2162b3a9/attachment-0003.bin> -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-drm-amdgpu-add-new-semaphore-object-in-kernel-side.patch Type: text/x-patch Size: 16382 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161201/2162b3a9/attachment-0004.bin> -------------- next part -------------- A non-text attachment was scrubbed... Name: 0002-tests-amdgpu-add-sem-test.patch Type: text/x-patch Size: 5917 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161201/2162b3a9/attachment-0005.bin>
