On 2012-02-10 20:31, Luiz Capitulino wrote: > The Monitor object is passed back and forth within the migration/savevm > code so that it can print errors and progress to the user. > > However, that approach assumes a HMP monitor, being completely invalid > in QMP. > > This commit drops almost every single usage of the Monitor object, all > monitor_printf() calls have been converted into DPRINTF() ones.
Particularly NACK on this. Either the information is useless anyway, then remove it. Otherwise, keep it for channels that can properly display it (AKA HMP). I bet the latter can easily be achieved by providing non-printing Monitor objects over QMP instances. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux
