Timur Tabi wrote: > Geoff Thorpe wrote: > >> So from this user's perspective (FWIW), it would come as a surprise if >> the return value reflected the evaluated expression rather than what >> happened w.r.t. the spin/timeout. > > It shouldn't come as a surprise because I've thoroughly documented the > behavior. I also think returning the actual value of the expression is > better than a return code. Remember, the primary purpose of this macro is to > wait for a hardware register to change. Contrast this to wait_event_xxx, > which usually queries a variable. Therefore, the hardware register may set > multiple bits. For instance, you could do this: > > ret = spin_event_timeout(in_be32(x) & 0x14, ...); > > if (ret & 0x10) > do something here > > if (ret & 0x04) > do something else here > > I think the ability to do this is more important than making the code as > similar as possible to wait_event_xxx.
I was just providing feedback after (recently) coming to grips with the subtleties of similar assists in the kernel. Documentation helps when you don't know what to expect - but if you're expecting a certain behaviour (consistency of success==0), it's easy to misread the doc, or just not read it at all. BTW, your example above does not illustrate the use of the timeout, and you'd have to infer it because the return value does not provide any direct indication. As for access to the evaluated expression, I believe the following is functionally equivalent; rc = spin_event_timeout((ret = in_be32(x) & 0x14), ...); You can ignore the return value if you don't care, which gives the same thing as your original example, but with the 'ret' assignment inside the spin call instead of outside it. Anyway, just my $0.02 FWIW. I have no strongly-held opinion here, especially as the closest precedent I know of is wait_event_timeout(), which also does something unexpected, but it doesn't do the same unexpected thing as your function does. :-) It's based on jiffies rather than microseconds, and it returns zero if the timeout elapses otherwise it returns the number of remaining jiffies when the condition evaluated true. Cheers, Geoff _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
