[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <y0m3ba2jw4w.fsf@ton.toronto.redhat.com>
Date: 05 Oct 2006 17:22:23 -0400
From: fche@...hat.com (Frank Ch. Eigler)
To: Mike Mason <mmlnx@...ibm.com>
Cc: jrs@...ibm.com, Irfan Habib <irfan.habib@...il.com>,
Linux kernel <linux-kernel@...r.kernel.org>,
SystemTAP <systemtap@...rces.redhat.com>
Subject: Re: Fwd: Any way to find the network usage by a process?
Mike Mason <mmlnx@...ibm.com> writes:
> Here's a variation of Jose's script that uses the networking tapset
> and prints top-like output for transmits and receives. [...]
Thanks for posting it to the systemtap wiki.
Some minor style suggestions follow:
> [...]
> ifxmit_p[pid(), dev_name] ++
> ifxmit_b[pid(), dev_name] += length
These could be collapsed into a single statistics-aggregate array:
# ifxmit[pid(), dev_name] <<< length
Then the printing routine would use @count(ifxmit[...]) and @sum(ifxmit[...])
to extract the two values. Same of course for ifrecv.
> execname[pid()] = execname()
> user[pid()] = uid()
> ifdevs[pid(), dev_name] = dev_name
Calling pid() so many times is worse than calling it once and caching
the result in a local variable ("p = pid()").
The way that the script tracks pid-to-uid and pid-to-execname mappings
is not bad, though if that part were moved to new probes on fork or
exec, it would allow the network-related probes to run concurrently on
an SMP without fighting over locks.
- FChE
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists