[openib-general] [PATCH][RFC] kDAPL: remove dat wrapper funct ion dat_ia_query()

Itamar Rabenstein itamar at mellanox.co.il
Thu Jun 9 22:33:44 PDT 2005


 
>> Does that sound reasonable?

>Ok, I can wait.  I definitely understand the need to stabilize the
>provider and not introduce any new bugs.
>
>I will start playing around again with getting rid of dapl's own linked
>lists.
>
>-tduffy 

Hi All,

James,
 can you close the issue of merging files into one file ?
it is hard to create a big patch when you know that the file you are working
on
is going to be deleted or merged with other file.
I think we only left ia and cr 
I think that sp+psp+rsp should be one file dapl_sp.c

Tom, 
dapl has many lists and I think that most of them should be kernel lists.
I think that the following list :
in struct dapl_sp : struct dapl_llist_entry *cr_list_head;	/* CR
pending queue */
should be replaced with kernel rb tree because 
we search in this list in the hot path of the connect-disconnect flow.

we have agreed in the past that evd ring buffer should be replaced also with
kernel lists.

I will prepare a patch that will remove dapl_hash (no need for this hash).

 Itamar
  



More information about the general mailing list