From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mta1.migadu.com (out-91.mta1.migadu.com [95.215.58.91]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C1F452C0F7F for ; Tue, 1 Sep 2026 06:31:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=95.215.58.91 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788244284; cv=none; b=szyHcbFy6A2ex6NbtVhgyV0SLPAP0U4fRj6NQSTsjI6HQoVA+eSqkuhF96QQuIwahhmjX0i5WvXXpIHSnzYPnthvktKSegT42ACYNJ01bXICFwhO2jL1wh3zEApEZnpggmAliQzbXqsXPnT20YILeJFxbmmQVZsm/skgXVLWYxQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788244284; c=relaxed/simple; bh=jwf8WmkbwBdcg/b7Qk7yq4ug2VUbkvCrJJWEgI/fq2A=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=T42VlNU8XN+b1gSyVaXW/ERXnsDJqZolS/69huG+z3WV+H6Se1AADwLn8oN47bNl68JRhP+IK2qGzkKTeiJBsBJFQTNvTdizzvgYdsCPpmBT3hXTOX2L6/cBL2leWOaset10+sk74C9L5H43f7BaAMv/H1NAQMaVIY9LQ38ZvlU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev; spf=pass smtp.mailfrom=linux.dev; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b=QIVYW0eh; arc=none smtp.client-ip=95.215.58.91 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linux.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux.dev header.i=@linux.dev header.b="QIVYW0eh" X-Envelope-To: linux-kselftest@vger.kernel.org DKIM-Signature: a=rsa-sha256; bh=jwf8WmkbwBdcg/b7Qk7yq4ug2VUbkvCrJJWEgI/fq2A=; c=simple/simple; d=linux.dev; h=from:to:subject:date:message-id:mime-version:content-type; s=key1; t=1788244278; v=1; x=1788849078; b=QIVYW0eh1c63vhzod0ESQueinBsZpovLXZ7kmqnqOG9TscGvHk51sGa/4qZeczIoTRRXdt6L TcSvpCEEo2quZLJzGu65N7LNR35vArnmgbTHRmmfjLtOuquUHcpqGR0PZ563Uywy+twkDc1smjW CooiB2f9WVjoWQ8R+qhamUBo= X-Envelope-To: linux-kselftest@vger.kernel.org Received: by smtp.migadu.com with ESMTPS id 77e0f26e5b15dcd1; Tue, 01 Sep 2026 06:31:18 +0000 X-Mizu-Trace-ID: 77e0f26e5b15dcd1 X-Migadu-Flow: FLOW_OUT Date: Tue, 1 Sep 2026 14:31:06 +0800 From: Hangbin Liu To: Andrew Lunn Cc: Jakub Kicinski , Andrew Lunn , "David S. Miller" , Eric Dumazet , Paolo Abeni , Shuah Khan , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, Hangbin Liu Subject: Re: [PATCH net-next 1/2] netdevsim: add link speed support Message-ID: References: <20260831-nsim_speed-v1-0-7a651ee65738@kylinos.cn> <20260831-nsim_speed-v1-1-7a651ee65738@kylinos.cn> <043abd17-c896-40a7-901d-a3fc943151bc@lunn.ch> Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <043abd17-c896-40a7-901d-a3fc943151bc@lunn.ch> Hi Andrew, On Mon, Aug 31, 2026 at 02:24:54PM +0200, Andrew Lunn wrote: > > +static int nsim_get_link_ksettings(struct net_device *dev, > > + struct ethtool_link_ksettings *cmd) > > +{ > > + struct netdevsim *ns = netdev_priv(dev); > > + > > + cmd->base.speed = ns->ethtool.speed; > > + cmd->base.duplex = ns->ethtool.duplex; > > + cmd->base.port = PORT_OTHER; > > + cmd->base.autoneg = AUTONEG_DISABLE; > > Here you hard code autoneg as disabled. > > > +static int nsim_set_link_ksettings(struct net_device *dev, > > + const struct ethtool_link_ksettings *cmd) > > +{ > > + struct netdevsim *ns = netdev_priv(dev); > > + > > + if (cmd->base.speed > NSIM_LINK_SPEED_MAX) > > + return -EINVAL; > > + > > + return ethtool_virtdev_set_link_ksettings(dev, cmd, &ns->ethtool.speed, > > + &ns->ethtool.duplex); > > but here you don't appear to block enabling autoneg. I would expect a > test and EOPNOTSUPP. ethtool_virtdev_set_link_ksettings will check the cmds and only allow user to set speed and duplex. If a user tries to set autoneg it will return -EINVAL. Do you want nsim to return -EOPNOTSUPP specifically? Thanks Hangbin