Hi Goldwyn,
Goldwyn Rodrigues wrote:
> On 08/03/2015 06:31 AM, Guoqing Jiang wrote:
>> Let libcmap lib and related funs also only need one-time
>> setup during mdadm running period.
>>
>> Signed-off-by: Guoqing Jiang <gqjiang@xxxxxxxx>
>> ---
>> mdadm.c | 23 +++++++++++--------
>> mdadm.h | 20 ++++++++++++++++
>> util.c | 81
>> +++++++++++++++++++++++++++++++++++++++--------------------------
>> 3 files changed, 82 insertions(+), 42 deletions(-)
>>
[snip]
>> +
>> +void set_hookers(void)
>> +{
>> + set_dlm_hookers();
>> + set_cmap_hookers();
>> +}
>> +
>> +void free_hookers(void)
>> +{
>> + free_dlm_hookers();
>> + free_cmap_hookers();
>> +}
>>
>
> You could put both functions in a single structure called cluster_hooks.
>
Yes, it could be. If there more libs are needed in future, so I prefer
separate them according to different libraries.
Thanks,
Guoqing
--
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
