[PATCH v7 0/5] net: can: isotp-fixes
Oliver Hartkopp via B4 Relay <[email protected]>
| Newsgroups | org.kernel.vger.linux-can,org.kernel.feeds.b4-sent |
|---|---|
| Message-ID | <[email protected]> |
As sashiko-bot was not able to check the second patch this bundle is re-posted with b4 preparation. Fixed the remarks reported by sashiko-bot of V6 patchset. Signed-off-by: Oliver Hartkopp <[email protected]> --- Changes in v7: - patch 3: add helper so->tx_gen to monitor new ISOTP_SENDING attempts - patch 4: prevent missasignment of sk_err report with new so->tx_gen - Link to v6: https://patch.msgid.link/[email protected] Changes in v6: - patch 2: check for rx state and tx state to be IDLE in isotp_bind() (sashiko-bot) - patch 3: double check rx/tx states after all hrtimer_cancel() calls (sashiko-bot) move ISOTP_IDLE in err_event_drop for the same reason - patch 4: use READ_ONCE() to prevent compiler load tearing as remarked by sashiko-bot - Link to v5: https://patch.msgid.link/[email protected] Changes in v5: - patch 3: new: state change re-check after hrtimer_cancel() - patch 4: new: handle state transistion with cmpxchg() - Link to v4: https://patch.msgid.link/[email protected] Changes in v4: - patch 2: use a simpler method to reject a re-binding that can be implemented inside the lock: No waitqueue waiting just exit. - Link to v3: https://patch.msgid.link/[email protected] Changes in v3: - patch 2: go back to initial idea without hard resetting the state machine and timers but let these come to a graceful end. Even a new bind() at netdev shutdown now leads to this graceful statemachine shutdown via the wait queue. - Link to v2: https://patch.msgid.link/[email protected] Changes in v2: - patch 2: shutdown state machine to fix sashiko-bot complains Link: https://lore.kernel.org/linux-can/[email protected]/ - Link to v1: https://patch.msgid.link/[email protected] --- Oliver Hartkopp (5): can: isotp: use unconditional synchronize_rcu() in isotp_release() can: isotp: fix use-after-free race with concurrent NETDEV_UNREGISTER can: isotp: fix race between RX/TX timers and frame reception can: isotp: fix lock-free state transition in tx timer handler can: isotp: fix state machine corruption on signal interruption net/can/isotp.c | 176 +++++++++++++++++++++++++++++++++++++++++++------------- 1 file changed, 135 insertions(+), 41 deletions(-) --- base-commit: a635d6748234582ea287c5ffeae28b9b23f91c7e change-id: 20260710-isotp-fixes-ead7480d67bf Best regards, -- Oliver Hartkopp <[email protected]>