On Wednesday, November 27, 2013 09:22:01 PM Viresh Kumar wrote: > On 27 November 2013 19:52, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote: > > And here my question was: Is it safe to continue at all in that case? > > Hmm.. Honestly speaking I haven't thought about it earlier. And from > the kind of inputs we got from Nishanth its not safe at all and so we > really need a BUG_ON in this case, instead of WARN_ON. > > What do you say? Yeah, BUG_ON() sounds like the right thing to do here. I have a concern that on some systems you can't really say what frequency you're running at the moment, however. So there should be a flag for drivers indicating whether or not frequencies (or operation points in general) are directly testable and the check should only be done for the drivers with the flag set. Thanks! -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
