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 -- To unsubscribe from this list: send the line "unsubscribe trinity" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
