From dd0ee2ac23662c9234fd6873d8239c20f644c2f4 Mon Sep 17 00:00:00 2001
From: yoheimuta <yoheimuta@gmail.com>
Date: Sun, 19 Jun 2016 12:55:09 +0900
Subject: [PATCH 1/2] Added an include-dead-children flag

---
 linux.go | 20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/linux.go b/linux.go
index d477fa7..f2d70f9 100644
--- a/linux.go
+++ b/linux.go
@@ -52,8 +52,12 @@ var cpuTick float64
 var metricKey string
 
 // CPUUsage retrieve cpu usage
-func (p *Process) CPUUsage() float64 {
-	total := p.UTime + p.STime + p.CUTime + p.CSTime
+func (p *Process) CPUUsage(includeDeadChildren bool) float64 {
+	total := p.UTime + p.STime
+	if includeDeadChildren {
+		total += p.CUTime + p.CSTime
+	}
+
 	sec := uptime - (p.StartTime / cpuTick)
 	return 100 * ((total / cpuTick) / sec)
 }
@@ -64,6 +68,7 @@ type LinuxProcStatsPlugin struct {
 	Pid                  string
 	Pids                 []string
 	FollowChildProcesses bool
+	IncludeDeadChildren  bool
 }
 
 // FetchMetrics interface for mackerelplugin
@@ -96,7 +101,7 @@ func (lp LinuxProcStatsPlugin) FetchMetrics() (stats map[string]interface{}, err
 		}
 	}
 
-	running, numThreads, vSize, rss, cpuUsage := sumStats(ps)
+	running, numThreads, vSize, rss, cpuUsage := sumStats(ps, lp.IncludeDeadChildren)
 	stats = make(map[string]interface{})
 	stats["running"] = running
 	stats["processes"] = float64(len(ps))
@@ -272,7 +277,10 @@ func childStats(ps Processes, options []string) (Processes, error) {
 	return ps, err
 }
 
-func sumStats(ps Processes) (running, numThreads, vSize, rss, cpuUsage float64) {
+func sumStats(
+	ps Processes,
+	includeDeadChildren bool,
+) (running, numThreads, vSize, rss, cpuUsage float64) {
 	for _, p := range ps {
 		if p.State == `R` {
 			running++
@@ -280,7 +288,7 @@ func sumStats(ps Processes) (running, numThreads, vSize, rss, cpuUsage float64)
 		numThreads += p.NumThreads
 		vSize += p.VSize
 		rss += p.RSS
-		cpuUsage += p.CPUUsage()
+		cpuUsage += p.CPUUsage(includeDeadChildren)
 	}
 
 	return running, numThreads, vSize, rss, cpuUsage
@@ -381,6 +389,7 @@ func main() {
 	optFollowChildProcesses := flag.Bool("follow-child-processes", false, "Follow child processes")
 	optMetricKey := flag.String("metric-key-prefix", "", "Metric key prefix")
 	optVersion := flag.Bool("version", false, "Version")
+	optIncludeDeadChildren := flag.Bool("include-dead-children", true, "Include dead kids in cpu sum")
 	flag.Parse()
 
 	if *optVersion {
@@ -414,6 +423,7 @@ func main() {
 	procStats.Pid = pid
 	procStats.Pids = pids
 	procStats.FollowChildProcesses = *optFollowChildProcesses
+	procStats.IncludeDeadChildren = *optIncludeDeadChildren
 
 	uptime, err = getUptime()
 	if err != nil {

From 50cba71a5a89d9d08e5e10e1e674ce073ebf5bfb Mon Sep 17 00:00:00 2001
From: yoheimuta <yoheimuta@gmail.com>
Date: Sun, 19 Jun 2016 13:52:45 +0900
Subject: [PATCH 2/2] Updated README

---
 README.md | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 3afb209..94e6e68 100644
--- a/README.md
+++ b/README.md
@@ -4,7 +4,7 @@ Linux processes metrics plugin for mackerel.io agent.
 ## Synopsis
 
 ```shell
-mackerel-plugin-linux-proc-stats -pid=<pid>|-pidfile=<pidfile>|-process-pattern=<process pattern> [-metric-key-prefix=<metric-key-prefix>] [-tempfile=<tempfile>] [-follow-child-processes]
+mackerel-plugin-linux-proc-stats -pid=<pid>|-pidfile=<pidfile>|-process-pattern=<process pattern> [-metric-key-prefix=<metric-key-prefix>] [-tempfile=<tempfile>] [-follow-child-processes] [-include-dead-children=0]
 ```
 
 ```shell
@@ -12,6 +12,8 @@ $ ./mackerel-plugin-linux-proc-stats --help
 Usage of ./mackerel-plugin-linux-proc-stats:
   -follow-child-processes
         Follow child processes
+  -include-dead-children
+        Include dead kids in cpu sum (default true)
   -metric-key-prefix string
         Metric key prefix
   -pid string