[openib-general] [PATCH] Replace cl_memory.h by string.h[was:[PATCH] OpenSM: Use memory routines directly and eliminatecl_mem*routines]

[Hal Rosenstock]

On Thu, 2006-05-18 at 13:28, Eitan Zahavi wrote:
> I think we need to change major version of libosmcomp such that
> applications compiled against the previous API will fail to link...

Right (missed that). Will be done shortly.

-- Hal

> Eitan Zahavi
> Senior Engineering Director, Software Architect
> Mellanox Technologies LTD
> Tel:+972-4-9097208
> Fax:+972-4-9593245
> P.O. Box 586 Yokneam 20692 ISRAEL
> 
> 
> > -----Original Message-----
> > From: Hal Rosenstock [mailto:halr at voltaire.com]
> > Sent: Thursday, May 18, 2006 6:07 PM
> > To: Eitan Zahavi
> > Cc: Sasha Khapyorsky; openib-general at openib.org
> > Subject: RE: [openib-general] [PATCH] Replace cl_memory.h by
> > string.h[was:[PATCH] OpenSM: Use memory routines directly and
> > eliminatecl_mem*routines]
> > 
> > Hi Eitan,
> > 
> > On Thu, 2006-05-18 at 11:00, Eitan Zahavi wrote:
> > > Hi Sasha, Hal,
> > >
> > > There several applications (ibis and ibmgtsim) that depend on
> complib,
> > > The changes of cleaning up the cl_memory API affect these utilities.
> > > Can you please provide the list of APIs removed and their
> replacements ?
> > 
> > cl_memset -> memset
> > cl_memclr(x, y) -> memset ( x, 0, y)
> > cl_memcpy -> memcpy
> > 
> > Soon cl_malloc/cl_zalloc/cl_free will change (and the memory tracking
> > will be removed).
> > 
> > -- Hal
> > 
> > > Also if we eventually converge on a single complib for windows and
> linux
> > > then the Windows stack is going to be affected by these changes too.
> > >
> > > EZ
> > >
> > > Eitan Zahavi
> > > Senior Engineering Director, Software Architect
> > > Mellanox Technologies LTD
> > > Tel:+972-4-9097208
> > > Fax:+972-4-9593245
> > > P.O. Box 586 Yokneam 20692 ISRAEL
> > >
> > >
> > > > -----Original Message-----
> > > > From: openib-general-bounces at openib.org [mailto:openib-general-
> > > > bounces at openib.org] On Behalf Of Sasha Khapyorsky
> > > > Sent: Thursday, May 18, 2006 1:03 AM
> > > > To: Hal Rosenstock
> > > > Cc: openib-general at openib.org
> > > > Subject: [openib-general] [PATCH] Replace cl_memory.h by string.h
> > > [was:[PATCH]
> > > > OpenSM: Use memory routines directly and eliminatecl_mem*
> routines]
> > > >
> > > > On 12:14 Wed 17 May     , Hal Rosenstock wrote:
> > > > > OpenSM: Use memory routines directly and eliminate cl_mem*
> routines
> > > > > as these routines are part of ISO C
> > > > >
> > > > > Signed-off-by: Hal Rosenstock <halr at voltaire.com>
> > > >
> > > > Following Hal's cleanup this includes string.h header file for
> proper
> > > > mem*() functions prototype definitions where necessary,
> > > removes/includes
> > > > cl_memory.h as needed. Also couple of unistd.h additions for
> close(),
> > > > sleep() and unlink() calls.
> > > >
> > > > Signed-off-by: Sasha Khapyorsky <sashak at voltaire.com>