Skip to contentSkip to conversation

linux-next: kvm tree build failure

PATCH Applied Stephen Rothwell 3 participants 3 messages Dec 18, 2008

Subsystems (6)
KVM PowerPC Linux-next tree s390 +2 more subsystems
lore
Share & tools
lore
#0 lore raw

Hi Avi,

Today's linux-next build (powerpc ppc44x_config) failed like this:

In file included from include/linux/kvm_host.h:27,

                 from arch/powerpc/include/asm/kvm_44x.h:23,
                 from arch/powerpc/kernel/asm-offsets.c:52:

arch/powerpc/include/asm/kvm_host.h:203: error: redefinition of 'struct kvm_guest_debug'

Caused by commit 887e823e3f9c402ff2fe1ea2dc25072ebe81b5b1 ("KVM: New guest debug interface") which added struct kvm_guest_debug to include/linux/kvm.h but didn't remove it from arch/powerpc/include/asm/kvm_host.h. (ia64 and s390 probably have the same problem.)

I have reverted the kvm tree for today.

Signature · 3 lines
Cheers,
Stephen Rothwell                    [email protected]
http://www.canb.auug.org.au/~sfr/
#1 lore raw

PATCH 1/1 KVM: Remove old kvm_guest_debug structs (was: linux-next: kvm tree build failure)

Quoted 9 lines · Stephen Rothwell

Hi Avi,

Today's linux-next build (powerpc ppc44x_config) failed like this:

In file included from include/linux/kvm_host.h:27,

                 from arch/powerpc/include/asm/kvm_44x.h:23,
                 from arch/powerpc/kernel/asm-offsets.c:52:

arch/powerpc/include/asm/kvm_host.h:203: error: redefinition of 'struct kvm_guest_debug'

Quoted 7 lines

Caused by commit 887e823e3f9c402ff2fe1ea2dc25072ebe81b5b1 ("KVM: New guest debug interface") which added struct kvm_guest_debug to include/linux/kvm.h but didn't remove it from arch/powerpc/include/asm/kvm_host.h. (ia64 and s390 probably have the same problem.)

I have reverted the kvm tree for today.

Yes, missed them. This patch should cure the issue.

Jan

--------->

Remove the remaining arch fragments of the old guest debug interface that now break non-x86 builds.

---

 arch/ia64/include/asm/kvm_host.h    |    2 --
 arch/powerpc/include/asm/kvm_host.h |    6 ------
 arch/s390/include/asm/kvm_host.h    |    3 ---
 3 files changed, 0 insertions(+), 11 deletions(-)
arch/ia64/include/asm/kvm_host.h +0 -2
index 0560f3f..f262efe 100644
@@ -235,8 +235,6 @@ struct kvm_vm_data {
 
 struct kvm;
 struct kvm_vcpu;
-struct kvm_guest_debug{
-};
 
 struct kvm_mmio_req {
 	uint64_t addr;          /*  physical address		*/
arch/powerpc/include/asm/kvm_host.h +0 -6
index c1e436f..a226005 100644
@@ -200,10 +200,4 @@ struct kvm_vcpu_arch {
 	unsigned long pending_exceptions;
 };
 
-struct kvm_guest_debug {
-	int enabled;
-	unsigned long bp[4];
-	int singlestep;
-};
-
 #endif /* __POWERPC_KVM_HOST_H__ */
arch/s390/include/asm/kvm_host.h +0 -3
index 3c55e41..c6e674f 100644
@@ -21,9 +21,6 @@
 /* memory slots that does not exposed to userspace */
 #define KVM_PRIVATE_MEM_SLOTS 4
 
-struct kvm_guest_debug {
-};
-
 struct sca_entry {
 	atomic_t scn;
 	__u64	reserved;
Signature · 2 lines
Siemens AG, Corporate Technology, CT SE 26
Corporate Competence Center Embedded Linux
Applied
#2 lore raw

Re: [PATCH] KVM: Remove old kvm_guest_debug structs

Quoted 4 lines · Jan Kiszka

Remove the remaining arch fragments of the old guest debug interface that now break non-x86 builds.

Applied, thanks.

Signature · 1 line
error compiling committee.c: too many arguments to function