mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Tejun Heo <tj@kernel.org>
To: Alexandre Demers <alexandre.f.demers@gmail.com>
Cc: linux-kernel@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Subject: Re: [Regression, bisected 9e30cc] "sysfs, kernfs: no need to kern_mount() sysfs from sysfs_init()" prevents system from booting correctly
Date: Fri, 14 Mar 2014 10:08:04 -0400	[thread overview]
Message-ID: <20140314140804.GG12613@htj.dyndns.org> (raw)
In-Reply-To: <CAPEhTTFP3N-ReasmgL5n82mve8p8M3crqmaMvzV+F2p5JCSRbQ@mail.gmail.com>

Hello, Alexandre.

On Fri, Mar 14, 2014 at 09:27:35AM -0400, Alexandre Demers wrote:
> I was told to send this bug
> (https://bugzilla.kernel.org/show_bug.cgi?id=72061) directly by email.
> 
> So, I've been struggling with something that looks like a bug, but I
> may be wrong. Since 3.14-rc1 up until yesterday's latest commit, I
> can't boot my system correctly. The kernel will boot, but when it
> comes to mounting the disk's partitions, the computer seem to hit
> something while mounting the root partition: the light associated to
> the disk's activity stays on; if I hit ctrl+alt+del in the first 10 to
> 15 seconds, the computer will restart; beyond that point, the system
> just continues to work on the disk for a long period (I was painting a
> door the other day and it continued reading or writing the whole time)
> until it begins outputting errors. I'll have to add it in another
> comment later. It is about not being able to copy because disk is full
> (my partitions have plenty of free space). So I've been bisecting
> until I found out when the problem had begun.
> 
> ------
> Bisecting points to the following first bad commit:
> 9e30cc9595303b27b48be49b7bcd4d0679e34253 is the first bad commit
> commit 9e30cc9595303b27b48be49b7bcd4d0679e34253
> Author: Tejun Heo <tj@kernel.org>
> Date:   Thu Nov 28 14:54:38 2013 -0500
> 
>     sysfs, kernfs: no need to kern_mount() sysfs from sysfs_init()
> 
>     It has been very long since sysfs depended on vfs to keep track of
>     internal states and whether sysfs is mounted or not doesn't make any
>     difference to sysfs's internal operation.
> 
>     In addition to init and filesystem type registration, sysfs_init()
>     invokes kern_mount() to create in-kernel mount of sysfs.  This
>     internal mounting doesn't server any purpose anymore.  Remove it.
> 
>     Signed-off-by: Tejun Heo <tj@kernel.org>
>     Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> 
> :040000 040000 d85991e8a4e333ad651dc593cacff5e6b8d4f916
> d3bd9c50967807c9145a7bd9d3ec7978101c93f3 M fs
> ------
> 
> Using any kernel compiled before that commit boots fine. Am I missing something?

Hah, that's interesting and completely unexpected.  What userland are
you using?  Also, after such failure fills up the filesystem, can you
determine which files are filling it up?  We'll need to restore the
original behavior but need to first find out how userland depends on
it at all.

Thanks.

-- 
tejun

  reply	other threads:[~2014-03-14 14:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-14 13:27 Alexandre Demers
2014-03-14 14:08 ` Tejun Heo [this message]
2014-03-14 15:48   ` Alexandre Demers
2014-03-17 20:13     ` Tejun Heo
2014-03-18  2:08       ` Alexandre Demers
2014-03-26  1:03         ` Alexandre Demers
2014-03-26 13:18           ` Tejun Heo
2014-03-27 17:47             ` Alexandre Demers
2014-03-29 18:12               ` Tejun Heo
2014-03-29 22:08                 ` Alexandre Demers
2014-03-30 12:31                   ` Tejun Heo
2014-03-30 16:53                     ` Alexandre Demers
2014-04-02  5:42                       ` Alexandre Demers

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=20140314140804.GG12613@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

Powered by JetHome