Re: [PATCH] powerpc/kvm/booke: Fix build break due to stack frame size warning

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

 



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




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux