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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.