From: Tejun Heo <tj@kernel.org>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Alexandre Demers <alexandre.f.demers@gmail.com>,
linux-kernel@vger.kernel.org
Subject: [PATCH driver-core-next] sysfs: restore in-kernel mount of sysfs
Date: Sun, 30 Mar 2014 08:30:36 -0400 [thread overview]
Message-ID: <20140330123036.GA8942@htj.dyndns.org> (raw)
Long ago, sysfs used to depend on vfs data structures for internal
representation and thus needed to make a hidden mount to make them
available. sysfs stopped depending on vfs for internal representation
long ago and 9e30cc959530 ("sysfs, kernfs: no need to kern_mount()
sysfs from sysfs_init()") removed the internal mount.
This doesn't seem to cause wide-spread failure but there is one
failure reported. During boot, userland somehow falls into infinite
loop, fills up one of the filesystems used during boot (possibly a
tmpfs) and then hangs. It isn't yet clear what's going on or why
userland would depend on internal kernel mount of sysfs.
This needs further digging but let's restore the internal mount in the
simplest way.
Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Alexandre Demers <alexandre.f.demers@gmail.com>
---
fs/sysfs/mount.c | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/fs/sysfs/mount.c b/fs/sysfs/mount.c
index a66ad61..dd6d455 100644
--- a/fs/sysfs/mount.c
+++ b/fs/sysfs/mount.c
@@ -75,5 +75,14 @@ int __init sysfs_init(void)
return err;
}
+ /*
+ * XXX: Create an in-kernel mount of sysfs. It isn't clear yet why
+ * this makes any difference; however, there is one report of
+ * userland falling into an infinite loop during boot without it.
+ *
+ * http://lkml.kernel.org/g/CAPEhTTFP3N-ReasmgL5n82mve8p8M3crqmaMvzV+F2p5JCSRbQ@mail.gmail.com
+ */
+ WARN_ON(IS_ERR(kern_mount(&sysfs_fs_type)));
+
return 0;
}
next reply other threads:[~2014-03-30 12:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-30 12:30 Tejun Heo [this message]
2014-03-30 13:13 ` [PATCH driver-core-next] sysfs: restore in-kernel mount of sysfs Tejun Heo
2014-03-30 16:26 ` Greg Kroah-Hartman
2014-03-30 22:04 ` Tejun Heo
2014-04-02 17:05 ` Tejun Heo
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=20140330123036.GA8942@htj.dyndns.org \
--to=tj@kernel.org \
--cc=alexandre.f.demers@gmail.com \
--cc=gregkh@linuxfoundation.org \
--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