Changes in uspace/lib/c/generic/thread/tls.c [32254d6:ffccdff0] in mainline

File:

• uspace/lib/c/generic/thread/tls.c (modified) (5 diffs)

uspace/lib/c/generic/thread/tls.c

@@ -59 +59 @@
 #endif
 
-static ptrdiff_t _tcb_data_offset(const void *elf)
+static ptrdiff_t _tcb_data_offset(void)
 {
         const elf_segment_header_t *tls =
-            elf_get_phdr(elf, PT_TLS);
+            elf_get_phdr(__progsymbols.elfstart, PT_TLS);
 
         size_t tls_align = tls ? tls->p_align : 1;
@@ -74 +74 @@
 }
 
-/** Get address of static TLS block - only when RTLD is not initialized  */
+/** Get address of static TLS block */
 void *tls_get(void)
 {
@@ -80 +80 @@
         assert(runtime_env == NULL);
 #endif
-        return (uint8_t *)__tcb_get() + _tcb_data_offset(__progsymbols.elfstart);
+        return (uint8_t *)__tcb_get() + _tcb_data_offset();
 }
 
 static tcb_t *tls_make_generic(const void *elf, void *(*alloc)(size_t, size_t))
 {
-        /*
-         * See also rtld/module.c -> modules_process_tls(), where we have less
-         * messy code for the dynamic-linking version of this.
-         */
         assert(!elf_get_phdr(elf, PT_DYNAMIC));
 #ifdef CONFIG_RTLD
@@ -104 +100 @@
         assert(tls_align <= PAGE_SIZE);
 
-        /*
-         * FIXME: the calculation of alloc_size shouldn't include the alignment
-         * of tcb_t (at least in Variant II)
-         * See https://github.com/HelenOS/helenos/pull/240/files/4ef27ebf98a0656e09889b7d00efdec03343f1aa#r1929592924
-         * (you will also need to fix _tcb_data_offset)
-         */
-
 #ifdef CONFIG_TLS_VARIANT_1
         size_t alloc_size =
@@ -125 +114 @@
 #ifdef CONFIG_TLS_VARIANT_1
         tcb_t *tcb = area;
-        uint8_t *data = (uint8_t *)tcb + _tcb_data_offset(elf);
+        uint8_t *data = (uint8_t *)tcb + _tcb_data_offset();
         memset(tcb, 0, sizeof(*tcb));
 #else
         uint8_t *data = area;
-        tcb_t *tcb = (tcb_t *) (data - _tcb_data_offset(elf));
+        tcb_t *tcb = (tcb_t *) (data - _tcb_data_offset());
         memset(tcb, 0, sizeof(tcb_t));
         tcb->self = tcb;