pcp
[Top] [All Lists]

Re: [pcp] pcp updates - interp.c is the biggie

To: Mark Goodwin <mgoodwin@xxxxxxxxxx>, pcp@xxxxxxxxxxx
Subject: Re: [pcp] pcp updates - interp.c is the biggie
From: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Thu, 23 Apr 2015 15:27:15 +1000
Cc: "Frank Ch. Eigler" <fche@xxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <553849A3.9060002@xxxxxxxxxx>
References: <55382F13.8080906@xxxxxxxxxxxxxxxx> <553849A3.9060002@xxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0
On 23/04/15 11:23, Mark Goodwin wrote:
Hi Ken, just trying to review these changes ...

Thanks.

...  - I guess the crux
of the issue falls on the "fixed logic errors elsewhere", as mentioned :
...

These changes are in the previous half dozen or so commits involve interp.c

I built and installed with these changes and checked qa/787 passes.
But I've lost Frank and Paul's recipe for reproducing the archive replay
performance issue - anyone have a reference to that handy?

qa/787 uses qa/archives/bug-1044 which is a pmlogextract'd version of Frank's original super-SLOW-merged-archive-20140124.225919 archive. A tarball of the original archive is on oss.sgi.com:~kenj if you need/want it (~kenj/super-slow-archive.tar.xz). With this archive unpacked:

$ time pmval -t 60 -a super-SLOW-merged-archive-20140124.225919 pmcd.numagents

should finish in seconds, not minutes!

Paul's one is trickier ... I don't have an automated test for that and I've been testing interactively also with pmlogextract'd subset of the metrics ... gimme a while to snoop the details out.

<Prev in Thread] Current Thread [Next in Thread>