]> xenbits.xensource.com Git - libvirt.git/commit
qemu: Warn verbosely if using old loader:nvram pairs
authorMichal Privoznik <mprivozn@redhat.com>
Mon, 11 Nov 2019 16:22:50 +0000 (17:22 +0100)
committerMichal Privoznik <mprivozn@redhat.com>
Tue, 12 Nov 2019 07:34:37 +0000 (08:34 +0100)
commit75597f022a69bc8000ddcdea857b77fe45f0d874
tree16baf3e3355199c81ea878814d307bfa019a1f51
parente39d3424e329308d9e02b6df774f706a007ffd30
qemu: Warn verbosely if using old loader:nvram pairs

There are two ways for specifying loader:nvram pairs:

  1) --with-loader-nvram configure option
  2) nvram variable in qemu.conf

Since we have FW descriptors, using this old style is
discouraged, but not as strong as one would expect. Produce more
warnings:

  1) produce a warning if somebody tries the configure option
  2) produce a warning if somebody sets nvram variable and at
     least on FW descriptor was found

The reason for producing warning in case 1) is that package
maintainers, who set the configure option in the first place
should start moving towards FW descriptors and abandon the
configure option. After all, the warning is printed into config
output only in this case.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1763477

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
m4/virt-loader-nvram.m4
src/qemu/qemu.conf
src/qemu/qemu_conf.c