Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/23917

to look at the new patch set (#2).

Change subject: PROTOTYPE: libpfm demo
......................................................................

PROTOTYPE: libpfm demo

This transplants / transforms some code from Google Benchmark
to use libpfm to collect hardware performance counters for the
current process. As a demo, this collects a counter for some
backend tests and prints it to the INFO log at the end.

This requires that the user have permissions to collect performance
counters. On Ubuntu, this requires /proc/sys/kernel/perf_event_paranoid
to be <= 2 (default is usually 4).

These counters could be useful for identifying cache misses and
TLB misses.

TODO:
 - Add comments explaining that the code is adapted from Google Benchmark
   and what actually changed from the original code
 - Try to get hardware performance counters for an individual query,
   convey back to the coordinator, and aggregate into the profile. This
   may not be limited to the threads used by the query, so it is most
   exact when doing single-query-at-a-time testing.
 - Maybe a system-wide capability would be useful for concurrency testing.

Change-Id: I75172f69fba2b87bf97696583344830b6005158f
---
M CMakeLists.txt
M be/CMakeLists.txt
M be/src/service/unified-betest-main.cc
M be/src/util/CMakeLists.txt
A be/src/util/perf-counters.cc
A be/src/util/perf-counters.h
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
A cmake_modules/FindLibPfm.cmake
9 files changed, 521 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/17/23917/2
--
To view, visit http://gerrit.cloudera.org:8080/23917
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I75172f69fba2b87bf97696583344830b6005158f
Gerrit-Change-Number: 23917
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>

Reply via email to