Hi, > > + if (!qxl->migration_blocker) { > > + Error *local_err = NULL; > > + error_setg(&qxl->migration_blocker, > > + "qxl: guest bug: command not in ram bar"); > > + migrate_add_blocker(qxl->migration_blocker, > > &local_err); > > > > What do you do with the local_err? error_report_err() perhaps ?
We can't error out at that point, unlike most migration blockers this isn't added at device initialization time. So, yes, we could error_report it, but the message would end up in the logfile unnoticed, so I'm not sure how useful that is ... cheers, Gerd