On Mon, Aug 12, 2024 at 03:44:55PM +0200, Alexander Lobakin wrote: > From: Michal Kubiak <[email protected]> > > commit 3cc88e8405b8d55e0ff035e31971aadd6baee2b6 upstream. > > The initialization of vport interrupt consists of two functions: > 1) idpf_vport_intr_init() where a generic configuration is done > 2) idpf_vport_intr_req_irq() where the irq for each q_vector is > requested. > > The first function used to create a base name for each interrupt using > "kasprintf()" call. Unfortunately, although that call allocated memory > for a text buffer, that memory was never released. > > Fix this by removing creating the interrupt base name in 1). > Instead, always create a full interrupt name in the function 2), because > there is no need to create a base name separately, considering that the > function 2) is never called out of idpf_vport_intr_init() context. > > Fixes: d4d558718266 ("idpf: initialize interrupts and enable vport") > Cc: [email protected] # 6.7 > Signed-off-by: Michal Kubiak <[email protected]> > Reviewed-by: Pavan Kumar Linga <[email protected]> > Signed-off-by: Alexander Lobakin <[email protected]> > Reviewed-by: Simon Horman <[email protected]> > Tested-by: Krishneil Singh <[email protected]> > Signed-off-by: Tony Nguyen <[email protected]> > Link: > https://patch.msgid.link/[email protected] > Signed-off-by: Jakub Kicinski <[email protected]> > --- > drivers/net/ethernet/intel/idpf/idpf_txrx.c | 19 ++++++++----------- > 1 file changed, 8 insertions(+), 11 deletions(-) >
Now queued up, thanks. greg k-h
