diff mbox

include qemu-kvm.h at all cases.

Message ID 1234470982-330-1-git-send-email-glommer@redhat.com (mailing list archive)
State Accepted
Headers show

Commit Message

Glauber Costa Feb. 12, 2009, 8:36 p.m. UTC
We don't have any kind of kvm support for linux-user targets,
but conditionalizing the inclusion of kvm header make
linux-user compilation break. And we don't have a reason to
prevent that, do we?

Signed-off-by: Glauber Costa <glommer@redhat.com>
---
 qemu/gdbstub.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Dustin Kirkland Feb. 17, 2009, 12:01 a.m. UTC | #1
On 2/12/09, Glauber Costa <glommer@redhat.com> wrote:
> We don't have any kind of kvm support for linux-user targets,
>  but conditionalizing the inclusion of kvm header make
>  linux-user compilation break. And we don't have a reason to
>  prevent that, do we?

Ack, again, same problem here.  The lack of inclusion of this header
breaks the build of kvm-84 for the Ubuntu package.

:-Dustin
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Avi Kivity Feb. 17, 2009, 6 p.m. UTC | #2
Glauber Costa wrote:
> We don't have any kind of kvm support for linux-user targets,
> but conditionalizing the inclusion of kvm header make
> linux-user compilation break. And we don't have a reason to
> prevent that, do we?
>   

Applied, thanks.
diff mbox

Patch

diff --git a/qemu/gdbstub.c b/qemu/gdbstub.c
index c4d4361..a23e916 100644
--- a/qemu/gdbstub.c
+++ b/qemu/gdbstub.c
@@ -33,8 +33,8 @@ 
 #include "qemu-char.h"
 #include "sysemu.h"
 #include "gdbstub.h"
-#include "qemu-kvm.h"
 #endif
+#include "qemu-kvm.h"
 
 #define MAX_PACKET_LENGTH 4096