Re: [PATCH 5/7] kvm/arm: Fix missing include build error

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

 



On Thu, Apr 4, 2013 at 5:04 PM, Geoff Levand <geoff@xxxxxxxxxxxxx> wrote:
> Hi Christoffer,
>
> On Thu, 2013-04-04 at 16:51 -0700, Christoffer Dall wrote:
>> On Thu, Apr 4, 2013 at 4:33 PM, Geoff Levand <geoff@xxxxxxxxxxxxx> wrote:
>> > Include linux/cpu.h in kvm/arm.c.  Fixes build errors like
>> > these with ARCH=arm64:
>> >
>> > arch/arm/kvm/arm.c: error: ‘CPU_STARTING_FROZEN’ undeclared
>> >
>> > Signed-off-by: Geoff Levand <geoff@xxxxxxxxxxxxx>
>> > ---
>> >  arch/arm/kvm/arm.c |    1 +
>> >  1 file changed, 1 insertion(+)
>> >
>> > diff --git a/arch/arm/kvm/arm.c b/arch/arm/kvm/arm.c
>> > index 5a93698..49b4bc6 100644
>> > --- a/arch/arm/kvm/arm.c
>> > +++ b/arch/arm/kvm/arm.c
>> > @@ -24,6 +24,7 @@
>> >  #include <linux/fs.h>
>> >  #include <linux/mman.h>
>> >  #include <linux/sched.h>
>> > +#include <linux/cpu.h>
>> >  #include <linux/kvm.h>
>> >  #include <trace/events/kvm.h>
>> >
>> > --
>> >
>>
>> I'm confused, I don't see this symbol in arm.c - against which tree do
>> these patches apply exactly? If this is something introduced by arm64,
>> then it's premature, and should be added to that series.
>
> Sorry, this one is from Marc's kvm-for-next branch.  I'll send it
> to him to include, unless you just want to take it in preparation.
>
Hmm, not sure what the status of his branch is exactly. Does it
contain the arm64 stuff?

I don't really want to merge this before it's needed.

-Christoffer
--
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