[Mesa-dev] [PATCH 1/2] glapi: replace 'user' with 'context' in u_current.[ch] code
Chia-I Wu
olvaffe at gmail.com
Wed Mar 5 19:19:49 PST 2014
On Wed, Mar 5, 2014 at 10:48 PM, Brian Paul <brianp at vmware.com> wrote:
> To make the functions more understandable.
Both patches are
Reviewed-by: Chia-I Wu <olv at lunarg.com>
> ---
> src/mapi/glapi/glapi.c | 2 +-
> src/mapi/mapi_glapi.c | 2 +-
> src/mapi/u_current.c | 32 ++++++++++++++++----------------
> src/mapi/u_current.h | 20 ++++++++++----------
> 4 files changed, 28 insertions(+), 28 deletions(-)
>
> diff --git a/src/mapi/glapi/glapi.c b/src/mapi/glapi/glapi.c
> index 3a0b638..231bdad 100644
> --- a/src/mapi/glapi/glapi.c
> +++ b/src/mapi/glapi/glapi.c
> @@ -54,7 +54,7 @@ _glapi_check_multithread(void)
> void
> _glapi_set_context(void *context)
> {
> - u_current_set_user((const void *) context);
> + u_current_set_context((const void *) context);
> }
>
> void
> diff --git a/src/mapi/mapi_glapi.c b/src/mapi/mapi_glapi.c
> index 925caf3..d345f2a 100644
> --- a/src/mapi/mapi_glapi.c
> +++ b/src/mapi/mapi_glapi.c
> @@ -58,7 +58,7 @@ _glapi_check_multithread(void)
> void
> _glapi_set_context(void *context)
> {
> - u_current_set_user((const void *) context);
> + u_current_set_context((const void *) context);
> }
>
> void
> diff --git a/src/mapi/u_current.c b/src/mapi/u_current.c
> index 0721338..72190fe 100644
> --- a/src/mapi/u_current.c
> +++ b/src/mapi/u_current.c
> @@ -103,18 +103,18 @@ __thread struct mapi_table *u_current_table
> __attribute__((tls_model("initial-exec")))
> = (struct mapi_table *) table_noop_array;
>
> -__thread void *u_current_user
> +__thread void *u_current_context
> __attribute__((tls_model("initial-exec")));
>
> #else
>
> struct mapi_table *u_current_table =
> (struct mapi_table *) table_noop_array;
> -void *u_current_user;
> +void *u_current_context;
>
> #ifdef THREADS
> struct u_tsd u_current_table_tsd;
> -static struct u_tsd u_current_user_tsd;
> +static struct u_tsd u_current_context_tsd;
> static int ThreadSafe;
> #endif /* THREADS */
>
> @@ -127,7 +127,7 @@ u_current_destroy(void)
> {
> #if defined(THREADS) && defined(_WIN32)
> u_tsd_destroy(&u_current_table_tsd);
> - u_tsd_destroy(&u_current_user_tsd);
> + u_tsd_destroy(&u_current_context_tsd);
> #endif
> }
>
> @@ -138,7 +138,7 @@ static void
> u_current_init_tsd(void)
> {
> u_tsd_init(&u_current_table_tsd);
> - u_tsd_init(&u_current_user_tsd);
> + u_tsd_init(&u_current_context_tsd);
> }
>
> /**
> @@ -169,7 +169,7 @@ u_current_init(void)
> else if (knownID != u_thread_self()) {
> ThreadSafe = 1;
> u_current_set(NULL);
> - u_current_set_user(NULL);
> + u_current_set_context(NULL);
> }
> u_mutex_unlock(ThreadCheckMutex);
> }
> @@ -191,17 +191,17 @@ u_current_init(void)
> * void from the real context pointer type.
> */
> void
> -u_current_set_user(const void *ptr)
> +u_current_set_context(const void *ptr)
> {
> u_current_init();
>
> #if defined(GLX_USE_TLS)
> - u_current_user = (void *) ptr;
> + u_current_context = (void *) ptr;
> #elif defined(THREADS)
> - u_tsd_set(&u_current_user_tsd, (void *) ptr);
> - u_current_user = (ThreadSafe) ? NULL : (void *) ptr;
> + u_tsd_set(&u_current_context_tsd, (void *) ptr);
> + u_current_context = (ThreadSafe) ? NULL : (void *) ptr;
> #else
> - u_current_user = (void *) ptr;
> + u_current_context = (void *) ptr;
> #endif
> }
>
> @@ -211,16 +211,16 @@ u_current_set_user(const void *ptr)
> * void to the real context pointer type.
> */
> void *
> -u_current_get_user_internal(void)
> +u_current_get_context_internal(void)
> {
> #if defined(GLX_USE_TLS)
> - return u_current_user;
> + return u_current_context;
> #elif defined(THREADS)
> return (ThreadSafe)
> - ? u_tsd_get(&u_current_user_tsd)
> - : u_current_user;
> + ? u_tsd_get(&u_current_context_tsd)
> + : u_current_context;
> #else
> - return u_current_user;
> + return u_current_context;
> #endif
> }
>
> diff --git a/src/mapi/u_current.h b/src/mapi/u_current.h
> index f9cffd8..6709af2 100644
> --- a/src/mapi/u_current.h
> +++ b/src/mapi/u_current.h
> @@ -11,14 +11,14 @@
>
> #ifdef GLX_USE_TLS
> #define u_current_table _glapi_tls_Dispatch
> -#define u_current_user _glapi_tls_Context
> +#define u_current_context _glapi_tls_Context
> #else
> #define u_current_table _glapi_Dispatch
> -#define u_current_user _glapi_Context
> +#define u_current_context _glapi_Context
> #endif
>
> #define u_current_get_internal _glapi_get_dispatch
> -#define u_current_get_user_internal _glapi_get_context
> +#define u_current_get_context_internal _glapi_get_context
>
> #define u_current_table_tsd _gl_DispatchTSD
>
> @@ -33,13 +33,13 @@ struct mapi_table;
> extern __thread struct mapi_table *u_current_table
> __attribute__((tls_model("initial-exec")));
>
> -extern __thread void *u_current_user
> +extern __thread void *u_current_context
> __attribute__((tls_model("initial-exec")));
>
> #else /* GLX_USE_TLS */
>
> extern struct mapi_table *u_current_table;
> -extern void *u_current_user;
> +extern void *u_current_context;
>
> #endif /* GLX_USE_TLS */
>
> @@ -58,10 +58,10 @@ struct mapi_table *
> u_current_get_internal(void);
>
> void
> -u_current_set_user(const void *ptr);
> +u_current_set_context(const void *ptr);
>
> void *
> -u_current_get_user_internal(void);
> +u_current_get_context_internal(void);
>
> static INLINE const struct mapi_table *
> u_current_get(void)
> @@ -75,12 +75,12 @@ u_current_get(void)
> }
>
> static INLINE const void *
> -u_current_get_user(void)
> +u_current_get_context(void)
> {
> #ifdef GLX_USE_TLS
> - return u_current_user;
> + return u_current_context;
> #else
> - return likely(u_current_user) ? u_current_user : u_current_get_user_internal();
> + return likely(u_current_context) ? u_current_context : u_current_get_context_internal();
> #endif
> }
>
> --
> 1.7.10.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
--
olv at LunarG.com
More information about the mesa-dev
mailing list