Eitan, I did not removed anything from the existing VS project. I have only changed the compiler settings, visible from project's properties --> Configuration properties --> C/C++ --> Preprocessor --> Preprocessor Definitions.
For Win8 Release & Win8 Debug the above value was not changed and it is the following: "%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1". For Win8.1 Release & Win8.1 Debug the above value was changed into the following: "%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1". All remaining settings were not changed. For example, when selecting 'Win8.1 Release' build configuration, because of the NDIS640=1 setting, the headers for NDIS 6.40 will be included into the OVSEXT project, making the OVS extension an NDIS 6.40 compliant driver. -Sorin -----Original Message----- From: Eitan Eliahu [mailto:[email protected]] Sent: Monday, 24 November, 2014 19:08 To: Sorin Vinturis; [email protected] Subject: RE: [PATCH v2] datapath-windows: Update OVSEXT VS project to support 6.40 Sorin, did you remove Win8 project? Can we have 8.1 WInDDK libraries for both 6.30 and 6.40 drivers? Thanks, Eitan -----Original Message----- From: dev [mailto:[email protected]] On Behalf Of Sorin Vinturis Sent: Monday, November 24, 2014 9:03 AM To: [email protected] Subject: [ovs-dev] [PATCH v2] datapath-windows: Update OVSEXT VS project to support 6.40 Added support for creating OVS extension driver NDIS 6.40 compliant. Currently the OVSEXT Visual Studio project has four build configurations, 'Win8 Release', 'Win8 Debug', 'Win8.1 Release' and 'Win8.1 Debug'. All of them are creating a binary that is NDIS 6.30 compliant. I have changed the Win8.1 build configurations in order to create a binary that is NDIS 6.40 compliant. In this way, the OVSEXT project is able to create a release/debug binary that is NDIS 6.30 compliant, using the 'Win8 Release' and 'Win8 Debug' build configurations, as well as a release/debug binary that is NDIS 6.40 compliant, using the 'Win8.1 Release' and 'Win8.1 Debug' build configurations. Signed-off-by: Sorin Vinturis <[email protected]> Acked-by: Eitan Eliahu <[email protected]> --- datapath-windows/ovsext/ovsext.vcxproj | 35 +++++++++++++++++++++++++++++++++- 1 file changed, 34 insertions(+), 1 deletion(-) diff --git a/datapath-windows/ovsext/ovsext.vcxproj b/datapath-windows/ovsext/ovsext.vcxproj index 88c9122..693bc50 100644 --- a/datapath-windows/ovsext/ovsext.vcxproj +++ b/datapath-windows/ovsext/ovsext.vcxproj @@ -102,7 +102,7 @@ <PropertyGroup> <TargetName>OVSExt</TargetName> </PropertyGroup> - <ItemDefinitionGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Win8 + Release|x64'"> <ClCompile> <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1</PreprocessorDefinitions> </ClCompile> @@ -113,6 +113,39 @@ <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1</PreprocessorDefinitions> </ResourceCompile> </ItemDefinitionGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Win8.1 Release|x64'"> + <ClCompile> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1</PreprocessorDefinitions> + </ClCompile> + <Midl> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1</PreprocessorDefinitions> + </Midl> + <ResourceCompile> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1</PreprocessorDefinitions> + </ResourceCompile> + </ItemDefinitionGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Win8 Debug|x64'"> + <ClCompile> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1</PreprocessorDefinitions> + </ClCompile> + <Midl> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1</PreprocessorDefinitions> + </Midl> + <ResourceCompile> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS630=1</PreprocessorDefinitions> + </ResourceCompile> + </ItemDefinitionGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Win8.1 Debug|x64'"> + <ClCompile> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1</PreprocessorDefinitions> + </ClCompile> + <Midl> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1</PreprocessorDefinitions> + </Midl> + <ResourceCompile> + <PreprocessorDefinitions>%(PreprocessorDefinitions);NDIS_WDM=1;NDIS640=1</PreprocessorDefinitions> + </ResourceCompile> + </ItemDefinitionGroup> <ItemDefinitionGroup> <Link> <AdditionalDependencies>%(AdditionalDependencies);$(DDK_LIB_PATH)\ndis.lib;$(DDK_LIB_PATH)\fwpkclnt.lib;$(SDK_LIB_PATH)\uuid.lib;$(DDK_LIB_PATH)\netio.lib</AdditionalDependencies> -- 1.9.0.msysgit.0 _______________________________________________ dev mailing list [email protected] https://urldefense.proofpoint.com/v2/url?u=http-3A__openvswitch.org_mailman_listinfo_dev&d=AAIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=CWsgHUxi6ExLXY798tmo3LJ4e3geGYp56lkcH-5cLCY&m=LrfnoGch9G6_TPkJ5ZHRsNTI3exzEFBiWaD6dUH_zFQ&s=n4bOIxBT0f4aoMlS1R30ZavdqgdgJLZAj9W5yK21nAM&e= _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
