Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   HMS Incident Management <Incidentmanagement@hms.se>
To:     "mkl@pengutronix.de" <mkl@pengutronix.de>,
        "linux-can@vger.kernel.org" <linux-can@vger.kernel.org>,
        "Thomas.Kopp@microchip.com" <Thomas.Kopp@microchip.com>
CC:     "socketcan@hartkopp.net" <socketcan@hartkopp.net>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "marex@denx.de" <marex@denx.de>,
        "simon.horman@corigine.com" <simon.horman@corigine.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "mailhol.vincent@wanadoo.fr" <mailhol.vincent@wanadoo.fr>
Subject: [PATCH v5 1/3] can: length: fix bitstuffing count
Thread-Topic: [PATCH v5 1/3] can: length: fix bitstuffing count
Thread-Index: AQHZnwB80da4Mh/1cU+tsGKhSv3dyA==
Date:   Wed, 14 Jun 2023 20:40:42 +0000
Message-ID: <PAVPR10MB7209CEA1F5AD12B2E5C8ED86B15AA@PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?BqXCG+HsRuUS1UNaaP4YE39VapRH1uouqTJegB8HsEQ978OUNdVUi86SyH1P?=
 =?us-ascii?Q?RcWZhgesAE1qgMUz3DUZCZy9KqopC/k1pN+dxR5bGjBaj4rhkNba1kkT83HJ?=
 =?us-ascii?Q?cY4VWq9nFzruc4lG2Je3MowWP7UMRU0y89NA/9kXonHBmkWYOXI7dqZPSIZ7?=
 =?us-ascii?Q?KgHdIIR4ryljT8OtkEnaPYL0wB86+wCoBYxx4o9Mv2wDDIU31oeEzlbYMjen?=
 =?us-ascii?Q?tgGDa+IGpNJsRp1GJuwvaJvCCrRORtUoG+gi2hy4oHHUNN1Ng5fhPeCgL115?=
 =?us-ascii?Q?NHg3FoZ9I13cSA/ccbfVgIGQjYsTpk/byJ2QO33THuQOlQztNweBkON52Ap2?=
 =?us-ascii?Q?/02U5WRw503mmMWFkwgbToSF/15IG70WcO8IEDcIvbl2cQktHrqi7Bp0+P9F?=
 =?us-ascii?Q?TZ3gd7V5PZt0FFitgMuweiTj2y8jGxZvQwjjSzVKDF2Eun9kTGKMhnL71yzB?=
 =?us-ascii?Q?peUMQXfEkxnqEwRj70H6kQd8cOCYaYYdXZ5pLaKutIO34xzIcZGdCYBWEftW?=
 =?us-ascii?Q?KB5qY/9nMMC6iuGckn6xucVxQeimEoE9ETNE82+r0kpyM+xAbfRm8q8CNvVb?=
 =?us-ascii?Q?aNSL5JLD3zY7xhQr6HCKLeZlovq1kExejit7DcZC3wiJWZPO/c9Ni1mrvEQx?=
 =?us-ascii?Q?1AKbBeidYsWLO2dNKA7S1zb4qG1HQe2jm8wROFI0ajr9z4mz73hrGBGdrNVf?=
 =?us-ascii?Q?y/ZCZNC1r1seBJjQv+vPqFIJphbjJb98ckU8RrjrJY292xp05QqVpI+oB1ro?=
 =?us-ascii?Q?s2tXRMReWB+Lypl3Rxfzu6X0J0MAgrFwNKFm574AC44WpBWvWqFLo5FFzsOX?=
 =?us-ascii?Q?zwyfHtXl+wdQpiUQHpZRGkZgirWAujfO5hM+wM87UbqmMH3d0XmhpzKLLUD1?=
 =?us-ascii?Q?UQa7Oi6FEH15TpbZYCKzxCBlpYr18uVHhJ/eU2a18V1AVy8+dt29MZ+F60Co?=
 =?us-ascii?Q?VklDJ4RrLd1jj1G/Qfke0ksuBNjDTZ7bqw7Aeb5NtkkFUrfWRNXnQ3ML1kM5?=
 =?us-ascii?Q?xs9SVbv8MFBCRVYXoB6+6k7HWiLsFr7nuGJrXjrQ6S9pN+afocCXQ0+gF/4H?=
 =?us-ascii?Q?1r6Aa2JG0wSAUFhSur6+kkQHstArvAAZTxtPunHx6u+YLMQ42uk6yFn8vFyg?=
 =?us-ascii?Q?Is3lVrbSuScIunvRci7NC+vas/lAzBeO20/jCv0njRy9VugOus+1alsRHOpl?=
 =?us-ascii?Q?mTMfbLhIMnSDoYHeb5nO6W+jd7ngs1szvvZyg4dRXTiVVK8JSi14oi5y0+MU?=
 =?us-ascii?Q?5sBajsWlIXVgLTidxMa6a7tyBv1ea2p8Kt0XpQF4rEyRo4AzzBfWYVGKdS+h?=
 =?us-ascii?Q?ZyJH5FIFZy9szImqsDuOwOqIDOyUhWdAk4CECKzKdH1YHao3ZpK1miav6KdQ?=
 =?us-ascii?Q?O+G7K+wFerQpQeg0HDsoYB6LnlUITCjr/3w26ngWKLlnsKZL8u5hFvYFirtx?=
 =?us-ascii?Q?hHKqXtrTSMBIk5Xp7tNDw66yoTwXRAvXgc+KczIkoU1+cenPJ6o4tRYmC+A1?=
 =?us-ascii?Q?hgblzNRDR440v+RqnhtFyyDsDdH/wZeZeTpYGxNvnSOS1ZIAesmAtDig1W3r?=
 =?us-ascii?Q?Gi+8aGEkQUt+n376xBW2i+AX4kb0WYkLAq5usvg0Q3FHv4RZ0HjN1K2QFMHa?=
 =?us-ascii?Q?uc3QBZ1OGIUop/GItYOGZ3DQnQm+azoYEZdlqw55/YaQ?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: hms.se
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: 96670e17-0a6f-4aa8-faf5-08db6d179f6e
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jun 2023 20:40:42.9112
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5c7c1590-4488-4e42-bc9c-15218f8ac994
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: w6c2URbV/sIOG45KA42N+RLD3jvnbPav/sUgGxJimWvGTsANd+VX33yS4WhJTmf0gYy901afUh9RBqQhp4erlt5C2WU/eElRIkuOfST1EdOUCBPYAHiCpTzJTvdo/5xXO6nn2h/vD/Xc+0nJeHEmbQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS2PR10MB7226
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

**We apologize for the delay in delivering this email, which was caused by =
a mail incident that occurred over the weekend on June 10th. This email was=
 originally sent from vincent.mailhol@gmail.com on 06/11/2023 02:58:08=20

The Stuff Bit Count is always coded on 4 bits [1]. Update the Stuff
Bit Count size accordingly.

In addition, the CRC fields of CAN FD Frames contain stuff bits at
fixed positions called fixed stuff bits [2]. The CRC field starts with
a fixed stuff bit and then has another fixed stuff bit after each
fourth bit [2], which allows us to derive this formula:

  FSB count =3D 1 + round_down(len(CRC field)/4)

The length of the CRC field is [1]:

  len(CRC field) =3D len(Stuff Bit Count) + len(CRC)
                 =3D 4 + len(CRC)

with len(CRC) either 17 or 21 bits depending of the payload length.

In conclusion, for CRC17:

  FSB count =3D 1 + round_down((4 + 17)/4)
            =3D 6

and for CRC 21:

  FSB count =3D 1 + round_down((4 + 21)/4)
            =3D 7

Add a Fixed Stuff bits (FSB) field with above values and update
CANFD_FRAME_OVERHEAD_SFF and CANFD_FRAME_OVERHEAD_EFF accordingly.

[1] ISO 11898-1:2015 section 10.4.2.6 "CRC field":

  The CRC field shall contain the CRC sequence followed by a recessive
  CRC delimiter. For FD Frames, the CRC field shall also contain the
  stuff count.

  Stuff count

  If FD Frames, the stuff count shall be at the beginning of the CRC
  field. It shall consist of the stuff bit count modulo 8 in a 3-bit
  gray code followed by a parity bit [...]

[2] ISO 11898-1:2015 paragraph 10.5 "Frame coding":

  In the CRC field of FD Frames, the stuff bits shall be inserted at
  fixed positions; they are called fixed stuff bits. There shall be a
  fixed stuff bit before the first bit of the stuff count, even if the
  last bits of the preceding field are a sequence of five consecutive
  bits of identical value, there shall be only the fixed stuff bit,
  there shall not be two consecutive stuff bits. A further fixed stuff
  bit shall be inserted after each fourth bit of the CRC field [...]

Fixes: 85d99c3e2a13 ("can: length: can_skb_get_frame_len(): introduce funct=
ion to get data length of frame in data link layer")
Suggested-by: Thomas Kopp=20
Signed-off-by: Vincent Mailhol=20
Reviewed-by: Thomas Kopp=20
---
 include/linux/can/length.h | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/include/linux/can/length.h b/include/linux/can/length.h
index 69336549d24f..b8c12c83bc51 100644
--- a/include/linux/can/length.h
+++ b/include/linux/can/length.h
@@ -72,17 +72,18 @@
  * Error Status Indicator (ESI)		1
  * Data length code (DLC)		4
  * Data field				0...512
- * Stuff Bit Count (SBC)		0...16: 4 20...64:5
+ * Stuff Bit Count (SBC)		4
  * CRC					0...16: 17 20...64:21
  * CRC delimiter (CD)			1
+ * Fixed Stuff bits (FSB)		0...16: 6 20...64:7
  * ACK slot (AS)			1
  * ACK delimiter (AD)			1
  * End-of-frame (EOF)			7
  * Inter frame spacing			3
  *
- * assuming CRC21, rounded up and ignoring bitstuffing
+ * assuming CRC21, rounded up and ignoring dynamic bitstuffing
  */
-#define CANFD_FRAME_OVERHEAD_SFF DIV_ROUND_UP(61, 8)
+#define CANFD_FRAME_OVERHEAD_SFF DIV_ROUND_UP(67, 8)
=20
 /*
  * Size of a CAN-FD Extended Frame
@@ -101,17 +102,18 @@
  * Error Status Indicator (ESI)		1
  * Data length code (DLC)		4
  * Data field				0...512
- * Stuff Bit Count (SBC)		0...16: 4 20...64:5
+ * Stuff Bit Count (SBC)		4
  * CRC					0...16: 17 20...64:21
  * CRC delimiter (CD)			1
+ * Fixed Stuff bits (FSB)		0...16: 6 20...64:7
  * ACK slot (AS)			1
  * ACK delimiter (AD)			1
  * End-of-frame (EOF)			7
  * Inter frame spacing			3
  *
- * assuming CRC21, rounded up and ignoring bitstuffing
+ * assuming CRC21, rounded up and ignoring dynamic bitstuffing
  */
-#define CANFD_FRAME_OVERHEAD_EFF DIV_ROUND_UP(80, 8)
+#define CANFD_FRAME_OVERHEAD_EFF DIV_ROUND_UP(86, 8)
=20
 /*
  * Maximum size of a Classical CAN frame
--=20
2.39.3=