<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 21 April 2015 at 03:14, Thomas Gleixner <span dir="ltr"><<a href="mailto:tglx@linutronix.de" target="_blank">tglx@linutronix.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Mon, 20 Apr 2015, Baolin Wang wrote:<br>
> This patch introduces the 'struct itimerspec64' for 64bit to replace itimerspec,<br>
> and also introduces the conversion methods: itimerspec64_to_itimerspec() and<br>
> itimerspec_to_itimerspec64(), that makes itimerspec to ready for 2038 year.<br>
><br>
> Signed-off-by: Baolin Wang <<a href="mailto:baolin.wang@linaro.org">baolin.wang@linaro.org</a>><br>
> ---<br>
>  include/linux/time64.h |   13 +++++++++++++<br>
>  1 file changed, 13 insertions(+)<br>
><br>
> diff --git a/include/linux/time64.h b/include/linux/time64.h<br>
> index a383147..3647bdd 100644<br>
> --- a/include/linux/time64.h<br>
> +++ b/include/linux/time64.h<br>
> @@ -18,6 +18,11 @@ struct timespec64 {<br>
>  };<br>
>  #endif<br>
><br>
> +struct itimerspec64 {<br>
> +     struct timespec64 it_interval;  /* timer period */<br>
> +     struct timespec64 it_value;     /* timer expiration */<br>
> +};<br>
> +<br>
>  /* Parameters used to convert the timespec values: */<br>
>  #define MSEC_PER_SEC 1000L<br>
>  #define USEC_PER_MSEC        1000L<br>
> @@ -187,4 +192,12 @@ static __always_inline void timespec64_add_ns(struct timespec64 *a, u64 ns)<br>
><br>
>  #endif<br>
><br>
> +#define itimerspec64_to_itimerspec(its64) \<br>
<br>
</div></div>> +#define itimerspec_to_itimerspec64(its) \<br>
<br>
1.) Make these static inlines please. These macros are not typesafe.<br>
<br>
2.) Use pointers to the input value.<br>
<br>
Thanks.<br>
<br>
        tglx<br>
</blockquote></div><br><br clear="all"><div>Thanks for your comments, i'll fix in next patch.</div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Baolin.wang<br></div>Best Regards<br></div></div>
</div></div>