On 29 August 2007 at 11:34, Manuel Prinz wrote:
| Am Montag, den 27.08.2007, 08:14 -0500 schrieb Dirk Eddelbuettel:
| > I'd vote for actually 'Debian patching' the config file to set ^openib, and
| > adding a line there "comment out if you have infinit band and want to use 
it".
| 
| I checked in a patch that modifies the config and documents the issue in
| README.Debian. I wasn't able to test it yet, so I'm not tagging this bug
| as pending.
| 
| Nevertheless, while patching I started not feeling very comfortable with
| this, for the following reason: If one has InfiniBand, it will not be
| working anymore, maybe without warning. To enable it again, one has to
| do so on all nodes. If one doesn't have InfiniBand, all one gets is a
| quite annoying warning but that doesn't influence the behavior at all.
| 
| So the question I asked myself is: Isn't it more reasonable to comment
| out the configuration setting and add a line "uncomment if you want to
| shut up libibverbs". This would not influence the runtime behavior at
| all. Opinions?

It's tricky.  We could also use a debconf question once, and remember the
value to rewrite the conf file, but maybe that makes life to complicated.

Empirically thinking, I'd say leave the patch as we have yet to meet actual
infiniband users.  But we need the big fat warning somewhere. README.Debian
is a start.  In a way, though, the problem is with the ibverbs library, and I
seem to recall that its upstream author told me he'd change the behaviour.
But I can't recall the details now...

Dirk 

-- 
Three out of two people have difficulties with fractions.


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to