home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1993 #1 / NN_1993_1.iso / spool / comp / unix / bsd / 10981 < prev    next >
Encoding:
Internet Message Format  |  1993-01-06  |  6.6 KB
  1. Path: sparky!uunet!olivea!mintaka.lcs.mit.edu!ai-lab!hal.gnu.ai.mit.edu!mycroft
  2. From: mycroft@hal.gnu.ai.mit.edu (Charles Hannum)
  3. Newsgroups: comp.unix.bsd
  4. Subject: Memory leak in kvm_getprocs() and top 3.0
  5. Message-ID: <1ig4u5INNstu@life.ai.mit.edu>
  6. Date: 7 Jan 93 02:35:49 GMT
  7. Organization: MIT Artificial Intelligence Lab
  8. Lines: 253
  9. NNTP-Posting-Host: hal.gnu.ai.mit.edu
  10.  
  11.  
  12. Here are two sets of patches.  The first fixes a memory leak in
  13. kvm_getprocs(), and the second is to convent m_bsd44.c to m_386bsd.c,
  14. in the top 3.0 distribution.
  15.  
  16. Note that while I've tried to take advantage of virtual memory
  17. metering, this is not yet implemented in 386BSD.  I'm also not entirely
  18. certain why the resident size always shows up as `0'.  I'll post a
  19. patch when I decide to think about it again.
  20.  
  21. -----8<-----snip-----8<-----snip-----8<-----snip-----8<-----snip-----8<-----
  22. *** /usr/src/lib/libutil/kvm.c~    Fri Nov 27 21:16:35 1992
  23. --- /usr/src/lib/libutil/kvm.c    Wed Jan  6 20:32:34 1993
  24. ***************
  25. *** 99,105 ****
  26.   /*
  27.    * state
  28.    */
  29. ! static    struct kinfo_proc *kvmprocbase, *kvmprocptr;
  30.   static    int kvmnprocs;
  31.   /*
  32.    * u. buffer
  33. --- 99,105 ----
  34.   /*
  35.    * state
  36.    */
  37. ! static    struct kinfo_proc *kvmprocbase = NULL, *kvmprocptr;
  38.   static    int kvmnprocs;
  39.   /*
  40.    * u. buffer
  41. ***************
  42. *** 406,411 ****
  43. --- 406,413 ----
  44.               return (-1);
  45.           }
  46.           copysize = ret;
  47. +         if (kvmprocbase)
  48. +             free(kvmprocbase);
  49.           if ((kvmprocbase = (struct kinfo_proc *)malloc(copysize)) 
  50.                == NULL) {
  51.               seterr("out of memory");
  52. -----8<-----snip-----8<-----snip-----8<-----snip-----8<-----snip-----8<-----
  53.  
  54. -----8<-----snip-----8<-----snip-----8<-----snip-----8<-----snip-----8<-----
  55. *** m_bsd44.c    Sat May  9 15:09:54 1992
  56. --- m_386bsd.c    Wed Jan  6 21:28:02 1993
  57. ***************
  58. *** 2,17 ****
  59.    * top - a top users display for Unix
  60.    *
  61. !  * SYNOPSIS:  For a pre-release 4.4BSD system
  62. !  *          Note memory statistisc and process sizes could be wrong,
  63. !  *          by ps gets them wrong too...
  64.    *
  65.    * DESCRIPTION:
  66. !  * This is the machine-dependent module for BSD4.4 
  67. !  * Works for:
  68. !  *    hp300
  69.    *
  70. !  * LIBS: -lkvm
  71. !  *
  72. !  * AUTHOR:  Christos Zoulas <christos@ee.cornell.edu>
  73.    */
  74.   
  75. --- 2,13 ----
  76.    * top - a top users display for Unix
  77.    *
  78. !  * SYNOPSIS:  For 386BSD 0.1
  79.    *
  80.    * DESCRIPTION:
  81. !  * This is the machine-dependent module for 386BSD
  82.    *
  83. !  * AUTHOR:  Charles Hannum <mycroft@ai.mit.edu>
  84. !  *        based on m_bsd44.c, by
  85. !  *        Christos Zoulas <christos@ee.cornell.edu>
  86.    */
  87.   
  88. ***************
  89. *** 27,30 ****
  90. --- 23,27 ----
  91.   #include <sys/kinfo.h>
  92.   #include <sys/kinfo_proc.h>
  93. + #include <sys/vmmeter.h>
  94.   #ifdef notyet
  95.   #define time __time
  96. ***************
  97. *** 45,50 ****
  98.   #include "machine.h"
  99.   
  100. ! #define VMUNIX    "/vmunix"
  101. ! #define KMEM    "/dev/kmem"
  102.   #define MEM    "/dev/mem"
  103.   #ifdef DOSWAP
  104. --- 42,47 ----
  105.   #include "machine.h"
  106.   
  107. ! #define VMUNIX    "/386bsd"
  108. ! #define KMEM    NULL
  109.   #define MEM    "/dev/mem"
  110.   #ifdef DOSWAP
  111. ***************
  112. *** 79,82 ****
  113. --- 76,80 ----
  114.   #define X_HZ        2
  115.   #define X_AVENRUN    3
  116. + #define X_TOTAL        4
  117.   
  118.   static struct nlist nlst[] = {
  119. ***************
  120. *** 85,88 ****
  121. --- 83,87 ----
  122.       { "_hz" },            /* 2 */
  123.       { "_averunnable" },        /* 3 */
  124. +     { "_total" },        /* 4 */
  125.       { 0 }
  126.   };
  127. ***************
  128. *** 111,116 ****
  129.   
  130.   
  131. - static kvm_t *kd;
  132. - 
  133.   /* values that we stash away in _init and use in later routines */
  134.   
  135. --- 110,113 ----
  136. ***************
  137. *** 185,194 ****
  138.       register int pagesize;
  139.   
  140. !     if ((kd = kvm_open(VMUNIX, MEM, SWAP, O_RDONLY, "kvm_open")) == NULL)
  141.       return -1;
  142.   
  143. - 
  144.       /* get the list of symbols we want to access in the kernel */
  145. !     (void) kvm_nlist(kd, nlst);
  146.       if (nlst[0].n_type == 0)
  147.       {
  148. --- 182,190 ----
  149.       register int pagesize;
  150.   
  151. !     if (kvm_openfiles(VMUNIX, KMEM, SWAP) == -1)
  152.       return -1;
  153.   
  154.       /* get the list of symbols we want to access in the kernel */
  155. !     (void) kvm_nlist(nlst);
  156.       if (nlst[0].n_type == 0)
  157.       {
  158. ***************
  159. *** 302,307 ****
  160.       int size;
  161.   
  162. !     /* get total -- systemwide main memory usage structure */
  163. !     getkerninfo(KINFO_METER, &total, &size, 0);
  164.       /* convert memory stats to Kbytes */
  165.       memory_stats[0] = -1;
  166. --- 298,303 ----
  167.       int size;
  168.   
  169. !         (void) getkval(nlst[X_TOTAL].n_value, (int *)(&total), sizeof(total),
  170. !                nlst[X_TOTAL].n_name);
  171.       /* convert memory stats to Kbytes */
  172.       memory_stats[0] = -1;
  173. ***************
  174. *** 343,354 ****
  175.   
  176.       
  177. !     pbase = kvm_getprocs(kd, KINFO_PROC_ALL, 0, &nproc);
  178. !     if (nproc > onproc)
  179. !     pref = (struct kinfo_proc **) realloc(pref, sizeof(struct kinfo_proc *)
  180. !         * (onproc = nproc));
  181. !     if (pref == NULL || pbase == NULL) {
  182. !     (void) fprintf(stderr, "top: Out of memory.\n");
  183.       quit(23);
  184.       }
  185.       /* get a pointer to the states summary array */
  186.       si->procstates = process_states;
  187. --- 339,355 ----
  188.   
  189.       
  190. !     if ((nproc = kvm_getprocs(KINFO_PROC_ALL, 0)) == -1) {
  191. !         (void) fprintf(stderr, "top: kvm_getprocs() failed: %s\n", kvm_geterr());
  192. !     quit(23);
  193. !     }
  194. ! 
  195. !     if (pref)
  196. !         free(pref);
  197. ! 
  198. !     if ((pref = (struct kinfo_proc **) malloc(sizeof(struct kinfo_proc *) * nproc)) ==  NULL) {
  199. !         (void) fprintf(stderr, "top: out of memory\n");
  200.       quit(23);
  201.       }
  202. + 
  203.       /* get a pointer to the states summary array */
  204.       si->procstates = process_states;
  205. ***************
  206. *** 365,369 ****
  207.       memset((char *)process_states, 0, sizeof(process_states));
  208.       prefp = pref;
  209. !     for (pp = pbase, i = 0; i < nproc; pp++, i++)
  210.       {
  211.       /*
  212. --- 366,370 ----
  213.       memset((char *)process_states, 0, sizeof(process_states));
  214.       prefp = pref;
  215. !     for (kvm_setproc(); pp = (struct kinfo_proc *) kvm_nextproc();)
  216.       {
  217.       /*
  218. ***************
  219. *** 520,524 ****
  220.   
  221.   {
  222. !     if (kvm_read(kd, offset, (char *) ptr, size) != size)
  223.       {
  224.       if (*refstr == '!')
  225. --- 521,525 ----
  226.   
  227.   {
  228. !     if (kvm_read((void *) offset, (char *) ptr, size) != size)
  229.       {
  230.       if (*refstr == '!')
  231. ***************
  232. *** 562,575 ****
  233.   proc_compare(pp1, pp2)
  234.   
  235. ! struct kinfo_proc **pp1;
  236. ! struct kinfo_proc **pp2;
  237.   
  238.   {
  239. !     register struct kinfo_proc *p1;
  240. !     register struct kinfo_proc *p2;
  241.       register int result;
  242.       register pctcpu lresult;
  243.   
  244. -     /* remove one level of indirection */
  245.       p1 = *pp1;
  246.       p2 = *pp2;
  247. --- 563,573 ----
  248.   proc_compare(pp1, pp2)
  249.   
  250. ! struct kinfo_proc **pp1, **pp2;
  251.   
  252.   {
  253. !     register struct kinfo_proc *p1, *p2;
  254.       register int result;
  255.       register pctcpu lresult;
  256.   
  257.       p1 = *pp1;
  258.       p2 = *pp2;
  259. -----8<-----snip-----8<-----snip-----8<-----snip-----8<-----snip-----8<-----
  260. -- 
  261.  \  /   Charles Hannum, mycroft@ai.mit.edu
  262.  /\ \   PGP public key available on request.  MIME, AMS, NextMail accepted.
  263. Scheme  White heterosexual atheist male (WHAM) pride!
  264.