From 28f8dbd7b941648aea311bb0cf331f88c02441b6 Mon Sep 17 00:00:00 2001 From: qmuntal Date: Fri, 2 Dec 2022 09:25:26 +0100 Subject: runtime,cmd/internal/obj/x86: use TEB TLS slots on windows/i386 This CL redesign how we get the TLS pointer on windows/i386. It applies the same changes as done in CL 431775 for windows/amd64. We were previously reading it from the [TEB] arbitrary data slot, located at 0x14(FS), which can only hold 1 TLS pointer. With this CL, we will read the TLS pointer from the TEB TLS slot array, located at 0xE10(GS). The TLS slot array can hold multiple TLS pointers, up to 64, so multiple Go runtimes running on the same thread can coexists with different TLS. Each new TLS slot has to be allocated via [TlsAlloc], which returns the slot index. This index can then be used to get the slot offset from GS with the following formula: 0xE10 + index*4. The slot index is fixed per Go runtime, so we can store it in runtime.tls_g and use it latter on to read/update the TLS pointer. Loading the TLS pointer requires the following asm instructions: MOVQ runtime.tls_g, AX MOVQ AX(FS), AX Notice that this approach will now be implemented in all the supported windows arches. [TEB]: https://en.wikipedia.org/wiki/Win32_Thread_Information_Block [TlsAlloc]: https://learn.microsoft.com/en-us/windows/win32/api/processthreadsapi/nf-processthreadsapi-tlsalloc Change-Id: If4550b0d44694ee6480d4093b851f4991a088b32 Reviewed-on: https://go-review.googlesource.com/c/go/+/454675 Reviewed-by: Michael Pratt Run-TryBot: Quim Muntal Reviewed-by: Cherry Mui TryBot-Result: Gopher Robot --- src/runtime/cgo/gcc_windows_386.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'src/runtime/cgo') diff --git a/src/runtime/cgo/gcc_windows_386.c b/src/runtime/cgo/gcc_windows_386.c index 56fbaac9b8..0f4f01c7c0 100644 --- a/src/runtime/cgo/gcc_windows_386.c +++ b/src/runtime/cgo/gcc_windows_386.c @@ -12,10 +12,12 @@ #include "libcgo_windows.h" static void threadentry(void*); +static DWORD *tls_g; void -x_cgo_init(G *g) +x_cgo_init(G *g, void (*setg)(void*), void **tlsg, void **tlsbase) { + tls_g = (DWORD *)tlsg; } @@ -39,10 +41,10 @@ threadentry(void *v) * Set specific keys in thread local storage. */ asm volatile ( - "movl %0, %%fs:0x14\n" // MOVL tls0, 0x14(FS) - "movl %%fs:0x14, %%eax\n" // MOVL 0x14(FS), tmp - "movl %1, 0(%%eax)\n" // MOVL g, 0(FS) - :: "r"(ts.tls), "r"(ts.g) : "%eax" + "movl %0, %%fs:0(%1)\n" // MOVL tls0, 0(tls_g)(FS) + "movl %%fs:0(%1), %%eax\n" // MOVL 0(tls_g)(FS), tmp + "movl %2, 0(%%eax)\n" // MOVL g, 0(AX) + :: "r"(ts.tls), "r"(*tls_g), "r"(ts.g) : "%eax" ); crosscall_386(ts.fn); -- cgit v1.3