Re: [PATCH kvm-unit-tests] x86/hyperv: pass hv_vpindex cpu flag along with hv_synic

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

 



Roman Kagan <rkagan@xxxxxxxxxxxxx> writes:

> On Fri, Nov 09, 2018 at 02:59:49PM +0100, Vitaly Kuznetsov wrote:
>> Since Qemu commit 9b4cf107b09d ("hyperv: only add SynIC in compatible
>>  configurations") hv_vpindex flag is mandatory if hv_synic is used:
>> 
>> $ qemu-system-x86_64 -nodefaults -cpu kvm64,hv_time,hv_synic,hv_stimer  \
>>      -device hyperv-testdev -device pc-testdev -device isa-debug-exit,  \
>>      iobase=0xf4,iosize=0x4 -vnc none -serial stdio -device pci-testdev \
>>      -machine accel=kvm -kernel x86/hyperv_stimer.flat
>> 
>> Results in:
>> 
>> Hyper-V SynIC (requested by 'hv-synic' cpu flag) requires Hyper-V VP_INDEX
>>  ('hv-vpindex')
>> qemu-system-x86_64: kvm_init_vcpu failed: Function not implemented
>> 
>> Fix the configuration.
>> 
>> Signed-off-by: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx>
>> ---
>>  x86/unittests.cfg | 6 +++---
>>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> I wrote a nearly identical one back in May:
>
> https://patchwork.kernel.org/patch/10439445/
>

Sorry to hear that :-( I checked the mailing list archives before
posting this patch but I have to admin I only checked last couple
months.

> within a small cleanup series starting with
> 20180530165257.18292-1-rkagan@xxxxxxxxxxxxx
>
> It has attracted no attention since then :(
>
> Maybe you'll have better luck, so
>
> Reviewed-by: Roman Kagan <rkagan@xxxxxxxxxxxxx>

Thanks!

-- 
Vitaly



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux