Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 04 Oct 2016 18:05:16 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-bugs@FreeBSD.org
Subject:   [Bug 213201] [bhyve] virtio-net requires more than one descriptor per tx operation
Message-ID:  <bug-213201-8@https.bugs.freebsd.org/bugzilla/>

next in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D213201

            Bug ID: 213201
           Summary: [bhyve] virtio-net requires more than one descriptor
                    per tx operation
           Product: Base System
           Version: CURRENT
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Some People
          Priority: ---
         Component: misc
          Assignee: freebsd-bugs@FreeBSD.org
          Reporter: ricarkol@gmail.com

Hello,

We noticed that the Tx path for bhyve virtio-net devices
(pci_virtio_net.c:pci_vtnet_proctx) assumes that all Tx'es will have a first
descriptor with the header alone
(https://svnweb.freebsd.org/base/head/usr.sbin/bhyve/pci_virtio_net.c?view=
=3Dmarkup#l609):

        sc->pci_vtnet_tx(sc, &iov[1], n - 1, plen); // Skips the header
descriptor

The issue is that the virtio 1.0 specification allows for having the net he=
ader
and the data in a single descriptor. See section 5.1.6.2 of the virtio spec
(http://docs.oasis-open.org/virtio/virtio/v1.0/cs04/virtio-v1.0-cs04.html#x=
1-1680006).

Thanks,
Ricardo

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-213201-8>