[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1205962750.4758.17.camel@frecb07144>
Date: Wed, 19 Mar 2008 22:39:10 +0100
From: Laurent Vivier <Laurent.Vivier@...l.net>
To: Andrew Morton <akpm@...ux-foundation.org>
Cc: randy.dunlap@...cle.com, jens.axboe@...cle.com,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH][v4] Modify loop device to be able to manage
partitions of the disk image
Le mercredi 19 mars 2008 à 14:28 -0700, Andrew Morton a écrit :
> On Wed, 19 Mar 2008 21:24:41 +0100
> Laurent Vivier <Laurent.Vivier@...l.net> wrote:
>
> > Le mercredi 19 mars 2008 __ 13:11 -0700, Randy Dunlap a __crit :
> > > On Wed, 19 Mar 2008 13:36:07 +0100 Laurent Vivier wrote:
> > >
> > > > This patch allows to use loop device with partitionned disk image.
> > > >
> > > > Original behavior of loop is not modified.
> > > >
> > > > A new parameter is introduced to define how many partition we want to be
> > > > able to manage per loop device. This parameter is "max_part".
> > >
> > > What happened to the update to Documentation/kernel-parameters.txt
> > > that was in v3?
> >
> > Well, perhaps I didn't understand the comment of Andrew:
> >
> > "This shouldn't be needed."
> >
> > I though it means I should remove it. So, Andrew ???
>
> No, given that all module_param() options are available via the boot
> command line when the module is linked into vmlinux, we don't document them
> separately.
"No" is "To document max_part is not needed"
or
"No" is "you must not remove parameter documentation from your patch" ?
> There should be a way of auto-generating all the documentation for all the
> module parameters from their MODULE_PARM_DESC's. And there probably is,
> but I'm not sure how this is done (?)
>
> (does `make help', fails to spot it).
>
> You can do `modinfo loop' but that probably doesn't work if
> CONFIG_BLK_DEV_LOOP=y?
>
>
>
> I assume you tested the "loop.max_part=N" option?
Yes, I did (with N=63)
Regards,
Laurent
--
------------- Laurent.Vivier@...l.net ---------------
"The best way to predict the future is to invent it."
- Alan Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists