[Differential] D20276: [bhyve][virtio-net] Allow guest VM's to set JUMBO MTU in case of using the VALE switch.

[aleksandr.fedorov_itglobal.com (Aleksandr Fedorov)]

aleksandr.fedorov_itglobal.com added inline comments.

INLINE COMMENTS

> v.maffione_gmail.com wrote in pci_virtio_net.c:407
> nmd->cur_tx_ring
> 
> Why did you drop the iterating over all the possible tx rings?

I drop it to simplify the code under the assumption that current setup code

  void
   pci_vtnet_netmap_setup(struct pci_vtnet_softc *sc, char *ifname)
   {
  ...
        sc->vsc_nmd = nm_open(ifname, NULL, 0, 0);
  ...
  }

always sets the number of  TX/RX rings to one. And there is no way to change it using bhyve(8).

In the future, I want to replace the legacy API and add the ability to specify not only the VALE switch, but also other types of netmap ports, and return the iteration through the rings.

Or I can return this code now. What do you think?

CHANGES SINCE LAST ACTION
  https://reviews.freebsd.org/D20276/new/

REVISION DETAIL
  https://reviews.freebsd.org/D20276

EMAIL PREFERENCES
  https://reviews.freebsd.org/settings/panel/emailpreferences/

To: aleksandr.fedorov_itglobal.com, #bhyve, jhb, rgrimes, krion, v.maffione_gmail.com
Cc: novel, olevole_olevole.ru, freebsd-virtualization-list, evgueni.gavrilov_itglobal.com, bcran