On Wed, 16 Jan 2008, Mathieu Desnoyers wrote: > > > - cycle_t offset = (now - cs->cycle_last) & cs->mask; > > + /* First update the monotonic base portion. > > + * The dual array update method allows for lock-free reading. > > + */ > > + int num = !cs->base_num; > > + cycle_t offset = (now - cs->base[!num].cycle_base_last); > > !0 is not necessarily 1. This is why I use cpu_synth->index ? 0 : 1 in
How about simply "cpu_synth->index ^ 1"? Seems the best choice if you ask me, if all you are doing is changing it from 1 to zero and back to 1. -- Steve > my code. The two previous lines seems buggy. (I did the same mistake in > my first implementation) ;) > > > + offset &= cs->mask; > > + cs->base[num].cycle_base = cs->base[!num].cycle_base + offset; > > Here too. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
