Re: [PATCH] trace: Make trace_hwlat timestamp y2038 safe
From: Steven Rostedt
Date: Mon Mar 27 2017 - 12:12:04 EST
On Mon, 27 Mar 2017 17:35:13 +0200
Arnd Bergmann <arnd@xxxxxxxx> wrote:
> On Mon, Mar 27, 2017 at 5:30 PM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> > On Mon, 27 Mar 2017 16:53:09 +0200
> >> We could probably introduce a %pts format string for timespec64
> >> and have that pretty-printed.
> >
> > Hmm, probably don't want a %p as that suggests its a pointer, which it
> > should not be. Unless we pass in the address of the number.
>
> The special format strings that the kernel defines all start with %p and
> require passing by reference so we don't get a warning from gcc. We can't
> just make up new format strings otherwise, but we can create new meaning
> for special pointers as we do for struct resource and others.
>
That's fine, but we need to be careful when it comes to tracing.
Passing in the address of a structure in the ring buffer may be fine,
but we need to make sure that an address pointing to something other
than the ring buffer is forbidden.
I'll need to update libtraceevent to handle such cases too.
-- Steve