net/ice: fix DCF memory leaks
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest8 patches 2 repliesIn review
- v18 patches 8 repliesSuperseded
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | net/ice: fix DCF hardware init error pathdrivers/ | +10 | 0 replies | |
| 2/8 | net/ice: fix DCF parent adapter init error pathdrivers/ | +7 | 0 replies | |
| 3/8 | net/ice: fix DCF queue config buffer leakdrivers/ | +2 | 0 replies | |
| 4/8 | net/ice: fix DCF QoS buffer leak on commitdrivers/ | +2 | 0 replies | |
| 5/8 | net/ice: fix DCF QoS bandwidth replay config leakdrivers/ drivers/ drivers/ +1 more | +32 | 1 reply | |
| 6/8 | net/ice: fix DCF TM config leak on closedrivers/ | +4 | 0 replies | |
| 7/8 | net/ice: fix DCF representor info leak on closedrivers/ | +15 | 0 replies | |
| 8/8 | net/ice: fix DCF port leak on probe failuredrivers/ | +20 | 0 replies |
Reviews trailers per patch
All 8 patches have a Reviewed-by or Acked-by.
| Patch | Bruce Richardson |
|---|---|
| 1/8 | A-b |
| 2/8 | A-b |
| 3/8 | A-b |
| 4/8 | A-b |
| 5/8 | A-b |
| 6/8 | A-b |
| 7/8 | A-b |
| 8/8 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v2 0/8] net/ice: fix DCF memory leaks
V2-changes:
- Addressed Bruce Richardson's review comments.
- Patch-1: use rte_intr_callback_unregister_sync() in the new error path.
- Patch-5: fix the re-commit leak by swapping the saved config under the lock the VF reset replay holds.
This patch series fixes memory leaks, and related error path issues, in the ice DCF driver.
1. The VF VSI map leaked on DCF hardware init failure, and a failed VLAN capability query left the interrupt callback registered.
2. A failed DCF parent adapter init leaked the dummy main VSI, and a failed flow init also leaked the DDP parser and flow engine state.
3. The queue config message leaked when the required RXDID was not supported.
4. The temporary QoS bandwidth buffers still leaked on every successful hierarchy commit.
5. The QoS bandwidth config saved for replay leaked on every commit, close and reset.
6. TM nodes and shaper profiles leaked on close when the hierarchy was never committed.
7. The representor info leaked when the DCF port was closed.
8. A failed probe left the DCF port and its representors behind. A representor type other than VF is now rejected before the DCF port is created.
Anurag Mandal (8):
net/ice: fix DCF hardware init error path net/ice: fix DCF parent adapter init error path net/ice: fix DCF queue config buffer leak net/ice: fix DCF QoS buffer leak on commit net/ice: fix DCF QoS bandwidth replay config leak net/ice: fix DCF TM config leak on close net/ice: fix DCF representor info leak on close net/ice: fix DCF port leak on probe failure
drivers/net/intel/ice/ice_dcf.c | 27 ++++++++++++--- drivers/net/intel/ice/ice_dcf_ethdev.c | 48 +++++++++++++++++++------- drivers/net/intel/ice/ice_dcf_ethdev.h | 2 ++ drivers/net/intel/ice/ice_dcf_parent.c | 24 +++++++++++-- drivers/net/intel/ice/ice_dcf_sched.c | 19 +++++----- 5 files changed, 92 insertions(+), 28 deletions(-)
Signature · 1 line
2.25.1