Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1320393

Re: [PATCH] cputime: Fix timeval-->cputime conversion

From Thomas Gleixner <tglx@linutronix.de>
Newsgroups linux.kernel
Subject Re: [PATCH] cputime: Fix timeval-->cputime conversion
Date 2016-01-28 09:30 +0100
Message-ID <qVQ7N-FX-25@gated-at.bofh.it> (permalink)
References <qVOSl-8jL-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Cc'ing Arnd

On Thu, 28 Jan 2016, zengtao wrote:

> The structure:
> struct timeval {
> 	__kernel_time_t		tv_sec;		/* seconds */
> 	__kernel_suseconds_t	tv_usec;	/* microseconds */
> };
> both __kernel_time_t and __kernel_suseconds_t are short than u64
> when it is 32bit platform, so force u64 conversion here.
> 
> Signed-off-by: zengtao <prime.zeng@huawei.com>

Reviewed-by: Thomas Gleixner <tglx@linutronix.de>

> ---
>  include/asm-generic/cputime_nsecs.h | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/include/asm-generic/cputime_nsecs.h b/include/asm-generic/cputime_nsecs.h
> index 0419485..e2f7ff9 100644
> --- a/include/asm-generic/cputime_nsecs.h
> +++ b/include/asm-generic/cputime_nsecs.h
> @@ -91,7 +91,8 @@ static inline void cputime_to_timespec(const cputime_t ct, struct timespec *val)
>   */
>  static inline cputime_t timeval_to_cputime(const struct timeval *val)
>  {
> -	u64 ret = val->tv_sec * NSEC_PER_SEC + val->tv_usec * NSEC_PER_USEC;
> +	u64 ret = (u64)val->tv_sec * NSEC_PER_SEC +
> +			val->tv_usec * NSEC_PER_USEC;
>  	return (__force cputime_t) ret;
>  }
>  static inline void cputime_to_timeval(const cputime_t ct, struct timeval *val)
> -- 
> 1.9.1
> 
> 

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] cputime: Fix timeval-->cputime conversion zengtao <prime.zeng@huawei.com> - 2016-01-28 08:10 +0100
  Re: [PATCH] cputime: Fix timeval-->cputime conversion Thomas Gleixner <tglx@linutronix.de> - 2016-01-28 09:30 +0100
    Re: [PATCH] cputime: Fix timeval-->cputime conversion Arnd Bergmann <arnd@arndb.de> - 2016-01-28 13:00 +0100
      RE: [PATCH] cputime: Fix timeval-->cputime conversion "Zengtao (B)" <prime.zeng@hisilicon.com> - 2016-01-29 04:20 +0100
        Re: [PATCH] cputime: Fix timeval-->cputime conversion Arnd Bergmann <arnd@arndb.de> - 2016-01-29 09:50 +0100
          RE: [PATCH] cputime: Fix timeval-->cputime conversion "Zengtao (B)" <prime.zeng@hisilicon.com> - 2016-01-30 03:40 +0100
          RE: [PATCH] cputime: Fix timeval-->cputime conversion "Zengtao (B)" <prime.zeng@hisilicon.com> - 2016-02-01 05:00 +0100
            RE: [PATCH] cputime: Fix timeval-->cputime conversion Thomas Gleixner <tglx@linutronix.de> - 2016-02-01 09:50 +0100
              RE: [PATCH] cputime: Fix timeval-->cputime conversion "Zengtao (B)" <prime.zeng@hisilicon.com> - 2016-02-02 04:00 +0100

csiph-web