virtio-win / kvm-guest-drivers-windows

Windows paravirtualized drivers for QEMU\KVM
https://www.linux-kvm.org/page/WindowsGuestDrivers
BSD 3-Clause "New" or "Revised" License
1.92k stars 377 forks source link

Win 8/8.1 solution cleanup #994

Closed kostyanf14 closed 8 months ago

YanVugenfirer commented 8 months ago

fwcfg buildall.bat has Windows 8.1 (6.3) paramter in the call to inf2cat. Can be removed later.

YanVugenfirer commented 8 months ago

Another point for removal of unneeded stuff: netkvm-base.txt, netkvm-add-rsc.txt, netkvm-add-rss.txt - can be consulidated. RSS and RSC will be now always supported. netkvm-add-uso.txt - might not be supported on early Windows 10 versions

Device.c in ivsjmem has inconcistence line endings

YanVugenfirer commented 8 months ago

fwcfg buildall.bat has Windows 8.1 (6.3) paramter in the call to inf2cat. Can be removed later.

same in \pciserial and \Q35 directories.

YanVugenfirer commented 8 months ago

\viocrypt still has Win8 config, can be fixed in the next PR

YanVugenfirer commented 8 months ago

When opening main solution file: "C:\Users\yan\dev\kvm-guest-drivers-windows\NetKVM\CoInstaller\netkvmco-vs2015.vcxproj : warning : Wildcards in project items are not supported in VC projects and can cause unexpected behavior, instability and performance degradation in the Visual Studio IDE. Please refer to https://aka.ms/cpp/projectwildcards for supported options. "

YanVugenfirer commented 8 months ago

ARM64 config is failing for some of the drivers when building from the main solution:

ParaNdis_Debug.cpp 5>ParaNdis_Oid.cpp 5>ParaNdis_Protocol.cpp 4>LINK : fatal error LNK1104: cannot open file 'atls.lib' 4>Done building project "ProtocolService.vcxproj" -- FAILED.

YanVugenfirer commented 8 months ago

ARM64 config is failing for some of the drivers when building from the main solution:

ParaNdis_Debug.cpp 5>ParaNdis_Oid.cpp 5>ParaNdis_Protocol.cpp 4>LINK : fatal error LNK1104: cannot open file 'atls.lib' 4>Done building project "ProtocolService.vcxproj" -- FAILED.

This is the main issue in my opinion, other stuff can be fixed later.

YanVugenfirer commented 8 months ago

ARM64 config is failing for some of the drivers when building from the main solution:

ParaNdis_Debug.cpp 5>ParaNdis_Oid.cpp 5>ParaNdis_Protocol.cpp 4>LINK : fatal error LNK1104: cannot open file 'atls.lib' 4>Done building project "ProtocolService.vcxproj" -- FAILED.

This is the main issue in my opinion, other stuff can be fixed later.

Fixed. Spectre mitigation was not installed for VS2019

YanVugenfirer commented 8 months ago

@vrozenfe - up to you when to merge.