Re: RFS: libpsm2
Hi Mehdi,
On Wed, Dec 20, 2017 at 5:31 PM, Mehdi Dogguy <mehdi@dogguy.org> wrote:
> Hi Brian,
>
> On 08/12/2017 20:15, Brian Smith wrote:
>> libpsm2-2-compat - Preload library to provide PSM2 support for PSM
>> applications
>>
>
> 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
> easier.
>
> 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.
>
> Cheers,
>
> --
> Mehdi
Reply to: