From: David Miller <davem@davemloft.net>
To: linux-sh@vger.kernel.org
Subject: Re: NFS root failing
Date: Sun, 30 Dec 2007 02:55:01 +0000 [thread overview]
Message-ID: <20071229.185501.146899859.davem@davemloft.net> (raw)
In-Reply-To: <1198940473.6278.20.camel@localhost.localdomain>
From: Adrian McMenamin <adrian@newgolddream.dyndns.info>
Date: Sat, 29 Dec 2007 15:01:13 +0000
> I've been getting nowhere with it for a few days now: I had this (NFS
> root) working very well until earlier this week when I upgraded the
> Busybox config on target system. Now it fails - any experts able to
> help?
This patch should fix it, I'll be pushing it to Linus shortly.
[IPV4] Fix ip=dhcp regression
David Brownell pointed out a regression in my recent "Fix ip command
line processing" patch. It turns out to be a fairly blatant oversight on
my part whereby ic_enable is never set, and thus autoconfiguration is
never enabled. Clearly my testing was broken :-(
The solution that I have is to set ic_enable to 1 if we hit
ip_auto_config_setup(), which basically means that autoconfiguration is
activated unless told otherwise. I then flip ic_enable to 0 if ip=off,
ip=none, ip=::::::off or ip=::::::none using ic_proto_name();
The incremental patch is below, let me know if a non-incremental version
is prepared, as I did as for the original patch to be reverted pending a
fix.
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
---
net/ipv4/ipconfig.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/net/ipv4/ipconfig.c b/net/ipv4/ipconfig.c
index 420f56a..56a6757 100644
--- a/net/ipv4/ipconfig.c
+++ b/net/ipv4/ipconfig.c
@@ -1440,6 +1440,7 @@ static int __init ip_auto_config_setup(char *addrs)
int num = 0;
ic_set_manually = 1;
+ ic_enable = 1;
if (ic_proto_name(addrs))
return 1;
--
1.5.4.rc2
next prev parent reply other threads:[~2007-12-30 2:55 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-29 15:01 NFS root failing Adrian McMenamin
2007-12-29 22:14 ` Manuel Lauss
2007-12-29 22:50 ` Adrian McMenamin
2007-12-30 2:55 ` David Miller [this message]
2007-12-30 2:56 ` David Miller
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=20071229.185501.146899859.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=linux-sh@vger.kernel.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