Re: [PATCH v3 9/9] oxenstored: reduce syslog call overhead (fix typo)

[Zheng Li <dev@zheng.li>]

On 26/09/2014 17:29, Zheng Li wrote:
> On 26/09/2014 16:57, Ian Jackson wrote:
>> Zheng Li writes ("Re: [PATCH v3 9/9] oxenstored: reduce syslog call overhead (fix typo)"):
>>> Please see the 3rd format (:characters format) in the linked document below
>>> http://www.gnu.org/software/libc/manual/html_node/TZ-Variable.html
>>
>> That's great for glibc-based guests.  But what about the BSDs ?
>
> This code does xenstored access logging in Dom0. Looking at the supported Dom0 distros page on the wiki, its seems that only NetBSD is the concern here?
>
>> I looked at
>>    http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap08.html#tag_08
>> and it just says that the : syntax is `implementation defined'.
>
> I guess most BSDs adopt the same semantics. I searched for NetBSD libc manual and found this (starting from "if the value does not begin with a colon ...")

Sorry, should be "if the value begins with a colon ...". From what I read, even if it doesn't start with a colon, it will still be treated as a time conversion file as the first resort.

Also, the syslog implementation on BSD might be different (e.g. might not have the "stat" issue at all). So as far as setting the TZ variable won't cause any problem for the xenstored process, we should be fine with it.

Cheers,
Zheng