[ofw] RE: [REPOST] [PATCH] Add ND files

Leonid Keller leonid at mellanox.co.il
Sun Aug 10 05:30:20 PDT 2008


SB


________________________________

	From: Smith, Stan [mailto:stan.smith at intel.com] 
	Sent: Thursday, August 07, 2008 8:09 PM
	To: Leonid Keller; Reuven Amitai; Tzachi Dar; Fab Tillier
	Cc: ofw at lists.openfabrics.org
	Subject: RE: [ofw] RE: [REPOST] [PATCH] Add ND files
	
	
	Thanks Leonid, some additional questions inline.

________________________________

	From: Leonid Keller [mailto:leonid at mellanox.co.il] 
	Sent: Thursday, August 07, 2008 2:23 AM
	To: Smith, Stan; Reuven Amitai; Tzachi Dar; Fab Tillier
	Cc: ofw at lists.openfabrics.org
	Subject: RE: [ofw] RE: [REPOST] [PATCH] Add ND files
	
	
	see inline


________________________________

		From: ofw-bounces at lists.openfabrics.org
[mailto:ofw-bounces at lists.openfabrics.org] On Behalf Of Smith, Stan
		Sent: Thursday, August 07, 2008 12:45 AM
		To: Reuven Amitai; Tzachi Dar; Fab Tillier
		Cc: ofw at lists.openfabrics.org
		Subject: [ofw] RE: [REPOST] [PATCH] Add ND files
		
		
		Just to make sure I understand from an install
perspective, who/how/when & why is ndinstall.exe executed?
		[Leonid Keller] See ND as another flavour of WSD.
		It has to be installed from MSI the same way as WSD upon
user's request or by default.  
		 
		The implication here is ndinstall.exe must be executed
during the ND install, similar to WSD.
		ndinstall.exe is not and should not be referenced in a
.inf or .cat file; like installsp.exe.
		
		[Leonid Keller] It is referenced because we want be able
to install all from inf, without MSI.
		In that case one will need too install ND manually:
"ndinstall -i".
		 
		Why is nd so closely tied to IPoIB install?  Precludes
nd as optional?
		Why is nd not installed standalone independent of IPoIB?
A WIX 'feature'?
		I assume your intention is for nd to be a 'standard'
feature of WinOF 2.0?
		
		[Leonid Keller] ND, like WSD, requires IPoIB for its
work.
		But it tends to replace WSD, so, yes, we want it to be
a standard feature. 
		 
		Am I to assume that ND is limited to only those
environments where WSD is supported?
		Specifically, XP doesn't support WSD and should not
support ND?
		
		[Leonid Keller]  No, ND is not limited. It works on XP.
		 
		 
		I assume ibndprov.dll is intended to be installed in
%windir% ?
		
		[Leonid Keller] In %windir%\system32 like all other
dlls.  
		 
		DAT/DAPL .dll files are installed in %windir%, was the
reason for asking?
		I was under the impression system32\ was for mainly
driver files, with ibndprov.dll being a provider.
		Should the DAT/DAPL files be relocated to system32?
		
		[Leonid Keller]  We install all DLLs into \system32.
Drivers reside in \system32\drivers.
		 
		Could ndinstall.exe live in %ProgramFiles(x86)%\WinOF\ ?
		
		[Leonid Keller] It seems reasonable to put it in the
same place with WSD stuff, especially instalsp.exe 
		 
		Will install ndinstall.exe into WinOF\ folder
		 
		Why is nd, an upper level protocol, not resident in
trunk\ulp\nd\user ?
		
		[Leonid Keller] In our internal SVN tree it resides
really there ...
		 
		  Eventually the source for the nd provider will be in
svn, so from a minimize modifications to WIX src files, why not place
binaries where the src will live?  For the near term, we could construct
a sources/makefile to copy .dll & .exe to trunk\bin\user\...
		
		[Leonid Keller] When the source comes, we'll remove the
binaries ...
		I'd suggest to put ND binaries in trunk\bin\..., i.e. in
those same places, where they would come while generated from the source
code. 
		 
		ND binaries should live in trunk\ulp\nd\user along with
SOURCES file to copy into trunk\bin\user\.... 
		I'll set this up ASAP.
		 
		stan.

________________________________

		From: Reuven Amitai [mailto:reuven at mellanox.co.il] 
		Sent: Wednesday, August 06, 2008 4:11 AM
		To: Smith, Stan; Tzachi Dar; Fab Tillier
		Cc: ofw at lists.openfabrics.org
		Subject: RE: [REPOST] [PATCH] Add ND files
		
		
		Hi,
		 
		The patch add ND files (ibndprov.dll, ndinstall.exe) to
netipoib.inx, ipoib.cdf, ipoib32.cdf
		Currently IA64 platform isn't supported (HPC supports
Windows 2008 x64). netipoib-xp32.inf wasn't changed (same reason)
		Attached nd-part.diff (diff for netipoib.inx, ipoib.cdf,
ipoib32.cdf alone) and nd -bin .zip which contains ibndprov.dll and
ndinstall.exe
		(remove .zip suffix for each file). I think we should
add them under trunk\etc\bin\ directory (new directory).
		Do you have other suggestions for binaries file location
in the svn tree?
		 
		Thanks, Reuven.
		 
		>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
		 
		Index: ipoib.cdf

	
===================================================================

		--- ipoib.cdf     (revision 1466)

		+++ ipoib.cdf     (working copy)

		@@ -8,3 +8,6 @@

		 <hash>ipoib.sys=ipoib.sys

		 <hash>ibwsd.dll=ibwsd.dll

		 <hash>ibwsd32.dll=ibwsd32.dll

		+<hash>ibndprov.dll=ibndprov.dll

		+<hash>ibndprov32.dll=ibndprov32.dll

		+<hash>ndinstall.exe=ndinstall.exe

		Index: ipoib32.cdf

	
===================================================================

		--- ipoib32.cdf   (revision 1466)

		+++ ipoib32.cdf   (working copy)

		@@ -7,3 +7,6 @@

		 <hash>netipoib.inf=netipoib.inf

		 <hash>ipoib.sys=ipoib.sys

		 <hash>ibwsd.dll=ibwsd.dll

		+<hash>ibwsd.dll=ibwsd.dll

		+<hash>ibndprov.dll=ibndprov.dll

		+<hash>ndinstall.exe=ndinstall.exe

		Index: netipoib.inx

	
===================================================================

		--- netipoib.inx  (revision 1466)

		+++ netipoib.inx  (working copy)

		@@ -32,14 +32,16 @@

		 Characteristics = 0x81 ; NCF_HAS_UI | NCF_VIRTUAL

		 AddReg          = IpoibAddReg

		 CopyFiles       = IpoibCopyFiles

		-CopyFiles        = WsdCopyFiles

		+CopyFiles       = WsdCopyFiles

		+CopyFiles       = NdCopyFiles

		 

		 [Ipoib.DDInstall.ntamd64]

		 Characteristics = 0x81 ; NCF_HAS_UI | NCF_VIRTUAL

		 AddReg          = IpoibAddReg

		 CopyFiles       = IpoibCopyFiles

		-CopyFiles        = WsdCopyFiles

		-CopyFiles        = WOW64CopyFiles

		+CopyFiles       = WsdCopyFiles

		+CopyFiles       = NdCopyFiles

		+CopyFiles       = WOW64CopyFiles

		 

		 [Ipoib.DDInstall.ntia64]

		 Characteristics = 0x81 ; NCF_HAS_UI | NCF_VIRTUAL

		@@ -161,8 +163,13 @@

		 [WsdCopyFiles]

		 ibwsd.dll,,,0x00000002

		 

		+[NdCopyFiles]

		+ibndprov.dll,,,0x00000002

		+ndinstall.exe,,,0x00000002

		+

		 [WOW64CopyFiles]

		 ibwsd.dll,ibwsd32.dll,,0x00000002

		+ibndprov.dll,ibndprov32.dll,,0x00000002

		 

		 [SourceDisksNames.x86]

		 1 = %IcsDisk1%,,,""

		@@ -176,11 +183,16 @@

		 [SourceDisksFiles.x86]

		 ipoib.sys = 1

		 ibwsd.dll = 1

		+ibndprov.dll = 1

		+ndinstall.exe = 1

		 

		 [SourceDisksFiles.amd64]

		 ipoib.sys = 1

		 ibwsd.dll = 1

		 ibwsd32.dll = 1

		+ibndprov.dll = 1

		+ibndprov32.dll = 1

		+ndinstall.exe = 1

		 

		 [SourceDisksFiles.ia64]

		 ipoib.sys = 1

		@@ -190,6 +202,7 @@

		 [DestinationDirs]

		 IpoibCopyFiles    = %DIRID_DRIVERS%

		 WsdCopyFiles      = %DIRID_SYSTEM%

		+NdCopyFiles       = %DIRID_SYSTEM%

		 WOW64CopyFiles    = %DIRID_SYSTEM_X86%

		 DefaultDestDir    = %DIRID_SYSTEM%

		 

		

		 

________________________________

		From: Smith, Stan [mailto:stan.smith at intel.com] 
		Sent: Tuesday, July 29, 2008 9:55 PM
		To: Reuven Amitai; Tzachi Dar; Fab Tillier
		Cc: ofw at lists.openfabrics.org
		Subject: RE: [Patch] Add ND files into Netipoib.inx
		
		
		The only problem here not all of us have the ND files,
hence will error out on install!
		Please forward binaries and/or place binaries in svn
until ND sources are available to build.
		 
		Please do not inject known broken patches into svn
without suggesting a global fix for all.
		 
		thanks,
		 
		stan.

________________________________

		From: Reuven Amitai [mailto:reuven at mellanox.co.il] 
		Sent: Tuesday, July 29, 2008 10:12 AM
		To: Tzachi Dar; Smith, Stan; Fab Tillier
		Cc: ofw at lists.openfabrics.org
		Subject: [Patch] Add ND files into Netipoib.inx
		
		
		Hi,
		 
		The patch add ND files to netipoib.inx since in the
coming release ND will be part of WinOF.
		IbNdProv.dll is the Provider dll, ndinstall.exe is ND
installer.
		 
		Thanks, Reuven.
		 
		>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
		 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20080810/e36c8445/attachment.html>


More information about the ofw mailing list