[netperf-talk] PAP_MAX_CACHE_LEVELS and psp_iticksperclktick in file netsys_hpux11i.c

[Rick Jones]

Alexander Vinokur wrote:
> ---------------------------------------------
> 
> if (ret > 0) {
> 
> #ifdef *PAP_MAX_CACHE_LEVELS*
> 
>...
> 
> #else
> 
>     /* older OSes were "known" to be on CPUs where the itick was
>        1to1 here */
> 
>     *cpu_frequency = rint(((double)processor_info.*psp_iticksperclktick* * 
> 
>                           (double)sysconf(_SC_CLK_TCK)) / 1000000.0);
> 
> #endif 
> 
>   }
> 
> -------------------------
> Two notes-questions:
> 
> 1.    Did one mean *PSP_MAX_CACHE_LEVELS? *Not *PAP_MAX_CACHE_LEVELS?*

Yes.  I will fix that in the top-of-trunk.

> *   *2. Use of* psp_iticksperclktick *to get cpu speed on Itanium
>       produces invalid value; See:
> 
>       http://groups.google.com/group/comp.sys.hp.hpux/browse_frm/thread/e027d6a404a5019f

True - note though the comment in the #else clause above.  Only on Itanium-based 
systems is it the case that the interval counter (iticks) and the processor 
frequency are not 1:1.  The first OS rev to support Itanium is, for all incense 
and porpoises, 11.23 (I don't  concern myself with 11.20 or 11.22).  IIRC 11.23 
includes PSP_MAX_CACHE_LEVELS.  So, after your fix for PAP to PSP, anything 
compiled on 11.23 or later will get the right thing by using the updated pstat 
info, and anything compiled on HP-UX 11.11 will get the right thing by running 
only on a system where the iticks tick at the same frequency as the CPU.

At least, that was the idea :)

happy benchmarking,

rick jones