[ofw] RE: Building the correct IPoIB version for the correct OS

Tzachi Dar tzachid at mellanox.co.il
Mon Nov 2 09:29:30 PST 2009


I go to the ipoib directory and build with 6001.18001 compiler. Here is
the result

BUILD: Examining q:\openib\trunk\ulp\ipoib_ndis6_cm\kernel directory for
files to compile.
    q:\openib\trunk\ulp\ipoib_ndis6_cm\kernel - 1 Pass Zero files (334
lines)
BUILD: Saving c:\winddk\6001.18001\build.dat...
BUILD: Building generated files in
q:\openib\trunk\ulp\ipoib_ndis6_cm\kernel directory
errors in directory q:\openib\trunk\ulp\ipoib_ndis6_cm\kernel
c:\winddk\6001.18001\bin\makefile.def(50) : error U1023: syntax error in
expression
errors in directory q:\openib\trunk\ulp\ipoib_ndis6_cm\kernel
nmake.exe /nologo BUILDMSG=Stop. -i BUILD_PASS=PASS0 NOLINK=1
PASS0ONLY=1 MAKEDI
R_RELATIVE_TO_BASEDIR= failed - rc = 2 

Thanks
Tzacih

> -----Original Message-----
> From: ofw-bounces at lists.openfabrics.org 
> [mailto:ofw-bounces at lists.openfabrics.org] On Behalf Of Smith, Stan
> Sent: Monday, November 02, 2009 7:26 PM
> To: Tzachi Dar
> Cc: ofw at lists.openfabrics.org
> Subject: [ofw] RE: Building the correct IPoIB version for the 
> correct OS
> 
> Tzachi Dar wrote:
> > It seems that build doesn't work after your change.
> >
> > Can you please verify this?
> 
> Odd, I tested building a full WinOF release on 3 different 
> occasions along with numerous arch specific cmd-window builds 
> (wnet,wxp,wlh & win7) before submitting the patch.
> 
> When you say the build doesn't work, what fails?
> An architecture specific build or a full WinOF build via 
> winof\buildrelease.bat?
> 
> 
> 
> >
> > Thanks
> > Tzachi
> >
> >> -----Original Message-----
> >> From: Stan C. Smith [mailto:stan.smith at intel.com]
> >> Sent: Tuesday, October 27, 2009 1:46 AM
> >> To: Tzachi Dar
> >> Cc: ofw at lists.openfabrics.org
> >> Subject: RE: Building the correct IPoIB version for the correct OS
> >>
> >> Tzachi Dar wrote:
> >>> There was no real reason for that. I guess that we have copied it 
> >>> from somewhere.
> >>>
> >>> Feel free to change it.
> >>>
> >>> Thanks
> >>> Tzachi
> >>
> >> OK, thank you.
> >>
> >> Revision: 2518
> >> Author: stansmith
> >> Date: 4:40:57 PM, Monday, October 26, 2009
> >> Message:
> >> [IPOIB] NDIS 5 & NDIS 6 versions: use a symbolic constant 
> to identify 
> >> Windows version in makefile.
> >> ----
> >> Modified : /gen1/trunk/ulp/ipoib/kernel/makefile
> >> Modified : /gen1/trunk/ulp/ipoib_NDIS6_CM/kernel/makefile
> >>
> >> Signed-off-by: Tzachi Dar <tzachid at mellanox.co.il> and stan smith 
> >> <stan.smith at intel.com>
> >>
> >> --- C:/Documents and Settings/scsmith/Local
> >> Settings/Temp/makefi-revBASE.svn002.tmp      Mon Oct 26 
> 16:26:03 2009
> >> +++ C:/Documents and Settings/scsmith/My
> >> Documents/openIB-windows/SVN/gen1/trunk/ulp/ipoib/kernel/m
> >> akefile      Mon Oct 26 13:10:20 2009
> >> @@ -8,6 +8,6 @@
> >>  # prevent the Build utility from building the driver for later  # 
> >> version of the operating system than windows 2003  #
> >> -MAXIMUM_NT_TARGET_VERSION=0x502
> >> +MAXIMUM_NT_TARGET_VERSION=$(_NT_TARGET_VERSION_WS03)
> >>
> >>  !INCLUDE ..\..\..\inc\openib.def
> >>
> >>
> >> --- C:/Documents and Settings/scsmith/Local
> >> Settings/Temp/makefil-revBASE.svn001.tmp     Mon Oct 26 
> 16:20:35 2009
> >> +++ C:/Documents and Settings/scsmith/My
> >> Documents/openIB-windows/SVN/gen1/trunk/ulp/ipoib_NDIS6_CM/ker
> >> nel/makefile Fri Oct 23 12:32:21 2009 @@ -8,6 +8,6 @@  # 
> prevent the 
> >> Build utility from building the driver for an earlier  # 
> version of 
> >> the operating system than vista/Win 2008.  # 
> >> -MINIMUM_NT_TARGET_VERSION=0x600
> >> +MINIMUM_NT_TARGET_VERSION=$(_NT_TARGET_VERSION_VISTA)
> >>
> >>  !INCLUDE ..\..\..\inc\openib.def
> >>
> >>
> >>
> >>
> >>>
> >>>> -----Original Message-----
> >>>> From: Smith, Stan [mailto:stan.smith at intel.com]
> >>>> Sent: Monday, October 26, 2009 5:56 PM
> >>>> To: Tzachi Dar
> >>>> Cc: ofw at lists.openfabrics.org
> >>>> Subject: RE: Building the correct IPoIB version for the 
> correct OS
> >>>>
> >>>> Hello,
> >>>>   Comments below.
> >>>>
> >>>> Tzachi Dar wrote:
> >>>>> Yes, you're right:
> >>>>> 1. The "dirs" file should contains ipoib_NDIS6_CM 2. 
> The add-on to 
> >>>>> makefile of ipoib based on NDIS 5:
> >>>>> #
> >>>>> # prevent the Build utility from building the driver 
> for later # 
> >>>>> version of the operating system than windows 2003 #
> >>>>> MAXIMUM_NT_TARGET_VERSION=0x502
> >>>>>
> >>>>> !INCLUDE ..\..\..\inc\openib.def
> >>>>>
> >>>>> 3. The add-on to makefile of ipoib based on NDIS 6: # # prevent 
> >>>>> the Build utility from building the driver for an earlier # 
> >>>>> version of the operating system than vista/Win 2008. # 
> >>>>> MINIMUM_NT_TARGET_VERSION=0x600
> >>>>>
> >>>>> !INCLUDE ..\..\..\inc\openib.def
> >>>>>
> >>>>> We already updated SVN with these changes on revision 2506.
> >>>>
> >>>> OK.
> >>>>
> >>>> One small question.
> >>>> What is the logic in using a hard coded OS version 
> constant instead 
> >>>> of the self-documenting, MSFT suggested symbolic constant?
> >>>>
> >>>> $(_NT_TARGET_VERSION_WS03)
> >>>> $(_NT_TARGET_VERSION_VISTA)
> >>>>
> >>>>
> >>>>>
> >>>>> Thanks
> >>>>> Tzachi
> >>>>>
> >>>>>> -----Original Message-----
> >>>>>> From: Stan C. Smith [mailto:stan.smith at intel.com]
> >>>>>> Sent: Friday, October 23, 2009 9:50 PM
> >>>>>> To: Tzachi Dar
> >>>>>> Cc: ofw at lists.openfabrics.org
> >>>>>> Subject: Building the correct IPoIB version for the correct OS
> >>>>>>
> >>>>>>
> >>>>>> Hello,
> >>>>>>   Are you the correct person to be fielding IPoIB 
> questions these 
> >>>>>> days? I'll assume so for now.
> >>>>>>
> >>>>>> Given the requirement NDIS6 IPoIB is only for 
> Vista/Svr2008 and 
> >>>>>> beyond, then to prevent ipoib.sys name space 
> overlap/overwrite, 
> >>>>>> is the following what you had in mind for NDIS5.3 ipoib builds?
> >>>>>> Assumption is the ulp\dirs file contains entries for 
> both ipoib & 
> >>>>>> ipoib_ndis6_cm.
> >>>>>>
> >>>>>> --- C:/Documents and Settings/scsmith/Local
> >>>>>> Settings/Temp/make-revBASE.svn004.tmp        Fri Oct 
> 23 12:43:20
> >>>>>> 2009 +++ C:/Documents and Settings/scsmith/My 
> >>>>>> Documents/openIB-windows/SVN/gen1/trunk/ulp/ipoib/kernel/m
> >>>>>> akefile      Fri Oct 23 12:44:57 2009
> >>>>>> @@ -4,4 +4,8 @@
> >>>>>>  # that is shared by all the driver components of the OpenIB 
> >>>>>> Windows project. #
> >>>>>>
> >>>>>> +# Build this NDIS 5.3 ipoib for Windows versions prior to
> >>>>>>  Vista/Svr +2008 +
> >>>>>> +#MAXIMUM_NT_TARGET_VERSION=$(_NT_TARGET_VERSION_WS03) +
> >>>>>> !INCLUDE ..\..\..\inc\openib.def
> 
> _______________________________________________
> ofw mailing list
> ofw at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
> 



More information about the ofw mailing list