Re: dlm: use KMEM_CACHE() in dlm_memory_init()

[Markus Elfring]

>>>> https://elixir.bootlin.com/linux/v6.10-rc4/source/fs/dlm/memory.c#L27
>>>> https://elixir.bootlin.com/linux/v6.10-rc4/source/mm/slab_common.c#L362
>>>
>>> probably only for "dlm_cb" that turns into "dlm_callback".
>> …
>>
>> Will the development attention grow for deviations of passed name strings
>> from applied data structure identifiers?
>
> My initial purpose is to replace kmem_cache_create() with KMEM_CACHE().

Do you take any help from advanced analysis tools into account
for such transformation attempts?

Example for the semantic patch language (Coccinelle software):
[PATCH v2] Coccinelle: api: Add SmPL script “use_KMEM_CACHE.cocci”
https://lore.kernel.org/cocci/b08603d6-cac1-4876-a56c-30c680d5dc52@xxxxxx/
https://sympa.inria.fr/sympa/arc/cocci/2024-02/msg00000.html


> In my view，there is no problem in unifying passed name strings and
> structure identifiers.
> Maybe that's wrong，

I suggest to take another look also at feedback which other contributors
(from your Linux software distribution) got for similar change suggestions.


> I will resubmit a patch that does not change "dlm_cb" to "dlm_callback".

Do you see further opportunities to improve the change description accordingly?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.10-rc4#n94

Regards,
Markus