[dpdk-dev] vhost: fix build error for old kernels

Message ID 1457427081-7290-1-git-send-email-yuanhan.liu@linux.intel.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers

Commit Message

Yuanhan Liu March 8, 2016, 8:51 a.m. UTC
  VIRTIO_NET_F_GUEST_ANNOUNCE is a new feature introduced since kernel
v3.5. For older kernels (or more precisely, old distributions), we
could simply define it manually, to fix the "macro not defined" error.

Fixes: d293dac8f30e ("vhost: claim support of guest announce")

Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
---
 lib/librte_vhost/rte_virtio_net.h | 5 +++++
 1 file changed, 5 insertions(+)
  

Comments

Thomas Monjalon March 11, 2016, 3:29 p.m. UTC | #1
2016-03-08 16:51, Yuanhan Liu:
> VIRTIO_NET_F_GUEST_ANNOUNCE is a new feature introduced since kernel
> v3.5. For older kernels (or more precisely, old distributions), we
> could simply define it manually, to fix the "macro not defined" error.
> 
> Fixes: d293dac8f30e ("vhost: claim support of guest announce")
> 
> Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>

Applied, thanks
  

Patch

diff --git a/lib/librte_vhost/rte_virtio_net.h b/lib/librte_vhost/rte_virtio_net.h
index 7d1fde2..443245d 100644
--- a/lib/librte_vhost/rte_virtio_net.h
+++ b/lib/librte_vhost/rte_virtio_net.h
@@ -97,6 +97,11 @@  struct vhost_virtqueue {
 	struct buf_vector	buf_vec[BUF_VECTOR_MAX];	/**< for scatter RX. */
 } __rte_cache_aligned;
 
+/* Old kernels have no such macro defined */
+#ifndef VIRTIO_NET_F_GUEST_ANNOUNCE
+ #define VIRTIO_NET_F_GUEST_ANNOUNCE 21
+#endif
+
 
 /*
  * Make an extra wrapper for VIRTIO_NET_F_MQ and