From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mta1.migadu.com (out-92.mta1.migadu.com [95.215.58.92]) (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 8F29E2F616B for ; Tue, 1 Sep 2026 06:31:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=95.215.58.92 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788244283; cv=none; b=iGjfJFAfrY42sS/Edr+0zOOivSe3KsttdvXN90TZiZgd59UTA1sAOolfZ35fP1MjOwYy10JzCFraycc1xmnaL+m1vi0K02jraflOD84z8QxAsKD+ec9v8iwEf6KC2+awDgR3ubeb86jKqTHtuh+d/ANMrcPHas1ezpHlXwkH+6E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788244283; 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=jIZjnFzQvL9U318lt1APi7dr4RtKx+DHBG0kd0PX/ZPKCCtsig/fAgekQlyFJ9+XD7KQ7AwlDQJoKM+vz4jsTMxFMgGxXnD35idsCCllxeEk+rJQhbDWD3bt96BzR/8xQfZaLsCn0MYo3M1uaLTDj3mxS/Ms/FcU89ZEDVRTtaM= 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.92 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-kernel@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-kernel@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-kernel@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