public inbox for glibc-cvs@sourceware.org
help / color / mirror / Atom feed
* [glibc] hurd: Move thread state manipulation into _hurd_tls_new ()
@ 2023-02-19 23:32 Samuel Thibault
0 siblings, 0 replies; only message in thread
From: Samuel Thibault @ 2023-02-19 23:32 UTC (permalink / raw)
To: glibc-cvs
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=e48f33e76be2a3baa920b7e9f472354f7ff0c0a6
commit e48f33e76be2a3baa920b7e9f472354f7ff0c0a6
Author: Sergey Bugaev <bugaevc@gmail.com>
Date: Sat Feb 18 23:37:09 2023 +0300
hurd: Move thread state manipulation into _hurd_tls_new ()
This is going to be done differently on x86_64.
Signed-off-by: Sergey Bugaev <bugaevc@gmail.com>
Message-Id: <20230218203717.373211-2-bugaevc@gmail.com>
Diff:
---
mach/setup-thread.c | 18 ++----------------
sysdeps/mach/hurd/i386/tls.h | 25 ++++++++++++++++++++-----
2 files changed, 22 insertions(+), 21 deletions(-)
diff --git a/mach/setup-thread.c b/mach/setup-thread.c
index 6ce5c13d37..ae24a14955 100644
--- a/mach/setup-thread.c
+++ b/mach/setup-thread.c
@@ -83,25 +83,11 @@ weak_alias (__mach_setup_thread, mach_setup_thread)
kern_return_t
__mach_setup_tls (thread_t thread)
{
- kern_return_t error;
- struct machine_thread_state ts;
- mach_msg_type_number_t tssize = MACHINE_THREAD_STATE_COUNT;
- tcbhead_t *tcb;
-
- tcb = _dl_allocate_tls (NULL);
+ tcbhead_t *tcb = _dl_allocate_tls (NULL);
if (tcb == NULL)
return KERN_RESOURCE_SHORTAGE;
- if (error = __thread_get_state (thread, MACHINE_THREAD_STATE_FLAVOR,
- (natural_t *) &ts, &tssize))
- return error;
- assert (tssize == MACHINE_THREAD_STATE_COUNT);
-
- _hurd_tls_new (thread, &ts, tcb);
-
- error = __thread_set_state (thread, MACHINE_THREAD_STATE_FLAVOR,
- (natural_t *) &ts, tssize);
- return error;
+ return _hurd_tls_new (thread, tcb);
}
weak_alias (__mach_setup_tls, mach_setup_tls)
diff --git a/sysdeps/mach/hurd/i386/tls.h b/sysdeps/mach/hurd/i386/tls.h
index 590abd47fe..0f8dd24145 100644
--- a/sysdeps/mach/hurd/i386/tls.h
+++ b/sysdeps/mach/hurd/i386/tls.h
@@ -378,16 +378,25 @@ _hurd_tls_fork (thread_t child, thread_t orig, struct i386_thread_state *state)
}
static inline kern_return_t __attribute__ ((unused))
-_hurd_tls_new (thread_t child, struct i386_thread_state *state, tcbhead_t *tcb)
+_hurd_tls_new (thread_t child, tcbhead_t *tcb)
{
+ error_t err;
+ /* Fetch the target thread's state. */
+ struct i386_thread_state state;
+ mach_msg_type_number_t state_count = i386_THREAD_STATE_COUNT;
+ err = __thread_get_state (child, i386_REGS_SEGS_STATE,
+ (thread_state_t) &state,
+ &state_count);
+ if (err)
+ return err;
+ assert (state_count == i386_THREAD_STATE_COUNT);
/* Fetch the selector set by _hurd_tls_init. */
int sel;
asm ("mov %%gs, %w0" : "=q" (sel) : "0" (0));
- if (sel == state->ds) /* _hurd_tls_init was never called. */
+ if (sel == state.ds) /* _hurd_tls_init was never called. */
return 0;
HURD_TLS_DESC_DECL (desc, tcb);
- error_t err;
tcb->tcb = tcb;
tcb->self = child;
@@ -397,8 +406,14 @@ _hurd_tls_new (thread_t child, struct i386_thread_state *state, tcbhead_t *tcb)
else
err = __i386_set_gdt (child, &sel, desc);
- state->gs = sel;
- return err;
+ if (err)
+ return err;
+
+ /* Update gs to use the selector. */
+ state.gs = sel;
+ return __thread_set_state (child, i386_REGS_SEGS_STATE,
+ (thread_state_t) &state,
+ state_count);
}
/* Global scope switch support. */
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2023-02-19 23:32 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-19 23:32 [glibc] hurd: Move thread state manipulation into _hurd_tls_new () Samuel Thibault
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).