Uploaded image for project: 'OASIS Virtual I/O Device (VIRTIO) TC'
  1. OASIS Virtual I/O Device (VIRTIO) TC
  2. VIRTIO-134

Make it clear how chained indirect descriptors must work.

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: virtio 1.0 csprd03, virtio 1.0 cs01, virtio 1.0 cs02
    • Fix Version/s: virtio 1.0 cs03
    • Labels:
      None
    • Environment:

      Rusty Russell <rusty@rustcorp.com.au>

      Description

      Chained indirect descriptors were partially banned by virtio-15, which said how devices would handle them (by terminating the descriptor when it hits the end of the indirect descriptors). It didn't spell out that (obviously) they shouldn't be generated by drivers!

      It also leaves open the possibility of one or more direct descriptors followed by a trailing indirect descriptor. All implementations I know of handle this, though none I know of generate such things. My test lguest implementation didn't (now fixed); we should spell out explicitly that it's possible.

        Attachments

          Activity

          rusty Rusty Russell (Inactive) created issue -
          rusty Rusty Russell (Inactive) made changes -
          rusty Rusty Russell (Inactive) made changes -
          Status New [ 10000 ] Open [ 1 ]
          mstsirkin Michael Tsirkin (Inactive) made changes -
          Fix Version/s virtio 1.0 cs03 [ 12760 ]
          mstsirkin Michael Tsirkin (Inactive) made changes -
          Fix Version/s virtio 1.0 cs03 [ 12760 ]
          mstsirkin Michael Tsirkin (Inactive) made changes -
          Fix Version/s virtio 1.0 cs03 [ 12760 ]
          Assignee Michael Tsirkin [ mstsirkin ]
          Environment Rusty Russell <rusty@rustcorp.com.au>
          mstsirkin Michael Tsirkin (Inactive) made changes -
          Resolution r521
          mstsirkin Michael Tsirkin (Inactive) made changes -
          Assignee Michael Tsirkin [ mstsirkin ] Rusty Russell [ rusty ]
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]

            People

            • Assignee:
              rusty Rusty Russell (Inactive)
              Reporter:
              rusty Rusty Russell (Inactive)
            • Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: