[Openib-windows] Debug prints in free version

[Tzachi Dar]

Printing (even only in the case of errors) will hurt performance for the
following reasons:
 
Today, these macros are being removed out at compile time. If they are there
in release, the check that is made if to print or not will still be
executed. 
 
Having these functions in our code also means bigger foot print of our
functions and therefore another hit in performance (for example more cache
misses).
 
Please also note that your idea is not enough since the IPOIB code is using
the macro IPOIB_TRACE which should also be changed to exist in free also.
(There is a list of macros that should be changed).
 
We might do a separate build that is free but still has the prints, but
debug messages should be omitted from the binaries that we give to our
customers.
 
Thanks
Tzachi
 
-----Original Message-----
From: Yossi Leybovich [mailto:sleybo at mellanox.co.il] 
Sent: Monday, August 08, 2005 3:05 PM
To: Fab Tillier
Cc: openib-windows at openib.org
Subject: [Openib-windows] Debug prints in free version
 
Fab 
I think it will make it easy to all of us if the stack will generate printf
in case of error. 
In the current defines in order to debug any kernel level component you need
to replace the sys file to the debug version 
It will be better if : 
CL_PRINT in case of ERROR_DBG should use cl_msg_out so it will generate
print even in free version 
It should not heart the data flow (we print just in case of error) 
What do you think ? 
Yossi 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/ofw/attachments/20050808/3d610dbc/attachment.html>