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

VIRTIO Spec feedback #4 From: Haggai Eran

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: virtio 1.0 csprd01
    • Fix Version/s: virtio 1.0 csprd02
    • Labels:
      None
    • Proposal:
      Hide

      Proposal:

      diff --git a/content.tex b/content.tex
      index 374bc02..f048a88 100644
      — a/content.tex
      +++ b/content.tex
      @@ -425,7 +425,7 @@ written by the driver and read by the device.
      The “idx” field indicates where the driver would put the next descriptor
      entry in the ring (modulo the queue size). This starts at 0, and increases.

      -If the VIRTIO_RING_F_INDIRECT_DESC feature bit is not negotiated, the
      +If the VIRTIO_RING_F_EVENT_IDX feature bit is not negotiated, the
      “flags” field offers a crude interrupt control mechanism. The driver
      MUST set this to 0 or 1: 1 indicates that the device SHOULD NOT send
      an interrupt when it consumes a descriptor chain from the available
      @@ -473,7 +473,7 @@ for drivers using untrusted buffers: if you do not know exactly
      how much has been written by the device, you usually have to zero
      the buffer to ensure no data leakage occurs.

      -If the VIRTIO_RING_F_INDIRECT_DESC feature bit is not negotiated, the
      +If the VIRTIO_RING_F_EVENT_IDX feature bit is not negotiated, the
      “flags” field offers a crude interrupt control mechanism. The driver
      MUST initialize this to 0, the device MUST set this to 0 or 1: 1
      indicates that the driver SHOULD NOT send an notification when it adds

      Show
      Proposal: diff --git a/content.tex b/content.tex index 374bc02..f048a88 100644 — a/content.tex +++ b/content.tex @@ -425,7 +425,7 @@ written by the driver and read by the device. The “idx” field indicates where the driver would put the next descriptor entry in the ring (modulo the queue size). This starts at 0, and increases. -If the VIRTIO_RING_F_INDIRECT_DESC feature bit is not negotiated, the +If the VIRTIO_RING_F_EVENT_IDX feature bit is not negotiated, the “flags” field offers a crude interrupt control mechanism. The driver MUST set this to 0 or 1: 1 indicates that the device SHOULD NOT send an interrupt when it consumes a descriptor chain from the available @@ -473,7 +473,7 @@ for drivers using untrusted buffers: if you do not know exactly how much has been written by the device, you usually have to zero the buffer to ensure no data leakage occurs. -If the VIRTIO_RING_F_INDIRECT_DESC feature bit is not negotiated, the +If the VIRTIO_RING_F_EVENT_IDX feature bit is not negotiated, the “flags” field offers a crude interrupt control mechanism. The driver MUST initialize this to 0, the device MUST set this to 0 or 1: 1 indicates that the driver SHOULD NOT send an notification when it adds
    • Resolution:
      Hide

      Decision: 2014-02-11 minutes: Applied
      revision r235

      Show
      Decision: 2014-02-11 minutes: Applied revision r235

      Description

      Date: Tue, 28 Jan 2014 10:03:51 +0200
      Link to Mail: https://lists.oasis-open.org/archives/virtio-comment/201401/msg00045.html
      Commenter name: Haggai Eran <haggaie@mellanox.com>

      In Section 2.4.5, "The Virtqueue Available Ring", it says:

      > If the VIRTIO_RING_F_INDIRECT_DESC feature bit is not negotiated, the
      > “flags” field offers a crude interrupt

      Shouldn't it be VIRTIO_RING_F_EVENT_IDX instead of
      VIRTIO_RING_F_INDIRECT_DESC?
      The next paragraph says:

      > Otherwise, if the VIRTIO_RING_F_EVENT_IDX feature bit is negotiated, the
      > driver MUST set the "flags" field

      The same question also applies to Section 2.4.6.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              mstsirkin Michael Tsirkin (Inactive)
            • Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: