[PATCH] Prepend timestamp in msgbuf

[Garrett Cooper]

On Mon, Oct 17, 2011 at 5:18 PM, Arnaud Lacombe <lacombar at gmail.com> wrote:
> Hi,
>
> On Mon, Oct 17, 2011 at 6:22 PM, Ed Schouten <ed at 80386.nl> wrote:
>> Ah, missed something.
>>
>>> +             getnanouptime(&ts);
>>> +             err = snprintf(buf, sizeof buf, "[%zd.%.6ld] ",
>>> +                 ts.tv_sec, ts.tv_nsec / 1000);
>>
>> It seems we also have a getmicrouptime(), which returns a struct
>> timeval.
> fixed.
>
>> Also a more general question: is it actually safe to call
>> getnanouptime() here? This code gets executed from an arbitrary context,
>> right?
>>
> right, but getmicrouptime() is not doing much magic. Just reading a
> cached value, do an arithmetic conversion. I do not really see any
> unsafe part.

Based on glancing around other areas of the kernel, I'd assume that
using this KPI as-is is fine because I don't see any locking employed
elsewhere...
-Garrett