Messages in this thread | | | From | Naoya Horiguchi <> | Subject | Re: [PATCH v6 5/5] trace, ras: move ras_event.h under include/trace/events | Date | Thu, 21 May 2015 23:00:24 +0000 |
| |
On Thu, May 21, 2015 at 09:01:52AM -0400, Steven Rostedt wrote: > On Thu, 21 May 2015 11:24:37 +0200 > Borislav Petkov <bp@suse.de> wrote: > > > On Thu, May 21, 2015 at 11:41:25AM +0800, Xie XiuQi wrote: > > > From: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com> > > > > > > Most of header files for tracepoints are located to include/trace/events or > > > their relevant subdirectories under drivers/. One exception is > > > > That's simply not true. > > > > > include/ras/ras_events.h, which looks inconsistent. So let's move it to the > > > default places for such headers. > > > > No thanks - ras TPs can live just fine in include/ras/. > > > > I agree with Boris, the solution is not to move it. It's not > inconsistent, lots of places use it. Just do a git grep -l TRACE_EVENT > to see.
My bad, I withdraw this patch.
Thanks, Naoya Horiguchi
| |