[ofw] RE: x86 XP env - WSD & installsp fail to build, IPoIB fails to install.

[Tzachi Dar]

Hi Fab,

The real issue I can see here is where to put our resources:

One way, is to say that from now and on, we will have one 2 builds for
xp 2 for 2003, 2 for 2008 (and maybe even 2 for vista) (2 is because one
for 32 bit and one for amd64) => total 8 builds. People from Intel might
argue that we should also have an itanium build wherever it is
possibale. OFW has already done more complicated tasks and we can do
that as well. I believe that there is extra work in doing the builds and
the tests.

The other way is to say we only support 4 builds: xp and 2003 together
(32 and 64). And 2008/vista (32/64) (again Itanium might be added). The
extra time that will be freed can be used for other tasks. For example
making sure that ipoib can reach 10Gb/sec. Having only 4 builds will
mean that things will not be as perfect as they can be in one sense but
they will work better in another one.

So the real issue that has to be asked is do we want to write another
installer and do more builds?
If the answer is yes, than we will change the inf (and don't build WSD
for xp). If the answer is no than some simple work around should be
used. This is the real question here.

As for the perfmon registry on xp: Well it just wouldn't be created for
the first place, so no real issue here.

The printf is to tell the user that WSD doesn't work on XP, and can be
removed of course.

Thoughts 
Tzachi

> -----Original Message-----
> From: Fab Tillier [mailto:ftillier at windows.microsoft.com] 
> Sent: Wednesday, May 28, 2008 8:06 PM
> To: Tzachi Dar; Smith, Stan
> Cc: ofw at lists.openfabrics.org
> Subject: RE: [ofw] RE: x86 XP env - WSD & installsp fail to 
> build, IPoIB fails to install.
> 
> > From: Tzachi Dar [mailto:tzachid at mellanox.co.il]
> > Sent: Wednesday, May 28, 2008 8:49 AM
> > To: Fab Tillier; Smith, Stan
> > Cc: ofw at lists.openfabrics.org
> > Subject: RE: [ofw] RE: x86 XP env - WSD & installsp fail to build, 
> > IPoIB fails to install.
> >
> > The problem was indeed in xp32 but WSD will not run in any 
> version of 
> > XP, so I guess that my fix is still correct.
> 
> <snip...>
> 
> > 2) Make sure that WSD will be built (without the other functions). 
> > This is what I did.
> >
> > I'm fine with all options.
> 
> But what happens when someone installs the XP drivers on 
> 2003?  They get improper behavior (perfmon registry entry 
> stays around when it should be cleaned up?)  I think adding a 
> XP-specific entry to the INF file that doesn't copy the WSD 
> DLL would be better.
> 
> > > > @@ -691,6 +693,11 @@
> > > >  {
> > > >         WSADATA wsd;
> > > >
> > > > +#ifdef WINXP_BUILD
> > > > +       printf("WSD doesn't run on windows XP\n");
> 
> I missed this earlier - printf isn't appropriate here.
> 
> > > > +       return 1;
> > > > +#endif
> > > > +
>