[openib-general] RE: [PATCH][kdapl] Integrate dapl_hca_alloc/dapl_hca_free to dapl _provider.c

James Lentini jlentini at netapp.com
Mon Jun 27 15:12:24 PDT 2005


Ok. Removed in revision 2723.

On Sun, 26 Jun 2005, Itamar Rabenstein wrote:

>> uDAPL used the dapl_hca structure's ia_list to cleanup IA
>> resources when the user space process forked. When the code was ported
>> to the kernel, the list was retained, but the cleanup wasn't.
>> Suppose a kDAPL consumer (another kernel module) is unloaded and
>> forgets to close an IA. What do we want to have happen when the kDAPL
>> module is unloaded? Don't we want the resources associated with any
>> open IA's cleaned up?
>>
>> james
>>
>
> NO!!!
>
> If other modules has a bug (forgot to close the IA) we are not going to hide
> it.
> ib_dat_provider module will not be able to go down (be unloaded) until the
> bug will fixed.
> Kernel is not a place where you clean other modules BUGS.
>
> Udapl need this feature not Kdapl.
>
> Please delete this list.
>
> Itamar
>



More information about the general mailing list