Re: trace/events: DECLARE vs DEFINE semantic
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: trace/events: DECLARE vs DEFINE semantic
- From: "H. Peter Anvin" <hpa@xxxxxxxxx>
- Date: Wed, 02 Dec 2009 15:08:57 -0800
- Cc: Steven Rostedt <rostedt@xxxxxxxxxxx>, Masami Hiramatsu <mhiramat@xxxxxxxxxx>, akpm@xxxxxxxxxxxxxxxxxxxx, Ingo Molnar <mingo@xxxxxxx>, mingo@xxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, randy.dunlap@xxxxxxxxxx, wcohen@xxxxxxxxxx, fweisbec@xxxxxxxxx, tglx@xxxxxxxxxxxxx, jbaron@xxxxxxxxxx, linux-tip-commits@xxxxxxxxxxxxxxx, Christoph Hellwig <hch@xxxxxx>
- In-reply-to: <20091202225736.GA14770@Krystal>
- References: <20091202144334.GA30359@xxxxxxx> <1259765735.12870.42.camel@xxxxxxxxxxxxxxxxxxx> <20091202162715.GB9710@Krystal> <1259773888.12870.61.camel@xxxxxxxxxxxxxxxxxxx> <20091202180633.GF9710@Krystal> <1259777987.12870.70.camel@xxxxxxxxxxxxxxxxxxx> <20091202190135.GA23316@Krystal> <1259781578.12870.78.camel@xxxxxxxxxxxxxxxxxxx> <4B16EC06.6010706@xxxxxxxxxx> <1259794005.12870.102.camel@xxxxxxxxxxxxxxxxxxx> <20091202225736.GA14770@Krystal>
- User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20091014 Fedora/3.0-2.8.b4.fc11 Thunderbird/3.0b4
On 12/02/2009 02:57 PM, Mathieu Desnoyers wrote:
>>
>> TRACE_CLASS - Declares a class
>> TRACE_CLASS_EVENT - defines an event for said class
>> TRACE_EVENT - Declares a class and defines an event (as is today)
>
> Yep, it looks good! It's self-descriptive and don't require to explain
> what the thing is doing each time we refer to it. (however I feel a bit
> sad for Fred, Wilma and Barney) ;)
>
Although you have to admit it's a bit confusing that:
TRACE_EVENT = TRACE_CLASS + TRACE_CLASS_EVENT
... as opposed to ...
TRACE_CLASS_EVENT = TRACE_CLASS + TRACE_EVENT
-hpa
--
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel. I don't speak on their behalf.
--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]