On Dec 21, 2017, at 6:41 PM, Andres Rodriguez 
<andre...@gmail.com<mailto:andre...@gmail.com>> wrote:

Rename fences -> semaphores in preparation for upgrading fence
functionality.

This series renames the following symbols:
 - pipe_fence_handle -> pipe_semaphore_handle
 - fence_server_sync -> semaphore_server_sync
 - create_fence_fd -> create_semaphore_fd
 - fence_reference -> semaphore_reference
 - fence_finish -> semaphore_finish
 - fence_get_fd -> semaphore_get_fd
 - PIPE_FLUSH_FENCE_FD -> PIPE_FLUSH_SEMAPHORE_FD
 - PIPE_CAP_NATIVE_FENCE_FD -> PIPE_CAP_NATIVE_SEMAPHORE_FD

This patch does not introduce any functionality changes.

Hi Andres.  This rename leaves the drivers in a weird mixed nomenclature between
fence and semaphore.  Is there a plan to clean this up?

Example, pulled from lp_screen.c:

* Fence reference counting.
*/
static void
-llvmpipe_fence_reference(struct pipe_screen *screen,
-                         struct pipe_fence_handle **ptr,
-                         struct pipe_fence_handle *fence)
+llvmpipe_semaphore_reference(struct pipe_screen *screen,
+                         struct pipe_semaphore_handle **ptr,
+                         struct pipe_semaphore_handle *fence)
{
  struct lp_fence **old = (struct lp_fence **) ptr;
  struct lp_fence *f = (struct lp_fence *) fence;

-   lp_fence_reference(old, f);
+   lp_semaphore_reference(old, f);
}

We're casting semaphore handles to lp_fence pointers and passing
fences into lp_semaphore_reference.  A confusing mix of terminology.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to