public inbox for libc-ports@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] tile: use _dl_static_init to set GLRO(gl_pagesize)
@ 2013-07-03 20:21 Chris Metcalf
  2013-07-03 20:34 ` Carlos O'Donell
  0 siblings, 1 reply; 3+ messages in thread
From: Chris Metcalf @ 2013-07-03 20:21 UTC (permalink / raw)
  To: libc-ports

A recently-added test (dlfcn/tststatic5) pointed out that tile was not
properly initializing the variable pagesize in certain cases.  This
change just copies the existing code from MIPS.
---
I assume this change can be committed to master for inclusion in 2.18,
as it fixes a (new) testsuite failure?

 ports/ChangeLog.tile                           |    6 ++
 ports/sysdeps/unix/sysv/linux/tile/Makefile    |    6 ++
 ports/sysdeps/unix/sysv/linux/tile/Versions    |    6 ++
 ports/sysdeps/unix/sysv/linux/tile/dl-static.c |   84 ++++++++++++++++++++++++
 ports/sysdeps/unix/sysv/linux/tile/ldsodefs.h  |   32 +++++++++
 5 files changed, 134 insertions(+), 0 deletions(-)
 create mode 100644 ports/sysdeps/unix/sysv/linux/tile/dl-static.c
 create mode 100644 ports/sysdeps/unix/sysv/linux/tile/ldsodefs.h

2013-07-03  Chris Metcalf  <cmetcalf@tilera.com>

	* sysdeps/unix/sysv/linux/tile/dl-static.c: New file to support
	variable page size for tile.
	* sysdeps/unix/sysv/linux/tile/ldsodefs.h: Likewise.
	* sysdeps/unix/sysv/linux/tile/Makefile: Build dl-static in elf.
	* sysdeps/unix/sysv/linux/tile/Versions: Add _dl_var_init.

diff --git a/ports/sysdeps/unix/sysv/linux/tile/Makefile b/ports/sysdeps/unix/sysv/linux/tile/Makefile
index 1c0c968..0bcb95f 100644
--- a/ports/sysdeps/unix/sysv/linux/tile/Makefile
+++ b/ports/sysdeps/unix/sysv/linux/tile/Makefile
@@ -16,4 +16,10 @@ endif
 
 ifeq ($(subdir),elf)
 sysdep_routines += dl-vdso
+ifeq ($(build-shared),yes)
+# This is needed for DSO loading from static binaries.
+sysdep-dl-routines += dl-static
+sysdep_routines += dl-static
+sysdep-rtld-routines += dl-static
+endif
 endif
diff --git a/ports/sysdeps/unix/sysv/linux/tile/Versions b/ports/sysdeps/unix/sysv/linux/tile/Versions
index 1df7518..9b40d28 100644
--- a/ports/sysdeps/unix/sysv/linux/tile/Versions
+++ b/ports/sysdeps/unix/sysv/linux/tile/Versions
@@ -1,3 +1,9 @@
+ld {
+  GLIBC_PRIVATE {
+    # used for loading by static libraries
+    _dl_var_init;
+  }
+}
 libc {
   GLIBC_2.12 {
     _flush_cache;
diff --git a/ports/sysdeps/unix/sysv/linux/tile/dl-static.c b/ports/sysdeps/unix/sysv/linux/tile/dl-static.c
new file mode 100644
index 0000000..5b59a04
--- /dev/null
+++ b/ports/sysdeps/unix/sysv/linux/tile/dl-static.c
@@ -0,0 +1,84 @@
+/* Variable initialization.  Tile version.
+   Copyright (C) 2001-2013 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library.  If not, see
+   <http://www.gnu.org/licenses/>.  */
+
+#include <ldsodefs.h>
+
+#ifdef SHARED
+
+void
+_dl_var_init (void *array[])
+{
+  /* It has to match "variables" below. */
+  enum
+    {
+      DL_PAGESIZE = 0
+    };
+
+  GLRO(dl_pagesize) = *((size_t *) array[DL_PAGESIZE]);
+}
+
+#else
+
+static void *variables[] =
+{
+  &GLRO(dl_pagesize)
+};
+
+static void
+_dl_unprotect_relro (struct link_map *l)
+{
+  ElfW(Addr) start = ((l->l_addr + l->l_relro_addr)
+		      & ~(GLRO(dl_pagesize) - 1));
+  ElfW(Addr) end = ((l->l_addr + l->l_relro_addr + l->l_relro_size)
+		    & ~(GLRO(dl_pagesize) - 1));
+
+  if (start != end)
+    __mprotect ((void *) start, end - start, PROT_READ | PROT_WRITE);
+}
+
+void
+_dl_static_init (struct link_map *l)
+{
+  struct link_map *rtld_map = l;
+  struct r_scope_elem **scope;
+  const ElfW(Sym) *ref = NULL;
+  lookup_t loadbase;
+  void (*f) (void *[]);
+  size_t i;
+
+  loadbase = _dl_lookup_symbol_x ("_dl_var_init", l, &ref, l->l_local_scope,
+				  NULL, 0, 1, NULL);
+
+  for (scope = l->l_local_scope; *scope != NULL; scope++)
+    for (i = 0; i < (*scope)->r_nlist; i++)
+      if ((*scope)->r_list[i] == loadbase)
+	{
+	  rtld_map = (*scope)->r_list[i];
+	  break;
+	}
+
+  if (ref != NULL)
+    {
+      f = (void (*) (void *[])) DL_SYMBOL_ADDRESS (loadbase, ref);
+      _dl_unprotect_relro (rtld_map);
+      f (variables);
+      _dl_protect_relro (rtld_map);
+    }
+}
+
+#endif
diff --git a/ports/sysdeps/unix/sysv/linux/tile/ldsodefs.h b/ports/sysdeps/unix/sysv/linux/tile/ldsodefs.h
new file mode 100644
index 0000000..92877e5
--- /dev/null
+++ b/ports/sysdeps/unix/sysv/linux/tile/ldsodefs.h
@@ -0,0 +1,32 @@
+/* Run-time dynamic linker data structures for loaded ELF shared objects. Tile.
+   Copyright (C) 2001-2013 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <http://www.gnu.org/licenses/>.  */
+
+#ifndef	_LDSODEFS_H
+
+/* Get the real definitions.  */
+#include_next <ldsodefs.h>
+
+/* Now define our stuff.  */
+
+/* We need special support to initialize DSO loaded for statically linked
+   binaries.  */
+extern void _dl_static_init (struct link_map *map);
+#undef DL_STATIC_INIT
+#define DL_STATIC_INIT(map) _dl_static_init (map)
+
+#endif /* ldsodefs.h */
-- 
1.7.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] tile: use _dl_static_init to set GLRO(gl_pagesize)
  2013-07-03 20:21 [PATCH] tile: use _dl_static_init to set GLRO(gl_pagesize) Chris Metcalf
@ 2013-07-03 20:34 ` Carlos O'Donell
  2013-07-03 20:42   ` Chris Metcalf
  0 siblings, 1 reply; 3+ messages in thread
From: Carlos O'Donell @ 2013-07-03 20:34 UTC (permalink / raw)
  To: Chris Metcalf; +Cc: libc-ports

On 07/03/2013 02:48 PM, Chris Metcalf wrote:
> A recently-added test (dlfcn/tststatic5) pointed out that tile was not
> properly initializing the variable pagesize in certain cases.  This
> change just copies the existing code from MIPS.
> ---
> I assume this change can be committed to master for inclusion in 2.18,
> as it fixes a (new) testsuite failure?

Yes.

Bug fixes (not new features) are the whole point of the freeze ;-)

If you have something that's a bit of both you need to include
the release manager (David Miller) and argue your point.

Cheers,
Carlos.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] tile: use _dl_static_init to set GLRO(gl_pagesize)
  2013-07-03 20:34 ` Carlos O'Donell
@ 2013-07-03 20:42   ` Chris Metcalf
  0 siblings, 0 replies; 3+ messages in thread
From: Chris Metcalf @ 2013-07-03 20:42 UTC (permalink / raw)
  To: Carlos O'Donell; +Cc: libc-ports

On 7/3/2013 4:34 PM, Carlos O'Donell wrote:
> On 07/03/2013 02:48 PM, Chris Metcalf wrote:
>> A recently-added test (dlfcn/tststatic5) pointed out that tile was not
>> properly initializing the variable pagesize in certain cases.  This
>> change just copies the existing code from MIPS.
>> ---
>> I assume this change can be committed to master for inclusion in 2.18,
>> as it fixes a (new) testsuite failure?
> Yes.
>
> Bug fixes (not new features) are the whole point of the freeze ;-)
>
> If you have something that's a bit of both you need to include
> the release manager (David Miller) and argue your point.

I'm comfortable that this one is a pure bug fix.

-- 
Chris Metcalf, Tilera Corp.
http://www.tilera.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-07-03 20:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-03 20:21 [PATCH] tile: use _dl_static_init to set GLRO(gl_pagesize) Chris Metcalf
2013-07-03 20:34 ` Carlos O'Donell
2013-07-03 20:42   ` Chris Metcalf

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).