On Mon, 2013-11-25 at 04:26 -0600, Bharat Bhushan wrote: > > > -----Original Message----- > > From: Wood Scott-B07421 > > Sent: Saturday, November 23, 2013 3:22 AM > > To: Alexander Graf > > Cc: kvm-ppc@xxxxxxxxxxxxxxx; kvm@xxxxxxxxxxxxxxx; Wood Scott-B07421; Bhushan > > Bharat-R65777 > > Subject: [PATCH] powerpc/kvm/booke: Fix build break due to stack frame size > > warning > > > > Commit ce11e48b7fdd256ec68b932a89b397a790566031 ("KVM: PPC: E500: Add > > userspace debug stub support") added "struct thread_struct" to the > > stack of kvmppc_vcpu_run(). thread_struct is 1152 bytes on my build, > > compared to 48 bytes for the recently-introduced "struct debug_reg". > > Use the latter instead. > > > > This fixes the following error: > > > > cc1: warnings being treated as errors > > arch/powerpc/kvm/booke.c: In function 'kvmppc_vcpu_run': > > arch/powerpc/kvm/booke.c:760:1: error: the frame size of 1424 bytes is larger > > than 1024 bytes > > make[2]: *** [arch/powerpc/kvm/booke.o] Error 1 > > make[1]: *** [arch/powerpc/kvm] Error 2 > > make[1]: *** Waiting for unfinished jobs.... > > > > Signed-off-by: Scott Wood <scottwood@xxxxxxxxxxxxx> > > Cc: Bharat Bhushan <r65777@xxxxxxxxxxxxx> > > --- > > Build tested only. Bharat, please test. > > Tested with qemu debug stub; It works fine > > -Bharat Alex, are you going to take this through your tree? -Scott -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html