[Qemu-devel] [PATCH] User qemu profiling

["Laurent Desnogues" <laurent.desnogues@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 383 bytes --]

Hello,

this patch:

  - Makefile.target:  re-enable profiling for user qemu.  It seems
    profiling was (accidently?) removed by commit 3937

  - syscall.c:
      * add an include to get _mcleanup prototype
      * add a call to _mcleanup for exit_group in a way
        similar to what is done for exit

BTW, I wonder why -p is preferred over -pg for profiling.

Thanks,

Laurent

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: user-profile.patch --]
[-- Type: text/x-patch; name=user-profile.patch, Size: 1032 bytes --]

Index: Makefile.target
===================================================================
--- Makefile.target	(revision 5545)
+++ Makefile.target	(working copy)
@@ -394,6 +394,12 @@
 endif
 endif
 
+# profiling code
+ifdef TARGET_GPROF
+LDFLAGS+=-p
+CFLAGS+=-p
+endif
+
 OBJS= main.o syscall.o strace.o mmap.o signal.o path.o thunk.o \
       elfload.o linuxload.o uaccess.o
 LIBS+= $(AIOLIBS)
Index: linux-user/syscall.c
===================================================================
--- linux-user/syscall.c	(revision 5566)
+++ linux-user/syscall.c	(working copy)
@@ -54,6 +54,9 @@
 #include <netinet/ip.h>
 #include <netinet/tcp.h>
 #include <qemu-common.h>
+#ifdef HAVE_GPROF
+#include <sys/gmon.h>
+#endif
 
 #define termios host_termios
 #define winsize host_winsize
@@ -4864,6 +4867,9 @@
 #ifdef __NR_exit_group
         /* new thread calls */
     case TARGET_NR_exit_group:
+#ifdef HAVE_GPROF
+        _mcleanup();
+#endif
         gdb_exit(cpu_env, arg1);
         ret = get_errno(exit_group(arg1));
         break;