From: Jakub Kicinski <kuba@kernel•org>
To: Richard Cochran <richardcochran@gmail•com>
Cc: "Vladimir Oltean" <vladimir.oltean@nxp•com>,
"Köry Maincent" <kory.maincent@bootlin•com>,
"Oleksij Rempel" <o.rempel@pengutronix•de>,
"Horatiu Vultur" <horatiu.vultur@microchip•com>,
"Michael Walle" <michael@walle•cc>,
"Andrew Lunn" <andrew@lunn•ch>,
"Florian Fainelli" <f.fainelli@gmail•com>,
"Heiner Kallweit" <hkallweit1@gmail•com>,
netdev@vger•kernel.org, devicetree@vger•kernel.org,
linux-kernel@vger•kernel.org, linux-doc@vger•kernel.org,
linux-omap@vger•kernel.org,
"Maxime Chevallier" <maxime.chevallier@bootlin•com>,
thomas.petazzoni@bootlin•com,
"Russell King" <linux@armlinux•org.uk>,
"David S. Miller" <davem@davemloft•net>,
"Eric Dumazet" <edumazet@google•com>,
"Paolo Abeni" <pabeni@redhat•com>,
"Rob Herring" <robh+dt@kernel•org>,
"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro•org>,
"Jonathan Corbet" <corbet@lwn•net>,
"Jay Vosburgh" <j.vosburgh@gmail•com>,
"Veaceslav Falico" <vfalico@gmail•com>,
"Andy Gospodarek" <andy@greyhouse•net>,
"Claudiu Manoil" <claudiu.manoil@nxp•com>,
"Alexandre Belloni" <alexandre.belloni@bootlin•com>,
UNGLinuxDriver@microchip•com,
"Minghao Chi" <chi.minghao@zte•com.cn>,
"Jie Wang" <wangjie125@huawei•com>,
"Oleksij Rempel" <linux@rempel-privat•de>,
"Sean Anderson" <sean.anderson@seco•com>,
"Gustavo A. R. Silva" <gustavoars@kernel•org>,
"Wolfram Sang" <wsa+renesas@sang-engineering•com>,
"Alexander Lobakin" <alexandr.lobakin@intel•com>,
"Marco Bonelli" <marco@mebeim•net>
Subject: Re: [PATCH v3 3/5] net: Let the active time stamping layer be selectable.
Date: Fri, 17 Mar 2023 21:03:06 -0700 [thread overview]
Message-ID: <20230317210306.346e80ea@kernel.org> (raw)
In-Reply-To: <ZBUyST3kDP1ZE1lF@hoboy.vegasvil.org>
On Fri, 17 Mar 2023 20:38:49 -0700 Richard Cochran wrote:
> On Fri, Mar 17, 2023 at 05:21:50PM +0200, Vladimir Oltean wrote:
> > On Thu, Mar 16, 2023 at 04:09:20PM +0100, Köry Maincent wrote:
> > > Was there any useful work that could be continued on managing timestamp through
> > > NDOs. As it seem we will made some change to the timestamp API, maybe it is a
> > > good time to also take care of this.
> >
> > Not to my knowledge. Yes, I agree that it would be a good time to add an
> > NDO for hwtimestamping (while keeping the ioctl fallback), then
> > transitioning as many devices as we can, and removing the fallback when
> > the transition is complete.
>
> Um, user space ABI cannot be removed.
NDO meaning a dedicated callback in struct net_device_ops, so at least
for netdevs we can copy the data from user space, validate in the core
and then call the driver with a normal kernel pointer. So just an
internal refactoring, no uAPI changes.
next prev parent reply other threads:[~2023-03-18 4:03 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-08 13:59 [PATCH v3 0/5] net: Make MAC/PHY time stamping selectable Köry Maincent
2023-03-08 13:59 ` [PATCH v3 1/5] net: ethtool: Refactor identical get_ts_info implementations Köry Maincent
2023-03-08 13:59 ` [PATCH v3 2/5] net: Expose available time stamping layers to user space Köry Maincent
2023-03-08 22:54 ` Vladimir Oltean
2023-03-08 13:59 ` [PATCH v3 3/5] net: Let the active time stamping layer be selectable Köry Maincent
2023-03-08 15:28 ` Willem de Bruijn
2023-03-10 14:41 ` Köry Maincent
2023-03-10 14:59 ` Willem de Bruijn
2023-03-10 15:32 ` Andrew Lunn
2023-03-08 18:26 ` kernel test robot
2023-03-08 23:03 ` Vladimir Oltean
2023-03-10 10:48 ` Köry Maincent
2023-03-10 11:35 ` Vladimir Oltean
2023-03-10 12:15 ` Michael Walle
2023-03-10 13:15 ` Horatiu Vultur
2023-03-10 13:34 ` Michael Walle
2023-03-10 14:04 ` Köry Maincent
2023-03-10 15:05 ` Richard Cochran
2023-03-10 15:24 ` Andrew Lunn
2023-03-10 16:06 ` Vladimir Oltean
2023-03-10 20:48 ` Michael Walle
2023-03-10 16:44 ` Vladimir Oltean
2023-03-13 8:17 ` Horatiu Vultur
2023-03-13 8:40 ` Oleksij Rempel
2023-03-14 11:02 ` Köry Maincent
2023-03-16 15:09 ` Köry Maincent
2023-03-17 15:21 ` Vladimir Oltean
2023-03-17 19:07 ` Jakub Kicinski
2023-03-17 19:43 ` Max Georgiev
2023-03-30 12:38 ` Köry Maincent
2023-03-30 16:26 ` Jakub Kicinski
2023-03-31 5:05 ` Max Georgiev
2023-03-31 5:07 ` Max Georgiev
2023-04-02 17:12 ` Vladimir Oltean
2023-04-03 9:27 ` Köry Maincent
2023-03-18 3:38 ` Richard Cochran
2023-03-18 4:03 ` Jakub Kicinski [this message]
2023-03-18 11:54 ` Vladimir Oltean
2023-03-24 10:25 ` Maxime Chevallier
2023-04-02 17:36 ` Vladimir Oltean
2023-03-09 6:13 ` kernel test robot
2023-03-09 17:33 ` kernel test robot
2023-03-08 13:59 ` [PATCH v3 4/5] net: fix up drivers WRT phy time stamping Köry Maincent
2023-03-08 13:59 ` [PATCH v3 5/5] dt-bindings: net: phy: add timestamp preferred choice property Köry Maincent
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=20230317210306.346e80ea@kernel.org \
--to=kuba@kernel$(echo .)org \
--cc=UNGLinuxDriver@microchip$(echo .)com \
--cc=alexandr.lobakin@intel$(echo .)com \
--cc=alexandre.belloni@bootlin$(echo .)com \
--cc=andrew@lunn$(echo .)ch \
--cc=andy@greyhouse$(echo .)net \
--cc=chi.minghao@zte$(echo .)com.cn \
--cc=claudiu.manoil@nxp$(echo .)com \
--cc=corbet@lwn$(echo .)net \
--cc=davem@davemloft$(echo .)net \
--cc=devicetree@vger$(echo .)kernel.org \
--cc=edumazet@google$(echo .)com \
--cc=f.fainelli@gmail$(echo .)com \
--cc=gustavoars@kernel$(echo .)org \
--cc=hkallweit1@gmail$(echo .)com \
--cc=horatiu.vultur@microchip$(echo .)com \
--cc=j.vosburgh@gmail$(echo .)com \
--cc=kory.maincent@bootlin$(echo .)com \
--cc=krzysztof.kozlowski+dt@linaro$(echo .)org \
--cc=linux-doc@vger$(echo .)kernel.org \
--cc=linux-kernel@vger$(echo .)kernel.org \
--cc=linux-omap@vger$(echo .)kernel.org \
--cc=linux@armlinux$(echo .)org.uk \
--cc=linux@rempel-privat$(echo .)de \
--cc=marco@mebeim$(echo .)net \
--cc=maxime.chevallier@bootlin$(echo .)com \
--cc=michael@walle$(echo .)cc \
--cc=netdev@vger$(echo .)kernel.org \
--cc=o.rempel@pengutronix$(echo .)de \
--cc=pabeni@redhat$(echo .)com \
--cc=richardcochran@gmail$(echo .)com \
--cc=robh+dt@kernel$(echo .)org \
--cc=sean.anderson@seco$(echo .)com \
--cc=thomas.petazzoni@bootlin$(echo .)com \
--cc=vfalico@gmail$(echo .)com \
--cc=vladimir.oltean@nxp$(echo .)com \
--cc=wangjie125@huawei$(echo .)com \
--cc=wsa+renesas@sang-engineering$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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