From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: unlisted-recipients:; (no To-header on input)
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>,
Linux Edac Mailing List <linux-edac@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: [PATCH v3 02/31] events/hw_event: use __string() trace macros for events
Date: Thu, 9 Feb 2012 22:01:01 -0200 [thread overview]
Message-ID: <1328832090-9166-3-git-send-email-mchehab@redhat.com> (raw)
In-Reply-To: <1328832090-9166-1-git-send-email-mchehab@redhat.com>
Some data there uses temporary alloced space. Just attributing
string pointers directly won't work on such cases.
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
---
include/trace/events/hw_event.h | 78 +++++++++++++++++++-------------------
1 files changed, 39 insertions(+), 39 deletions(-)
diff --git a/include/trace/events/hw_event.h b/include/trace/events/hw_event.h
index 3735c6f..078a099 100644
--- a/include/trace/events/hw_event.h
+++ b/include/trace/events/hw_event.h
@@ -20,17 +20,17 @@ DECLARE_EVENT_CLASS(hw_event_class,
TP_ARGS(type, instance),
TP_STRUCT__entry(
- __field( const char *, type )
+ __string( type, type )
__field( unsigned int, instance )
),
TP_fast_assign(
- __entry->type = type;
+ __assign_str(type, type);
__entry->instance = instance;
),
TP_printk("Initialized %s#%d\n",
- __entry->type,
+ __get_str(type),
__entry->instance)
);
@@ -70,8 +70,8 @@ TRACE_EVENT(mc_corrected_error,
__field( unsigned long, syndrome )
__field( int, row )
__field( int, channel )
- __field( const char *, label )
- __field( const char *, msg )
+ __string( label, mci->csrows[row].channels[channel].label)
+ __string( msg, msg )
),
TP_fast_assign(
@@ -82,16 +82,16 @@ TRACE_EVENT(mc_corrected_error,
__entry->syndrome = syndrome;
__entry->row = row;
__entry->channel = channel;
- __entry->label = mci->csrows[row].channels[channel].label;
- __entry->msg = msg;
+ __assign_str(label, mci->csrows[row].channels[channel].label);
+ __assign_str(msg, msg);
),
TP_printk(HW_ERR "mce#%d: Corrected error %s on label \"%s\" "
"(page 0x%lux, offset 0x%lux, grain %ud, "
"syndrome 0x%lux, row %d, channel %d)\n",
__entry->mc_index,
- __entry->msg,
- __entry->label,
+ __get_str(msg),
+ __get_str(label),
__entry->page_frame_number,
__entry->offset_in_page,
__entry->grain,
@@ -116,8 +116,8 @@ TRACE_EVENT(mc_uncorrected_error,
__field( unsigned long, offset_in_page )
__field( u32, grain )
__field( int, row )
- __field( const char *, msg )
- __field( const char *, label )
+ __string( msg, msg )
+ __string( label, label )
),
TP_fast_assign(
@@ -126,15 +126,15 @@ TRACE_EVENT(mc_uncorrected_error,
__entry->offset_in_page = offset_in_page;
__entry->grain = mci->csrows[row].grain;
__entry->row = row;
- __entry->msg = msg;
- __entry->label = label;
+ __assign_str(msg, msg);
+ __assign_str(label, label);
),
TP_printk(HW_ERR "mce#%d: Uncorrected error %s on label \"%s\""
"(page 0x%lux, offset 0x%lux, grain %ud, row %d)\n",
__entry->mc_index,
- __entry->msg,
- __entry->label,
+ __get_str(msg),
+ __get_str(label),
__entry->page_frame_number,
__entry->offset_in_page,
__entry->grain,
@@ -160,23 +160,23 @@ TRACE_EVENT(mc_corrected_error_fbd,
__field( unsigned int, mc_index )
__field( int, row )
__field( int, channel )
- __field( const char *, label )
- __field( const char *, msg )
+ __string( label, mci->csrows[row].channels[channel].label)
+ __string( msg, msg )
),
TP_fast_assign(
__entry->mc_index = mci->mc_idx;
__entry->row = row;
__entry->channel = channel;
- __entry->label = mci->csrows[row].channels[channel].label;
- __entry->msg = msg;
+ __assign_str(label, mci->csrows[row].channels[channel].label);
+ __assign_str(msg, msg);
),
TP_printk(HW_ERR "mce#%d: Corrected Error %s on label \"%s\" "
"(row %d, channel %d)\n",
__entry->mc_index,
- __entry->msg,
- __entry->label,
+ __get_str(msg),
+ __get_str(label),
__entry->row,
__entry->channel)
);
@@ -194,8 +194,8 @@ TRACE_EVENT(mc_uncorrected_error_fbd,
__field( int, row )
__field( int, channela )
__field( int, channelb )
- __field( const char *, msg )
- __field( const char *, label )
+ __string( msg, msg )
+ __string( label, label )
),
TP_fast_assign(
@@ -203,15 +203,15 @@ TRACE_EVENT(mc_uncorrected_error_fbd,
__entry->row = row;
__entry->channela = channela;
__entry->channelb = channelb;
- __entry->msg = msg;
- __entry->label = label;
+ __assign_str(msg, msg);
+ __assign_str(label, label);
),
TP_printk(HW_ERR "mce#%d: Uncorrected Error %s on label \"%s\" "
"(row %d, channels: %d, %d)\n",
__entry->mc_index,
- __entry->msg,
- __entry->label,
+ __get_str(msg),
+ __get_str(label),
__entry->row,
__entry->channela,
__entry->channelb)
@@ -236,8 +236,8 @@ TRACE_EVENT(mc_out_of_range,
TP_ARGS(mci, type, field, invalid_val, min, max),
TP_STRUCT__entry(
- __field( const char *, type )
- __field( const char *, field )
+ __string( type, type )
+ __string( field, field )
__field( unsigned int, mc_index )
__field( int, invalid_val )
__field( int, min )
@@ -245,8 +245,8 @@ TRACE_EVENT(mc_out_of_range,
),
TP_fast_assign(
- __entry->type = type;
- __entry->field = field;
+ __assign_str(type, type);
+ __assign_str(field, field);
__entry->mc_index = mci->mc_idx;
__entry->invalid_val = invalid_val;
__entry->min = min;
@@ -255,8 +255,8 @@ TRACE_EVENT(mc_out_of_range,
TP_printk(HW_ERR "mce#%d %s: %s=%d is not between %d and %d\n",
__entry->mc_index,
- __entry->type,
- __entry->field,
+ __get_str(type),
+ __get_str(field),
__entry->invalid_val,
__entry->min,
__entry->max)
@@ -274,18 +274,18 @@ TRACE_EVENT(mc_corrected_error_no_info,
TP_ARGS(mci, msg),
TP_STRUCT__entry(
- __field( const char *, msg )
+ __string( msg, msg )
__field( unsigned int, mc_index )
),
TP_fast_assign(
- __entry->msg = msg;
+ __assign_str(msg, msg);
__entry->mc_index = mci->mc_idx;
),
TP_printk(HW_ERR "mce#%d: Corrected Error: %s\n",
__entry->mc_index,
- __entry->msg)
+ __get_str(msg))
);
TRACE_EVENT(mc_uncorrected_error_no_info,
@@ -294,18 +294,18 @@ TRACE_EVENT(mc_uncorrected_error_no_info,
TP_ARGS(mci, msg),
TP_STRUCT__entry(
- __field( const char *, msg )
+ __string( msg, msg )
__field( unsigned int, mc_index )
),
TP_fast_assign(
- __entry->msg = msg;
+ __assign_str(msg, msg);
__entry->mc_index = mci->mc_idx;
),
TP_printk(HW_ERR "mce#%d: Uncorrected Error: %s\n",
__entry->mc_index,
- __entry->msg)
+ __get_str(msg))
);
--
1.7.8
next prev parent reply other threads:[~2012-02-10 0:01 UTC|newest]
Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-10 0:00 [PATCH v3 00/31] Hardware Events Report Mecanism (HERM) Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 01/31] events/hw_event: Create a " Mauro Carvalho Chehab
2012-02-10 13:41 ` Borislav Petkov
2012-02-10 14:17 ` Mauro Carvalho Chehab
2012-02-12 12:48 ` Borislav Petkov
2012-02-12 17:21 ` Mauro Carvalho Chehab
2012-02-12 18:44 ` Borislav Petkov
2012-02-12 19:38 ` Mauro Carvalho Chehab
2012-02-13 9:21 ` Borislav Petkov
2012-02-13 10:23 ` Mauro Carvalho Chehab
2012-02-10 0:01 ` Mauro Carvalho Chehab [this message]
2012-02-10 0:01 ` [PATCH v3 03/31] hw_event: Consolidate uncorrected/corrected error msgs into one Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 04/31] drivers/edac: rename channel_info to csrow_channel_info Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 05/31] edac: Create a dimm struct and move the labels into it Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 06/31] edac: Add per dimm's sysfs nodes Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 07/31] edac: Prepare to push down to drivers the filling of the dimm_info Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 08/31] edac: Better describe the memory concepts The memory terms changed along the time, since when EDAC were originally written: new concepts were introduced, and some things have different meanings, depending on the memory architecture. Better define those terms, and better describe each supported memory type Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 09/31] i5400_edac: Convert it to report memory with the new location Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 10/31] i7300_edac: " Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 11/31] edac: move dimm properties to struct dimm_info Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 12/31] edac: Don't initialize csrow's first_page & friends when not needed Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 13/31] edac: move nr_pages to dimm struct Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 14/31] edac: Add per-dimm sysfs show nodes Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 15/31] edac: DIMM location cleanup Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 16/31] edac/ppc4xx_edac: Fix compilation Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 17/31] edac-mc: Allow reporting errors on a non-csrow oriented way Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 18/31] edac.h: Use kernel-doc-nano-HOWTO.txt notation for enums Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 19/31] edac: rework memory layer hierarchy description Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 20/31] edac: Export MC hierarchy counters for CE and UE Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 21/31] hw_event: Add x86 MCE events on it Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 22/31] amd64_edac: convert it to use the MCE log tracepoint where applicable Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 23/31] edac: Simplify logs for i7core and sb edac drivers Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 24/31] edac_mc: Some clenups at the log message Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 25/31] edac: Add a sysfs node to test the EDAC error report facility Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 26/31] edac_mc: Fix the enable label filter logic Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 27/31] edac: Initialize the dimm label with the known information Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 28/31] edac: don't OOPS if the csrow is not visible Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 29/31] edac: Fix sysfs csrow?/*ce*count counters Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 30/31] edac: Fix new error counts Mauro Carvalho Chehab
2012-02-10 0:01 ` [PATCH v3 31/31] edac: Fix per layer error count counters Mauro Carvalho Chehab
2012-02-10 13:26 ` [PATCH v3 00/31] Hardware Events Report Mecanism (HERM) Borislav Petkov
2012-02-10 16:39 ` Mauro Carvalho Chehab
2012-02-12 12:08 ` Borislav Petkov
2012-02-12 17:10 ` Mauro Carvalho Chehab
2012-02-13 21:29 ` Luck, Tony
2012-02-10 16:48 ` [PATCH v3 32/31] edac: restore mce.h file Mauro Carvalho Chehab
2012-02-13 9:23 ` [PATCH v3 00/31] Hardware Events Report Mecanism (HERM) Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1328832090-9166-3-git-send-email-mchehab@redhat.com \
--to=mchehab@redhat.com \
--cc=linux-edac@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).