On Wed, Aug 21, 2013 at 2:52 PM, Davide Italiano <dav...@freebsd.org> wrote: > On Wed, Aug 21, 2013 at 2:47 PM, Adrian Chadd <adr...@freebsd.org> wrote: >> Author: adrian >> Date: Wed Aug 21 21:47:56 2013 >> New Revision: 254616 >> URL: http://svnweb.freebsd.org/changeset/base/254616 >> >> Log: >> Change the name of this particular event to reflect the name used in >> Linux and Intel examples. >> >> Sourced: >> >> * https://github.com/andikleen/pmu-tools/blob/master/snb-client.csv >> * http://software.intel.com/en-us/comment/1747932#comment-1747932 >> >> Note: >> >> * It's not currently in the Intel SDM; I need to chase down what's >> going on. >> >> Tested: >> >> * Sandy Bridge >> >> Modified: >> head/sys/dev/hwpmc/pmc_events.h >> >> Modified: head/sys/dev/hwpmc/pmc_events.h >> ============================================================================== >> --- head/sys/dev/hwpmc/pmc_events.h Wed Aug 21 21:30:56 2013 >> (r254615) >> +++ head/sys/dev/hwpmc/pmc_events.h Wed Aug 21 21:47:56 2013 >> (r254616) >> @@ -3304,7 +3304,7 @@ __PMC_EV_ALIAS("MEM_LOAD_UOPS_LLC_HIT_RE >> IAP_EVENT_D2H_04H) \ >> __PMC_EV_ALIAS("MEM_LOAD_UOPS_LLC_HIT_RETIRED.XSNP_NONE", \ >> IAP_EVENT_D2H_08H) \ >> -__PMC_EV_ALIAS("MEM_LOAD_UOPS_LLC_HIT_RETIRED.LLC_MISS", \ >> +__PMC_EV_ALIAS("MEM_LOAD_UOPS_MISC_RETIRED.LLC_MISS", \ >> IAP_EVENT_D4H_02H) \ >> __PMC_EV_ALIAS("L2_TRANS.DEMAND_DATA_RD", IAP_EVENT_F0H_01H) \ >> __PMC_EV_ALIAS("L2_TRANS.RFO", IAP_EVENT_F0H_02H) \ >> @@ -3538,7 +3538,7 @@ __PMC_EV_ALIAS("MEM_LOAD_UOPS_LLC_HIT_RE >> IAP_EVENT_D2H_08H) \ >> __PMC_EV_ALIAS("MEM_LOAD_UOPS_LLC_HIT_RETIRED.ALL", \ >> IAP_EVENT_D2H_0FH) \ >> -__PMC_EV_ALIAS("MEM_LOAD_UOPS_LLC_HIT_RETIRED.LLC_MISS", \ >> +__PMC_EV_ALIAS("MEM_LOAD_UOPS_MISC_RETIRED.LLC_MISS", \ >> IAP_EVENT_D4H_02H) \ >> __PMC_EV_ALIAS("MEM_LOAD_UOPS_LLC_MISS_RETIRED.LOCAL_DRAM", \ >> IAP_EVENT_D3H_01H) \ > > Can you update the manpage as well? > > -- > Davide > > "There are no solved problems; there are only problems that are more > or less solved" -- Henri Poincare
Also for r254476, as already requested by Hiren, because everyone looks at the manpage to understand what metrice the event measures before running pmcstat, so having manpages update when new events are added is almost necessary. Thanks, -- Davide "There are no solved problems; there are only problems that are more or less solved" -- Henri Poincare _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"