Re: RFS: libpsm2
On Wed, Dec 20, 2017 at 5:31 PM, Mehdi Dogguy <firstname.lastname@example.org> wrote:
> Hi Brian,
> On 08/12/2017 20:15, Brian Smith wrote:
>> libpsm2-2-compat - Preload library to provide PSM2 support for PSM
> Wouldn't it be better if this shared library is handled using alternatives?
> This will allow a user to take advantage of this library without the need
> to fiddle with environment variables and makes its deployment even more
> My proposal is implemented in the patch attached. I've used this on an HPC
> cluster and found it neat. It we agree on this, I can add it to infinipath-psm
> package as well.
I'll take a look at testing the alternatives patch you provided.
> Besides, libpsm2-2 and libpsm2-2-compat provide udev rules but do not call
> "udevadm control --reload-rules >/dev/null 2>&1" in postinst.
> Is udev auto-detecting new rules now?
libpsm2-2's 40-psm.rules sets the permissions for the hfi1 devices to
0666. This appears to be unnecessary, as the hfi1 module now has code
to set the 0666 permission in the hfi1_devnode callback. Therefore, I
believe that 40-psm.rules should be removed from libpsm2-2.
40-psm-compat.rules creates the /dev/ipath symlink, so it is still
needed by libpsm2-2-compat. I will take a look at that in concert with
testing the alternatives patch.