Re: [RFT/PATCH v2 4/6] x86-64: vclock_gettime(CLOCK_MONOTONIC) can'tever see nsec < 0

From: Ingo Molnar
Date: Thu Apr 07 2011 - 03:57:23 EST



* Andy Lutomirski <luto@xxxxxxx> wrote:

> +
> + /* wall_time_nsec, vgetns(), and wall_to_monotonic.tv_nsec
> + * are all guaranteed to be nonnegative.
> + */

> + /* wall_time_nsec and wall_to_monotonic.tv_nsec are
> + * guaranteed to be between 0 and NSEC_PER_SEC.
> + */

Nor have you addressed stylistic review feedback i gave before ...

Really, we absolutely must insist on 110% attention to small details for
patches that modify assembly code.

Thanks,

Ingo
--
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/