From: Tejun Heo <tj@kernel.org>
To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: Jens Axboe <jens.axboe@oracle.com>,
Stephen Rothwell <sfr@canb.auug.org.au>,
linux-next@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
Ingo Molnar <mingo@elte.hu>,
dm-devel@redhat.com
Subject: Re: linux-next: Tree for August 27
Date: Sat, 30 Aug 2008 11:10:25 +0200 [thread overview]
Message-ID: <48B90E81.5060905@kernel.org> (raw)
In-Reply-To: <200808292123.07246.bzolnier@gmail.com>
Hello,
Bartlomiej Zolnierkiewicz wrote:
>>> Hmmm... it broke booting? Bart, can you please elaborate a bit what got
>>> broken? Did it fail to locate the root partition or were you using
>>> static device nodes?
>> It works fine for me, fwiw.
>
> It fails to locate the root partition if you are using the default root
> device on x86 (the one hard-wired into bzImage, it uses hard-coded 8-bit
> major/minor numbers).
Right. That would break it too.
> Now I see that the issue is easily fixed by passing the root device name
> with "root=" (since it will figure out the actual major/minor numbers
> later) and that the default root device is said to be deprecated according
> to Documentation/x86/boot.txt (OTOH there is no warning at all that it is
> deprecated and that you should be using "root=" so I'm not convinced that
> I'm the only one who would hit the unfortunate combination).
Yeap, in retrospect, enabling BLOCK_EXT_DEVT by default seems too
disruptive. There also are people running off static /dev. I hope
distros enable it during their alphas and betas tho so that they can
spot problems in configuration utilities and stuff early.
> Actually it seems that CONFIG_DEBUG_BLOCK_EXT_DEVT introduction (which I
> think is a nice step forward) is a good oportunity to put the deprecation
> of the default root device further, i.e. it may be worth to hook the
> default root device check and/or CONFIG_DEBUG_BLOCK_EXT_DEVT one into
> mount_block_root().
Maybe we can add a warning there if BLOCK_EXT_DEVT is enabled or do
you have something else on mind?
Thanks.
--
tejun
next prev parent reply other threads:[~2008-08-30 9:12 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-27 8:34 linux-next: Tree for August 27 Stephen Rothwell
2008-08-27 10:06 ` Bartlomiej Zolnierkiewicz
2008-08-27 14:57 ` Bartlomiej Zolnierkiewicz
2008-08-27 19:45 ` Bartlomiej Zolnierkiewicz
2008-08-29 0:10 ` Bartlomiej Zolnierkiewicz
2008-08-29 7:05 ` Jens Axboe
2008-08-29 8:36 ` Stephen Rothwell
2008-08-29 9:35 ` [PATCH] block: don't test for partition size in bdget_disk() and blk_lookup_devt() Tejun Heo
2008-08-29 9:37 ` linux-next: Tree for August 27 Tejun Heo
2008-08-29 9:42 ` Jens Axboe
2008-08-29 19:23 ` Bartlomiej Zolnierkiewicz
2008-08-30 9:10 ` Tejun Heo [this message]
2008-08-31 13:38 ` Bartlomiej Zolnierkiewicz
2008-09-01 11:00 ` [PATCH #blk-for-2.8] init: DEBUG_BLOCK_EXT_DEVT requires explicit root= param Tejun Heo
2008-09-01 11:16 ` Jens Axboe
2008-09-01 11:20 ` Tejun Heo
2008-09-01 11:30 ` Jens Axboe
2008-09-01 11:38 ` Tejun Heo
2008-09-01 16:34 ` Bartlomiej Zolnierkiewicz
2008-08-28 1:25 ` linux-next: Tree for August 27 Stephen Rothwell
2008-08-28 9:08 ` Ingo Molnar
-- strict thread matches above, loose matches on Subject: below --
2010-08-27 2:12 Stephen Rothwell
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=48B90E81.5060905@kernel.org \
--to=tj@kernel.org \
--cc=bzolnier@gmail.com \
--cc=dm-devel@redhat.com \
--cc=jens.axboe@oracle.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=sfr@canb.auug.org.au \
/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;
as well as URLs for NNTP newsgroup(s).