From: Oliver Hartkopp <socketcan@hartkopp.net>
To: linux-can@vger.kernel.org
Cc: Oliver Hartkopp <socketcan@hartkopp.net>
Subject: [PATCH 0/3] Revert "can: raw: instantly reject unsupported CAN frames"
Date: Fri, 9 Jan 2026 15:41:32 +0100 [thread overview]
Message-ID: <20260109144135.8495-1-socketcan@hartkopp.net> (raw)
This reverts commit 1a620a723853a0f49703c317d52dc6b9602cbaa8
and its follow-up fixes for the introduced dependency issues.
commit 1a620a723853 ("can: raw: instantly reject unsupported CAN frames")
commit cb2dc6d2869a ("can: Kconfig: select CAN driver infrastructure by default")
commit 6abd4577bccc ("can: fix build dependency")
commit 5a5aff6338c0 ("can: fix build dependency")
The reverted patch was accessing CAN device internal data structures
from the network layer because it needs to know about the CAN protocol
capabilities of the CAN devices.
This data access caused build problems between the CAN network and the
CAN driver layer which introduced unwanted Kconfig dependencies and fixes.
The patches 2 & 3 implement a better approach which makes use of the
CAN specific ml_priv data which is accessible from both sides.
With this change the CAN network layer can check the required features
and the decoupling of the driver layer and network layer is restored.
Oliver Hartkopp (3):
Revert "can: raw: instantly reject unsupported CAN frames"
can: propagate CAN device capabilities via ml_priv
can: raw: instantly reject disabled CAN frames
drivers/net/can/Kconfig | 7 +++--
drivers/net/can/Makefile | 2 +-
drivers/net/can/dev/Makefile | 5 ++--
drivers/net/can/dev/dev.c | 27 +++++++++++++++++++
drivers/net/can/dev/netlink.c | 1 +
drivers/net/can/vcan.c | 15 +++++++++++
drivers/net/can/vxcan.c | 15 +++++++++++
include/linux/can/can-ml.h | 24 +++++++++++++++++
include/linux/can/dev.h | 8 +-----
net/can/raw.c | 49 ++++++-----------------------------
10 files changed, 100 insertions(+), 53 deletions(-)
--
2.47.3
next reply other threads:[~2026-01-09 14:45 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-09 14:41 Oliver Hartkopp [this message]
2026-01-09 14:41 ` [PATCH 1/3] Revert "can: raw: instantly reject unsupported CAN frames" Oliver Hartkopp
2026-01-09 14:41 ` [PATCH 2/3] can: propagate CAN device capabilities via ml_priv Oliver Hartkopp
2026-01-09 14:41 ` [PATCH 3/3] can: raw: instantly reject disabled CAN frames Oliver Hartkopp
2026-01-09 15:19 ` [PATCH 0/3] Revert "can: raw: instantly reject unsupported CAN frames" Marc Kleine-Budde
2026-01-09 15:25 ` Marc Kleine-Budde
2026-01-09 15:42 ` Oliver Hartkopp
2026-01-09 15:48 ` Marc Kleine-Budde
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260109144135.8495-1-socketcan@hartkopp.net \
--to=socketcan@hartkopp.net \
--cc=linux-can@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox