From: Rusty Russell <rusty@rustcorp.com.au>
To: Satyam Sharma <satyam.sharma@gmail.com>
Cc: Gabriel C <nix.or.die@googlemail.com>,
Linus Torvalds <torvalds@linux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
lguest <lguest@ozlabs.org>
Subject: Re: [PATCH] Make lguest compile with CONFIG_BLOCK=n and CONFIG_NET=n
Date: Mon, 30 Jul 2007 11:07:49 +1000 [thread overview]
Message-ID: <1185757669.12151.108.camel@localhost.localdomain> (raw)
In-Reply-To: <a781481a0707291748x5a70c2d3i83ef54d7f2336b33@mail.gmail.com>
On Mon, 2007-07-30 at 06:18 +0530, Satyam Sharma wrote:
> On 7/30/07, Rusty Russell <rusty@rustcorp.com.au> wrote:
> > +config LGUEST_NET
> > + tristate
> > + depends on LGUEST_GUEST && NET
>
> default y ?
Thanks, that does work better. Clean configs get an "N" otherwise 8(
> I /think/ the default "y" would also automatically become "m" if any of the
> dependencies are modules ... probably need to test this, though.
No, it doesn't seem to. But LGUEST_GUEST is a bool anyway, so it's OK.
Thanks!
Rusty.
==
Make lguest compile with CONFIG_BLOCK=n and CONFIG_NET=n
Gabriel C reports lguest doesn't compile with CONFIG_BLOCK=n. Fix
this by introducing a config var for the block device, which depends
on LGUEST && BLOCK. Do the same for the net driver, rather then
depending gratuitously on CONFIG_NET.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
diff -r 9e987fcabb16 drivers/block/Makefile
--- a/drivers/block/Makefile Mon Jul 30 09:47:25 2007 +1000
+++ b/drivers/block/Makefile Mon Jul 30 10:02:32 2007 +1000
@@ -31,4 +31,4 @@ obj-$(CONFIG_BLK_DEV_UB) += ub.o
obj-$(CONFIG_BLK_DEV_UB) += ub.o
obj-$(CONFIG_XEN_BLKDEV_FRONTEND) += xen-blkfront.o
-obj-$(CONFIG_LGUEST_GUEST) += lguest_blk.o
+obj-$(CONFIG_LGUEST_BLOCK) += lguest_blk.o
diff -r 9e987fcabb16 drivers/lguest/Kconfig
--- a/drivers/lguest/Kconfig Mon Jul 30 09:47:25 2007 +1000
+++ b/drivers/lguest/Kconfig Mon Jul 30 10:03:39 2007 +1000
@@ -1,6 +1,6 @@ config LGUEST
config LGUEST
tristate "Linux hypervisor example code"
- depends on X86 && PARAVIRT && NET && EXPERIMENTAL && !X86_PAE
+ depends on X86 && PARAVIRT && EXPERIMENTAL && !X86_PAE
select LGUEST_GUEST
select HVC_DRIVER
---help---
@@ -18,3 +18,13 @@ config LGUEST_GUEST
The guest needs code built-in, even if the host has lguest
support as a module. The drivers are tiny, so we build them
in too.
+
+config LGUEST_NET
+ tristate
+ default y
+ depends on LGUEST_GUEST && NET
+
+config LGUEST_BLOCK
+ default y
+ tristate
+ depends on LGUEST_GUEST && BLOCK
diff -r 9e987fcabb16 drivers/net/Makefile
--- a/drivers/net/Makefile Mon Jul 30 09:47:25 2007 +1000
+++ b/drivers/net/Makefile Mon Jul 30 10:02:22 2007 +1000
@@ -177,7 +177,7 @@ obj-$(CONFIG_HPLANCE) += hplance.o 7990.
obj-$(CONFIG_HPLANCE) += hplance.o 7990.o
obj-$(CONFIG_MVME147_NET) += mvme147.o 7990.o
obj-$(CONFIG_EQUALIZER) += eql.o
-obj-$(CONFIG_LGUEST_GUEST) += lguest_net.o
+obj-$(CONFIG_LGUEST_NET) += lguest_net.o
obj-$(CONFIG_MIPS_JAZZ_SONIC) += jazzsonic.o
obj-$(CONFIG_MIPS_AU1X00_ENET) += au1000_eth.o
obj-$(CONFIG_MIPS_SIM_NET) += mipsnet.o
next prev parent reply other threads:[~2007-07-30 1:08 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-07-29 15:18 [PATCH] BLOCK=n , LGUEST=m/y compile error Gabriel C
2007-07-30 0:25 ` [PATCH] Make lguest compile with CONFIG_BLOCK=n and CONFIG_NET=n Rusty Russell
2007-07-30 0:48 ` Satyam Sharma
2007-07-30 1:07 ` Rusty Russell [this message]
2007-07-30 10:09 ` Gabriel C
2007-07-31 10:11 ` [PATCH] Fix lguest driver config options Rusty Russell
2007-07-30 1:29 ` [PATCH] Make lguest compile with CONFIG_BLOCK=n and CONFIG_NET=n Gabriel C
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=1185757669.12151.108.camel@localhost.localdomain \
--to=rusty@rustcorp.com.au \
--cc=lguest@ozlabs.org \
--cc=linux-kernel@vger.kernel.org \
--cc=nix.or.die@googlemail.com \
--cc=satyam.sharma@gmail.com \
--cc=torvalds@linux-foundation.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