2009-06-22 15:18:52

by Jaswinder Singh Rajput

Subject: [PATCH -tip] perf_counter util: set alias for page-faults

faults should be alias for page-faults

Also fixed alignment and 80 characters issue

Signed-off-by: Jaswinder Singh Rajput <[email protected]>
---
tools/perf/util/parse-events.c | 36 ++++++++++++++++++------------------
1 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c
index f569548..06af2fa 100644
--- a/tools/perf/util/parse-events.c
+++ b/tools/perf/util/parse-events.c
@@ -33,8 +33,7 @@ static struct event_symbol event_symbols[] = {

{ CSW(CPU_CLOCK), "cpu-clock", "" },
{ CSW(TASK_CLOCK), "task-clock", "" },
- { CSW(PAGE_FAULTS), "page-faults", "" },
- { CSW(PAGE_FAULTS), "faults", "" },
+ { CSW(PAGE_FAULTS), "page-faults", "faults" },
{ CSW(PAGE_FAULTS_MIN), "minor-faults", "" },
{ CSW(PAGE_FAULTS_MAJ), "major-faults", "" },
{ CSW(CONTEXT_SWITCHES), "context-switches", "cs" },
@@ -71,24 +70,24 @@ static char *sw_event_names[] = {

#define MAX_ALIASES 8

-static char *hw_cache [][MAX_ALIASES] = {
- { "L1-data" , "l1-d", "l1d" },
- { "L1-instruction" , "l1-i", "l1i" },
- { "L2" , "l2" },
- { "Data-TLB" , "dtlb", "d-tlb" },
- { "Instruction-TLB" , "itlb", "i-tlb" },
- { "Branch" , "bpu" , "btb", "bpc" },
+static char *hw_cache[][MAX_ALIASES] = {
+ { "L1-data", "l1-d", "l1d" },
+ { "L1-instruction", "l1-i", "l1i" },
+ { "L2", "l2" },
+ { "Data-TLB", "dtlb", "d-tlb" },
+ { "Instruction-TLB", "itlb", "i-tlb" },
+ { "Branch", "bpu" , "btb", "bpc" },
};

-static char *hw_cache_op [][MAX_ALIASES] = {
- { "Load" , "read" },
- { "Store" , "write" },
- { "Prefetch" , "speculative-read", "speculative-load" },
+static char *hw_cache_op[][MAX_ALIASES] = {
+ { "Load", "read" },
+ { "Store", "write" },
+ { "Prefetch", "speculative-read", "speculative-load" },
};

-static char *hw_cache_result [][MAX_ALIASES] = {
- { "Reference" , "ops", "access" },
- { "Miss" },
+static char *hw_cache_result[][MAX_ALIASES] = {
+ { "Reference", "ops", "access" },
+ { "Miss" },
};

char *event_name(int counter)
@@ -160,7 +159,8 @@ static int parse_aliases(const char *str, char *names[][MAX_ALIASES], int size)
return -1;
}

-static int parse_generic_hw_symbols(const char *str, struct perf_counter_attr *attr)
+static int
+parse_generic_hw_symbols(const char *str, struct perf_counter_attr *attr)
{
int cache_type = -1, cache_op = 0, cache_result = 0;

@@ -201,7 +201,7 @@ static int check_events(const char *str, unsigned int i)

if (strlen(event_symbols[i].alias))
if (!strncmp(str, event_symbols[i].alias,
- strlen(event_symbols[i].alias)))
+ strlen(event_symbols[i].alias)))
return 1;
return 0;
}
--
1.6.0.6

2009-06-22 15:28:30

by Jaswinder Singh Rajput

Subject: [tip:perfcounters/urgent] perf_counter tools: Set alias for page-faults

Commit-ID: c0c22dbfa8ba3c5045eeb9c76d2822ffc44fefc3
Gitweb: http://git.kernel.org/tip/c0c22dbfa8ba3c5045eeb9c76d2822ffc44fefc3
Author: Jaswinder Singh Rajput <[email protected]>
AuthorDate: Mon, 22 Jun 2009 20:47:26 +0530
Committer: Ingo Molnar <[email protected]>
CommitDate: Mon, 22 Jun 2009 17:25:53 +0200

perf_counter tools: Set alias for page-faults

"faults" should be alias for "page-faults"

Also fixed alignment and 80 characters issue

Signed-off-by: Jaswinder Singh Rajput <[email protected]>
Cc: Peter Zijlstra <[email protected]>
LKML-Reference: <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>


---
tools/perf/util/parse-events.c | 36 ++++++++++++++++++------------------
1 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/tools/perf/util/parse-events.c b/tools/perf/util/parse-events.c
index f569548..06af2fa 100644
--- a/tools/perf/util/parse-events.c
+++ b/tools/perf/util/parse-events.c
@@ -33,8 +33,7 @@ static struct event_symbol event_symbols[] = {

{ CSW(CPU_CLOCK), "cpu-clock", "" },
{ CSW(TASK_CLOCK), "task-clock", "" },
- { CSW(PAGE_FAULTS), "page-faults", "" },
- { CSW(PAGE_FAULTS), "faults", "" },
+ { CSW(PAGE_FAULTS), "page-faults", "faults" },
{ CSW(PAGE_FAULTS_MIN), "minor-faults", "" },
{ CSW(PAGE_FAULTS_MAJ), "major-faults", "" },
{ CSW(CONTEXT_SWITCHES), "context-switches", "cs" },
@@ -71,24 +70,24 @@ static char *sw_event_names[] = {

#define MAX_ALIASES 8

-static char *hw_cache [][MAX_ALIASES] = {
- { "L1-data" , "l1-d", "l1d" },
- { "L1-instruction" , "l1-i", "l1i" },
- { "L2" , "l2" },
- { "Data-TLB" , "dtlb", "d-tlb" },
- { "Instruction-TLB" , "itlb", "i-tlb" },
- { "Branch" , "bpu" , "btb", "bpc" },
+static char *hw_cache[][MAX_ALIASES] = {
+ { "L1-data", "l1-d", "l1d" },
+ { "L1-instruction", "l1-i", "l1i" },
+ { "L2", "l2" },
+ { "Data-TLB", "dtlb", "d-tlb" },
+ { "Instruction-TLB", "itlb", "i-tlb" },
+ { "Branch", "bpu" , "btb", "bpc" },
};

-static char *hw_cache_op [][MAX_ALIASES] = {
- { "Load" , "read" },
- { "Store" , "write" },
- { "Prefetch" , "speculative-read", "speculative-load" },
+static char *hw_cache_op[][MAX_ALIASES] = {
+ { "Load", "read" },
+ { "Store", "write" },
+ { "Prefetch", "speculative-read", "speculative-load" },
};

-static char *hw_cache_result [][MAX_ALIASES] = {
- { "Reference" , "ops", "access" },
- { "Miss" },
+static char *hw_cache_result[][MAX_ALIASES] = {
+ { "Reference", "ops", "access" },
+ { "Miss" },
};

char *event_name(int counter)
@@ -160,7 +159,8 @@ static int parse_aliases(const char *str, char *names[][MAX_ALIASES], int size)
return -1;
}

-static int parse_generic_hw_symbols(const char *str, struct perf_counter_attr *attr)
+static int
+parse_generic_hw_symbols(const char *str, struct perf_counter_attr *attr)
{
int cache_type = -1, cache_op = 0, cache_result = 0;

@@ -201,7 +201,7 @@ static int check_events(const char *str, unsigned int i)

if (strlen(event_symbols[i].alias))
if (!strncmp(str, event_symbols[i].alias,
- strlen(event_symbols[i].alias)))
+ strlen(event_symbols[i].alias)))
return 1;
return 0;
}