[ofa-general] Re: [PATCH 3/7] Move nodenamemap out of infiniband-diags into libosmcomp

[Sasha Khapyorsky]

On 08:21 Fri 09 Nov     , Ira Weiny wrote:
> On Fri, 9 Nov 2007 13:46:42 +0200
> Sasha Khapyorsky <sashak at voltaire.com> wrote:
> 
> > On 19:16 Wed 07 Nov     , Ira Weiny wrote:
> > > > 
> > > > > Perhaps <sysconfdir>/ib-node-name-map?
> > > > 
> > > > Or under /etc/ofa/ ?
> > > 
> > > I think <sysconfdir> is appropriate.  OFED can specify /etc/ofa if they wish.
> > 
> > But then they cannot do it in configure time.
> > 
> > I think I will apply the patch as is now. And later I will add something
> > like --opensm-config-dir for all config files, not just for node name
> > map.
> > 
> 
> I don't understand what you mean?  They can use:
> 
>   --sysconfdir=DIR       read-only single-machine data [PREFIX/etc]
> 
> This works for me.

Great!

> Perhaps I should change the other config files to use
> sysconfdir?

Yes, that what I thought. But clearly it is general issue and not
related to node name map.

> I have not done so because of legacy reasons, but I think it would
> be more correct, no?

I think so. Or maybe OpenSM own parameter (which will be defaulted to
$sysconfdir/opensm or $syconfdir/ofa) - didn't think about it a lot yet.

Sasha