Changeset d6f9fff in mainline for kernel/arch/ia32/include

Timestamp:

2016-04-27T12:39:14Z (10 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

1a5eca4

Parents:

8a36bc1e

Message:

ia32: Make TLS settable from uspace

The TLS document[1] mandates that %gs[0] is the thread pointer on ia32.
That is good as it allows userspace-only TLS management for fibrils:
fibril_save/restore() simply manipulate the thread pointer in %gs:0 and
don't need to ask the kernel to modify %gs's base. The kernel treats
%gs:0 as another preserved register and preserves it across context
switches. GCC gets in the way a little bit because it by default assumes
that TLS is accessible from negative %gs offsets (which would
necessitate a kernel-assisted solution). Fortunately, there is a GCC
option to suppress this assumption.

• Introduce the concept of virtual registers, with VREG_TP (thread pointer) being the first of them
• Preserve VREG_TP in context_save/restore()
• Stop using sys_tls_set() in favour of using %gs:0 as the thread pointer
• Make GCC generate code that always goes through %gs:0 to access TLS

[1] Drepper, U.: ELF Handling For Thread-Local Storage

Location:

kernel/arch/ia32/include/arch

Files:

• asm.h (modified) (1 diff)
• context_struct.ag (modified) (1 diff)
• pm.h (modified) (2 diffs)
• proc/thread.h (modified) (1 diff)
• vreg.h (added)

kernel/arch/ia32/include/arch/asm.h

@@ -439 +439 @@
         asm volatile (
                 "ltr %[sel]"
+                :: [sel] "r" (sel)
+        );
+}
+
+/** Load GS from descriptor table.
+ *
+ * @param sel Selector specifying descriptor of the GS segment.
+ *
+ */
+NO_TRACE static inline void gs_load(uint16_t sel)
+{
+        asm volatile (
+                "mov %[sel], %%gs"
                 :: [sel] "r" (sel)
         );

kernel/arch/ia32/include/arch/context_struct.ag

@@ -66 +66 @@
                 },
                 {
+                        name : tp,
+                        type : uint32_t
+                },
+
+                {
                         name : ipl,
                         type : ipl_t

kernel/arch/ia32/include/arch/pm.h

@@ -45 +45 @@
 #define UDATA_DES  4
 #define TSS_DES    5
-#define TLS_DES    6  /* Pointer to Thread-Local-Storage data */
+#define VREG_DES   6  /* Virtual registers */
 
 #ifdef CONFIG_FB
@@ -169 +169 @@
 
 extern void tss_initialize(tss_t *t);
-extern void set_tls_desc(uintptr_t tls);
 
 #endif /* __ASM__ */

kernel/arch/ia32/include/arch/proc/thread.h

@@ -39 +39 @@
 
 typedef struct {
-        sysarg_t tls;
 } thread_arch_t;