qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] ef4929: dataplane: use linux-headers/ for vir


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] ef4929: dataplane: use linux-headers/ for virtio includes
Date: Thu, 03 Jan 2013 06:00:08 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: ef4929fb3c25e03deca76c7f5d22fba08edf864f
      
https://github.com/qemu/qemu/commit/ef4929fb3c25e03deca76c7f5d22fba08edf864f
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2013-01-03 (Thu, 03 Jan 2013)

  Changed paths:
    M hw/Makefile.objs
    M hw/dataplane/Makefile.objs

  Log Message:
  -----------
  dataplane: use linux-headers/ for virtio includes

The hw/dataplane/vring.c code includes linux/virtio_ring.h.  Ensure that
we use linux-headers/ instead of the system-wide headers, which may be
out-of-date on older distros.

This resolves the following build error on Debian 6:

  CC    hw/dataplane/vring.o
cc1: warnings being treated as errors
hw/dataplane/vring.c: In function 'vring_enable_notification':
hw/dataplane/vring.c:71: error: implicit declaration of function 
'vring_avail_event'
hw/dataplane/vring.c:71: error: nested extern declaration of 'vring_avail_event'
hw/dataplane/vring.c:71: error: lvalue required as left operand of assignment

Note that we now build dataplane/ for each target instead of only once.
There is no way around this since linux-headers/ is only available for
per-target objects - and it's how virtio, vfio, kvm, and friends are
built.

Signed-off-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Anthony Liguori <address@hidden>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]