From: Marc Zyngier <marc.zyngier@arm.com>
To: Christoffer Dall <cdall@cs.columbia.edu>,
Geoff Levand <geoff@infradead.org>
Cc: Michal Marek <mmarek@suse.cz>,
kvmarm@lists.cs.columbia.edu, linux-kbuild@vger.kernel.org
Subject: Re: kbuild not creating output dirs
Date: Thu, 04 Apr 2013 13:05:13 +0200 [thread overview]
Message-ID: <7122dbf190b8fbc7b734ac0956a9df89@localhost> (raw)
In-Reply-To: <CAEDV+gJog=7dyq=uMf69dAchbdP2boaqXs9ydWmiq-XN+yJ66Q@mail.gmail.com>
On Wed, 3 Apr 2013 16:34:45 -0700, Christoffer Dall
<cdall@cs.columbia.edu>
wrote:
> there was a similar problem with the kvm 32-bit makefile at some
> point, the fix looks ok to me.
>
> -Christoffer
>
> On Wed, Apr 3, 2013 at 4:28 PM, Geoff Levand <geoff@infradead.org>
wrote:
>> Hi Michal,
>>
>> I was building Marc's kvm development kernel outside of the
>> sources (make O=), and I got the following build error:
>>
>> arch/arm64/kvm/../../../arch/arm/kvm/mmio.c: fatal error: opening
>> dependency file arch/arm64/kvm/../../../arch/arm/kvm/.mmio.o.d: No
such
>> file or directory
>>
>> The cause was that kbuild did not create the output directory
>> arch/arm64/kvm/../../../arch/arm/kvm, and so gcc failed.
>>
>> Here is what was in his makefile:
>>
>> obj-$(CONFIG_KVM_ARM_HOST) += $(addprefix ../../../virt/kvm/,
>> kvm_main.o coalesced_mmio.o)
>>
>> I changed this to build a composite and kbuild then created
>> the output directories:
>>
>> obj-$(CONFIG_KVM_ARM_HOST) += kvm.o
>> kvm-$(CONFIG_KVM_ARM_HOST) += $(addprefix ../../../virt/kvm/,
>> kvm_main.o coalesced_mmio.o)
>>
>> I guess kbuild used some other rules and saw the dependency on
>> the output directory.
>>
>> So, my questions are:
>>
>> Is it expected that the original rule should fail?
>> Is my fix the correct way?
>> Can I use some other way to get those output directories created?
Yeah, this looks like the thing to do. I'll fix the KVM/arm64 makefile.
Thanks for investigating.
M.
--
Fast, cheap, reliable. Pick two.
prev parent reply other threads:[~2013-04-04 11:23 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-03 23:28 kbuild not creating output dirs Geoff Levand
2013-04-03 23:34 ` Christoffer Dall
2013-04-04 11:05 ` Marc Zyngier [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7122dbf190b8fbc7b734ac0956a9df89@localhost \
--to=marc.zyngier@arm.com \
--cc=cdall@cs.columbia.edu \
--cc=geoff@infradead.org \
--cc=kvmarm@lists.cs.columbia.edu \
--cc=linux-kbuild@vger.kernel.org \
--cc=mmarek@suse.cz \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox