hotspot/agent/src/os/linux/ps_proc.c
author dsamersoff
Fri, 13 Jun 2014 05:10:44 -0700
changeset 25062 d38c0d15b3f7
parent 20295 a5dd1b071c32
child 28924 8943e95a2fb3
permissions -rw-r--r--
8038392: Generating prelink cache breaks JAVA 'jinfo' utility normal behaviour Summary: Better parsing of /proc/pid/maps in sa Reviewed-by: sspitsyn, sla
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
15734
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
     2
 * Copyright (c) 2003, 2013, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 670
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 670
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 670
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
489c9b5090e2 Initial load
duke
parents:
diff changeset
    25
#include <stdio.h>
489c9b5090e2 Initial load
duke
parents:
diff changeset
    26
#include <stdlib.h>
489c9b5090e2 Initial load
duke
parents:
diff changeset
    27
#include <string.h>
15734
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
    28
#include <signal.h>
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    29
#include <errno.h>
20295
a5dd1b071c32 8025613: clang: remove -Wno-unused-value
twisti
parents: 15734
diff changeset
    30
#include <sys/types.h>
a5dd1b071c32 8025613: clang: remove -Wno-unused-value
twisti
parents: 15734
diff changeset
    31
#include <sys/wait.h>
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    32
#include <sys/ptrace.h>
489c9b5090e2 Initial load
duke
parents:
diff changeset
    33
#include "libproc_impl.h"
489c9b5090e2 Initial load
duke
parents:
diff changeset
    34
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
#if defined(x86_64) && !defined(amd64)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
#define amd64 1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
#ifndef __WALL
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
#define __WALL          0x40000000  // Copied from /usr/include/linux/wait.h
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
// This file has the libproc implementation specific to live process
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
// For core files, refer to ps_core.c
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
static inline uintptr_t align(uintptr_t ptr, size_t size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
  return (ptr & ~(size - 1));
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
// ---------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
// ptrace functions
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
// ---------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
// read "size" bytes of data from "addr" within the target process.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    55
// unlike the standard ptrace() function, process_read_data() can handle
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
// unaligned address - alignment check, if required, should be done
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
// before calling process_read_data.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
static bool process_read_data(struct ps_prochandle* ph, uintptr_t addr, char *buf, size_t size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
  long rslt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  size_t i, words;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
  uintptr_t end_addr = addr + size;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
  uintptr_t aligned_addr = align(addr, sizeof(long));
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
  if (aligned_addr != addr) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
    char *ptr = (char *)&rslt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
    errno = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
    rslt = ptrace(PTRACE_PEEKDATA, ph->pid, aligned_addr, 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
    if (errno) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
      print_debug("ptrace(PTRACE_PEEKDATA, ..) failed for %d bytes @ %lx\n", size, addr);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
    for (; aligned_addr != addr; aligned_addr++, ptr++);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
    for (; ((intptr_t)aligned_addr % sizeof(long)) && aligned_addr < end_addr;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
        aligned_addr++)
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
       *(buf++) = *(ptr++);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
  words = (end_addr - aligned_addr) / sizeof(long);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  // assert((intptr_t)aligned_addr % sizeof(long) == 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  for (i = 0; i < words; i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
    errno = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
    rslt = ptrace(PTRACE_PEEKDATA, ph->pid, aligned_addr, 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
    if (errno) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
      print_debug("ptrace(PTRACE_PEEKDATA, ..) failed for %d bytes @ %lx\n", size, addr);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
    *(long *)buf = rslt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
    buf += sizeof(long);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
    aligned_addr += sizeof(long);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
  if (aligned_addr != end_addr) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
    char *ptr = (char *)&rslt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
    errno = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
    rslt = ptrace(PTRACE_PEEKDATA, ph->pid, aligned_addr, 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
    if (errno) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
      print_debug("ptrace(PTRACE_PEEKDATA, ..) failed for %d bytes @ %lx\n", size, addr);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
    for (; aligned_addr != end_addr; aligned_addr++)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
       *(buf++) = *(ptr++);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
// null implementation for write
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
static bool process_write_data(struct ps_prochandle* ph,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
                             uintptr_t addr, const char *buf , size_t size) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
// "user" should be a pointer to a user_regs_struct
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
static bool process_get_lwp_regs(struct ps_prochandle* ph, pid_t pid, struct user_regs_struct *user) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
  // we have already attached to all thread 'pid's, just use ptrace call
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  // to get regset now. Note that we don't cache regset upfront for processes.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
// Linux on x86 and sparc are different.  On x86 ptrace(PTRACE_GETREGS, ...)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
// uses pointer from 4th argument and ignores 3rd argument.  On sparc it uses
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
// pointer from 3rd argument and ignores 4th argument
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
#if defined(sparc) || defined(sparcv9)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
#define ptrace_getregs(request, pid, addr, data) ptrace(request, pid, addr, data)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
#define ptrace_getregs(request, pid, addr, data) ptrace(request, pid, data, addr)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
7415
e7c6833aac14 7003789: PTRACE_GETREGS problems with SA on Linux.
kevinw
parents: 6176
diff changeset
   127
#if defined(_LP64) && defined(PTRACE_GETREGS64)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
#define PTRACE_GETREGS_REQ PTRACE_GETREGS64
7415
e7c6833aac14 7003789: PTRACE_GETREGS problems with SA on Linux.
kevinw
parents: 6176
diff changeset
   129
#elif defined(PTRACE_GETREGS)
e7c6833aac14 7003789: PTRACE_GETREGS problems with SA on Linux.
kevinw
parents: 6176
diff changeset
   130
#define PTRACE_GETREGS_REQ PTRACE_GETREGS
e7c6833aac14 7003789: PTRACE_GETREGS problems with SA on Linux.
kevinw
parents: 6176
diff changeset
   131
#elif defined(PT_GETREGS)
e7c6833aac14 7003789: PTRACE_GETREGS problems with SA on Linux.
kevinw
parents: 6176
diff changeset
   132
#define PTRACE_GETREGS_REQ PT_GETREGS
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
#ifdef PTRACE_GETREGS_REQ
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
 if (ptrace_getregs(PTRACE_GETREGS_REQ, pid, user, NULL) < 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
   print_debug("ptrace(PTRACE_GETREGS, ...) failed for lwp %d\n", pid);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
   return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
 }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
 return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
#else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
 print_debug("ptrace(PTRACE_GETREGS, ...) not supported\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
 return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
15734
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   148
static bool ptrace_continue(pid_t pid, int signal) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   149
  // pass the signal to the process so we don't swallow it
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   150
  if (ptrace(PTRACE_CONT, pid, NULL, signal) < 0) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   151
    print_debug("ptrace(PTRACE_CONT, ..) failed for %d\n", pid);
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   152
    return false;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   153
  }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   154
  return true;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   155
}
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   156
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   157
// waits until the ATTACH has stopped the process
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   158
// by signal SIGSTOP
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   159
static bool ptrace_waitpid(pid_t pid) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   160
  int ret;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   161
  int status;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   162
  while (true) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   163
    // Wait for debuggee to stop.
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   164
    ret = waitpid(pid, &status, 0);
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   165
    if (ret == -1 && errno == ECHILD) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   166
      // try cloned process.
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   167
      ret = waitpid(pid, &status, __WALL);
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   168
    }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   169
    if (ret >= 0) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   170
      if (WIFSTOPPED(status)) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   171
        // Any signal will stop the thread, make sure it is SIGSTOP. Otherwise SIGSTOP
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   172
        // will still be pending and delivered when the process is DETACHED and the process
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   173
        // will go to sleep.
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   174
        if (WSTOPSIG(status) == SIGSTOP) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   175
          // Debuggee stopped by SIGSTOP.
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   176
          return true;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   177
        }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   178
        if (!ptrace_continue(pid, WSTOPSIG(status))) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   179
          print_error("Failed to correctly attach to VM. VM might HANG! [PTRACE_CONT failed, stopped by %d]\n", WSTOPSIG(status));
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   180
          return false;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   181
        }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   182
      } else {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   183
        print_debug("waitpid(): Child process exited/terminated (status = 0x%x)\n", status);
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   184
        return false;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   185
      }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   186
    } else {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   187
      switch (errno) {
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   188
        case EINTR:
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   189
          continue;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   190
          break;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   191
        case ECHILD:
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   192
          print_debug("waitpid() failed. Child process pid (%d) does not exist \n", pid);
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   193
          break;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   194
        case EINVAL:
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   195
          print_debug("waitpid() failed. Invalid options argument.\n");
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   196
          break;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   197
        default:
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   198
          print_debug("waitpid() failed. Unexpected error %d\n",errno);
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   199
          break;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   200
      }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   201
      return false;
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   202
    }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   203
  }
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   204
}
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   205
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
// attach to a process/thread specified by "pid"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
static bool ptrace_attach(pid_t pid) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
  if (ptrace(PTRACE_ATTACH, pid, NULL, NULL) < 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
    print_debug("ptrace(PTRACE_ATTACH, ..) failed for %d\n", pid);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
  } else {
15734
68f78eb98356 8008088: SA can hang the VM
rbackman
parents: 7662
diff changeset
   212
    return ptrace_waitpid(pid);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
// -------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
// functions for obtaining library information
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
// -------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
/*
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
 * splits a string _str_ into substrings with delimiter _delim_ by replacing old * delimiters with _new_delim_ (ideally, '\0'). the address of each substring
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
 * is stored in array _ptrs_ as the return value. the maximum capacity of _ptrs_ * array is specified by parameter _n_.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
 * RETURN VALUE: total number of substrings (always <= _n_)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
 * NOTE: string _str_ is modified if _delim_!=_new_delim_
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
static int split_n_str(char * str, int n, char ** ptrs, char delim, char new_delim)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
   int i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
   for(i = 0; i < n; i++) ptrs[i] = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
   if (str == NULL || n < 1 ) return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
   i = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
   // skipping leading blanks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
   while(*str&&*str==delim) str++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
   while(*str&&i<n){
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
     ptrs[i++] = str;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
     while(*str&&*str!=delim) str++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
     while(*str&&*str==delim) *(str++) = new_delim;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
   }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
   return i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
/*
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
 * fgets without storing '\n' at the end of the string
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
static char * fgets_no_cr(char * buf, int n, FILE *fp)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
   char * rslt = fgets(buf, n, fp);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
   if (rslt && buf && *buf){
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
       char *p = strchr(buf, '\0');
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
       if (*--p=='\n') *p='\0';
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
   }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
   return rslt;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
// callback for read_thread_info
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
static bool add_new_thread(struct ps_prochandle* ph, pthread_t pthread_id, lwpid_t lwp_id) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
  return add_thread_info(ph, pthread_id, lwp_id) != NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
static bool read_lib_info(struct ps_prochandle* ph) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
  char fname[32];
25062
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   266
  char buf[PATH_MAX];
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  FILE *fp = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  sprintf(fname, "/proc/%d/maps", ph->pid);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
  fp = fopen(fname, "r");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  if (fp == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
    print_debug("can't open /proc/%d/maps file\n", ph->pid);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
25062
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   276
  while(fgets_no_cr(buf, PATH_MAX, fp)){
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   277
    char * word[7];
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   278
    int nwords = split_n_str(buf, 7, word, ' ', '\0');
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   279
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   280
    if (nwords < 6) {
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   281
      // not a shared library entry. ignore.
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   282
      continue;
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   283
    }
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   284
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   285
    // SA does not handle the lines with patterns:
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   286
    //   "[stack]", "[heap]", "[vdso]", "[vsyscall]", etc.
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   287
    if (word[5][0] == '[') {
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   288
        // not a shared library entry. ignore.
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   289
        continue;
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   290
    }
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   291
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   292
    if (nwords > 6) {
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   293
      // prelink altered mapfile when the program is running.
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   294
      // Entries like one below have to be skipped
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   295
      //  /lib64/libc-2.15.so (deleted)
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   296
      // SO name in entries like one below have to be stripped.
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   297
      //  /lib64/libpthread-2.15.so.#prelink#.EECVts
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   298
      char *s = strstr(word[5],".#prelink#");
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   299
      if (s == NULL) {
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   300
        // No prelink keyword. skip deleted library
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   301
        print_debug("skip shared object %s deleted by prelink\n", word[5]);
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   302
        continue;
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   303
      }
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   304
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   305
      // Fall through
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   306
      print_debug("rectifying shared object name %s changed by prelink\n", word[5]);
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   307
      *s = 0;
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   308
    }
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   309
d38c0d15b3f7 8038392: Generating prelink cache breaks JAVA &#39;jinfo&#39; utility normal behaviour
dsamersoff
parents: 20295
diff changeset
   310
    if (find_lib(ph, word[5]) == false) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
       intptr_t base;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
       lib_info* lib;
6176
4d9030fe341f 6953477: Increase portability and flexibility of building Hotspot
bobv
parents: 5547
diff changeset
   313
#ifdef _LP64
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
       sscanf(word[0], "%lx", &base);
6176
4d9030fe341f 6953477: Increase portability and flexibility of building Hotspot
bobv
parents: 5547
diff changeset
   315
#else
4d9030fe341f 6953477: Increase portability and flexibility of building Hotspot
bobv
parents: 5547
diff changeset
   316
       sscanf(word[0], "%x", &base);
4d9030fe341f 6953477: Increase portability and flexibility of building Hotspot
bobv
parents: 5547
diff changeset
   317
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
       if ((lib = add_lib_info(ph, word[5], (uintptr_t)base)) == NULL)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
          continue; // ignore, add_lib_info prints error
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
       // we don't need to keep the library open, symtab is already
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
       // built. Only for core dump we need to keep the fd open.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
       close(lib->fd);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
       lib->fd = -1;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
  fclose(fp);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
// detach a given pid
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
static bool ptrace_detach(pid_t pid) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
  if (pid && ptrace(PTRACE_DETACH, pid, NULL, NULL) < 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
    print_debug("ptrace(PTRACE_DETACH, ..) failed for %d\n", pid);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
    return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
// detach all pids of a ps_prochandle
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
static void detach_all_pids(struct ps_prochandle* ph) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
  thread_info* thr = ph->threads;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
  while (thr) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
     ptrace_detach(thr->lwp_id);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
     thr = thr->next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
static void process_cleanup(struct ps_prochandle* ph) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
  detach_all_pids(ph);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
static ps_prochandle_ops process_ops = {
223
5c3b023117d9 6452081: 3/4 Allow for Linux builds with Sun Studio Linux compilers
dcubed
parents: 1
diff changeset
   355
  .release=  process_cleanup,
5c3b023117d9 6452081: 3/4 Allow for Linux builds with Sun Studio Linux compilers
dcubed
parents: 1
diff changeset
   356
  .p_pread=  process_read_data,
5c3b023117d9 6452081: 3/4 Allow for Linux builds with Sun Studio Linux compilers
dcubed
parents: 1
diff changeset
   357
  .p_pwrite= process_write_data,
5c3b023117d9 6452081: 3/4 Allow for Linux builds with Sun Studio Linux compilers
dcubed
parents: 1
diff changeset
   358
  .get_lwp_regs= process_get_lwp_regs
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
// attach to the process. One and only one exposed stuff
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
struct ps_prochandle* Pgrab(pid_t pid) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
  struct ps_prochandle* ph = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
  thread_info* thr = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
  if ( (ph = (struct ps_prochandle*) calloc(1, sizeof(struct ps_prochandle))) == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
     print_debug("can't allocate memory for ps_prochandle\n");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
     return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
  if (ptrace_attach(pid) != true) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
     free(ph);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
     return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
  // initialize ps_prochandle
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
  ph->pid = pid;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
  // initialize vtable
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
  ph->ops = &process_ops;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
  // read library info and symbol tables, must do this before attaching threads,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
  // as the symbols in the pthread library will be used to figure out
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
  // the list of threads within the same process.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
  read_lib_info(ph);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
  // read thread info
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
  read_thread_info(ph, add_new_thread);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
  // attach to the threads
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
  thr = ph->threads;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
  while (thr) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
     // don't attach to the main thread again
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
     if (ph->pid != thr->lwp_id && ptrace_attach(thr->lwp_id) != true) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
        // even if one attach fails, we get return NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
        Prelease(ph);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
        return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
     }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
     thr = thr->next;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
  return ph;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
}