----- Original Message ----- > No, the parser code is not supposed to do semantic validation like > this. This kind of check should be done exclusively in the driver > code which forms the command line args for the hypervisors which > don't support it. Oh, I just understand why there is no hypervisor type check in domain_conf.c. I will move this check to qemuBuildCommandLine(). Thanks for your review. Best wishes, Jincheng Miao > > > Regards, > Daniel > -- > |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| > |: http://libvirt.org -o- http://virt-manager.org :| > |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| > |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| > -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list