i965/drm: Port to Mesa's atomic header.
authorKenneth Graunke <kenneth@whitecape.org>
Tue, 21 Mar 2017 21:54:07 +0000 (14:54 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Mon, 10 Apr 2017 21:31:13 +0000 (14:31 -0700)
Drop xf86atomic.h in favor of Mesa's util/u_atomic.h.  We replace the
atomic_t wrapper struct with a bare integer, switch to the 'p_atomic'
naming conventions, and move over the one extra helper.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Jason Ekstrand <jason@jlekstrand.net>
src/mesa/drivers/dri/i965/Makefile.sources
src/mesa/drivers/dri/i965/intel_bufmgr_gem.c
src/mesa/drivers/dri/i965/xf86atomic.h [deleted file]

index fe20116b9bea1afff8f3c9b27914bf575eb67784..9ae764fed5877155b60ad5648e8cd5db269417e9 100644 (file)
@@ -165,8 +165,7 @@ i965_FILES = \
        intel_upload.c \
        libdrm_lists.h \
        libdrm_macros.h \
-       uthash.h \
-       xf86atomic.h
+       uthash.h
 
 i965_gen6_FILES = \
        genX_blorp_exec.c
index 4f9d6be7ac8d18d97f16f98ec43fc2e681817e7f..c67f813bdf55b3db1ee465aadbbfa3ae2379e9f1 100644 (file)
@@ -39,7 +39,7 @@
 #endif
 
 #include <xf86drm.h>
-#include <xf86atomic.h>
+#include <util/u_atomic.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <stdlib.h>
 #define ARRAY_SIZE(x) (sizeof(x) / sizeof((x)[0]))
 #define MAX2(A, B) ((A) > (B) ? (A) : (B))
 
+static inline int
+atomic_add_unless(int *v, int add, int unless)
+{
+   int c, old;
+   c = p_atomic_read(v);
+   while (c != unless && (old = p_atomic_cmpxchg(v, c, c + add)) != c)
+      c = old;
+   return c == unless;
+}
+
 /**
  * upper_32_bits - return bits 32-63 of a number
  * @n: the number we're accessing
@@ -110,7 +120,7 @@ struct drm_bacon_gem_bo_bucket {
 typedef struct _drm_bacon_bufmgr_gem {
        drm_bacon_bufmgr bufmgr;
 
-       atomic_t refcount;
+       int refcount;
 
        int fd;
 
@@ -169,7 +179,7 @@ typedef struct _drm_bacon_reloc_target_info {
 struct _drm_bacon_bo_gem {
        drm_bacon_bo bo;
 
-       atomic_t refcount;
+       int refcount;
        uint32_t gem_handle;
        const char *name;
 
@@ -472,7 +482,7 @@ drm_bacon_gem_bo_reference(drm_bacon_bo *bo)
 {
        drm_bacon_bo_gem *bo_gem = (drm_bacon_bo_gem *) bo;
 
-       atomic_inc(&bo_gem->refcount);
+       p_atomic_inc(&bo_gem->refcount);
 }
 
 /**
@@ -834,7 +844,7 @@ retry:
        }
 
        bo_gem->name = name;
-       atomic_set(&bo_gem->refcount, 1);
+       p_atomic_set(&bo_gem->refcount, 1);
        bo_gem->validate_index = -1;
        bo_gem->reloc_tree_fences = 0;
        bo_gem->used_as_reloc_target = false;
@@ -954,7 +964,7 @@ drm_bacon_gem_bo_alloc_userptr(drm_bacon_bufmgr *bufmgr,
        if (!bo_gem)
                return NULL;
 
-       atomic_set(&bo_gem->refcount, 1);
+       p_atomic_set(&bo_gem->refcount, 1);
        DRMINITLISTHEAD(&bo_gem->vma_list);
 
        bo_gem->bo.size = size;
@@ -1131,7 +1141,7 @@ drm_bacon_bo_gem_create_from_name(drm_bacon_bufmgr *bufmgr,
        if (!bo_gem)
                goto out;
 
-       atomic_set(&bo_gem->refcount, 1);
+       p_atomic_set(&bo_gem->refcount, 1);
        DRMINITLISTHEAD(&bo_gem->vma_list);
 
        bo_gem->bo.size = open_arg.size;
@@ -1402,8 +1412,8 @@ static void drm_bacon_gem_bo_unreference_locked_timed(drm_bacon_bo *bo,
 {
        drm_bacon_bo_gem *bo_gem = (drm_bacon_bo_gem *) bo;
 
-       assert(atomic_read(&bo_gem->refcount) > 0);
-       if (atomic_dec_and_test(&bo_gem->refcount))
+       assert(p_atomic_read(&bo_gem->refcount) > 0);
+       if (p_atomic_dec_zero(&bo_gem->refcount))
                drm_bacon_gem_bo_unreference_final(bo, time);
 }
 
@@ -1411,7 +1421,7 @@ static void drm_bacon_gem_bo_unreference(drm_bacon_bo *bo)
 {
        drm_bacon_bo_gem *bo_gem = (drm_bacon_bo_gem *) bo;
 
-       assert(atomic_read(&bo_gem->refcount) > 0);
+       assert(p_atomic_read(&bo_gem->refcount) > 0);
 
        if (atomic_add_unless(&bo_gem->refcount, -1, 1)) {
                drm_bacon_bufmgr_gem *bufmgr_gem =
@@ -1422,7 +1432,7 @@ static void drm_bacon_gem_bo_unreference(drm_bacon_bo *bo)
 
                pthread_mutex_lock(&bufmgr_gem->lock);
 
-               if (atomic_dec_and_test(&bo_gem->refcount)) {
+               if (p_atomic_dec_zero(&bo_gem->refcount)) {
                        drm_bacon_gem_bo_unreference_final(bo, time.tv_sec);
                        drm_bacon_gem_cleanup_bo_cache(bufmgr_gem, time.tv_sec);
                }
@@ -2657,7 +2667,7 @@ drm_bacon_bo_gem_create_from_prime(drm_bacon_bufmgr *bufmgr, int prime_fd, int s
        if (!bo_gem)
                goto out;
 
-       atomic_set(&bo_gem->refcount, 1);
+       p_atomic_set(&bo_gem->refcount, 1);
        DRMINITLISTHEAD(&bo_gem->vma_list);
 
        /* Determine size of bo.  The fd-to-handle ioctl really should
@@ -3368,7 +3378,7 @@ drm_bacon_bufmgr_gem_find(int fd)
 
        DRMLISTFOREACHENTRY(bufmgr_gem, &bufmgr_list, managers) {
                if (bufmgr_gem->fd == fd) {
-                       atomic_inc(&bufmgr_gem->refcount);
+                       p_atomic_inc(&bufmgr_gem->refcount);
                        return bufmgr_gem;
                }
        }
@@ -3384,7 +3394,7 @@ drm_bacon_bufmgr_gem_unref(drm_bacon_bufmgr *bufmgr)
        if (atomic_add_unless(&bufmgr_gem->refcount, -1, 1)) {
                pthread_mutex_lock(&bufmgr_list_mutex);
 
-               if (atomic_dec_and_test(&bufmgr_gem->refcount)) {
+               if (p_atomic_dec_zero(&bufmgr_gem->refcount)) {
                        DRMLISTDEL(&bufmgr_gem->managers);
                        drm_bacon_bufmgr_gem_destroy(bufmgr);
                }
@@ -3554,7 +3564,7 @@ drm_bacon_bufmgr_gem_init(int fd, int batch_size)
                goto exit;
 
        bufmgr_gem->fd = fd;
-       atomic_set(&bufmgr_gem->refcount, 1);
+       p_atomic_set(&bufmgr_gem->refcount, 1);
 
        if (pthread_mutex_init(&bufmgr_gem->lock, NULL) != 0) {
                free(bufmgr_gem);
diff --git a/src/mesa/drivers/dri/i965/xf86atomic.h b/src/mesa/drivers/dri/i965/xf86atomic.h
deleted file mode 100644 (file)
index a0e3909..0000000
+++ /dev/null
@@ -1,113 +0,0 @@
-/*
- * Copyright © 2009 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- *
- * Authors:
- *    Chris Wilson <chris@chris-wilson.co.uk>
- *
- */
-
-/**
- * @file xf86atomics.h
- *
- * Private definitions for atomic operations
- */
-
-#ifndef LIBDRM_ATOMICS_H
-#define LIBDRM_ATOMICS_H
-
-#ifdef HAVE_CONFIG_H
-#include "config.h"
-#endif
-
-#define HAS_ATOMIC_OPS 1
-
-typedef struct {
-       int atomic;
-} atomic_t;
-
-# define atomic_read(x) ((x)->atomic)
-# define atomic_set(x, val) ((x)->atomic = (val))
-# define atomic_inc(x) ((void) __sync_fetch_and_add (&(x)->atomic, 1))
-# define atomic_inc_return(x) (__sync_add_and_fetch (&(x)->atomic, 1))
-# define atomic_dec_and_test(x) (__sync_add_and_fetch (&(x)->atomic, -1) == 0)
-# define atomic_add(x, v) ((void) __sync_add_and_fetch(&(x)->atomic, (v)))
-# define atomic_dec(x, v) ((void) __sync_sub_and_fetch(&(x)->atomic, (v)))
-# define atomic_cmpxchg(x, oldv, newv) __sync_val_compare_and_swap (&(x)->atomic, oldv, newv)
-
-#endif
-
-#if HAVE_LIB_ATOMIC_OPS
-#include <atomic_ops.h>
-
-#define HAS_ATOMIC_OPS 1
-
-typedef struct {
-       AO_t atomic;
-} atomic_t;
-
-# define atomic_read(x) AO_load_full(&(x)->atomic)
-# define atomic_set(x, val) AO_store_full(&(x)->atomic, (val))
-# define atomic_inc(x) ((void) AO_fetch_and_add1_full(&(x)->atomic))
-# define atomic_inc_return(x) (AO_fetch_and_add1_full(&(x)->atomic) + 1)
-# define atomic_add(x, v) ((void) AO_fetch_and_add_full(&(x)->atomic, (v)))
-# define atomic_dec(x, v) ((void) AO_fetch_and_add_full(&(x)->atomic, -(v)))
-# define atomic_dec_and_test(x) (AO_fetch_and_sub1_full(&(x)->atomic) == 1)
-# define atomic_cmpxchg(x, oldv, newv) AO_compare_and_swap_full(&(x)->atomic, oldv, newv)
-
-#endif
-
-#if (defined(__sun) || defined(__NetBSD__)) && !defined(HAS_ATOMIC_OPS)  /* Solaris & OpenSolaris & NetBSD */
-
-#include <sys/atomic.h>
-#define HAS_ATOMIC_OPS 1
-
-#if defined(__NetBSD__)
-#define LIBDRM_ATOMIC_TYPE int
-#else
-#define LIBDRM_ATOMIC_TYPE uint_t
-#endif
-
-typedef struct { LIBDRM_ATOMIC_TYPE atomic; } atomic_t;
-
-# define atomic_read(x) (int) ((x)->atomic)
-# define atomic_set(x, val) ((x)->atomic = (LIBDRM_ATOMIC_TYPE)(val))
-# define atomic_inc(x) (atomic_inc_uint (&(x)->atomic))
-# define atomic_inc_return(x) (atomic_inc_uint_nv(&(x)->atomic))
-# define atomic_dec_and_test(x) (atomic_dec_uint_nv(&(x)->atomic) == 0)
-# define atomic_add(x, v) (atomic_add_int(&(x)->atomic, (v)))
-# define atomic_dec(x, v) (atomic_add_int(&(x)->atomic, -(v)))
-# define atomic_cmpxchg(x, oldv, newv) atomic_cas_uint (&(x)->atomic, oldv, newv)
-
-#endif
-
-#if ! HAS_ATOMIC_OPS
-#error libdrm requires atomic operations, please define them for your CPU/compiler.
-#endif
-
-static inline int atomic_add_unless(atomic_t *v, int add, int unless)
-{
-       int c, old;
-       c = atomic_read(v);
-       while (c != unless && (old = atomic_cmpxchg(v, c, c + add)) != c)
-               c = old;
-       return c == unless;
-}