[libvirt] [PATCH 1/4] qemu: Move checks for SMM from command-line creation into validation phase
Ján Tomko
jtomko at redhat.com
Thu Jun 7 19:31:50 UTC 2018
The commit summary speaks of moving checks, but the commit actually
relaxes them. I would not expect functional changes from that message.
How about:
qemu: relax and move SMM checks to validation phase
On Thu, Jun 07, 2018 at 10:37:40AM +0200, Martin Kletzander wrote:
>We are still hoping all of such checks will be moved there and this is one small
>step in that direction.
>
>One of the things that this is improving is the fact that instead of error
>message *that was wrong) you get when starting a domain with SMM and i440fx we
(that was wrong)
>allow the setting to go through. SMM option exists and makes sense on i440fx as
>well (basically whenever that _SMM_OPT capability is set).
>
>Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
>---
> src/qemu/qemu_capabilities.c | 11 -----------
> src/qemu/qemu_capabilities.h | 3 ---
> src/qemu/qemu_command.c | 12 ++----------
> src/qemu/qemu_domain.c | 15 ++++++++++++---
> 4 files changed, 14 insertions(+), 27 deletions(-)
>
With the commit summary adjusted or changes split in two:
Reviewed-by: Ján Tomko <jtomko at redhat.com>
Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180607/a1320b63/attachment-0001.sig>
More information about the libvir-list
mailing list