[openib-general] Re: [PATCH] cleanup dat provider registration
James Lentini
jlentini at netapp.com
Mon Jun 27 15:15:35 PDT 2005
On Sun, 26 Jun 2005, Christoph Hellwig wrote:
> On Wed, Jun 22, 2005 at 05:13:03PM -0400, James Lentini wrote:
>>
>> This is an excellent simplification. Committed in revision 2682 with a
>> few minor modifications:
>>
>> - kept printouts in dat_registry_add_provider,
>> dat_registry_remove_provider, and dat_registry_list_providers
>>
>> - updated printout in dat_ia_close (this wasn't something you
>> changed)
>>
>> - removed parens around sizeof
>
> kernel style is to have parants around it, but all of the openib code
> is different. well, let's keep it that way.
>
>> - removed space in front of labels
>
> lots of new kernel code uses the space, but again it's okay to stick
> to the surrounding code.
>
>> The last two are for consistency with the coding style we've been
>> using. If we've deviated from what is acceptable, let us know.
>>
>> Given this simplification, I can think of a few more changes:
>>
>> - rename api.c to registry.c
>
> Note yet. The code will get some major surgery still, and as part of
> that split into different files again maybe, just on very different
> boundaries.
>
>> - remove all "dictionary" references: rename dat_dictionary_search to
>> dat_provider_list_search, rename struct dat_dictionary_entry to
>> struct dat_provider_list_entry, rename
>> dat_dictionary_key_is_equal() to dat_provider_info_is_equal()
>
> I'll plan bigger changes in that area that will kill the dictionary
> term, so a simple search and replace is probably not worth it.
Oh well, I already did the cleanup.
I'll add an item to the TODO list to update the API with a
register/callback interface for accessing the provider.
More information about the general
mailing list