* [Patch] GETSP for MIPS
@ 2000-02-01 9:19 Andreas Jaeger
2000-02-02 2:16 ` Andreas Schwab
0 siblings, 1 reply; 4+ messages in thread
From: Andreas Jaeger @ 2000-02-01 9:19 UTC (permalink / raw)
To: GNU libc hacker
I got on MIPS the following warning:
memprof.c:146: warning: implicit declaration of function `GETSP'
We should check if we really have a definition for GETSP. I'm
appending a patch.
Andreas Schwab, could you add a m68k version?
Andreas
2000-02-01 Andreas Jaeger <aj@suse.de>
* malloc/memprof.c (GETSP): Add version for Mips, warn if GETSP is
not defined.
============================================================
Index: malloc/memprof.c
--- malloc/memprof.c 1999/10/12 00:19:09 1.2
+++ malloc/memprof.c 2000/02/01 17:10:37
@@ -1,5 +1,5 @@
/* Profile heap and stack memory usage of running program.
- Copyright (C) 1998, 1999 Free Software Foundation, Inc.
+ Copyright (C) 1998, 1999, 2000 Free Software Foundation, Inc.
This file is part of the GNU C Library.
Contributed by Ulrich Drepper <drepper@cygnus.com>, 1998.
@@ -85,6 +85,13 @@
#endif
#ifdef __powerpc__
# define GETSP() ({ register uintptr_t stack_ptr asm ("%r1"); stack_ptr; })
+#endif
+#ifdef __mips__
+# define GETSP() ({ register uintptr_t stack_ptr asm ("$29"); stack_ptr; })
+#endif
+#ifndef GETSP
+# warning "GETSP is not defined for this architecture."
+# define GETSP 0
#endif
#ifdef __i386__
--
Andreas Jaeger
SuSE Labs aj@suse.de
private aj@arthur.rhein-neckar.de
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Patch] GETSP for MIPS
2000-02-01 9:19 [Patch] GETSP for MIPS Andreas Jaeger
@ 2000-02-02 2:16 ` Andreas Schwab
2000-02-02 19:08 ` Richard Henderson
2000-02-02 22:56 ` Jakub Jelinek
0 siblings, 2 replies; 4+ messages in thread
From: Andreas Schwab @ 2000-02-02 2:16 UTC (permalink / raw)
To: Andreas Jaeger; +Cc: GNU libc hacker
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 11432 bytes --]
Andreas Jaeger <aj@suse.de> writes:
|> I got on MIPS the following warning:
|> memprof.c:146: warning: implicit declaration of function `GETSP'
|>
|> We should check if we really have a definition for GETSP. I'm
|> appending a patch.
The macro does not belong here, it should be moved to a sysdep header.
Andreas.
2000-02-02 Andreas Schwab <schwab@suse.de>
* malloc/Makefile (distribute): Add memprof.h
* malloc/memprof.c: Move definition of GETSP and GETTIME to new
header memprof.h.
* sysdeps/generic/memprof.h: New file.
* sysdeps/alpha/memprof.h: New file.
* sysdeps/i386/memprof.h: New file.
* sysdeps/m68k/memprof.h: New file.
* sysdeps/mips/memprof.h: New file.
* sysdeps/powerpc/memprof.h: New file.
* sysdeps/sparc/memprof.h: New file.
Index: malloc/Makefile
===================================================================
RCS file: /cvs/glibc/libc/malloc/Makefile,v
retrieving revision 1.21
diff -u -a -u -r1.21 malloc/Makefile
--- malloc/Makefile 1999/10/05 05:31:11 1.21
+++ malloc/Makefile 2000/02/02 09:56:09
@@ -27,7 +27,7 @@
headers := $(dist-headers) obstack.h mcheck.h
tests := mallocbug tst-malloc
-distribute = thread-m.h mtrace.pl mcheck-init.c stackinfo.h
+distribute = thread-m.h mtrace.pl mcheck-init.c stackinfo.h memprof.h
# Things which get pasted together into gmalloc.c.
gmalloc-routines := malloc morecore
Index: malloc/memprof.c
===================================================================
RCS file: /cvs/glibc/libc/malloc/memprof.c,v
retrieving revision 1.2
diff -u -a -u -r1.2 malloc/memprof.c
--- malloc/memprof.c 1999/10/12 00:19:09 1.2
+++ malloc/memprof.c 2000/02/02 09:55:06
@@ -28,6 +28,8 @@
#include <unistd.h>
#include <sys/time.h>
+#include <memprof.h>
+
/* Pointer to the real functions. These are determined used `dlsym'
when really needed. */
static void *(*mallocp) (size_t);
@@ -73,34 +75,6 @@
#define peak_heap peak_use[0]
#define peak_stack peak_use[1]
#define peak_total peak_use[2]
-
-#ifdef __i386__
-# define GETSP() ({ register uintptr_t stack_ptr asm ("esp"); stack_ptr; })
-#endif
-#ifdef __alpha__
-# define GETSP() ({ register uintptr_t stack_ptr asm ("$30"); stack_ptr; })
-#endif
-#ifdef __sparc__
-# define GETSP() ({ register uintptr_t stack_ptr asm ("%sp"); stack_ptr; })
-#endif
-#ifdef __powerpc__
-# define GETSP() ({ register uintptr_t stack_ptr asm ("%r1"); stack_ptr; })
-#endif
-
-#ifdef __i386__
-# define GETTIME(low,high) asm ("rdtsc" : "=a" (low), "=d" (high))
-#endif
-#ifndef GETTIME
-# define GETTIME(low,high) \
- { \
- struct timeval tval; \
- uint64_t usecs; \
- gettimeofday (&tval, NULL); \
- usecs = (uint64_t) tval.tv_usec + (uint64_t) tval.tv_usec * 1000000; \
- low = usecs & 0xffffffff; \
- high = usecs >> 32; \
- }
-#endif
#define DEFAULT_BUFFER_SIZE 1024
static size_t buffer_size;
--- /dev/null Tue Jan 18 16:58:34 2000
+++ sysdeps/alpha/memprof.h Wed Feb 2 10:52:05 2000
@@ -0,0 +1,21 @@
+/* Copyright (C) 2000 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 Library General Public License as
+ published by the Free Software Foundation; either version 2 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
+ Library General Public License for more details.
+
+ You should have received a copy of the GNU Library General Public
+ License along with the GNU C Library; see the file COPYING.LIB. If not,
+ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ Boston, MA 02111-1307, USA. */
+
+#define GETSP() ({ register uintptr_t stack_ptr asm ("$30"); stack_ptr; })
+
+#include <sysdeps/generic/memprof.h>
--- /dev/null Tue Jan 18 16:58:34 2000
+++ sysdeps/generic/memprof.h Wed Feb 2 10:56:39 2000
@@ -0,0 +1,35 @@
+/* Copyright (C) 2000 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 Library General Public License as
+ published by the Free Software Foundation; either version 2 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
+ Library General Public License for more details.
+
+ You should have received a copy of the GNU Library General Public
+ License along with the GNU C Library; see the file COPYING.LIB. If not,
+ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ Boston, MA 02111-1307, USA. */
+
+
+#ifndef GETSP
+# warning "GETSP is not defined for this architecture."
+# define GETSP 0
+#endif
+
+#ifndef GETTIME
+# define GETTIME(low,high) \
+ { \
+ struct timeval tval; \
+ uint64_t usecs; \
+ gettimeofday (&tval, NULL); \
+ usecs = (uint64_t) tval.tv_usec + (uint64_t) tval.tv_usec * 1000000; \
+ low = usecs & 0xffffffff; \
+ high = usecs >> 32; \
+ }
+#endif
--- /dev/null Tue Jan 18 16:58:34 2000
+++ sysdeps/i386/memprof.h Wed Feb 2 10:50:58 2000
@@ -0,0 +1,22 @@
+/* Copyright (C) 2000 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 Library General Public License as
+ published by the Free Software Foundation; either version 2 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
+ Library General Public License for more details.
+
+ You should have received a copy of the GNU Library General Public
+ License along with the GNU C Library; see the file COPYING.LIB. If not,
+ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ Boston, MA 02111-1307, USA. */
+
+#define GETSP() ({ register uintptr_t stack_ptr asm ("esp"); stack_ptr; })
+#define GETTIME(low,high) asm ("rdtsc" : "=a" (low), "=d" (high))
+
+#include <sysdeps/generic/memprof.h>
--- /dev/null Tue Jan 18 16:58:34 2000
+++ sysdeps/m68k/memprof.h Wed Feb 2 10:51:44 2000
@@ -0,0 +1,22 @@
+/* Copyright (C) 2000 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 Library General Public License as
+ published by the Free Software Foundation; either version 2 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
+ Library General Public License for more details.
+
+ You should have received a copy of the GNU Library General Public
+ License along with the GNU C Library; see the file COPYING.LIB. If not,
+ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ Boston, MA 02111-1307, USA. */
+
+
+#define GETSP() ({ register uintptr_t stack_ptr asm ("%sp"); stack_ptr; })
+
+#include <sysdeps/generic/memprof.h>
--- /dev/null Tue Jan 18 16:58:34 2000
+++ sysdeps/mips/memprof.h Wed Feb 2 10:53:29 2000
@@ -0,0 +1,21 @@
+/* Copyright (C) 2000 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 Library General Public License as
+ published by the Free Software Foundation; either version 2 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
+ Library General Public License for more details.
+
+ You should have received a copy of the GNU Library General Public
+ License along with the GNU C Library; see the file COPYING.LIB. If not,
+ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ Boston, MA 02111-1307, USA. */
+
+#define GETSP() ({ register uintptr_t stack_ptr asm ("$29"); stack_ptr; })
+
+#include <sysdeps/generic/memprof.h>
--- /dev/null Tue Jan 18 16:58:34 2000
+++ sysdeps/powerpc/memprof.h Wed Feb 2 10:53:03 2000
@@ -0,0 +1,21 @@
+/* Copyright (C) 2000 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 Library General Public License as
+ published by the Free Software Foundation; either version 2 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
+ Library General Public License for more details.
+
+ You should have received a copy of the GNU Library General Public
+ License along with the GNU C Library; see the file COPYING.LIB. If not,
+ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ Boston, MA 02111-1307, USA. */
+
+#define GETSP() ({ register uintptr_t stack_ptr asm ("%r1"); stack_ptr; })
+
+#include <sysdeps/generic/memprof.h>
--- /dev/null Tue Jan 18 16:58:34 2000
+++ sysdeps/sparc/memprof.h Wed Feb 2 10:52:32 2000
@@ -0,0 +1,21 @@
+/* Copyright (C) 2000 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 Library General Public License as
+ published by the Free Software Foundation; either version 2 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
+ Library General Public License for more details.
+
+ You should have received a copy of the GNU Library General Public
+ License along with the GNU C Library; see the file COPYING.LIB. If not,
+ write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ Boston, MA 02111-1307, USA. */
+
+#define GETSP() ({ register uintptr_t stack_ptr asm ("%sp"); stack_ptr; })
+
+#include <sysdeps/generic/memprof.h>
--
Andreas Schwab "And now for something
SuSE Labs completely different."
schwab@suse.de
SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Patch] GETSP for MIPS
2000-02-02 2:16 ` Andreas Schwab
@ 2000-02-02 19:08 ` Richard Henderson
2000-02-02 22:56 ` Jakub Jelinek
1 sibling, 0 replies; 4+ messages in thread
From: Richard Henderson @ 2000-02-02 19:08 UTC (permalink / raw)
To: Andreas Schwab; +Cc: Andreas Jaeger, GNU libc hacker
On Wed, Feb 02, 2000 at 11:15:55AM +0100, Andreas Schwab wrote:
> * malloc/memprof.c: Move definition of GETSP and GETTIME to new
> header memprof.h.
Instead of GETSP, I recommend __builtin_frame_address(0).
r~
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Patch] GETSP for MIPS
2000-02-02 2:16 ` Andreas Schwab
2000-02-02 19:08 ` Richard Henderson
@ 2000-02-02 22:56 ` Jakub Jelinek
1 sibling, 0 replies; 4+ messages in thread
From: Jakub Jelinek @ 2000-02-02 22:56 UTC (permalink / raw)
To: Andreas Schwab; +Cc: Andreas Jaeger, GNU libc hacker
On Wed, Feb 02, 2000 at 11:15:55AM +0100, Andreas Schwab wrote:
> Andreas Jaeger <aj@suse.de> writes:
>
> |> I got on MIPS the following warning:
> |> memprof.c:146: warning: implicit declaration of function `GETSP'
> |>
> |> We should check if we really have a definition for GETSP. I'm
> |> appending a patch.
>
> The macro does not belong here, it should be moved to a sysdep header.
sparc/sparc32/sparcv9 GETTIME could be:
#define GETTIME(low,high) \
{ \
register unsigned int lowtick asm ("g1"); \
asm ("rd %%tick, %%g1; srlx %%g1, 32, %1" \
: "=r" (lowtick), "=r" (high)); \
low = lowtick; \
}
sparc/sparc64 GETTIME:
#define GETTIME(low,high) \
{ \
uint64_t tickreg; \
asm ("rd %%tick, %0" : "=r" (tickreg)); \
low = tickreg & 0xffffffff; \
high = tickreg >> 32; \
}
Cheers,
Jakub
___________________________________________________________________
Jakub Jelinek | jakub@redhat.com | http://sunsite.mff.cuni.cz/~jj
Linux version 2.3.41 on a sparc64 machine (1343.49 BogoMips)
___________________________________________________________________
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2000-02-02 22:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2000-02-01 9:19 [Patch] GETSP for MIPS Andreas Jaeger
2000-02-02 2:16 ` Andreas Schwab
2000-02-02 19:08 ` Richard Henderson
2000-02-02 22:56 ` Jakub Jelinek
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).