mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Olof Johansson <olof@lixom.net>
To: Lei Wen <adrian.wenl@gmail.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org,
	Mandeep Baines <msb@chromium.org>
Subject: Re: [PATCH v3] mmc: add config and runtime option for number of mmcblk minors
Date: Wed, 8 Sep 2010 09:57:05 -0500	[thread overview]
Message-ID: <20100908145705.GC23745@lixom.net> (raw)
In-Reply-To: <AANLkTikLLwfSXbnjPByXTktEFFVueeAd7HZS5ft-rj-m@mail.gmail.com>

Hi,

On Wed, Sep 08, 2010 at 10:25:58PM +0800, Lei Wen wrote:

> The patch's purpose is good. As modern sd&mmc is used to host the file
> system, the 8 partition limitation is becoming a kind of bottleneck...
> But why not just add GENHD_FL_EXT_DEVT flag to allow mmc use extended
> partition numbers?

No need to quote the whole patch next time. :)

Given that MMC is sometimes used in deeply embedded environments where
udev might not be running to take care of dynamic device numbering,
I chose to stay with a static layout.

I could be convinced otherwise though. It would cause some additional
hassles for me since we start udev lateish during boot and have a
prepopulated /dev before that, but that can be dealt with.


-Olof


  reply	other threads:[~2010-09-08 14:57 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-18  4:13 [PATCH] mmc: make number of mmcblk minors configurable Olof Johansson
2010-08-19  0:16 ` Andrew Morton
2010-08-19  3:22   ` Olof Johansson
2010-08-20  1:02     ` [PATCH v2] mmc: add config and runtime option for number of mmcblk minors Olof Johansson
2010-08-20 22:13       ` [PATCH v3] " Olof Johansson
2010-08-20 22:18         ` sys_init_module system call runcoderen
2010-08-20 22:32           ` Randy Dunlap
2010-08-20 22:50             ` runcoderen
2010-09-08 14:25         ` [PATCH v3] mmc: add config and runtime option for number of mmcblk minors Lei Wen
2010-09-08 14:57           ` Olof Johansson [this message]
2010-09-08 15:19             ` Kay Sievers
2010-09-08 15:57               ` Olof Johansson
2010-09-08 16:48                 ` Colin Cross
2010-09-08 16:50                   ` [PATCH] mmc_block: Allow more than 8 partitions per card Colin Cross
2010-09-09  2:49                     ` Lei Wen
2010-09-09  7:54                     ` Adrian Hunter

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=20100908145705.GC23745@lixom.net \
    --to=olof@lixom.net \
    --cc=adrian.wenl@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=msb@chromium.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