Messages in this thread | | | Subject | Re: [net PATCH] octeontx2-pf: Fix resource leakage in VF driver unbind | From | Paolo Abeni <> | Date | Tue, 10 Jan 2023 11:43:28 +0100 |
| |
Hello,
On Tue, 2023-01-10 at 12:29 +0200, Leon Romanovsky wrote: > On Tue, Jan 10, 2023 at 10:20:15AM +0000, patchwork-bot+netdevbpf@kernel.org wrote: > > Hello: > > > > This patch was applied to netdev/net.git (master) > > by Paolo Abeni <pabeni@redhat.com>: > > > > On Mon, 9 Jan 2023 11:43:25 +0530 you wrote: > > > resources allocated like mcam entries to support the Ntuple feature > > > and hash tables for the tc feature are not getting freed in driver > > > unbind. This patch fixes the issue. > > > > > > Fixes: 2da489432747 ("octeontx2-pf: devlink params support to set mcam entry count") > > > Signed-off-by: Hariprasad Kelam <hkelam@marvell.com> > > > Signed-off-by: Sunil Kovvuri Goutham <sgoutham@marvell.com> > > > > > > [...] > > > > Here is the summary with links: > > - [net] octeontx2-pf: Fix resource leakage in VF driver unbind > > https://git.kernel.org/netdev/net/c/53da7aec3298 > > I don't think that this patch should be applied. > > It looks like wrong Fixes to me and I don't see clearly how structures > were allocated on VF which were cleared in this patch.
My understanding is that the resource allocation happens via:
otx2_dl_mcam_count_set()
which is registered as the devlink parameter write operation on the vf by the fixes commit - the patch looks legit to me.
Did I miss something?
Thanks!
Paolo
| |