From: Randy Dunlap <randy.dunlap@oracle.COM>
To: Greg KH <gregkh@suse.de>
Cc: Ian Abbott <abbotti@mev.co.uk>, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Support kernel/hotplug sysctl variable when !CONFIG_NET
Date: Mon, 7 Jun 2010 11:11:45 -0700 [thread overview]
Message-ID: <20100607111145.1b98fdb4.randy.dunlap@oracle.com> (raw)
In-Reply-To: <20100607174350.GA7807@suse.de>
On Mon, 7 Jun 2010 10:43:50 -0700 Greg KH wrote:
> On Mon, Jun 07, 2010 at 12:57:12PM +0100, Ian Abbott wrote:
> > From: Ian Abbott <abbotti@mev.co.uk>
> >
> > The kernel/hotplug sysctl variable (/proc/sys/kernel/hotplug file) was
> > made conditional on CONFIG_NET by commit
> > f743ca5e10f4145e0b3e6d11b9b46171e16af7ce (applied in 2.6.18) to fix
> > problems with undefined references in 2.6.16 when CONFIG_HOTPLUG=y &&
> > !CONFIG_NET, but this restriction is no longer needed.
Ack, builds for me on x86_64 when CONFIG_NET is not enabled.
> Why is this restriction no longer needed? What changed?
commit cd3772e6898c6386f21d2958346d6dd57d4204f5
Author: Ming Lei <tom.leiming@gmail.com>
Date: Sun Nov 16 18:22:09 2008 +0800
kernel/ksysfs.c:fix dependence on CONFIG_NET
Access to uevent_seqnum and uevent_helper does not need to
depend on CONFIG_NET, so remove it.
Signed-off-by: Ming Lei <tom.leiming@gmail.com>
Cc: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
> > This patch makes the kernel/hotplug sysctl variable depend only on
> > CONFIG_HOTPLUG.
>
> Why is this needed? Are you really running without a network and still
> want to change the default /sbin/hotplug binary?
>
> confused,
---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***
next prev parent reply other threads:[~2010-06-07 18:13 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-07 11:57 [PATCH] Support kernel/hotplug sysctl variable when !CONFIG_NET Ian Abbott
2010-06-07 12:14 ` Ian Abbott
2010-06-07 17:43 ` Greg KH
2010-06-07 18:11 ` Randy Dunlap [this message]
2010-06-07 18:16 ` Greg KH
2010-06-07 18:26 ` Randy Dunlap
2010-06-07 18:49 ` Greg KH
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=20100607111145.1b98fdb4.randy.dunlap@oracle.com \
--to=randy.dunlap@oracle.com \
--cc=abbotti@mev.co.uk \
--cc=gregkh@suse.de \
--cc=linux-kernel@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