[libvirt] [PATCH 1/4] qemu: Move checks for SMM from command-line creation into validation phase
Martin Kletzander
mkletzan at redhat.com
Thu Jun 7 21:04:48 UTC 2018
On Thu, Jun 07, 2018 at 09:31:50PM +0200, Ján Tomko wrote:
>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
>
Here you suggest changing the commit message to describe both changes.
>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:
>
Here you are suggesting to split it into two. I'm fine with both, just tell me
what you meant so I can fix it up, thanks.
>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/a265484e/attachment-0001.sig>
More information about the libvir-list
mailing list