Received-SPF: pass (google.com: domain of linux-kernel+bounces-79184-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <fb8c6ec5-05d0-46cc-8bb4-b97b3d77c02b@intel.com>
Date: Fri, 23 Feb 2024 13:12:52 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH RFC net-next v1 0/6] ethtool HW timestamping statistics
Content-Language: en-US
From: Jacob Keller <jacob.e.keller@intel.com>
To: Rahul Rameshbabu <rrameshbabu@nvidia.com>, Saeed Mahameed
	<saeed@kernel.org>, Leon Romanovsky <leon@kernel.org>, "David S. Miller"
	<davem@davemloft.net>, Eric Dumazet <edumazet@google.com>, Jakub Kicinski
	<kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>, Jonathan Corbet
	<corbet@lwn.net>, Richard Cochran <richardcochran@gmail.com>, Tariq Toukan
	<tariqt@nvidia.com>, Gal Pressman <gal@nvidia.com>, Vadim Fedorenko
	<vadim.fedorenko@linux.dev>, Andrew Lunn <andrew@lunn.ch>, Heiner Kallweit
	<hkallweit1@gmail.com>, Przemek Kitszel <przemyslaw.kitszel@intel.com>,
	"Ahmed Zaki" <ahmed.zaki@intel.com>, Alexander Lobakin
	<aleksander.lobakin@intel.com>, Hangbin Liu <liuhangbin@gmail.com>, "Paul
 Greenwalt" <paul.greenwalt@intel.com>, Justin Stitt <justinstitt@google.com>,
	Randy Dunlap <rdunlap@infradead.org>, Maxime Chevallier
	<maxime.chevallier@bootlin.com>, Kory Maincent <kory.maincent@bootlin.com>,
	Wojciech Drewek <wojciech.drewek@intel.com>, Vladimir Oltean
	<vladimir.oltean@nxp.com>, Jiri Pirko <jiri@resnulli.us>, Alexandre Torgue
	<alexandre.torgue@foss.st.com>, Jose Abreu <joabreu@synopsys.com>, "Dragos
 Tatulea" <dtatulea@nvidia.com>
CC: <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-doc@vger.kernel.org>
References: <20240223192658.45893-1-rrameshbabu@nvidia.com>
 <4c94d725-a903-4979-bcc6-9353e0869cd7@intel.com>
In-Reply-To: <4c94d725-a903-4979-bcc6-9353e0869cd7@intel.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?T2c3QVc4dDdNNE11bXFaVklNTEYzV3ErTFZwRGVXaWcvVFR6M1lNb210YUM2?=
 =?utf-8?B?MHdPTEthTXVTaUhJdFFpTEd3NWJNMllzamRueVc1bDBTUzNaZjZRRDdIQ2Uz?=
 =?utf-8?B?Zk54VWxVUzdCSUNzeFJWMVpqemZXTjZOMTk5K1c3T2Q4SXIzU1JPNitsSHR1?=
 =?utf-8?B?MUhXOEE1ZEpxc3FMbmlRMHcvLzA5M2xNbjU4WGptL25SRUdtcVZKZmVDOWox?=
 =?utf-8?B?VGhwMTRsWlJwdXlqZU0wR0M2WlZzKzlNMEFEQUx0Qi9lc0I4SDArUnV0azho?=
 =?utf-8?B?Q2VJZlpCNEtTUVE1dDhZbWJWcWdNS25rRVd5elZqV1R3bnN6Q3lXNzBucXdK?=
 =?utf-8?B?N016ZkVOVWVFenhFalZxTWg4Nk1QcDRmcDdNRW5kYW9STm8rV3ZURHRaSHNQ?=
 =?utf-8?B?U0gxSXN5Mks2bFNQNEI2MVp6Z01TYTR6NlVmSUNZYk9DWkYzVVAwaHViNDdt?=
 =?utf-8?B?N2FIRmVJY3NsTjdPbE1kcU1QOXVOa2ZTdUJMN3E0YTEzd255WjNWNUZoamJS?=
 =?utf-8?B?b3ZQRWx6RHlCdEhpaC9ucHlxbXNCUTVSbGJFOHA4M1BrODJmVTcyMG9jTXFv?=
 =?utf-8?B?Y0RIN3Z3MDdHbXpialIzMmEvYzZXNEh6d2xJWG9qakRBRFozQjJ4NlpSakNh?=
 =?utf-8?B?ZDdLTkpNM1FqcWFlZWRoR3ZHSVJyWVpnd3Z1SlRGL1ZjLy9iNXo4VUwzSWtL?=
 =?utf-8?B?T1JJTk5lckVwODJqdGhnWFJpUUtqaDI5N3hPZFA2ZzE3bmlYVTVMd1ViSTlK?=
 =?utf-8?B?QTdLb2t2Q1BSa1BLcWtCSm82WWRmUVl0aER6QXVCeW1jVWRtVE9TK1dMZEdP?=
 =?utf-8?B?N29PVit3SUZUekU3TzZ1WWk2ck5PR0tZTFZuNVJUbUlSY0o0a0hVNDdBYlJG?=
 =?utf-8?B?ejBzYzJwZlExYm44RHF1ekRiK3RmbDlCeENjdHpRZER3Zjk3blZveGtkcXg5?=
 =?utf-8?B?VFlSbU1LYWhIazdLckd6NG9wWk1FcVh3V1Bsb0tCeW5DWURlcjJ3UkFaeGFq?=
 =?utf-8?B?eGp6QWdxU2lFUUxySDgxS1FMcVVxSS9vM3g4eVYzdkI0N29lMDlTcjZPYWZX?=
 =?utf-8?B?MnlzaUxOSkQwWlgvaklyVDl1eGh4K053WmozRHVEai82WDFpN2RoTHNDYUVn?=
 =?utf-8?B?OTNqd3E1TGFLaFpzUExYRkI5U0RQUkd3SmFKaFBtZ3IzNDRIcFR6NE9GRVhE?=
 =?utf-8?B?TG40RHZrZkErYXhqWE9HMFlHeGJIMXNKMmZrMmw0c0tmSzBXZXRQZEZ5akhZ?=
 =?utf-8?B?Sk02M056Ry9sTzFFdFRXdVR1QnlDMGZQUE1xUG5sZ2lUYmgvMVVZeDk1TTNI?=
 =?utf-8?B?QkdXUVFiZFlXajFGbWpwK3dabXJTcFZtaFlmZFZ0Ym4xengwWFNIYlJrVHB1?=
 =?utf-8?B?T3dxU0xzNGdTSkxHdUprbmZYOUNvTlp0OGFoM000a0xPRERmR21nUXgwODBH?=
 =?utf-8?B?QUM2OUhENlZlUmFQOEZiMkpQN05IRWlMQlAvRjJVU3NucGFnb3F6dThIbWND?=
 =?utf-8?B?V0FJT0lKVEd5by9JWFZYTzNUaXpnUFhaMmVsTDNqWWJFSXRSa0szV1FzeGhk?=
 =?utf-8?B?cTlaeGNocnBIRmVzLyt0azM3ZGRNWHJYR056U0J4SHNGeTN5QWt4NGVVaHRS?=
 =?utf-8?B?TFY0SWdxcE9xQU5iU0hTL1Evd0NxQ1hldUdkWXBMUXQ4Q01Nb0FkaDExTDZQ?=
 =?utf-8?B?S1FtMDhCdUhrN1hXVmJ0dVZkclErTUNERTlEQXJJYjdwR3Z3RzY1Z2JTbzN3?=
 =?utf-8?B?d0s2RjBrRHQyWTZ4RDlOSFlRK1hzcnN2OURFdWNITnRtWUZEQU1uV2NwUmRy?=
 =?utf-8?B?MUg1bmhKSUV3bDhvRnR0Qk92N3l1MGpRck13QWhwMGR4S3lsajg2cjFNeEl4?=
 =?utf-8?B?NXVRb3h6eG5VajlxVE9QL3k3eE1tU0dURzVJaEJ5R2dlbHlDRS9tY0lZb2JC?=
 =?utf-8?B?VWJIQ1BrN2JZNVVyaVBLM295cS9HeDVvUWhlbHdKQy8yN1N0b0xyYUVtQU82?=
 =?utf-8?B?UmR4aC9MNkdMN0dxdTN6WHh3ckY0c0dFNnZJQVlEcjI1TGxSN2JsMHpDTENI?=
 =?utf-8?B?OGNYRGpXdFE5UGM2WjJJcTR2NWdoZzZQa0lhSnBtcTRNTlk2NnJ3T1JyY3I3?=
 =?utf-8?B?TlVId1RJVy9uZk1IUmFkSklKeDJRa2UyVmFOUkVLTTNWeGpIU2tkUGVxd2VD?=
 =?utf-8?B?V1E9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: c9850344-70f3-4c56-2120-08dc34b433bd
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Feb 2024 21:12:54.7448
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: NRCyXwHe4DHXDylIlEVpwDq1idEyOcyntuXr9SVUqU1gZafnZtUW0VJfwph1hYP2MFzZf3U2B+mimXf9I6KDbaBAvgUzTMkG5MOfRVBg31o=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4737
X-OriginatorOrg: intel.com


On 2/23/2024 1:00 PM, Jacob Keller wrote:
> 
> 
> On 2/23/2024 11:24 AM, Rahul Rameshbabu wrote:
>> The goal of this patch series is to introduce a common set of ethtool statistics
>> for hardware timestamping that a driver implementer can hook into. The
>> statistics counters added are based on what I believe are common
>> patterns/behaviors found across various hardware timestamping implementations
>> seen in the kernel tree today. The mlx5 family of devices is used as the PoC for
>> this patch series. Other vendors are more than welcome to chim in on this
>> series.
>>
>> Statistics can be queried from either the DMA or PHY layers. I think this
>> concept of layer-based statistics selection and the general timestamping layer
>> selection work Kory Maincent is working on can be converged together [1].
>>
>> [1] https://lore.kernel.org/netdev/20240216-feature_ptp_netnext-v8-0-510f42f444fb@bootlin.com/
>>
> 
> Makes sense! I like this direction, I had meant to work on this for the
> Intel parts, but got sidetracked by other tasks. I look forward to
> seeing what you've done here.
> 

I'm fairly happy with the series overall, and the entire thing read
pretty straight forward.

I'd be happy to follow up with patches to convert the ice driver to
report these statistics as well.