[PATCH]: ipoib with mlx4 initialisation ordering

[Hans Petter Selasky]

On 2020-02-26 22:30, Andreas Kempe wrote:
> On Wed, Feb 26, 2020 at 10:08:09PM +0100, Hans Petter Selasky wrote:
>> On 2020-02-26 22:05, Andreas Kempe wrote:
>>> +module_exit_order(ipoib_cleanup_module, SI_ORDER_FIRST);
>>
>> Try:
>>
>> module_exit_order(ipoib_cleanup_module, SI_ORDER_FOURTH);
>>
> 
> Tried it, but the machine still hung. See patch:
> 
> Index: sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_main.c
> ===================================================================
> --- sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_main.c	(revision 356611)
> +++ sys/ofed/drivers/infiniband/ulp/ipoib/ipoib_main.c	(working copy)
> @@ -1739,7 +1739,7 @@
>   }
>   
>   module_init(ipoib_init_module);
> -module_exit(ipoib_cleanup_module);
> +module_exit_order(ipoib_cleanup_module, SI_ORDER_FOURTH);
>   
>   static int
>   ipoib_evhand(module_t mod, int event, void *arg)
> 
>>
>> Fourth is first,
>>
>> first is last.
>>
>> for exit.
>>
> 

I haven't yet found time to reproduce this issue.

Possibly you're right that the list order matters.

--HPS