I have pushed some changes to master branch which fixes the issue of -subtract. Also one more thing to remember is -subtract only works when you provide the node name, for example:
./ptlstats -collect ooocore/total -snapshot post -subtract pre stat_file.stats - Avadh On Thu, Feb 17, 2011 at 7:01 PM, Curtis Dunham <[email protected]>wrote: > On Fri, Feb 18, 2011 at 01:41:09AM +0000, Mu-Tien wrote: > > Timothy Hayes <hayesti <at> tcd.ie> writes: > > > > > > Hi, > > > > > > Thank you, that patch appears to fix the problem although it has > > > uncovered another one. I can't seem to get the -subtract argument to > > > work for ptlstats. > > > > > > -- > > > ./ptlstats -snapshot post -subtract pre ./kernel.stats > isolated.txt > > > ./ptlstats -snapshot post ./kernel.stats > post.txt > > > ./ptlstats -snapshot pre ./kernel.stats > pre.txt > > > -- > > > > > > All these files should theoretically be different but isolated.txt and > > > post.txt are exactly the same. > > > > > > Am I calling the tool wrong or is this a bug? > > > > > > Kind regards > > > Tim > > > > > > On 14 February 2011 18:12, avadh patel <avadh4all <at> gmail.com> > wrote: > > > > Hi, > > > > I fond the issue and its because the stats file was reopened every > time > > > > configuration is changed !! > > > > I have attached a small patch that should fix this issue. I don't > have a > > > > setup that call's ptlcall_capture_stats() function so can you please > test > > > > this patch and confirm that this fixes the issue. > > > > Thanks, > > > > Avadh > > > > > > > > I am also encountering the same problem when using the ptlstats > "-subtract" > > option. I add "-snapshot-cycles N" argument to simconfig when running > > simulation. I use > > ./ptlstast -snapshot final -subtract 0 test.stats > test.txt > > to get stats with simulation warm up, but the output (test.txt) isn't > > different from stats without "-subtract 0". What would be the proper way > > to get stats with simulation warm up? Thank you very much. > > > All, > I was similarly annoyed with this in PTLsim. > > > http://www.ptlsim.org/git/?p=ptlsim-stable.git;a=blobdiff;f=ptlstats.cpp;h=39767467c213b769177cb49b568c24608314eb23;hp=137d9711d6ef06802e4c853216d362867f8c6227;hb=011d62987afd28c972805e26caea671586aec579;hpb=4addb2ec347bb8fa586995645a0a3944b3a1fdd3 > > > Curtis > > _______________________________________________ > http://www.marss86.org > Marss86-Devel mailing list > [email protected] > https://www.cs.binghamton.edu/mailman/listinfo/marss86-devel >
_______________________________________________ http://www.marss86.org Marss86-Devel mailing list [email protected] https://www.cs.binghamton.edu/mailman/listinfo/marss86-devel
