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
_______________________________________________
freebsd-virtualization@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-virtualization
To unsubscribe, send any mail to 
"freebsd-virtualization-unsubscr...@freebsd.org"

Reply via email to