My only concern:

> 
>      struct evcnt {
>              u_int64_t       ev_count;      /* how many have occurred */
		  /|\
		   |

This may be sub-optimal on many architectures. On vax, for example, it
generates a bunch of extra insn's. If this is incremented often it 
would be not-so-good.

-- Ragge