[PATCH 04/19] staging: lustre: discard cfs_time_seconds()

2018-02-12 Thread NeilBrown
cfs_time_seconds() converts a number of seconds to the matching number of jiffies. The standard way to do this in Linux is "* HZ". So discard cfs_time_seconds() and use "* HZ" instead. Reviewed-by: James Simmons Signed-off-by: NeilBrown ---

Re: [PATCH 04/19] staging: lustre: discard cfs_time_seconds()

2018-01-17 Thread James Simmons
> cfs_time_seconds() converts a number of seconds to the > matching number of jiffies. > The standard way to do this in Linux is "* HZ". > So discard cfs_time_seconds() and use "* HZ" instead. This doesn't collide as bad as I thought it did. Also it builds now that the lnet seltest changes

Re: [PATCH 04/19] staging: lustre: discard cfs_time_seconds()

2018-01-09 Thread Greg Kroah-Hartman
On Mon, Jan 08, 2018 at 06:04:33PM +, James Simmons wrote: > > > On Mon, Jan 08, 2018 at 04:52:35PM +, James Simmons wrote: > > > > > > > cfs_time_seconds() converts a number of seconds to the > > > > matching number of jiffies. > > > > The standard way to do this in Linux is "* HZ". >

Re: [PATCH 04/19] staging: lustre: discard cfs_time_seconds()

2018-01-08 Thread James Simmons
> On Mon, Jan 08, 2018 at 04:52:35PM +, James Simmons wrote: > > > > > cfs_time_seconds() converts a number of seconds to the > > > matching number of jiffies. > > > The standard way to do this in Linux is "* HZ". > > > So discard cfs_time_seconds() and use "* HZ" instead. > > > > Just to

Re: [PATCH 04/19] staging: lustre: discard cfs_time_seconds()

2018-01-08 Thread Greg Kroah-Hartman
On Mon, Jan 08, 2018 at 04:52:35PM +, James Simmons wrote: > > > cfs_time_seconds() converts a number of seconds to the > > matching number of jiffies. > > The standard way to do this in Linux is "* HZ". > > So discard cfs_time_seconds() and use "* HZ" instead. > > Just to make you aware I

Re: [PATCH 04/19] staging: lustre: discard cfs_time_seconds()

2018-01-08 Thread James Simmons
> cfs_time_seconds() converts a number of seconds to the > matching number of jiffies. > The standard way to do this in Linux is "* HZ". > So discard cfs_time_seconds() and use "* HZ" instead. Just to make you aware I have been working for several months on moving lustre away from using

[PATCH 04/19] staging: lustre: discard cfs_time_seconds()

2018-01-07 Thread NeilBrown
cfs_time_seconds() converts a number of seconds to the matching number of jiffies. The standard way to do this in Linux is "* HZ". So discard cfs_time_seconds() and use "* HZ" instead. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_debug.h |4 ++--