[PVFS2-developers] Patch : Dynamic loading of BMI library

[Beilloin, David]

Hi all,

In attachment is a new patch to support BMI dynamic shared library,
Based on your previous input.

This options is off by default, and can be enabled at configure time with
--enable-dynamic-bmilib, 

When enabled ALLOW_DYNAMIC_BMILLIB is defined in CFLAGS and activate the
Small part of code for this features, and binary are link with -ldl.

When not no change are made compared to the current way it works.

Concerning scripts, pvfs2-genconfig is updated to check in the case where 
the choosen protocol is not tcp, gm or ib if a shared library is available
for the required protocol (base on its name libbmi_<protocol>) to do this 
pvfs2-genconfig call a simple program responsible for checking the existence
Of such library and its symbols.

Thanks in advance for your input and comments,
And hop it will fit your requirements.

Regards,
David Beilloin.


> -----Message d'origine-----
> De : pvfs2-developers-bounces at beowulf-underground.org 
> [mailto:pvfs2-developers-bounces at beowulf-underground.org] De 
> la part de Beilloin, David
> Envoyé : mercredi 2 mars 2005 18:16
> À : Phil Carns; Rob Ross
> Cc : PVFS2 Developers
> Objet : RE: [PVFS2-developers] Patch : Dynamic loading of BMI library
> 
> Ok, I understand your point of view, and I will try to do a 
> better Patch allowing optional features compilation and 
> script installation.
> 
> Thanks for your 2c :-)
> 
> David.
> 
> > -----Message d'origine-----
> > De : Phil Carns [mailto:pcarns at wastedcycles.org] Envoyé : 
> mercredi 2 
> > mars 2005 18:09 À : Rob Ross Cc : Beilloin, David; PVFS2 Developers 
> > Objet : Re: [PVFS2-developers] Patch : Dynamic loading of 
> BMI library
> > 
> > I tend to agree with Rob regarding the support issues 
> associated with 
> > dynamic module loading, so I would be worried about making this 
> > something that the general user community has to mess with.
> > 
> > However, it would clearly be useful for some advanced users or 
> > developers :)  And I think actually the biggest problem with our 
> > original ancient module loading approach in BMI was that it was
> > manditory- you had worry about module locations and synchronization 
> > issues even if you didn't care at all about actually using the 
> > feature.
> > 
> > Is it possible (or maybe your patch already does this, I haven't 
> > looked at it yet myself) to have the whole dynamic loading thing 
> > optional so that user's following the quickstart guide in 
> its current 
> > form aren't exposed to it?
> > In other words, default to compiling everything statically 
> and don't 
> > build or install any modules.
> > 
> > It might even be nice to go as far as to have some configure option 
> > (--enable-dynamic-loader or something) that you have to explicitly 
> > turn on to get the functionality.  That would also allow 
> you to skip 
> > the -ldl in the general case (for maximum portability) 
> unless whoever 
> > is compiling it knows that they want dynamic loading.
> > 
> > I think something along those lines would be ok if it doesn't 
> > complicate the code too much, but that's just my 2c :)
> > 
> > -Phil
> > 
> > Rob Ross wrote:
> > > Hi David,
> > > 
> > > Not yet, sorry.  I've been traveling for the last two 
> weeks (and am 
> > > leaving again today), and RobL has been focusing on the
> > BG/L platform.
> > > 
> > > I tend to not want to include dynamic loading of libraries,
> > because I
> > > think that if we get people using them we will probably 
> just end up 
> > > with a ton more bug reports because people don't keep the
> > libraries in
> > > sync.  But I do see how it might be a useful technology,
> > and I'll have
> > > a look.  It would help if you could get Phil or Pete on
> > your side :).
> > > 
> > > Please don't cross-post; we see mail even if it goes to
> > just one list :).
> > > 
> > > Rob
> > > 
> > > On Tue, 1 Mar 2005, Beilloin, David wrote:
> > > 
> > > 
> > >>Hi
> > >>Did anyone had some time to look at this ?
> > >>
> > >>Thanks for your answer,
> > >>David.
> > >>
> > 
> > 
> 
> _______________________________________________
> PVFS2-developers mailing list
> PVFS2-developers at beowulf-underground.org
> http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
> 
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pvfs2_dynamicbmi.patch
Type: application/octet-stream
Size: 11997 bytes
Desc: pvfs2_dynamicbmi.patch
Url : http://www.beowulf-underground.org/pipermail/pvfs2-developers/attachments/20050318/72bf7be6/pvfs2_dynamicbmi.obj