7 Sep
2022
7 Sep
'22
1:52 a.m.
Hi Mark
Current snd_soc_dai_link_event_pre_pmu() is using kzalloc() / kfree() for "params", but it is fixed size, and not used on private data. It is used for setup rtd at end of this function, just a local variable. We don't need to use kzalloc() / kfree() for it. This patch replace it as local variable.
The reason we're allocating it dynamically is that it's quite large (608 bytes on arm64) and is starting to get a bit excessive for allocation on the stack, especially when you're building with some of the KASAN type stuff that increases stack usage.
Oh, I see. Thank you for clearing the reason. I think it is very nice to have such comment on the code.
Thank you for your help !!
Best regards --- Kuninori Morimoto