Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Date:   Tue, 21 Nov 2023 00:05:49 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Jakub Kicinski <kuba@kernel.org>
Cc:     =?utf-8?B?S8O2cnk=?= Maincent <kory.maincent@bootlin.com>,
        Florian Fainelli <florian.fainelli@broadcom.com>,
        Broadcom internal kernel review list 
        <bcm-kernel-feedback-list@broadcom.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Paolo Abeni <pabeni@redhat.com>,
        Richard Cochran <richardcochran@gmail.com>,
        Radu Pirea <radu-nicolae.pirea@oss.nxp.com>,
        Jay Vosburgh <j.vosburgh@gmail.com>,
        Andy Gospodarek <andy@greyhouse.net>,
        Nicolas Ferre <nicolas.ferre@microchip.com>,
        Claudiu Beznea <claudiu.beznea@tuxon.dev>,
        Willem de Bruijn <willemdebruijn.kernel@gmail.com>,
        Jonathan Corbet <corbet@lwn.net>,
        Horatiu Vultur <horatiu.vultur@microchip.com>,
        UNGLinuxDriver@microchip.com, Simon Horman <horms@kernel.org>,
        Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-doc@vger.kernel.org,
        Maxime Chevallier <maxime.chevallier@bootlin.com>
Subject: Re: [PATCH net-next v7 15/16] net: ethtool: ts: Let the active time
 stamping layer be selectable
Message-ID: <20231120220549.cvsz2ni3wj7mcukh@skbuf>
References: <20231120105255.cgbart5amkg4efaz@skbuf>
 <20231120121440.3274d44c@kmaincent-XPS-13-7390>
 <20231120120601.ondrhbkqpnaozl2q@skbuf>
 <20231120144929.3375317e@kmaincent-XPS-13-7390>
 <20231120142316.d2emoaqeej2pg4s3@skbuf>
 <20231120093723.4d88fb2a@kernel.org>
 <20231120190023.ymog4yb2hcydhmua@skbuf>
 <20231120115839.74ee5492@kernel.org>
 <20231120211759.j5uvijsrgt2jqtwx@skbuf>
 <20231120133737.70dde657@kernel.org>
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20231120133737.70dde657@kernel.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?iso-8859-1?Q?FtK9LRRRGBEUMzidq2qT5bmZQAnNXu/co4U8YCR5LzmeDwvx2n1zFwVyKS?=
 =?iso-8859-1?Q?U5s/ytjCi9JfcttbmN+zQ1ud6gyYyqtui1LBwaWO6PEZiwTXScKF1o14lr?=
 =?iso-8859-1?Q?TL02ggf5AgdXOntQzI5YFVB+qE/6ItYMxiHbqLN92H5HPuLhm5Z2EDKTGp?=
 =?iso-8859-1?Q?PTehZXDXIQykEsuYbEJnjyPpQLZOP7s87XaO2/J4VfpBAZFn0FPZqYLwmf?=
 =?iso-8859-1?Q?MIQScll7+v2uZjRcA7mVZtCKXz6MCndyjhCsykxnJQKU8/F4Yo9nsVoByh?=
 =?iso-8859-1?Q?6bJ5KotUVEFQhtUvn4Ja6lEe33nhPd7qs65mGrEPAufCT9yJNWKV8c7Dmz?=
 =?iso-8859-1?Q?MgbZ02BJw4fkTaag9DVj00MCVBIqhIn5rG+twP7X6Jx0tjyLcK2z7iPeLJ?=
 =?iso-8859-1?Q?Wz9RI/TwKNoRLWLBoqAtiuraATVJmxPc1KbsXdkG6o+zOKrK2Z1byCVMub?=
 =?iso-8859-1?Q?vArYu9PmlC5VTpzdtKEuDsTb8wU1ub+UPX9qn9MCr8IeVWRoTsyV85snIt?=
 =?iso-8859-1?Q?TyLHqNNHGrs5I6ehpOb/P6kmFHh0cg7UTuqjHb26pU0XWYhSntdP20wwvm?=
 =?iso-8859-1?Q?pT5XQk6FW6sN2L9Hczr/xInxsApR6umWqJUEV8KFNq+mizGghL/RaW8gmB?=
 =?iso-8859-1?Q?wPo2d34sI0JToGEjINx+UDXJER0JcoUqCUFl8Wy7ZX0BXpJEXD1Si+rqiV?=
 =?iso-8859-1?Q?t4R8DY3IYboOcDJmOEwNFcLkEw/7ES+2czeVgdGd8rBbEcmgM/pOX5p6vI?=
 =?iso-8859-1?Q?FXGV9UauOLg1F8otM82mkiiE2Nf0UAvceLyQK5cIV3jCqcSl7rMAR4nxGL?=
 =?iso-8859-1?Q?4Ggi6rCJoANZRO982EO42AOcNF1WJEHdDjs2bsGnm0lZ3fR9rtPzfFNRjn?=
 =?iso-8859-1?Q?xZYLBqJ9lurQIuoKgLv890BOSIQQU7efsKjGeSMy/gywJW5uaaFZEVWy1M?=
 =?iso-8859-1?Q?PDxWyvAsvRACv+IgBnOmYghuHul2w/IfvlKmwtqc5Qtumq17CTFvDafPoA?=
 =?iso-8859-1?Q?lobxHMLrcND5r+fMGC1I2ir23n6/6zMq23fpMHmlmq1dZ9esJeqSnJlI66?=
 =?iso-8859-1?Q?Z3C2gAA2hXcb6r2hTOwLrjhgdZkp3Oo59rAcIdrX2zk9ZxV9Xs9sWJk4Bx?=
 =?iso-8859-1?Q?m3DjtfZ8mdWBvxO2NLMijSVGCBKwo2bsNf2PydlsW8OhbiIDMJzlGgVoP7?=
 =?iso-8859-1?Q?a/duEBLBTAMW6IKBdvY7OZFYSu6BGqGatluLXb5LHlRnmlCHe/eFcjhnKG?=
 =?iso-8859-1?Q?6ndHXW30kHzz6JVBUHQH5SCPTyTaiKTPlsJwta5nzoCsZF3wub4JlK83+u?=
 =?iso-8859-1?Q?ZZQQ+C9nDw9PB6d/GgjpImTVSwtJFL4aOkDcz+6GnS/RHhP3Rp9q1s+KCm?=
 =?iso-8859-1?Q?nIkVD6yNv1WdJBWKbmgkeIHQzYlCYlPHzgAsTTojvqvlb0mqahxyJ0arnL?=
 =?iso-8859-1?Q?50bjGD1pEbc7oHuSn1vb4K40fc4vsoPeXwEcdtai/QPaQwgTXCj+SKm+HK?=
 =?iso-8859-1?Q?8KRLlagZDvsTkB7M+uhLssL3NkXvBoNQTRofsnOTUyg955uhtgXuIYU6Wn?=
 =?iso-8859-1?Q?526G4CKaca/2FrtmaScP292RwSigtSEu9IQ1nDgmTkoF6k0xxkMB8KVDiV?=
 =?iso-8859-1?Q?a7feedUsxf6er1v19xrVL9LRJnIxI3xzxB/2wuNDrptcx8iDtOugXyeA?=
 =?iso-8859-1?Q?=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2aec3ddc-c197-4c35-df17-08dbea14dde7
X-MS-Exchange-CrossTenant-AuthSource: DB8PR04MB6459.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Nov 2023 22:05:54.7574
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: KRf301wSRCNSRu1jjcSUNCRirsMGduvJu1guWEjD+oA4vWXkTGX1WxpmsFMwzgNpPb8GHvjYe3lkwjSzrRbfBg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR04MB7258
X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00,DKIM_INVALID,
        DKIM_SIGNED,RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=no 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
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Mon, 20 Nov 2023 14:06:03 -0800 (PST)

On Mon, Nov 20, 2023 at 01:37:37PM -0800, Jakub Kicinski wrote:
> > Is it an accurate summary?
> 
> Yes.
> 
> For now we can impose the requirement that only one can be active 
> easily at the kernel level. But the uAPI should allow expressing more.

I see. That's quite something to think about for K?ry. In its defense,
I also agree that this idea seems the most orthogonal to everything else
that we have or may want to add in the future, and is not likely to
become obsoleted by some other mechanism that can achieve the same
thing, but in a more flexible way. It's just that it's quite the task.

I sense it may be time to dust off and submit the rest of my
ndo_hwtstamp_get()/ ndo_hwtstamp_set() conversions before a netlink
conversion of SIOCGHWTSTAMP/SIOCSHWTSTAMP could even take place...
https://github.com/vladimiroltean/linux/commits/ndo-hwtstamp-v9