public inbox for linux-kbuild@vger.kernel.org
 help / color / mirror / Atom feed
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.

      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