[netperf-talk] Solaris 9 compile issue

[Andrew Gallatin]

Rick Jones writes:
 > > Of course, I just now tried a TCP_SENDFILE test and it dumped core :)
 > 
 > Turns-out that in my case at least, the code for solaris sendfilev was 
 > bracketed by if defined(__sun__) and __sun__ isn't defined.  It seems 
 > that __sun is defined.
 > 
 > If you could check with gcc (I've not got gcc on my system) to see which 
 > of __sun__ and __sun are defined, that would help a lot.  At least with 
 > sendfile in general if not in your specific situation.

Hi Rick,

Indeed, the __sun__'s should be __sun.  I confess that I never tested
the original patch to add solaris sendfile support with cc, just gcc.
I have attached a patch to fix it.  I've tested this with both cc
("Sun C 5.7") as well as gcc on Solaris 10GA.


Drew

-------------- next part --------------
A non-text attachment was scrubbed...
Name: solaris_sendfile.diff
Type: application/octet-stream
Size: 1843 bytes
Desc: solaris sendfile fix
Url : http://www.netperf.org/pipermail/netperf-talk/attachments/20071120/90fa2a4b/attachment.obj