pcp
[Top] [All Lists]

pcp updates: fche merge, pmdaproc, qa

To: PCP Mailing List <pcp@xxxxxxxxxxx>
Subject: pcp updates: fche merge, pmdaproc, qa
From: Nathan Scott <nathans@xxxxxxxxxx>
Date: Fri, 20 Sep 2013 06:22:15 -0400 (EDT)
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <1284163470.25952473.1379672524477.JavaMail.root@xxxxxxxxxx>
Reply-to: Nathan Scott <nathans@xxxxxxxxxx>
Thread-index: 8gypYSg0v/p9BsekYYwFAzwDMXbg8g==
Thread-topic: pcp updates: fche merge, pmdaproc, qa
Changes committed to git://oss.sgi.com/pcp/pcp.git dev

 qa/.gitignore                   |    1 
 qa/022.linux.4                  |    2 
 qa/723                          |   49 +++++++++
 qa/723.out                      |    2 
 qa/735                          |   52 +++++++++
 qa/735.out                      |    2 
 qa/943                          |    8 +
 qa/943.out                      |  171 --------------------------------
 qa/943.out.1                    |  171 ++++++++++++++++++++++++++++++++
 qa/943.out.2                    |  175 +++++++++++++++++++++++++++++++++
 qa/group                        |    2 
 src/include/pcp/pmapi.h         |    2 
 src/libpcp/src/check-statics    |    1 
 src/libpcp/src/context.c        |   74 ++++++--------
 src/pmcollectl/pmcollectl.py    |   19 +--
 src/pmdas/linux_proc/clusters.h |    2 
 src/pmdas/linux_proc/help       |    4 
 src/pmdas/linux_proc/indom.h    |   10 +
 src/pmdas/linux_proc/pmda.c     |   74 +++++++++++++-
 src/pmdas/linux_proc/proc_pid.c |  209 +++++++++++++++++++++++++++++-----------
 src/pmdas/linux_proc/proc_pid.h |   62 ++++++++---
 src/pmdas/linux_proc/root_proc  |    4 
 22 files changed, 791 insertions(+), 305 deletions(-)

commit e00b973eb5be0e29fed5ad47c0400ef216bf3207
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Fri Sep 20 20:20:41 2013 +1000

    Reduce memory footprint of per-process metrics
    
    Collapse all (per-process) "fetched" state variables
    into a single integer, accessed via bitfield flags.

commit dd1d60a6034363acd1e7939158877350330f08d2
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Fri Sep 20 20:16:06 2013 +1000

    Update test 943 to cope with presence of additional proc metrics

commit ecc1f3a2936ffb1279148f2f619dcd1fd3fbbaff
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Fri Sep 20 16:10:11 2013 +1000

    Add proc.psinfo metrics for cgroups and security labels
    
    Extract Linux per-process metrics from procfs for each running
    process.  Observed that many, many processes have the same value
    for these (string) metrics, so optimise the memory used so that
    only a single copy of each different value is ever held in-core.
    We achieve this through (ab)use of the pmdaCache API for indoms,
    using a pseudo-indom to hold a string dictionary, and associate
    the (inst) identifier with each process rather than the string.
    
    The string format for each of these metrics is made to match the
    format that ps(1) reports - for security labels there is no need
    to tweak the kernel value, for cgroups some string manipulation
    is performed (via converting the multi-line string to a one-line
    semi-colon separated string without the cgroup-hierarchy-number).
    
    Tests 723 & 724 are added to exercise the extraction, comparing
    the pmdaproc values to those obtained from reading directly the
    procfs files.

commit 24b51166354ba6e9f39ced1466fc169f190fccec
Merge: 143a420 5ecb31c
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Thu Sep 19 13:32:04 2013 +1000

    Merge branch 'fche/dev' of ../pcpfans into dev

commit 5ecb31ccab273925edca9fcf18f11d4c08964bea
Author: Frank Ch. Eigler <fche@xxxxxxxxxx>
Date:   Wed Sep 18 12:07:47 2013 -0400

    pmcollectl: host name improvements
    
    - document the -h HOST option in --help usage()
    - default to local: rather than localhost
    - when creating archive folios (-f FILE option), use the
      pmGetContextHostName result consistently

commit 0eac9192a7514caf27c8cab162397af6df9379c7
Author: Frank Ch. Eigler <fche@xxxxxxxxxx>
Date:   Tue Sep 17 19:45:51 2013 -0400

    update check-statics to bless the new pmGetContextHostName static

commit 52979b09a55a08efb3ab681bd99d0a99ba91f1f9
Author: Frank Ch. Eigler <fche@xxxxxxxxxx>
Date:   Tue Sep 17 19:40:09 2013 -0400

    pmapi pmGetContextHostName: return to static char[] return value
    
    It seems somewhat better to have a controlled & limited race condition
    (over the contents of a string buffer), than a likely memory leak.

<Prev in Thread] Current Thread [Next in Thread>
  • pcp updates: fche merge, pmdaproc, qa, Nathan Scott <=