[ofw] RE: [PATCH] IPOIB_NDIS6_CM IA64 & ibndprov.dll interactions.

[Tzachi Dar]

If you are sure the changes will only affect the IA64, then please go
ahead.

Thanks
Tzachi 

-----Original Message-----
From: Stan C. Smith [mailto:stan.smith at intel.com] 
Sent: Friday, October 30, 2009 12:14 AM
To: Tzachi Dar
Cc: ofw at lists.openfabrics.org
Subject: RE: [PATCH] IPOIB_NDIS6_CM IA64 & ibndprov.dll interactions.

Hello,
  I did not hear back on this issue? Did you want me to go ahead and
commit the IA64 specific changes?


Stan C. Smith wrote:
> Hello,
>   The OpenIB SVN IA64 version of ibndprov.dll does not build by 
> design; blocked in makefile.
> The ipoib_NDIS6_CM src version of netipoib.inx references ibndprov.dll

> in the 'NdCopyFiles' section.
> When inf2cat attempts to sign any Windows OS version for IA64 arch, 
> inf2cat fails as there is no ibndprov.dll file present.
> 
> In the existing NDIS 5.3 ipoib src, netipoib.inx has specific 
> modifications for IA64 installation which remove the dependence on 
> ibndprov.dll.
> 
> I would like to suggest modifications to 
> ipoib_NDIS6_CM\kernel\netipoib.inx exactly like those done for 
> ipoib\kernel\netipoib.inx.
> 
> I'm told fixing the SVN version of IA64 ibndprov.dll build would be 
> difficult at best and time not well spent as ND for IA64 is not 
> officially supported by Microsoft. I'm guessing you do not build for
> IA64?   
> 
> With the following patches applied to
> ipoib_NDIS6_CM\kernel\netipoib.inx, WinOF installers can be generated 
> which install the correct NDIS version of IPoIB.sys.
> Current Svr2008 R2 testing shows the NDIS 6 ipoib driver to load and 
> execute without error.
> uDAPL testing shows no differences; such that IBAT is working 
> correctly.
> 
> Signed-off-by: stan smith <stan.smith at intel.com>
> 
> --- a/ulp/ipoib_NDIS6_CM/kernel/netipoib.inx	Mon Oct 26 15:56:15 2009
> +++ b/ulp/ipoib_NDIS6_CM/kernel/netipoib.inx	Mon Oct 26 15:59:59 2009
> @@ -57,8 +57,7 @@
>  AddReg          = IpoibAddReg
>  CopyFiles       = IpoibCopyFiles
>  CopyFiles       = WsdCopyFiles
> -CopyFiles       = NdCopyFiles
> -CopyFiles       = WOW64CopyFiles
> +CopyFiles       = WOW64WsdCopyFiles
>  *IfType         = 6		; IF_TYPE_ETHERNET_CSMACD
>  *MediaType      = 0		; NdisMedium802_3
>  *PhysicalMediaType = 14		; NdisPhysicalMedium802_3
> @@ -222,6 +221,9 @@
>  ibwsd.dll,ibwsd32.dll,,0x00000002
>  ibndprov.dll,ibndprov32.dll,,0x00000002
> 
> +[WOW64WsdCopyFiles]
> +ibwsd.dll,ibwsd32.dll,,0x00000002
> +
>  [SourceDisksNames.x86]
>  1 = %IcsDisk1%,,,""
> 
> @@ -249,15 +251,13 @@
>  ipoib.sys = 1
>  ibwsd.dll = 1
>  ibwsd32.dll = 1
> -ibndprov.dll = 1
> -ibndprov32.dll = 1
> -ndinstall.exe = 1
> 
>  [DestinationDirs]
>  IpoibCopyFiles    = %DIRID_DRIVERS%
>  WsdCopyFiles      = %DIRID_SYSTEM%
>  NdCopyFiles       = %DIRID_SYSTEM%
>  WOW64CopyFiles    = %DIRID_SYSTEM_X86%
> +WOW64WsdCopyFiles = %DIRID_SYSTEM_X86%
>  DefaultDestDir    = %DIRID_SYSTEM%
> 
>  [Strings]