> > - eventsel & ARCH_PERFMON_EVENTSEL_INT); > > + eventsel & ARCH_PERFMON_EVENTSEL_INT, > > + !!(eventsel & HSW_INTX), > > + !!(eventsel & HSW_INTX_CHECKPOINTED)); > > } > > > > Those !! are !necessary, since the formal argument is a bool.
bool is 1 or 0, x & y with y > 1 is not 1 or 0. iirc assigning > 1 to bool is undefined So I think they are necessary. Or did I misundersand you? -Andi -- [email protected] -- Speaking for myself only -- 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/
