Re: [PATCH 2/2] qemu: move qemuDomainDefValidateVideo into qemuDomainDeviceDefValidateVideo

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 12/18/2017 10:35 AM, Laine Stump wrote:
> 
> qemuDomainDefValidateVideo() is just a loop performing various checks
> on each video device. Rather than maintaining this outlyer function

*outlying

> called from qemuDomainDefValidateVideo(), just fold the validations
> into qemuDomainDeviceDefValidateVideo(), which is called once for each
> video device (my guess is that ...DeviceDefValidateVideo() didn't
> exist yet when ...DomainDefValidateVideo() was added, but I haven't

Everything between "(my guess is..." and "...verified this)." could have
been below the ---

Anyway, the simple answer is qemuDomainDefValidateVideo was introduced
in 2.4 as commit id 133fb140 and qemuDomainDeviceDefValidateVideo was
introduced in 3.10 as commit id ab948b629. Thankfully gitk makes this
determination really easy!  To take it one step further, I think you'll
recognize the committer of the original qemuDomainDeviceDefValidate
implementation in v2.0 as commit id d987f63a.

> verified this).
> ---
> 
> I randomly noticed this when looking up something else in the validation code...
> 
> 
>  src/qemu/qemu_domain.c | 151 +++++++++++++++++++++----------------------------
>  1 file changed, 66 insertions(+), 85 deletions(-)
> 

Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux