Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1725002

Re: virtio_net: ethtool supported link modes

Show all headers | View raw

On Fri, 2017-09-01 at 11:36 +0800, Jason Wang wrote:
> 
> On 2017年09月01日 01:04, Radu Rendec wrote:
> > Hello,
> > 
> > Looking at the code in virtnet_set_link_ksettings, it seems the speed
> > and duplex can be set to any valid value. The driver will "remember"
> > them and report them back in virtnet_get_link_ksettings.
> > 
> > However, the supported link modes (link_modes.supported in struct
> > ethtool_link_ksettings) is always 0, indicating that no speed/duplex
> > setting is supported.
> > 
> > Does it make more sense to set (at least a few of) the supported link
> > modes, such as 10baseT_Half ... 10000baseT_Full?
> > 
> > I would expect to see consistency between what is reported in
> > link_modes.supported and what can actually be set. Could you please
> > share your opinion on this?
> 
> I think the may make sense only if there's a hardware implementation for 
> virtio. And we probably need to extend virtio spec for adding new commands.

So you're saying that the "hardware" should provide the supported link
modes (e.g. by using feature bits at the virtio layer) and the
virtio_net driver should just translate them and expose them as
link_modes.supported?

Then for consistency, I assume setting speed/duplex via ethtool should
also go into the virtio layer (currently virtio_net seems to just store
them for future retrieval via ethtool).

Thanks,
Radu

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

virtio_net: ethtool supported link modes Radu Rendec <rrendec@arista.com> - 2017-08-31 19:10 +0200
  Re: virtio_net: ethtool supported link modes Jason Wang <jasowang@redhat.com> - 2017-09-01 05:40 +0200
    Re: virtio_net: ethtool supported link modes Radu Rendec <rrendec@arista.com> - 2017-09-01 14:10 +0200
  Re: virtio_net: ethtool supported link modes "Michael S. Tsirkin" <mst@redhat.com> - 2017-09-01 17:50 +0200
    Re: virtio_net: ethtool supported link modes Radu Rendec <rrendec@arista.com> - 2017-09-01 18:30 +0200
      Re: virtio_net: ethtool supported link modes "Michael S. Tsirkin" <mst@redhat.com> - 2017-09-01 19:50 +0200
        Re: virtio_net: ethtool supported link modes Radu Rendec <rrendec@arista.com> - 2017-09-04 17:00 +0200

csiph-web