From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH #blk-for-2.8] init: DEBUG_BLOCK_EXT_DEVT requires explicit root= param Date: Mon, 01 Sep 2008 13:20:58 +0200 Message-ID: <48BBD01A.7000603@kernel.org> References: <20080827183453.eb651f50.sfr@canb.auug.org.au> <200808292123.07246.bzolnier@gmail.com> <48B90E81.5060905@kernel.org> <200808311538.38918.bzolnier@gmail.com> <48BBCB62.4040309@kernel.org> <20080901111635.GN20055@kernel.dk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Return-path: Received: from hera.kernel.org ([140.211.167.34]:33255 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758662AbYIALXM (ORCPT ); Mon, 1 Sep 2008 07:23:12 -0400 In-Reply-To: <20080901111635.GN20055@kernel.dk> Sender: linux-next-owner@vger.kernel.org List-ID: To: Jens Axboe Cc: Bartlomiej Zolnierkiewicz , Stephen Rothwell , linux-next@vger.kernel.org, LKML , Ingo Molnar , dm-devel@redhat.com Jens Axboe wrote: >> There are other ways to specify device numbers directly (ie. root=8:0) >> and DEBUG_BLOCK_EXT_DEVT being a debug option I don't think it >> warrants a complex logic, so I just added simple warning message. > > How about just putting that in the help entry as well? Given that it's > now defaulting to 'n', people that enable it should read the help text > anyway. I'll add that to help message but I still think it'll be beneficial to have warning message printed to avoid getting people off guard. e.g. alpha/beta distro built with the debug option turned on to detect bugs. Thanks. -- tejun