From: Randy Dunlap <randy.dunlap@oracle.com>
To: Valdis.Kletnieks@vt.edu
Cc: Roman Zippel <zippel@linux-m68k.org>,
Michal Marek <mmarek@suse.cz>,
Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org, linux-kbuild@vger.kernel.org
Subject: Re: mmotm 2010-01-06-14-34 uploaded (squashfs+lib/decomp)
Date: Fri, 8 Jan 2010 08:37:17 -0800 [thread overview]
Message-ID: <20100108083717.31ff968c.randy.dunlap@oracle.com> (raw)
In-Reply-To: <24025.1262933694@localhost>
On Fri, 08 Jan 2010 01:54:54 -0500 Valdis.Kletnieks@vt.edu wrote:
> On Thu, 07 Jan 2010 08:04:02 PST, Randy Dunlap said:
> > On Wed, 06 Jan 2010 23:01:47 -0500 Valdis.Kletnieks@vt.edu wrote:
> >
> > > On Wed, 06 Jan 2010 22:48:16 EST, Valdis.Kletnieks@vt.edu said:
> > >
> > > > Building with CONFIG_SQUASHFS=m, CONFIG_SQUASHFS_LZMA=y ,
> > > > and CONFIG_DECOMPRESS_LZMA=n fails:
> > >
> > > > Looks like a missing select/depends for DECOMPRESS_LZMA. Somebody else can
> > > > decide which it should be...
> > >
> > > Digging further:
> > >
> > > x Symbol: DECOMPRESS_LZMA [=n] x
> > > x Selected by: RD_LZMA [=n] && BLK_DEV_INITRD [=y] || SQUASHFS_LZMA [=y x
> > >
> > > How the heck did this happen? Looks like a SELECT *is* there but it's
> > > not firing??!?
> >
> > I saw this build error in linux-next and sent a patch for it -- it's below.
> > However, it doesn't appear to be exactly the same config as yours.
>
> Yeah, that patch is how you ended up cc'ed - it was already in the linux-next
> patch in Andrew's -mmotm tarball. It looked like you addressed one corner case
> of the problem, and I tripped over another. But now I'm trying to get my
> brain wrapped around how I was able to get SQUASHFS_LZMA=y, and it had a
> select for DECOMPRESS_LZMA, but that *still* was 'n' anyhow? How did we end
> up with both && and || in that 'Selected by:' line? Is that saying that
> RD_LZMA needs to be on too?
The menuconfig display is truncated (use left/right arrow keys to pan it).
The xconfig display shows this:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
DECOMPRESS_LZMA
type: tristate
reverse dep: RD_LZMA && BLK_DEV_INITRD || SQUASHFS_LZMA && MISC_FILESYSTEMS && SQUASHFS
defined at lib/Kconfig:117
There is no help available for this kernel option.
Symbol: DECOMPRESS_LZMA [=n]
Selected by: RD_LZMA [=n] && BLK_DEV_INITRD [=n] || SQUASHFS_LZMA [=n] && MISC_FILESYSTEMS [=n] && SQUASHFS [=n]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In the "reverse dep:" line, && has higher precedence that || does, so it's:
reverse dep: (RD_LZMA && BLK_DEV_INITRD) || (SQUASHFS_LZMA && MISC_FILESYSTEMS && SQUASHFS)
So RD_LZMA and BLK_DEV_INITRD don't matter.
> Adding some plausible-sounding kbuild-related entries from MAINTAINERS to
> the cc: list, maybe they can explain it...
---
~Randy
next prev parent reply other threads:[~2010-01-08 16:38 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-06 22:34 mmotm 2010-01-06-14-34 uploaded akpm
2010-01-07 0:57 ` [PATCH -mmotm] memcg: implement memory thresholds document fixes Daisuke Nishimura
2010-01-07 1:18 ` KAMEZAWA Hiroyuki
2010-01-07 1:34 ` Daisuke Nishimura
2010-01-07 1:40 ` KAMEZAWA Hiroyuki
2010-01-07 8:54 ` Balbir Singh
2010-01-07 8:52 ` Balbir Singh
2010-01-07 9:04 ` KAMEZAWA Hiroyuki
2010-01-07 6:29 ` Kirill A. Shutemov
2010-01-07 1:08 ` mmotm 2010-01-06-14-34 uploaded (kernel/cgroup) Randy Dunlap
2010-01-07 2:31 ` KAMEZAWA Hiroyuki
2010-01-07 2:41 ` Li Zefan
2010-01-07 2:47 ` KAMEZAWA Hiroyuki
2010-01-07 3:04 ` Li Zefan
2010-01-07 16:47 ` Randy Dunlap
2010-01-07 17:29 ` Kirill A. Shutemov
2010-01-07 1:10 ` mmotm 2010-01-06-14-34 uploaded (mm/memcontrol) Randy Dunlap
2010-01-07 2:13 ` Daisuke Nishimura
2010-01-07 2:21 ` KAMEZAWA Hiroyuki
2010-01-07 2:59 ` Daisuke Nishimura
2010-01-07 3:02 ` KAMEZAWA Hiroyuki
2010-01-07 4:06 ` [PATCH -mmotm] build fix for memcg-move-charges-of-anonymous-swap.patch Daisuke Nishimura
2010-01-07 4:30 ` KAMEZAWA Hiroyuki
2010-01-07 5:07 ` KAMEZAWA Hiroyuki
2010-01-07 5:14 ` Daisuke Nishimura
2010-01-07 5:52 ` KAMEZAWA Hiroyuki
2010-01-07 11:23 ` How should we handle CONFIG_CGROUP_MEM_RES_CTRL_SWAP (Re: [PATCH -mmotm] build fix for memcg-move-charges-of-anonymous-swap.patch) Daisuke Nishimura
2010-01-07 12:38 ` Balbir Singh
2010-01-07 4:06 ` [PATCH -mmotm] build fix for memcg-improve-performance-in-moving-swap-charge.patch Daisuke Nishimura
2010-01-07 4:31 ` KAMEZAWA Hiroyuki
2010-01-07 3:48 ` mmotm 2010-01-06-14-34 uploaded Valdis.Kletnieks
2010-01-07 4:01 ` Valdis.Kletnieks
2010-01-07 16:04 ` mmotm 2010-01-06-14-34 uploaded (squashfs+lib/decomp) Randy Dunlap
2010-01-08 6:54 ` Valdis.Kletnieks
2010-01-08 16:37 ` Randy Dunlap [this message]
2010-01-07 17:13 ` mmotm 2010-01-06-14-34 uploaded Randy Dunlap
2010-01-08 22:21 ` mmotm 2010-01-06-14-34 - crash in CPUfreq Valdis.Kletnieks
2010-01-11 15:39 ` Minchan Kim
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=20100108083717.31ff968c.randy.dunlap@oracle.com \
--to=randy.dunlap@oracle.com \
--cc=Valdis.Kletnieks@vt.edu \
--cc=akpm@linux-foundation.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mmarek@suse.cz \
--cc=zippel@linux-m68k.org \
/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
Powered by JetHome