[PATCH 02/29] staging/lustre/lnet: Get rid of IOC_LIBCFS_DEBUG_PEER hack
Oleg Drokin
green at linuxhacker.ru
Mon Jan 4 17:48:14 UTC 2016
On Jan 4, 2016, at 12:45 PM, Simmons, James A. wrote:
>> From: Oleg Drokin <green at linuxhacker.ru>
>>
>> IOC_LIBCFS_DEBUG_PEER was added back in the stone ages to print debug
>> statistics on a peer when peer timeout happens.
>> Redo it properly as a separate LNet API call,
>> also get rid of "ioctl" forwarding into the underlying LNDs,
>> since no current LNDs implement this function anymore.
> ..
>> diff --git a/drivers/staging/lustre/include/linux/lnet/api.h b/drivers/staging/lustre/include/linux/lnet/api.h
>> index 75285fd..fa5fad3 100644
>> --- a/drivers/staging/lustre/include/linux/lnet/api.h
>> +++ b/drivers/staging/lustre/include/linux/lnet/api.h
>> @@ -197,6 +197,7 @@ int LNetGet(lnet_nid_t
>> int LNetSetLazyPortal(int portal);
>> int LNetClearLazyPortal(int portal);
>> int LNetCtl(unsigned int cmd, void *arg);
>> +void LNetDebugPeer(lnet_process_id_t id);
>
> We don't need this one line camel case wrapper.
>
>>
>> /** @} lnet_misc */
>>
> ...
>> @@ -1436,6 +1406,12 @@ LNetCtl(unsigned int cmd, void *arg)
>> }
>> EXPORT_SYMBOL(LNetCtl);
>>
>> +void LNetDebugPeer(lnet_process_id_t id)
>> +{
>> + lnet_debug_peer(id.nid);
>> +}
>> +EXPORT_SYMBOL(LNetDebugPeer);
>
> Lets just use lnet_debug_peer directly.
It's not exported and this is not how we export functionality out of the LNet.
The way I did it is the way LNet api is exposed. Why break the custom all of a sudden and
replace one hack with another?
More information about the devel
mailing list