On 20.06.2017 14:33, Paolo Bonzini wrote: > > > On 20/06/2017 11:10, Stefan Raspl wrote: >> Plus if we would prevent this command from doing anything when >> tracepoints are in use, then we should probably give the user some >> kind of feedback on why nothing changed upon his keypress - but I >> wouldn't know a place in kvm_stats GUI to do that. So if we leave as >> is, at least the user gets some immediate feedback and hopefully >> figure what's going on, and that he might want to rather restart >> with debugfs only. > > What about writing a message in the last line? kvm_stat fills the entire screen till the last line. We'd have to make any message stick out, probably by erasing the line before and after, too - and then make sure that the message remains visible for at least 3 seconds, i.e. re-apply the change after refreshing the body till 3 sec expired (since users could set the refresh interval to e.g. 0.5 sec via 's', whereby it would be really hard to spot the message within the 0.5 sec that it is visible on screen). So that would require some kind of time measuring logic. Which also would need to consider add'l key presses before those 3 secs expire. In short: We're talking quite a bit of more messy code for little benefit. Then again: How about simply drawing a separate screen a la the help (see 'h') with whatever error message we need, and which is exited upon a key press? In a generic way, of course, so we could re-use for future likewise cases. Ciao, Stefan
