On 1/27/25 07:06, Singh, Krishneil K wrote:
-----Original Message-----
From: Tarun K Singh <tarun.k.si...@intel.com>
Sent: Tuesday, November 5, 2024 10:49 AM
To: intel-wired-...@lists.osuosl.org
Cc: net...@vger.kernel.org
Subject: [PATCH iwl-net v1 0/4] fix locking issue
This series fix deadlock issues in the driver. The first patch changes
argument of function 'idpf_vport_ctrl_lock' to adapter. The second patch
renames 'vport_ctrl_lock' to 'vport_cfg_lock'. The first 2 patches make the
third patch easier to review. The third patch fixes the locking issue,
and the fourth patch prevents lockdep from complaining.
Ahmed Zaki (1):
idpf: add lock class key
Tarun K Singh (3):
idpf: Change function argument
idpf: rename vport_ctrl_lock
idpf: Add init, reinit, and deinit control lock
drivers/net/ethernet/intel/idpf/idpf.h | 49 ++++++++----
.../net/ethernet/intel/idpf/idpf_ethtool.c | 59 +++++++-------
drivers/net/ethernet/intel/idpf/idpf_lib.c | 80 +++++++++++++------
drivers/net/ethernet/intel/idpf/idpf_main.c | 39 ++++++---
4 files changed, 149 insertions(+), 78 deletions(-)
--
2.46.0
After additional testing on this [series|patch] we discovered an issue after doing
ifdown while device is in a reset, which can result in a <null pointer
dereference(?)>.
Thank you for the report,
Could you please check if this issue could be reproduced without the
series?
We have yet another patch by Emil, that aims to fix such issues by
detaching the device prior to the reset.
@netdev
I would like to consider adding "in reset" state for the netdev,
to prevent such behavior in more civilized way though.