<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 12 May 2015 at 17:39, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Monday 11 May 2015 19:08:38 Baolin Wang wrote:<br>
> This patch series changes the 32-bit time type (timespec/itimerspec) to the 64-bit one<br>
> (timespec64/itimerspec64), since 32-bit time types will break in the year 2038.<br>
><br>
> This patch series introduces new methods with timespec64/itimerspec64 type,<br>
> and removes the old ones with timespec/itimerspec type for posix_clock_operations<br>
> and k_clock structure.<br>
><br>
> Also introduces some new functions with timespec64/itimerspec64 type, like current_kernel_time64(),<br>
> hrtimer_get_res64(), cputime_to_timespec64() and timespec64_to_cputime().<br>
><br>
> Changes since v2:<br>
>       -Split the syscall conversion patch into small some patches.<br>
><br>
><br>
> Baolin Wang (22):<br>
>   linux/time64.h:Introduce the 'struct itimerspec64' for 64bit<br>
>   timekeeping:Introduce the current_kernel_time64() function with<br>
>     timespec64 type<br>
>   time/hrtimer:Introduce hrtimer_get_res64() with timespec64 type for<br>
>     getting the timer resolution<br>
>   posix-timers:Split out the guts of the syscall and change the<br>
>     implementation for timer_gettime<br>
>   posix-timers:Convert to the 64bit methods for the timer_gettime<br>
>     syscall function<br>
<br>
</span>I have two more very general comments about the series:<br>
<br>
a) something has gone wrong with your submission in v2 and v3 but was<br>
   working earlier: normally all emails should be sent by git-send-email<br>
   as replies to the [patch 00/22] mail. This is the default, and it<br>
   is enabled by the '--thread --no-chain-reply' options. Please try<br>
   to get this to work again.<br>
<br>
b) it would be better to have a little shorter subject lines, to avoid<br>
   line-wrapping in the list above. Here are some examples what you could<br>
   use to replace the lines above:<br>
<br>
   timekeeping: introduce struct itimerspec64<br>
   timekeeping: introduce current_kernel_time64()<br>
   hrtimer: introduce hrtimer_get_res64()<br>
   posix-timers: split up sys_timer_gettime()<br>
   posix-timers: convert timer_gettime() to timespec64<br>
<br>
   In general, try to come up with the shortest description that<br>
   uniquely describes what your patch does, and move any details into<br>
   the longer patch description.<br>
<span class="HOEnZb"><font color="#888888"><br>
        Arnd<br>
</font></span></blockquote></div><div class="gmail_extra"><br></div>OK, i'll fix these in next patch series.Thanks for your comments.<br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Baolin.wang<br></div>Best Regards<br></div></div>
</div></div>