On Wed, Jan 12, 2022 at 2:56 PM Jakub Kicinski [off-list ref] wrote:

On Sun,  9 Jan 2022 17:56:31 -0800 Dimitris Michailidis wrote:

quoted

+static int fun_hwtstamp_set(struct net_device *dev, struct ifreq *ifr)
+{
+     struct funeth_priv *fp = netdev_priv(dev);
+     struct hwtstamp_config cfg;
+
+     if (copy_from_user(&cfg, ifr->ifr_data, sizeof(cfg)))
+             return -EFAULT;
+
+     if (cfg.flags)           /* flags is reserved, must be 0 */
+             return -EINVAL;
+

This check was moved to the core in 9c9211a3fc7a ("net_tstamp: add new
flag HWTSTAMP_FLAG_BONDED_PHC_INDEX")

Thanks, I've changed it.