Received-SPF: pass (google.com: domain of linux-kernel+bounces-157911-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: "Trexel, Ed" <ed.trexel@hp.com>
To: Ravi Gunasekaran <r-gunasekaran@ti.com>, "s-vadapalli@ti.com"
	<s-vadapalli@ti.com>, "rogerq@kernel.org" <rogerq@kernel.org>
CC: "davem@davemloft.net" <davem@davemloft.net>, "edumazet@google.com"
	<edumazet@google.com>, "kuba@kernel.org" <kuba@kernel.org>,
	"pabeni@redhat.com" <pabeni@redhat.com>, "richardcochran@gmail.com"
	<richardcochran@gmail.com>, "jreeder@ti.com" <jreeder@ti.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "srk@ti.com"
	<srk@ti.com>
Subject: RE: [PATCH net v2] net: ethernet: ti: am65-cpts: Fix PTPv1 message
 type on TX packets
Thread-Topic: [PATCH net v2] net: ethernet: ti: am65-cpts: Fix PTPv1 message
 type on TX packets
Thread-Index: AQHalhdcGrDqLZJbJk++v+oR/kZMA7F4NTNg
Date: Thu, 25 Apr 2024 01:36:17 +0000
Message-ID: <MW5PR84MB157272BC110ED170C35983F790172@MW5PR84MB1572.NAMPRD84.PROD.OUTLOOK.COM>
References: <20240424071626.32558-1-r-gunasekaran@ti.com>
In-Reply-To: <20240424071626.32558-1-r-gunasekaran@ti.com>
Accept-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?TUT/t+CFEwrZIdCHUqCETKv+NiE4Wqml7fBkYEpyXTBl1nRpbNt2GFwqsbBU?=
 =?us-ascii?Q?WB29uK0XX9Lni1S391aQnLIWJEVPfrnDTzErpInQknSryJo4ZdgEpFk1i6dh?=
 =?us-ascii?Q?aPJg4/7XNON92Un7PMU3Ax7zdpVb8nDZhQ16xqpCidmW3xgtRcmfsQV/jaAa?=
 =?us-ascii?Q?cOp4Tc04fd5MZtvoqYCrswdREEF/KsjGOWlVLgri1+KnQm2C2P69SoMaa8Dk?=
 =?us-ascii?Q?w2j5H813CZU6CNOP+p38vWFvGN+goXiUBGD2YFehMQCH82Ab5ol2uIYUSlM4?=
 =?us-ascii?Q?ntpoe75TsCSmd09L1Ae0kWOFzoYm8ix9VwA3W1qPpWz2mLVfG5i0RnkAUZsy?=
 =?us-ascii?Q?4J0JRqnTU/2NkcS6LDklhhyVXm7IFnpLm8o7PnRmrWu3xDt7zGiuOYNaW7Wv?=
 =?us-ascii?Q?kFIuPnmHvvFi2qiZ8VLEggZJ410aqO8xh3/PEOnPB7FoL7R2O0lLmW9tXu8x?=
 =?us-ascii?Q?n7sa8wEJyyAahnHOPWnPa7GH8sMcPLafty+Y7V9toh+5bZJPXUgOK4PZXiPb?=
 =?us-ascii?Q?QhIfzFY6mBzCXCidAAfP+mvx6pvXa87q3kI5T0jmorrAklXvia53G+9nnten?=
 =?us-ascii?Q?qf5hyFVQQ5tbamWLKi1lvtEb9XL2OAcnZyHsU4GY/RmIu8RtJYKdbydZocqk?=
 =?us-ascii?Q?fO5rNA47YT3MKCgKLIgAKa8nvDF75q7/96OwHxCcIic7Yt1vAJ+3FrGjabpN?=
 =?us-ascii?Q?9/c7e7U5GsW22vyNU68J9ivgWXsT4hEsbUr/eA8eZApg1+qg0TNgVGhZ3q4H?=
 =?us-ascii?Q?EUoDO4vKt+ufgbW6myTj60wnnm8pO2UM49ao641Ss0NjpTPh7DyB60FE7yw7?=
 =?us-ascii?Q?4F65nkZ8wxw8lWUfcU1F4JaRNl6qwniiC2LE0xYsJnESSE59AeTGdFiXQjQM?=
 =?us-ascii?Q?+UpQPeh8BNHWFzzXrEcFja5goiuoRtN4H1SGoISiS1abp/S+Ox2HEIOR6cEx?=
 =?us-ascii?Q?OTQKrt1JsWRybGFI23Bc0OgDwSW7wJ613j3xFXk8X6j5ofh+oAqdwtwi/8xO?=
 =?us-ascii?Q?QlrK2qHOFn8L96qMv7N3vsI960Ts4EVPzxASRmOVW9oxjP2trT2isyebYTv6?=
 =?us-ascii?Q?DzGrkIHC6KScURP0spbKbY108uPnqyzcgae3cpW9gIiAPE4SCR9ThaTjTj7S?=
 =?us-ascii?Q?QWX9KbfLtEROOD2JWSJps1IloDC+W736EDv/AvQ8cwO1O46gL0NQlikUlBLs?=
 =?us-ascii?Q?yzDC99dNm058EEuv7PEHO/XGnAxPIbj5+SaVlYCVrv/C3/qUBNd0myrcbZ9a?=
 =?us-ascii?Q?E/SfH6jGH2tmoutziKZfsuB59S5WxOm1sfvs2g/XI+vE3EmDRPcL6CwBiO2y?=
 =?us-ascii?Q?i0XqVLemsijEsisWUFGw/YMIJu11h1kSKZHXLcz3HyldH6Yc60efhn8FYxP1?=
 =?us-ascii?Q?hR0h5cK125RuGRIdcHOXjoUHkOAvzZzRFJ+PBhwMQcrurewVefY12B12quPY?=
 =?us-ascii?Q?eP2vg6wOjnGjc8HodiBezl5jlBJjcFxGL+mmLtduMMlk1ox3Rh+PVu6obKTe?=
 =?us-ascii?Q?ZOv35PxJXVtyyn3IC+rlus1PFtbDmMQpDYYTJpVat/Pi8LXRZeV1r8vP/fkV?=
 =?us-ascii?Q?XX9XLuScjCxufCHjFTQ=3D?=
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: hp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MW5PR84MB1572.NAMPRD84.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: 48d69516-007f-44d1-0005-08dc64c81a53
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Apr 2024 01:36:17.7353
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: ca7981a2-785a-463d-b82a-3db87dfc3ce6
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 1RtaIFC76cQM0i2VwW2KCOwBk+KAkZ5ygsMODgEbQMMY3qb8K4jMlPOcAmzi269WjvSx6v1QqqvzNXfOgsMRUA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: EA2PR84MB3376
X-Mimecast-Spam-Score: 0
X-Mimecast-Originator: hp.com
Content-Language: en-US
Content-Type: text/plain; charset=WINDOWS-1252
Content-Transfer-Encoding: quoted-printable

From: Ravi Gunasekaran <r-gunasekaran@ti.com>=20
Sent: Wednesday, April 24, 2024 1:16 AM
To: s-vadapalli@ti.com; rogerq@kernel.org; r-gunasekaran@ti.com
Cc: davem@davemloft.net; edumazet@google.com; kuba@kernel.org; pabeni@redha=
t.com; richardcochran@gmail.com; jreeder@ti.com; netdev@vger.kernel.org; li=
nux-kernel@vger.kernel.org; srk@ti.com; Trexel, Ed <ed.trexel@hp.com>
Subject: [PATCH net v2] net: ethernet: ti: am65-cpts: Fix PTPv1 message typ=
e on TX packets

CAUTION: External Email=20
From: Jason Reeder <mailto:jreeder@ti.com>

The CPTS, by design, captures the messageType (Sync, Delay_Req, etc.)
field from the second nibble of the PTP header which is defined in the
PTPv2 (1588-2008) specification. In the PTPv1 (1588-2002) specification
the first two bytes of the PTP header are defined as the versionType
which is always 0x0001. This means that any PTPv1 packets that are
tagged for TX timestamping by the CPTS will have their messageType set
to 0x0 which corresponds to a Sync message type. This causes issues
when a PTPv1 stack is expecting a Delay_Req (messageType: 0x1)
timestamp that never appears.

Fix this by checking if the ptp_class of the timestamped TX packet is
PTP_CLASS_V1 and then matching the PTP sequence ID to the stored
sequence ID in the skb->cb data structure. If the sequence IDs match
and the packet is of type PTPv1 then there is a chance that the
messageType has been incorrectly stored by the CPTS so overwrite the
messageType stored by the CPTS with the messageType from the skb->cb
data structure. This allows the PTPv1 stack to receive TX timestamps
for Delay_Req packets which are necessary to lock onto a PTP Leader.

Fixes: f6bd59526ca5 ("net: ethernet: ti: introduce am654 common platform ti=
me sync driver")
Signed-off-by: Jason Reeder <mailto:jreeder@ti.com>
Signed-off-by: Ravi Gunasekaran <mailto:r-gunasekaran@ti.com>
Tested-by: Ed Trexel <ed.trexel@hp.com>
---
Changes since v1:
-----------------
* Added Fixes tag as per Paolo's suggestion
* Rebased to latest tip

v1: https://lore.kernel.org/all/20240419080547.10682-1-r-gunasekaran@ti.com

drivers/net/ethernet/ti/am65-cpts.c | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/drivers/net/ethernet/ti/am65-cpts.c b/drivers/net/ethernet/ti/=
am65-cpts.c
index c66618d91c28..f89716b1cfb6 100644
--- a/drivers/net/ethernet/ti/am65-cpts.c
+++ b/drivers/net/ethernet/ti/am65-cpts.c
@@ -784,6 +784,11 @@ static bool am65_cpts_match_tx_ts(struct am65_cpts *cp=
ts,
struct am65_cpts_skb_cb_data *skb_cb =3D
(struct am65_cpts_skb_cb_data *)skb->cb;

+ if ((ptp_classify_raw(skb) & PTP_CLASS_V1) &&
+ ((mtype_seqid & AM65_CPTS_EVENT_1_SEQUENCE_ID_MASK) =3D=3D
+ (skb_cb->skb_mtype_seqid & AM65_CPTS_EVENT_1_SEQUENCE_ID_MASK)))
+ mtype_seqid =3D skb_cb->skb_mtype_seqid;
+
if (mtype_seqid =3D=3D skb_cb->skb_mtype_seqid) {
u64 ns =3D event->timestamp;


base-commit: a59668a9397e7245b26e9be85d23f242ff757ae8
--=20
2.17.1