Messages in this thread | | | From | Kalle Valo <> | Subject | Re: [PATCH] ath10k: allocate dummy net_device dynamically | Date | Wed, 20 Mar 2024 17:25:52 +0200 |
| |
Jeff Johnson <quic_jjohnson@quicinc.com> writes:
> On 3/19/2024 3:47 AM, Breno Leitao wrote: >> Embedding net_device into structures prohibits the usage of flexible >> arrays in the net_device structure. For more details, see the discussion >> at [1]. >> >> Un-embed the net_device from struct ath10k by converting it >> into a pointer. Then use the leverage alloc_netdev() to allocate the >> net_device object at ath10k_core_create(). The free of the device occurs >> at ath10k_core_destroy(). >> >> [1] https://lore.kernel.org/all/20240229225910.79e224cf@kernel.org/ >> >> Signed-off-by: Breno Leitao <leitao@debian.org> > > NAK this based upon the ath11k patch results. > > As suggested there we should just use kmalloc/kfree to match the existing logic.
BTW if the patch is not tested on a real device then it's good to document that in the commit message with "Compile tested only" or similar.
-- https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
| |