On Thu, Jul 12, 2018 at 9:41 PM, Sebastian Huber
<sebastian.hu...@embedded-brains.de> wrote:
>
>
>
> ----- Am 6. Jul 2018 um 15:11 schrieb Hesham Almatary 
> heshamelmat...@gmail.com:
>
> > On Fri, 6 Jul 2018 at 1:28 pm, Sebastian Huber <s...@rtems.org> wrote:
> [...]
> >> -CPU_Counter_ticks _CPU_Counter_read( void );
> >> +static inline CPU_Counter_ticks _CPU_Counter_read( void )
> >> +{
> >> +  unsigned long ticks;
> >> +
> >> +  __asm__ volatile ( "rdtime %0" : "=&r" ( ticks ) );
> >> +
> >
> > Shouldn’t this be “rdcycle” instead of “rdtime”?
>
> The rdcycle is affected by power saving states. If you want to use the CPU 
> counter for fast timestamps, then this is bad.

I agree. However, if we just want to read cycles/ticks (for
performance monitoring) by calling this function, "rdtime" wouldn't
report the desired result** in tthis case.


** From the spec: "The execution environment should provide a means of
determining the period of the real-time counter (seconds/tick)"


-- 
Hesham
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to