<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=us-ascii">
<META NAME="Generator" CONTENT="MS Exchange Server version 5.5.2654.45">
<TITLE>RE: [Openib-windows] Re: Complib changes to support OpenSM in OpenIB</TITLE>
</HEAD>
<BODY>
<P><FONT SIZE=2>Hello Fab,</FONT>
<BR><FONT SIZE=2>Attached is the cl_qlockpool.h file with the functions implemented as static inline.</FONT>
<BR><FONT SIZE=2>Thanks,</FONT>
<BR><FONT SIZE=2>Yael </FONT>
</P>
<P><FONT SIZE=2>-----Original Message-----</FONT>
<BR><FONT SIZE=2>From: Fab Tillier [<A HREF="mailto:ftillier@silverstorm.com">mailto:ftillier@silverstorm.com</A>]</FONT>
<BR><FONT SIZE=2>Sent: Tuesday, August 30, 2005 9:48 PM</FONT>
<BR><FONT SIZE=2>To: 'Eitan Zahavi'</FONT>
<BR><FONT SIZE=2>Cc: 'Tzachi Dar'; 'Aviram Gutman'; openib-windows@openib.org; Yael Kalka</FONT>
<BR><FONT SIZE=2>Subject: RE: [Openib-windows] Re: Complib changes to support OpenSM in</FONT>
<BR><FONT SIZE=2>OpenIB</FONT>
</P>
<BR>
<P><FONT SIZE=2>> From: Eitan Zahavi [<A HREF="mailto:eitan@mellanox.co.il">mailto:eitan@mellanox.co.il</A>]</FONT>
<BR><FONT SIZE=2>> Sent: Tuesday, August 30, 2005 11:31 AM</FONT>
<BR><FONT SIZE=2>> </FONT>
<BR><FONT SIZE=2>> Fab Tillier wrote:</FONT>
<BR><FONT SIZE=2>> >></FONT>
<BR><FONT SIZE=2>> >>>What is the problem with keeping qlockpool in opensm?</FONT>
<BR><FONT SIZE=2>> >></FONT>
<BR><FONT SIZE=2>> >>To have qlockpool be part of OpenSM we will need to:</FONT>
<BR><FONT SIZE=2>> >>1. Modify its implementation to be all static inline</FONT>
<BR><FONT SIZE=2>> ></FONT>
<BR><FONT SIZE=2>> ></FONT>
<BR><FONT SIZE=2>> > No, the static inline is a requirement I am imposing if qlockpool is going</FONT>
<BR><FONT SIZE=2>> > to be in the complib include directory.</FONT>
<BR><FONT SIZE=2>> Ok so if we provide header only you agree to place it in the complib dir?</FONT>
<BR><FONT SIZE=2>> This option is fine with me - we will do it. I did not understand this before.</FONT>
</P>
<P><FONT SIZE=2>As long as the header exports static inline functions and not CL_INLINE, then</FONT>
<BR><FONT SIZE=2>yes. This isn't my preferred solution, but it's time we moved on.</FONT>
</P>
<P><FONT SIZE=2>- Fab</FONT>
</P>
<P><FONT FACE="Arial" SIZE=2 COLOR="#000000"></FONT>
</BODY>
</HTML>