diff options
| author | Michal Schmidt <[email protected]> | 2023-10-19 07:13:46 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2023-10-20 10:45:07 +0000 |
| commit | 7db3111043885c146e795c199d39c3f9042d97c0 (patch) | |
| tree | b3f5a6e72fbbb2698b5cc1d94536f66e568dceb4 /drivers/net/dsa/dsa_loop.c | |
| parent | r8169: fix the KCSAN reported data race in rtl_rx while reading desc->opts1 (diff) | |
| download | kernel-7db3111043885c146e795c199d39c3f9042d97c0.tar.gz kernel-7db3111043885c146e795c199d39c3f9042d97c0.zip | |
iavf: initialize waitqueues before starting watchdog_task
It is not safe to initialize the waitqueues after queueing the
watchdog_task. It will be using them.
The chance of this causing a real problem is very small, because
there will be some sleeping before any of the waitqueues get used.
I got a crash only after inserting an artificial sleep in iavf_probe.
Queue the watchdog_task as the last step in iavf_probe. Add a comment to
prevent repeating the mistake.
Fixes: fe2647ab0c99 ("i40evf: prevent VF close returning before state transitions to DOWN")
Signed-off-by: Michal Schmidt <[email protected]>
Reviewed-by: Paul Menzel <[email protected]>
Reviewed-by: Przemek Kitszel <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'drivers/net/dsa/dsa_loop.c')
0 files changed, 0 insertions, 0 deletions