Re: [PATCH v22] edac, ras/hw_event.h: use events to handle hw issues

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Em 10-05-2012 19:46, Steven Rostedt escreveu:
> On Thu, 2012-05-10 at 17:55 -0300, Mauro Carvalho Chehab wrote:
> 
>>>>  EXPORT_SYMBOL_GPL(edac_mc_handle_error);
>>>> diff --git a/include/ras/hw_event.h b/include/ras/hw_event.h
>>>> new file mode 100644
>>>> index 0000000..66981ef
>>>> --- /dev/null
>>>> +++ b/include/ras/hw_event.h
>>>> @@ -0,0 +1,77 @@
>>>> +#undef TRACE_SYSTEM
>>>> +#define TRACE_SYSTEM hw_event
>>>
>>> 			ras
>>
>> Then, the header name should be called as "ras.h", otherwise an error happens:
>>
>> In file included from include/ras/hw_event.h:77:0,
>>                  from drivers/edac/edac_mc.c:38:
>> include/trace/define_trace.h:79:43: fatal error: ../../include/ras/ras.h: Arquivo ou diretório não encontrado
> 
> From samples/trace_events/trace-events-sample.h:
> 
> /*
>  * TRACE_INCLUDE_FILE is not needed if the filename and TRACE_SYSTEM are equal
>  */
> #define TRACE_INCLUDE_FILE trace-events-sample
> 
> Thus, you could do:
> 
> +++ b/include/ras/hw_event.h
> 
> +undef TRACE_SYSTEM
> +define TRACE_SYSTEM ras
> 
> [...]
> 
> #define TRACE_INCLUDE_FILE hw_event
> 
> 
> -- Steve

Thanks, Steve!

I'll rename it to include/ras/ras_event.h

Regards,
Mauro