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

RE: [PATCH] Fix compile on EWDK 22000


  • To: "paul@xxxxxxx" <paul@xxxxxxx>, "win-pv-devel@xxxxxxxxxxxxxxxxxxxx" <win-pv-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Owen Smith <owen.smith@xxxxxxxxxx>
  • Date: Fri, 6 Aug 2021 08:16:33 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=JRE95ehyHLQk8y7sIUE51gKY2bQTP9Owi7aqUrhVcSM=; b=CmmR91QyVxkS8l2PdMtkZJPmvc8N9f4ZhWwMe2dvNql9YMFqg6YUkNkLQ6KBOd2LcHx/aRbWP9HU3mYasNyP9lI7Sm/tYzo++buMIV9mNODSssxIDmhOxvFXZ3OKXqp1Dqd/beXs/56QlJ56LjmTbH59BqwVkZDXKK1uA9Q8M6g7asNwwissA/0pskg7R/vRDU7FX5jee1tnKimX7CB5y01Gv2wAoBBq0PiQaMbrjnPhgRx2Z1OiiZeyisJx934Yzwp1NuXcY0aqpbHHP6su7rWhJOw12lqLfCSf1gmD820EAdmbO2kuSoK2ii420wZ06MHxqQlEmaIZggCkiCplsA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K6ZZhMqPmyv+eHj8cckh9VhC6M2BkmGQKjFGmcLN4hhWEkaT4AB5Um5DGy1fIBzWDqNiSIxTnbry3hZpXWTvj9KKNLn2LMkw/PSAy2NfEmBK4jjlw/OZGbKBxzx9pTAUxti357jTP31CHIKXjE2jyHmXNSZ8Rdd2ennmxBC2oN7fNnu3b0Nl7ZtmsrViqu+nhJ3FOgd0RFiWmFBwepqPoMMH3u/MPZB+eow1xctCu1iz3gFl8L+yVpTPLU8FSFO00GmKN3aeIbaCk5I/hD+amA+/KCRyWxolEOpEy5BtWsIg4R7STpqJ/DlfLsdqENLa7Dfdn1iIwi5BooP91j3iQA==
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Delivery-date: Fri, 06 Aug 2021 08:16:43 +0000
  • Ironport-hdrordr: A9a23:BQzXvqwt3EcVkUCxVJ9+KrPw1r1zdoMgy1knxilNoHxuH/BwWf rPoB17726RtN91YhsdcL+7V5VoLUmzyXcX2/h1AV7BZniEhILAFugLgbcKqweKJ8SUzJ8+6U 4PSclD4N2bNykGsS75ijPIb+rJFrO8gd+VbeS19QYScelzAZsQiDuQkmygYzZLrA8tP+teKL OsovBpihCHYnotYsGyFhA+LpL+T42iruOeXfYebSRXkDWzsQ==
  • Ironport-sdr: D0QguzowzSlBtL88rlfqLovXmSm/F9ww0PTym3plIYQVbSmv9bhMC3hJE3VcHDv8WRcCZwkNUW YZKpDytYjLkDoxZUxgPm7FDU4HVt36q+Lv58m8A/MlvfawujoTk1QXVwLV+gHGb71FlnUz8AR+ h80pRNtyfB0cJV4Zp2iKvQUZxTDuI/7CSYWMcbM7mQBIYQPcS56IxE/8joRd8wEK8OSG93LEwK S1qlotZ2EXrhEj4iubtFekNnzJBpvm3UY5sU9+JOHXNPSfoYHGQ3qQnQ1cyXItF7oNR4tCDbO+ TlumW6zUDfP/J3pjo9JaOCKL
  • List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>
  • Thread-index: AQHXiHVJcxi3OnJWtEapR2lSY4u7W6tlOSAAgADrLfA=
  • Thread-topic: [PATCH] Fix compile on EWDK 22000

-----Original Message-----
From: win-pv-devel <win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx> On Behalf Of 
Paul Durrant
Sent: Thursday, August 5, 2021 7:09 PM
To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
Subject: Re: [PATCH] Fix compile on EWDK 22000

[CAUTION - EXTERNAL EMAIL] DO NOT reply, click links, or open attachments 
unless you have verified the sender and know the content is safe.

On 03/08/2021 15:38, Owen Smith wrote:
> - Adds warnings 4061 and 4062 to ignore list
> - Defines signtool file digest as SHA256
> - Adds alias for GetProjectInfoForReference target
>    It appears that VS2019/EWDK22000 changed a target name, and without the 
> alias
>    the build will fail.
> 
> Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>

Do the other drivers also suffer from this problem?

   Paul


All the drivers suffer the same problem, and I'm now looking at getting 
SDV/CodeQL/CodeAnalysis logs correct for the Static Tools Logo test in the 
Server 2022 HLK, which looks like more work is needed. From what I can tell, 
the DVL.XML logs should have an entry for CodeAnalysis and Semmle (CodeQL) 
indicating these tools have been run (I'm not sure yet if the logs need to be 
clean). So far, it looks like SDV with 20344 and 22000 cannot cope with 
RunCodeAnalysis=true and EnablePREFast=true at least during the "sdv /check" 
step. (Downside is testing this is very slow, as I must complete a SDV analysis 
cycle which can take several hours to complete)

Owen

> ---
>   vs2019/package/package.vcxproj             | 5 +++++
>   vs2019/version/version.vcxproj             | 6 ++++++
>   vs2019/xennet/xennet.vcxproj               | 7 ++++++-
>   vs2019/xennet_coinst/xennet_coinst.vcxproj | 5 +++++
>   4 files changed, 22 insertions(+), 1 deletion(-)
> 
> diff --git a/vs2019/package/package.vcxproj 
> b/vs2019/package/package.vcxproj index dd794b5..e4c6f0f 100644
> --- a/vs2019/package/package.vcxproj
> +++ b/vs2019/package/package.vcxproj
> @@ -56,6 +56,11 @@
>     <ItemGroup>
>       <PackageFiles Include="$(OutDir)\$(ProjectName)\*" />
>     </ItemGroup>
> +  <ItemDefinitionGroup>
> +    <DriverSign>
> +      <FileDigestAlgorithm>sha256</FileDigestAlgorithm>
> +    </DriverSign>
> +  </ItemDefinitionGroup>
>     <Target Name="Archive" AfterTargets="TestSign">
>       <Copy
>          SourceFiles="@(PackageFiles)"
> diff --git a/vs2019/version/version.vcxproj 
> b/vs2019/version/version.vcxproj index 9d149d0..b6ec6f3 100644
> --- a/vs2019/version/version.vcxproj
> +++ b/vs2019/version/version.vcxproj
> @@ -13,4 +13,10 @@
>     <Target Name="Build">
>       <Exec Command="powershell.exe -ExecutionPolicy Bypass -NoProfile 
> -NonInteractive -File $(Script) $(Platform) $(SolutionDir) $(IncludeDir) 
> $(SourceDir)" />
>     </Target>
> +  <Target Name="GetProjectInfoForReference"
> +          Returns="@(ProjectInfoForReference)">
> +    <ItemGroup>
> +      <ProjectInfoForReference Include="@(LibFullPath)" />
> +    </ItemGroup>
> +  </Target>
>   </Project>
> diff --git a/vs2019/xennet/xennet.vcxproj 
> b/vs2019/xennet/xennet.vcxproj index 027dc4a..beb5bb3 100644
> --- a/vs2019/xennet/xennet.vcxproj
> +++ b/vs2019/xennet/xennet.vcxproj
> @@ -26,7 +26,7 @@
>         <IntrinsicFunctions>true</IntrinsicFunctions>
>         
> <AdditionalIncludeDirectories>$(WindowsSdkDir)\include\km;..\..\include;..\..\include\xen;</AdditionalIncludeDirectories>
>         <WarningLevel>EnableAllWarnings</WarningLevel>
> -      
> <DisableSpecificWarnings>4464;4711;4770;4548;4820;4668;4255;5045;6001;6054;26451;28160;28196;30030;30029;%(DisableSpecificWarnings)</DisableSpecificWarnings>
> +      
> + <DisableSpecificWarnings>4061;4062;4464;4711;4770;4548;4820;4668;425
> + 5;5045;6001;6054;26451;28160;28196;30030;30029;%(DisableSpecificWarn
> + ings)</DisableSpecificWarnings>
>         <MultiProcessorCompilation>true</MultiProcessorCompilation>
>         <EnablePREfast>true</EnablePREfast>
>       </ClCompile>
> @@ -71,5 +71,10 @@
>     <ItemGroup>
>       <ResourceCompile Include="..\..\src\xennet\xennet.rc" />
>     </ItemGroup>
> +  <ItemDefinitionGroup>
> +    <DriverSign>
> +      <FileDigestAlgorithm>sha256</FileDigestAlgorithm>
> +    </DriverSign>
> +  </ItemDefinitionGroup>
>     <Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" />
>   </Project>
> diff --git a/vs2019/xennet_coinst/xennet_coinst.vcxproj 
> b/vs2019/xennet_coinst/xennet_coinst.vcxproj
> index a159abd..45ca05a 100644
> --- a/vs2019/xennet_coinst/xennet_coinst.vcxproj
> +++ b/vs2019/xennet_coinst/xennet_coinst.vcxproj
> @@ -53,5 +53,10 @@
>     <ItemGroup>
>       <None Include="..\..\src\coinst\xennet_coinst.def" />
>     </ItemGroup>
> +  <ItemDefinitionGroup>
> +    <DriverSign>
> +      <FileDigestAlgorithm>sha256</FileDigestAlgorithm>
> +    </DriverSign>
> +  </ItemDefinitionGroup>
>     <Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" />
>   </Project>
> 



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.