On Wed, 2006-16-08 at 12:33 +1000, Herbert Xu wrote:
> On Tue, Aug 15, 2006 at 09:58:19PM -0400, jamal wrote:

> > +static inline unsigned long make_jiffies(long secs)
> > +{
> > +   if (secs >= (MAX_SCHEDULE_TIMEOUT-1)/HZ)
> > +           return MAX_SCHEDULE_TIMEOUT-1;
> > +   else
> > +           return secs*HZ;
> > +}
> 
> This name is probably too generic for net/xfrm.h. 

agreed. 

>  How about moving
> it to linux/jiffies.h and rewrite in the style of msec_to_jiffies?
> 

Is there something other than the boundary check already done you
foresee being made? If yes, do you wanna take a crack at it?

cheers,
jamal

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to