Re: [PATCH] Re: boot time, process start time, and NOW time

[OGAWA Hirofumi]

George Anzinger <george@xxxxxxxxxx> writes:

> > The cause of this is
> >      INITIAL_JIFFIES % HZ (4294667296 % 1000)
> > because INITIAL_JIFFIES is unsigned long.
> > So, I guessed this is not intention.
> > Looks like this should be (-300*1000) % 1000.
> 
> What "should be"?

in time_init(), and hpet_time_init(),
        xtime.tv_nsec = (INITIAL_JIFFIES % HZ) * (NSEC_PER_SEC / HZ);
should be
        xtime.tv_nsec = ((long)INITIAL_JIFFIES % HZ) * (NSEC_PER_SEC / HZ);

because
	(INITIAL_JIFFIES % HZ) * (NSEC_PER_SEC / HZ)		== 296000000
and
	((long)INITIAL_JIFFIES % HZ) * (NSEC_PER_SEC / HZ)	== 0
-- 
OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/