[PATCH 2/2] i386: Use __seg_gs qualifier to cast access to TCB in THREAD_GSCOPE_RESET_FLAG()

[H.J. Lu]

On Wed, Sep 10, 2025 at 12:01 PM Uros Bizjak <ubizjak@gmail.com> wrote:

> Use the __seg_gs named address space qualifier to cast access to the
> gscope_flag in the TCB as a %gs: prefixed address.  This enables the
> use of the "m" operand constraint, which informs the compiler about
> memory access in the inline assembly.
>
> Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
> Cc: H.J.Lu <hjl.tools@gmail.com>
> Cc: Florian Weimer <fweimer@redhat.com>
> Cc: Carlos O'Donell <carlos@redhat.com>
> ---
>  sysdeps/i386/nptl/tls.h | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/sysdeps/i386/nptl/tls.h b/sysdeps/i386/nptl/tls.h
> index 35202b0964..5c266c7191 100644
> --- a/sysdeps/i386/nptl/tls.h
> +++ b/sysdeps/i386/nptl/tls.h
> @@ -255,9 +255,10 @@ tls_fill_user_desc (union user_desc_init *desc,
>  #define THREAD_GSCOPE_RESET_FLAG() \
>    do
>    \
>      { int __res;
>    \
> -      asm volatile ("xchgl %0, %%gs:%P1"
>    \
> +      asm volatile ("xchgl %1, %0"
>    \
>                     : "=r" (__res)
>     \
> -                   : "i" (offsetof (struct pthread,
> header.gscope_flag)),    \
> +                   : "m" (*(int __seg_gs *)
>     \
> +                          offsetof (struct pthread,
> header.gscope_flag)),    \
>                       "0" (THREAD_GSCOPE_FLAG_UNUSED));
>    \
>        if (__res == THREAD_GSCOPE_FLAG_WAIT)
>     \
>         lll_futex_wake (&THREAD_SELF->header.gscope_flag, 1,
> LLL_PRIVATE);    \
> --
> 2.51.0
>
>
LGTM.

Reviewed-by: H.J. Lu <hjl.tools@gmail.com>

Thanks.

-- 
H.J.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://sourceware.org/pipermail/libc-alpha/attachments/20250914/46139c55/attachment-0001.htm>