Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-27 Thread Oleg Nesterov
On 06/27, Chris Snook wrote: > > Oleg Nesterov wrote: > >On 06/26, Chris Snook wrote: > >>Oleg Nesterov wrote: > >>>on top of sys_time-speedup.patch > >>> > >>>Ingo Molnar wrote: > asmlinkage long sys_time(time_t __user * tloc) > { > - time_t i; > - struct timeval tv; > + /* >

Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-27 Thread Chris Snook
Oleg Nesterov wrote: On 06/26, Chris Snook wrote: Oleg Nesterov wrote: on top of sys_time-speedup.patch Ingo Molnar wrote: asmlinkage long sys_time(time_t __user * tloc) { - time_t i; - struct timeval tv; + /* +* We read xtime.tv_sec atomically - it's updated +

Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-27 Thread Oleg Nesterov
On 06/26, Chris Snook wrote: > > Oleg Nesterov wrote: > >on top of sys_time-speedup.patch > > > >Ingo Molnar wrote: > >> asmlinkage long sys_time(time_t __user * tloc) > >> { > >>- time_t i; > >>- struct timeval tv; > >>+ /* > >>+* We read xtime.tv_sec atomically - it's updated > >>+

Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-27 Thread Oleg Nesterov
On 06/26, Chris Snook wrote: Oleg Nesterov wrote: on top of sys_time-speedup.patch Ingo Molnar wrote: asmlinkage long sys_time(time_t __user * tloc) { - time_t i; - struct timeval tv; + /* +* We read xtime.tv_sec atomically - it's updated +* atomically by

Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-27 Thread Chris Snook
Oleg Nesterov wrote: On 06/26, Chris Snook wrote: Oleg Nesterov wrote: on top of sys_time-speedup.patch Ingo Molnar wrote: asmlinkage long sys_time(time_t __user * tloc) { - time_t i; - struct timeval tv; + /* +* We read xtime.tv_sec atomically - it's updated +

Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-27 Thread Oleg Nesterov
On 06/27, Chris Snook wrote: Oleg Nesterov wrote: On 06/26, Chris Snook wrote: Oleg Nesterov wrote: on top of sys_time-speedup.patch Ingo Molnar wrote: asmlinkage long sys_time(time_t __user * tloc) { - time_t i; - struct timeval tv; + /* + * We read xtime.tv_sec atomically - it's

Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-26 Thread Chris Snook
Oleg Nesterov wrote: on top of sys_time-speedup.patch Ingo Molnar wrote: asmlinkage long sys_time(time_t __user * tloc) { - time_t i; - struct timeval tv; + /* +* We read xtime.tv_sec atomically - it's updated +* atomically by update_wall_time(), so no need

[PATCH] sys_time-speedup-small-cleanup

2007-06-26 Thread Oleg Nesterov
on top of sys_time-speedup.patch Ingo Molnar wrote: > > asmlinkage long sys_time(time_t __user * tloc) > { > - time_t i; > - struct timeval tv; > + /* > + * We read xtime.tv_sec atomically - it's updated > + * atomically by update_wall_time(), so no need to > + * even

[PATCH] sys_time-speedup-small-cleanup

2007-06-26 Thread Oleg Nesterov
on top of sys_time-speedup.patch Ingo Molnar wrote: asmlinkage long sys_time(time_t __user * tloc) { - time_t i; - struct timeval tv; + /* + * We read xtime.tv_sec atomically - it's updated + * atomically by update_wall_time(), so no need to + * even

Re: [PATCH] sys_time-speedup-small-cleanup

2007-06-26 Thread Chris Snook
Oleg Nesterov wrote: on top of sys_time-speedup.patch Ingo Molnar wrote: asmlinkage long sys_time(time_t __user * tloc) { - time_t i; - struct timeval tv; + /* +* We read xtime.tv_sec atomically - it's updated +* atomically by update_wall_time(), so no need