Re: [patch] perf_event_open more likely tracepoint values

On Thu, 14 Nov 2013, Dave Jones wrote:

> On Thu, Nov 14, 2013 at 12:35:27PM -0500, Vince Weaver wrote:
>  
>  > Tracepoint ids tend to be relatively small (less than 1024 or so).
>  > This change makes it more likely to hit valid ones.
>  > 
>  > Once the fix to avoid the troublesome 0x18 (irq_work_exit)
>  > tracepoint gets into the kernel maybe we can start stressing
>  > things with trinity again.
>  
> applied, though fyi, this (and some other boolean choices) could
> be shorter with just
> 
> 	if (rand_bool())
> 		foo
> 	else
> 		bar
> 
> guess it depends if you foresee adding further case's though.

I wasn't sure of rand_bool() was only meant for setting boolean flag 
variables or if it should be also used as the generic "choose one of two 
options" selector.  

I'll start using it instead of rand()%2 in future patches.

Vince