virtio: fix build with debug
authorOuyang Changchun <changchun.ouyang@intel.com>
Thu, 24 Jul 2014 04:57:45 +0000 (12:57 +0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Fri, 1 Aug 2014 14:30:14 +0000 (16:30 +0200)
Fix 2 compilation issues in virtio PMD when dump option is enabled.

These errors were introduced by commits
    f37cdfde46a30 (remove unused virtqueue name)
and ce65e697c67ba (simplify the hardware structure).

Signed-off-by: Changchun Ouyang <changchun.ouyang@intel.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
lib/librte_pmd_virtio/virtio_ethdev.c
lib/librte_pmd_virtio/virtqueue.h

index b9f5529..6293ac6 100644 (file)
@@ -580,7 +580,7 @@ virtio_negotiate_features(struct virtio_hw *hw)
        /* Prepare guest_features: feature that driver wants to support */
        hw->guest_features = VTNET_FEATURES & ~mask;
        PMD_INIT_LOG(DEBUG, "guest_features before negotiate = %x",
-               guest_features);
+               hw->guest_features);
 
        /* Read device(host) feature bits */
        host_features = VIRTIO_READ_REG_4(hw, VIRTIO_PCI_HOST_FEATURES);
index 02fe00b..87db35f 100644 (file)
@@ -269,10 +269,10 @@ virtqueue_notify(struct virtqueue *vq)
        used_idx = (vq)->vq_ring.used->idx; \
        nused = (uint16_t)(used_idx - (vq)->vq_used_cons_idx); \
        PMD_INIT_LOG(DEBUG, \
-         "VQ: %s - size=%d; free=%d; used=%d; desc_head_idx=%d;" \
+         "VQ: - size=%d; free=%d; used=%d; desc_head_idx=%d;" \
          " avail.idx=%d; used_cons_idx=%d; used.idx=%d;" \
          " avail.flags=0x%x; used.flags=0x%x", \
-         (vq)->vq_name, (vq)->vq_nentries, (vq)->vq_free_cnt, nused, \
+         (vq)->vq_nentries, (vq)->vq_free_cnt, nused, \
          (vq)->vq_desc_head_idx, (vq)->vq_ring.avail->idx, \
          (vq)->vq_used_cons_idx, (vq)->vq_ring.used->idx, \
          (vq)->vq_ring.avail->flags, (vq)->vq_ring.used->flags); \