Re: [PATCH 1/2] MN10300: Move asm-arm/cnt32_to_63.h to include/linux/

From: Nicolas Pitre
Date: Fri Oct 03 2008 - 15:15:46 EST


On Thu, 2 Oct 2008, David Howells wrote:

> Nicolas Pitre <nico@xxxxxxx> wrote:
>
> > Oh, that's a possibility. In that case __m_cnt_hi will be reverted to a
> > previous state just like if cnt32_to_63() has not been called yet since
> > the new half period. And a subsequent call will fix that again.
>
> Surely that's not good enough. There's a 50% chance that reversion to a
> previous state will result in an extra increment of the __m_cnt_hi on the next
> call.

If so then you're using this interface in an inappropriate way.

The _absolute_ minimum frequency with which this should be fully
executed is once per half period of the base counter. I hope that in
practice it happens far more often than that.


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