On Thu, Mar 02, 2023 at 04:59:36PM +0100, Michal Prívozník wrote: > On 2/15/23 11:42, Andrea Bolognani wrote: > > +static int > > +qemuDomainDefBootPostParse(virDomainDef *def, > > + virQEMUDriverConfig *cfg) > > +{ > > + if (def->os.bootloader || def->os.bootloaderArgs) { > > + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", > > + _("bootloader is not supported by QEMU")); > > + return -1; > > + } > > Strictly speaking, this is a validate check. Haven't look into the > future, but feel free to move it there in a follow up patch. Right. I'll look into it. In the meantime, I've pushed the series. Thanks a ton for the review! -- Andrea Bolognani / Red Hat / Virtualization