Skip to contentSkip to conversation

Add KVM CFLAGS to vhost build

PATCH Applied Jan Kiszka 3 participants 4 messages Apr 6, 2010 – May 3, 2010 Kbuild

lore
Share & tools
lore

Jan Kiszka

author
#0 lore raw

PATCH 1/1 Add KVM CFLAGS to vhost build

The configure test of vhost uses KVM CFLAGS, so the build must use them as well. Otherwise we specifically miss what --kerneldir provides.

Signed-off-by Jan Kiszka <[email protected]>
---
 Makefile.target |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Makefile.target +1 -1
index 45cbe56..6fc3034 100644
@@ -32,7 +32,7 @@ PROGS=$(QEMU_PROG)
 
 LIBS+=-lm
 
-kvm.o kvm-all.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
+kvm.o kvm-all.o vhost.o vhost_net.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
 
 config-target.h: config-target.h-timestamp
 config-target.h-timestamp: config-target.mak
author
#1 lore raw
Quoted 3 lines · Jan Kiszka

The configure test of vhost uses KVM CFLAGS, so the build must use them as well. Otherwise we specifically miss what --kerneldir provides.

Ping.

Jan

Quoted 21 lines
Signed-off-by Jan Kiszka <[email protected]>
---
 Makefile.target |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Makefile.target +1 -1
index 45cbe56..6fc3034 100644
@@ -32,7 +32,7 @@ PROGS=$(QEMU_PROG)
 
 LIBS+=-lm
 
-kvm.o kvm-all.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
+kvm.o kvm-all.o vhost.o vhost_net.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
 
 config-target.h: config-target.h-timestamp
 config-target.h-timestamp: config-target.mak
Acked-by on v1 1/1
#2 lore raw
In reply to Jan Kiszka
Quoted 4 lines · Jan Kiszka

The configure test of vhost uses KVM CFLAGS, so the build must use them as well. Otherwise we specifically miss what --kerneldir provides.

Signed-off-by Jan Kiszka <[email protected]>
Acked-by Michael S. Tsirkin <[email protected]>
Quoted 17 lines
---
 Makefile.target |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Makefile.target +1 -1
index 45cbe56..6fc3034 100644
@@ -32,7 +32,7 @@ PROGS=$(QEMU_PROG)
 
 LIBS+=-lm
 
-kvm.o kvm-all.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
+kvm.o kvm-all.o vhost.o vhost_net.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
 
 config-target.h: config-target.h-timestamp
 config-target.h-timestamp: config-target.mak
Applied
#3 lore raw
In reply to Jan Kiszka
Quoted 5 lines · Jan Kiszka

The configure test of vhost uses KVM CFLAGS, so the build must use them as well. Otherwise we specifically miss what --kerneldir provides.

Signed-off-by Jan Kiszka<[email protected]>

Applied. Thanks.

Regards,

Anthony Liguori

Quoted 21 lines
---
  Makefile.target |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
Makefile.target +1 -1
index 45cbe56..6fc3034 100644
@@ -32,7 +32,7 @@ PROGS=$(QEMU_PROG)
 
  LIBS+=-lm
 
-kvm.o kvm-all.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
+kvm.o kvm-all.o vhost.o vhost_net.o: QEMU_CFLAGS+=$(KVM_CFLAGS)
 
  config-target.h: config-target.h-timestamp
  config-target.h-timestamp: config-target.mak